Skip to content
This repository has been archived by the owner on Oct 27, 2021. It is now read-only.

Major fixes to linear library #1081

Closed
wants to merge 167 commits into from
Closed

Conversation

evanshultz
Copy link
Collaborator

@evanshultz evanshultz commented Mar 1, 2017

There were a number of things wrong with this library so I tried to fix them as best as possible. I understand this is a large number of changes, but many of them are quite minor. Please take a look.

Except for EC03, which is often bogus, and a few notes below, all symbols pass KLC.

Changes:

  • Add MC33078, MC33079, MC33179
  • Add NE5534
  • Correct footprint info on OPA376IDCK
  • Add NC pin 2 to MAX472
  • Add keywords to MAX471/472
  • MAX4081F power pins to 'W' type
  • LTC6102* box outline width from 0 to 10mil
  • Add missing NC pins to LTC6082
  • Add datasheet links to LM311, LM319
  • LM319: Add NC pins and correct pin 8 name to Gnd2
  • LM311: GND pin to type 'W'
  • LM2903: One set of power pins for all units
  • Added LM3600 to DCM but without a datasheet because I don't find an "official" one. TI seems to have kept only LM13700.
  • Add missing pins and datasheet to LH0002H
  • LF398: Add missing pin 2 and update documentation
  • Add missing NC pins to CA3080
  • AD8213: Make one set of power pins for all units
  • 1408: Place pin 14 on 100mil grid and update documentation

Still to do:

  • Review and fix the many lame datasheet links
  • L0002H: expose E3/E4 pins

Questions:

  • 1408: Should it be renamed to "MC1408", does it really go in this library, and can I use a substitute datasheet (TI DAC0808)?
  • Can pin names be lowercase?

@evanshultz
Copy link
Collaborator Author

Please comment on the below symbol for LH0002H:
image

I added the E3 and E5 pins. Between all the pins and their names, I can't find anything I really like. This isn't terrible, but I hope somebody can find an even better arrangement.

Thanks!

@evanshultz
Copy link
Collaborator Author

I don't find a datasheet for LM13600 from an official source, which at this point is TI. What is the requirement for datasheet links? Must they come from the component vendor?

@yhlien
Copy link

yhlien commented Mar 2, 2017

I quickly checked LM13700 datasheet and found
""The output buffers of the LM13700 differe from those of the LM13600 in that their input biass currents....". The pins arrangement of these two are identical as well. So it might be a good idea to set LM13600 as an alias of LM13700?

@evanshultz
Copy link
Collaborator Author

@yhlien Thanks for the reply. Good point! Yes, they can be an ALIAS and that would make the library smaller. There are probably other cases where this can happen and I'll take a look thanks to your comment.

Still not sure about an "unofficial" datasheet...

@evanshultz
Copy link
Collaborator Author

Changes:

  • LM6171
  • LM6321
  • Eliminated high-temp versions of LM6361, which included all 10-pin versions

@SchrodingersGat @jkriege2
I know you guys are busy with the reorganization and other things, but I'd really like to know if we've sorted out the naming scheme. I wrote a reply at #1398 (comment) which I hope sums up what needs to be done. If that's right I will need to make changes before this gets merged.

@evanshultz evanshultz mentioned this pull request Jul 22, 2017
@evanshultz
Copy link
Collaborator Author

I realized there were a number of outstanding PRs that might fall into the linear library, so I searching around to incorporate them. Surprisingly, I found only one: OPA2691.

@evanshultz
Copy link
Collaborator Author

Made updates according to the latest KLC, which uses no suffix for generic parts with common pinouts, full MPN for unique parts (multiple symbols if there are different packages or pinouts), and package name suffix (_SOIC, for example) when there are multiple vendors with different package names for the same part.

Changes:

  • LF398: TI in TO can, TI and LT in DIP8, TI in SOIC14, LT in SOIC8
  • LM319: many vendors for SOIC14/DIP14, only TI in oddball TO can

I also found a duplicate DCM entry for LM733H.

@SchrodingersGat @jkriege2 @poeschlr
This is all the known changes I'm aware of for this library. I'll leave this PR idle for a couple days to gather comments, then probably start a clean PR to merge this major update. If there are any known or suspected gotchas when basically blowing out a whole library and starting over, please let me know.

@evanshultz
Copy link
Collaborator Author

I realized that there might be other parts in other libraries that belong in linear.lib. After searching by hand and for "op amp", "opamp", "amp", "comp", "current", and maybe some other terms, I found:

  • analog_devices.lib: Moved AD623 and several instrumentation amps
  • sensors.lib: AD8418 was duplicated here, moved INAxxx parts, moved ZXCT* parts

@evanshultz
Copy link
Collaborator Author

With #1437 merged, closing this. Phew!

@SchrodingersGat
Copy link
Contributor

What a saga! Thanks for your fine work :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants