On Mon, Mar 17, 2025 at 01:29:31PM +0100, Andrew Lunn wrote:

On Mon, Mar 17, 2025 at 10:13:19AM +0000, Sudeep Holla wrote:

quoted

The net fixed phy driver does not require the creation of a platform
device. Originally, this approach was chosen for simplicity when the
driver was first implemented.

With the introduction of the lightweight faux device interface, we now
have a more appropriate alternative. Migrate the driver to utilize the
faux bus, given that the platform device it previously created was not
a real one anyway. This will simplify the code, reducing its footprint
while maintaining functionality.

Cc: Andrew Lunn <andrew@lunn.ch>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Signed-off-by: Sudeep Holla <redacted>
---
 drivers/net/phy/fixed_phy.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

8 insertions, 8 deletions. How does this reduce its footprint?

Seems like pointless churn to me. Unless there is a real advantage to
faux bus you are not enumerating in your commit message.

It stops the abuse of using a platform device for something that is NOT
a platform device.  This file should have never used a platform device
for this in the first place, and this change is fixing that design bug.

thanks,

greg k-h