fix symbol to match footprint pr

[matthuszagh]

Modifies the SE5004L rf amp to match the proposed footprint PR.

---

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.

[myfreescalewebpage]

Hi @matthuszagh thanks for this fixes,

• The datasheet link seems broken, can you also update it ? "https://www.skyworksinc.com/-/media/SkyWorks/Documents/Products/601-700/SE5004L_202393B.pdf"
• I do not agree for changing pin 1, it is not connected in the datasheet... Or maybe you have another information ?
• Same comment for the EP, where have you found it is to be connected to GND ?

Joel

[matthuszagh]

Hey @myfreescalewebpage thanks for the review. Pin 1 etc and EP are connected. Check the land pattern in the first image of the footprint PR I referenced (taken from the datasheet). Let me know if any further concerns, I'll update the datasheet link.

[myfreescalewebpage]

I have seen you connected them on the footprint. But I refer to the datasheet. Why connecting these pins together ?

The question is for both the symbol and the footprint :)

[matthuszagh]

I'm referring to the datasheet too, which is what I was discussing in my last comment. I'll post the image from the datasheet again (page 7).

That first image on the left is the land pattern. As you can see, the EP and all ground pins are connected together. In order for the symbol to match this footprint, EP and ground must be connected.

Does this make sense, or am I missing your point?

[myfreescalewebpage]

No worry, I haven't checked the footprint in the datasheet, only the description of pins.
The symbol is not broken, so no need to wait V6, and I don't anything which need the footprint to be merged first.
Can you update the datasheet link first ?
Thanks,
Joel

[matthuszagh]

Datasheet updated.

[myfreescalewebpage]

Thanks, no more comment, merging.