Project

General

Profile

Bug #3271

Bug #3269: osmo-e1-xcvr has magnetics not placed the right way

Fix T1094NL footprint in library

Added by laforge 7 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
Start date:
05/15/2018
Due date:
% Done:

100%


PULSE_T1094NL.lbr PULSE_T1094NL.lbr 10.7 KB mschramm, 05/29/2018 06:11 PM

History

#1 Updated by mschramm 7 months ago

  • Status changed from New to In Progress

#2 Updated by mschramm 7 months ago

  • File PULSE_T1094NL.lbr added
  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

repaired PULSE_T1094NL footprint and symbol (solves OSM#3270)

... and OSM#3271. - The transformer symbol is made of two identical coils and hence it is not possible to mark the "2:" side (pins 9-11) separately. But the naming of this coil was changed in a way, that, when invoked, the "2" of the subpart name "TR1-1:2" shows towards the pins 9-11. Maybe we should evolve to a device symbol with two coils merged rather than two identical but separated, then an individual naming could honour the 2:1 side and can not be disturbed by e.g. subsequent mirroring. - Also repaired the name and value designators' layer association.

lib attached to ticket and will be updated in repo next week.

#3 Updated by laforge 7 months ago

Hi Martin,

thanks for your updates.

On Fri, May 18, 2018 at 08:04:27PM +0000, mschramm [REDMINE] wrote:

lib attached to ticket and will be updated in repo next week.

you now have push access to osmo-e1-xcvr.git, so it can be fixed there, too.

#4 Updated by mschramm 7 months ago

mschramm wrote:

lib attached to ticket and will be updated in repo next week.

done

#5 Updated by mschramm 6 months ago

had to touch this again:

commit 8ce574846d1f18d71b22fa811ebff4fd10694cf8
Author: Martin Schramm <mschramm@sysmocom.de>
Date:   Tue May 29 19:21:38 2018 +0200

    laforge.lbr: PULSE_T1094NL: better tPlace and tDocu layer (for OSM#3270)

#6 Updated by mschramm 6 months ago

  • File deleted (PULSE_T1094NL.lbr)

#7 Updated by mschramm 6 months ago

(updated discrete Pulse lib here too besides having this implicit in the laforge.lbr and in project's design files)

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)