Skip to content
This repository was archived by the owner on Oct 2, 2020. It is now read-only.

Conversation

adamheinrich
Copy link
Contributor

Stereo headphone amplifier with I2C volume control by TI.

There are two versions of the IC (BGA and QFN), this only includes the RTJ (WQFN-20) footprint.

I deliberately renamed pins LEFTINP/LEFTINM to LIN+/- and RIGHTINP/RIGHTINM to RIN+/- for clarity.

Also, pins 12 (AVDD) and 3, 9, 10, 13 (AGND) are called Vdd/GND in the datasheet but the datasheet clearly says they are Analog power/ground so I renamed them to distinguish them from pins 20 (Charge pump Vdd) and 19 (Charge pump GND).

Datasheet: http://www.ti.com/lit/ds/symlink/tpa6130a2.pdf

Associated footprint PR: KiCad/kicad-footprints#1471

TPA6130A2RTJ

TPA6130A2RTJ_pintable


All contributions to the kicad library must follow the KiCad library convention

Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items:

  • Provide a URL to a datasheet for the symbol(s) you are contributing
  • An example screenshot image is very helpful
  • Ensure that the associated footprints match the official footprint library
    • A new fitting footprint must be submitted if the library does not yet contain one.
  • If there are matching footprint PRs, provide link(s) as appropriate
  • Check the output of the Travis automated check scripts - fix any errors as required
  • Give a reason behind any intentional library convention rule violation.

Stereo headphone amplifier with I2C volume control by TI.

Signed-off-by: Adam Heinrich <adam@adamh.cz>
@myfreescalewebpage myfreescalewebpage added Pending reviewer A pull request waiting for a reviewer Addition Adds new symbols to library Pending footprint Pending footprint acceptance before merging labels Mar 10, 2019
@myfreescalewebpage myfreescalewebpage self-assigned this Mar 10, 2019
@myfreescalewebpage myfreescalewebpage removed the Pending reviewer A pull request waiting for a reviewer label Mar 10, 2019
@myfreescalewebpage
Copy link
Collaborator

myfreescalewebpage commented Mar 10, 2019

Hi @adamheinrich , thanks for contributing,

A few comments I have during my review:

  • Datasheet indicates that pins 3, 9, 10, 13, 21 (called AGND for you) must be connected to GND, so I suggest to stack them with pin 19.
  • Name of pins 1, 2, 4, 5 should be set according to the datasheet even if I understand your proposition, this is to avoid confusion for other user in the future. Increase size of the device vertically to achieve this.

Else, good design and good initiative for name of pin 12.

Cheers,
Joel

Signed-off-by: Adam Heinrich <adam@adamh.cz>
@adamheinrich
Copy link
Contributor Author

Hi @myfreescalewebpage, I grouped all grounds together and renamed pinds 1, 2, 4, 5 as you requested: 2bc02b0

Instead of resizing the device vertically, I put the I2C pins on top -- is that correct?

TPA6130A2RTJ

TPA6130A2RTJ_pintable

@myfreescalewebpage
Copy link
Collaborator

Thanks @adamheinrich that's ok for me like that, just need to wait the footprint now.

@myfreescalewebpage
Copy link
Collaborator

No news of the author on the footprint generator PR, indicate this is Abandoned

@myfreescalewebpage myfreescalewebpage added Abandoned Original author has stopped working on the PR and removed Abandoned Original author has stopped working on the PR labels Sep 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Abandoned Original author has stopped working on the PR Addition Adds new symbols to library Pending footprint Pending footprint acceptance before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants