KiCad / kicad-footprints

Official KiCad Footprint Libraries for Kicad version 5
https://kicad.github.io/footprints
Other
617 stars 714 forks source link

Texas S-PWSON-N10 issue #1536

Open myfreescalewebpage opened 5 years ago

myfreescalewebpage commented 5 years ago

Wanted to submit a new symbol, I have just checked if the footprint already exist. It is in the library but has not the right dimensions.

Existing footprint: Package_SON.pretty/Texas_S-PWSON-N10.kicad_mod Dimensions: 3x3mm Was added for: http://www.ti.com/lit/ds/symlink/tps63060.pdf

The device I want to submit: http://www.ti.com/lit/ds/symlink/tps54541.pdf It has the same package name but it is 4x4mm.

Seems Texas package naming is not so clear and will need to pay attention for future submissions.... How is it possible to handle this for my new device ?

Thanks, Joel

myfreescalewebpage commented 5 years ago

Proposition: keeping package size and pitch every time ? In this case:

Joel

poeschlr commented 5 years ago

In that case we can consider to completely drop the ti name alltogether as it then no longer realy adds anything (It can still be included in the description and keyword fields meaning a search will bring up the correct package)

We should however ensure that we do not change too many footprint names during a stable release phase. I might need to think about this a bit this weekend.

myfreescalewebpage commented 5 years ago

According to me keeping Texas is great to explain from where the "S-PWSON-N10" comes from, but I'm not absolutely attached to it.

Also, I was thinking that we have probably several verification to re-check to ensure the symbols are correctly associated to the footprints because this kind of can clearly be forgotten during symbol reviews...

Sure, this is something to think, take the time required.

In my opinion the v6 is the opportunity to do this kind of modifications. People who want a stable release install and use v5.1 and do not follow the github repos. I have already discuss of that with @evanshultz because I think there is several huge enhancements like that to think and it is not possible to handle them just before the v6 release, we have to do it in advance.

evanshultz commented 5 years ago

As I noted somewhere else, I think about a RF IC, TI told me the three letter name is the one to use. It is most descriptive and informative. You will see that play out again here where the footprint name shown above is the same but the three letter designation is different: DSC vs DPR.

In addition, the Texas_ prefix is valuable if the footprint isn't derived from a standard (like IPC) but instead adheres to the TI footprint given. In that case, we need the manufacturer prefix to indicate it's not a generic footprint but one meeting a specific manufacturer's recommendation.

evanshultz commented 5 years ago

Found it. https://github.com/KiCad/kicad-footprints/issues/606

myfreescalewebpage commented 5 years ago

Well done @evanshultz that's clearly the same issue applied to other devices :)

myfreescalewebpage commented 5 years ago

@poeschlr any news for this subject ? Thanks for your return, Joel

poeschlr commented 5 years ago

I think in future we simply go with the generic naming instead of using manufacturer specific conventions. (Unless it is a very specialized part.)

Edit: and include the JEDEC number and the TI number in the description/keyword field. (That way users can search for it.)

myfreescalewebpage commented 5 years ago

Thanks. For the referenced footprint in my first message, can you give an example of how they should be named ?

Edit: for these footprints, to avoid confusion, I suggest they should be renamed without waiting for the v6. What is your opinion ?

myfreescalewebpage commented 5 years ago

@poeschlr any help for my previous comment here ? Thanks so much, Joel

slowcoder commented 4 years ago

Just a heads up. The PWSON-N10 footprint still doesn't work for TPS54541. I found out the hard way.