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

update TMLM series and add missing symbols #427

Merged
merged 3 commits into from
Jul 30, 2018

Conversation

hvraven
Copy link
Contributor

@hvraven hvraven commented Mar 27, 2018

  • remove redundant aliases in favour of TMLM041xx
  • add missing symbols with higher output power
  • add dual output variants TMLM042xx
  • slightly update description

datasheet, footprints are already exsting

  • TMLM041xx (unchanged)
    tmlm041xx
  • Dual output variant
    tmlm042xx
  • Higher power variants (AC connections are not distinguished)
    tmlm051xx

@hvraven
Copy link
Contributor Author

hvraven commented Mar 27, 2018

This fixes issues #154 and #155

@Misca1234
Copy link
Collaborator

If these are galvanic isolated they should perhaps have a division line to mark that ?
(like some of the others ACDC, DCDC)

@hvraven
Copy link
Contributor Author

hvraven commented Mar 27, 2018

so far none of the ACDC converters have that line. I am not really sure if it helps adding it. If it should be added then it should most likely be added to all other ACDC converts as well (would need to check the datasheets for that).

@Misca1234
Copy link
Collaborator

Misca1234 commented Apr 8, 2018

It is not stated in the guidelines so I guess it is more of a matter of taste among the heavyweighters within the library admin team.

But antoniovazquezblanco open a issue on the matter
#429

@hvraven
Copy link
Contributor Author

hvraven commented May 28, 2018

As there seems to be some agreement by now I reworked the footprints. I also increased the height of the boxes a bit as I found them a bit to crowded.

tmlm042xx
tmlm041xx

@Misca1234
Copy link
Collaborator

@lorem-ipsum
You got a branch conflict in this push, it needs to be solved

- remove redundant aliases in favour of TMLM041xx
- add missing symbols with higher output power
- add dual output variants TMLM042xx
- slightly update description
@hvraven
Copy link
Contributor Author

hvraven commented Jul 25, 2018

I did the rebase and I think I didn't break anything

@Misca1234
Copy link
Collaborator

I have to dissagree a little with you about your clean up

Device TMLM041xx should be renamed into TMLM 04103
with all the other TMLM 041XX as aliases.
With aliases you also specify the output voltage seperatly

Same goes for TMLM042xx, TMLM051xx , TMLM101xx and TMLM201xx

What do you think about that ?

@hvraven
Copy link
Contributor Author

hvraven commented Jul 26, 2018

I don't really mind. It seems to be not really consistent throughout the library.

@Misca1234
Copy link
Collaborator

Misca1234 commented Jul 27, 2018

I guess people have different view on this, mine is from a documentation point of view.

You can see on the schematic directly what component it is and it makes the BOM generation more "automatic". Now, the BOM list generation could be better in kicad, but in the best of all worlds you would only need the schematic after 10 years when the PCB need a make over.

(Yes, you can change the name of course, on your own but if it is already there from the beginning)

@hvraven
Copy link
Contributor Author

hvraven commented Jul 30, 2018

Makes sense. Is it fine as is? I have changed the description from xW to yV zzzmA to make it more explicit. The power rating is still in the keywords.

@Misca1234
Copy link
Collaborator

Looks good, thx

@Misca1234 Misca1234 merged commit 91cddfb into KiCad:master Jul 30, 2018
@hvraven hvraven deleted the traco_tmlm_update branch July 30, 2018 15:16
@myfreescalewebpage myfreescalewebpage added Bug Fix symbol existing in the library Enhancement Improves existing symbol in the library Addition Adds new symbols to library and removed Bug Fix symbol existing in the library labels May 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Addition Adds new symbols to library Enhancement Improves existing symbol in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants