Hi Paolo

On Thu, Aug 01, 2024 at 01:32:56PM +0200, Paolo Abeni wrote:

On 7/29/24 14:23, Yanteng Si wrote:

quoted

The Loongson GMAC driver currently supports the network controllers
installed on the LS2K1000 SoC and LS7A1000 chipset, for which the GMAC
devices are required to be defined in the platform device tree source.
But Loongson machines may have UEFI (implies ACPI) or PMON/UBOOT
(implies FDT) as the system bootloaders. In order to have both system
configurations support let's extend the driver functionality with the
case of having the Loongson GMAC probed on the PCI bus with no device
tree node defined for it. That requires to make the device DT-node
optional, to rely on the IRQ line detected by the PCI core and to
have the MDIO bus ID calculated using the PCIe Domain+BDF numbers.

In order to have the device probe() and remove() methods less
complicated let's move the DT- and ACPI-specific code to the
respective sub-functions.

Signed-off-by: Feiyang Chen <redacted>
Signed-off-by: Yinggang Gu <redacted>
Acked-by: Huacai Chen <redacted>
Signed-off-by: Yanteng Si <redacted>

@Serge: I think this addresses your comment on the previous iteration, but
it would be great if you could have a look!

Thanks for reaching me out. I'll have a look at the series later
today.

-Serge(y)

Thanks,

Paolo