Re: [PATCH net] net: mdiobus: Debug print fwnode handle instead of raw pointer

From: Andrew Lunn
Date: Thu Sep 05 2024 - 10:54:28 EST


On Thu, Sep 05, 2024 at 04:32:47PM +0200, Alexander Dahl wrote:
> Was slightly misleading before, because printed is pointer to fwnode,
> not to phy device, as placement in message suggested. Include header
> for dev_dbg() declaration while at it.
>
> Output before:
>
> [ +0.001247] mdio_bus f802c000.ethernet-ffffffff: registered phy 2612f00a fwnode at address 3
>
> Output after:
>
> [ +0.001229] mdio_bus f802c000.ethernet-ffffffff: registered phy fwnode /ahb/apb/ethernet@f802c000/ethernet-phy@3 at address 3
>
> Signed-off-by: Alexander Dahl <ada@xxxxxxxxxxx>

The patch itself looks good. But i think this should be for net-next,
not net. '2612f00a' is not particularly useful, but i don't think it
causes problems for anybody as required by the Stable rules.

So the Subject should be [PATCH net-next]

https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>

Andrew