> I based my comments on the datasheet. For the LED_GPIO_CFG register, the > datasheet says: > > This register configures the external GPIO[2:0] pins. > > QFN package description also indicates GPIOs 0, 1 & 2. > As an example for the LAN9514, pin 22 of the QFN indicates: > > nSPD_LED/GPIO2 > > In LED_GPIO_CFG register, GPCTL2 description indicates: > > The value of this field determines the function of the external GPIO2 > > pin as follows > > Do you confirm it's actually GPIO 10, 9 and 8? > I think I may have misunderstood something. Sorry forgetting that you are referring RPi which uses LAN9514. Because these LEDs' GPIO can vary per chip (LAN9500, 9514..), it would be better not putting GPIO number. LAN9500 are GPIO 10/9/8 as described.
> While we are here, could you indicate the meaning of the bit 2 of > HW_CFG register (it's named HW_CFG_PSEL_)? It's the only bit I didn't > succeed to comment because I didn't find it in the datasheet. > I will then add it to the patch! It indicates internal & external phy, PSEL means PHY Select. You can find at LAN9500 doc in http://ww1.microchip.com/downloads/en/DeviceDoc/00001875C.pdf. > I'm also wondering what the meaning of STRAP_STATUS is. I could also > comment it if you or Steve can provide the information. It is marked as reserved in above LAN9500 manual. You may guess from configuration straps in the manual and define names. Woojung
