Re: [PATCH 3/6] net: bcmgenet: enable automatic phy discovery

Hi,

On 2/2/20 7:15 PM, Andrew Lunn wrote:

quoted

I though I should clarify the direct question here about ACPI. ACPI does
have the ability to do what you describe, but it a more rigorous way. If you
look at the ACPI GenericSerialBus abstraction you will see how ACPI would
likely handle this situation. I've been considering making a similar comment
in that large fwnode patch set posted the other day.

I should have been a lot more specific here, but I didn't want to write 
a book.

I know ~0 about ACPI. But it does not seem unreasonable to describe an
MDIO bus in the same way as an i2c bus, or an spi bus. Each can have
devices on it, at specific addresses. Each needs common properties
like interrupts, and each needs bus specific properties like SPI
polarity. And you need pointers to these devices, so that other
subsystems can use them.

So maybe the correct way to describe this is to use ACPI
GenericSerialBus?

AFAIK, not as the specification stands today.

First its not defined for MDIO (see 6-240 in acpi 6.3) , and secondly 
because its intended to be used from AML (one of the examples IIRC is to 
read battery vendor info). That implies to me, that the ACPI standards 
body's would also have to add some additional methods which configure 
and return state about the phys. AKA some of the linux phy_() functions 
would just redirect to AML equivalents the same way there are AML 
battery functions for returning status/etc.