[PATCH v3] net: mdio: mux-meson-gxl: set reversed bit when using internal phy
Simon Horman
horms at kernel.org
Mon Apr 28 11:12:42 PDT 2025
On Fri, Apr 25, 2025 at 03:20:09PM -0400, Da Xue wrote:
> This bit is necessary to receive packets from the internal PHY.
> Without this bit set, no activity occurs on the interface.
>
> Normally u-boot sets this bit, but if u-boot is compiled without
> net support, the interface will be up but without any activity.
>
> The vendor SDK sets this bit along with the PHY_ID bits.
I'd like to clarify that:
Without this patch the writel the patch is modifying will clear the PHY_ID bit.
But despite that the system works if at some point (uboot) set the PHY_ID bit?
>
> Fixes: 9a24e1ff4326 ("net: mdio: add amlogic gxl mdio mux support");
I don't think you need to resend because of this,
but the correct syntax is as follows. (No trailing ';'.)
Fixes: 9a24e1ff4326 ("net: mdio: add amlogic gxl mdio mux support")
> Signed-off-by: Da Xue <da at libre.computer>
> ---
> Changes since v2:
> * Rename REG2_RESERVED_28 to REG2_REVERSED
>
> Link to v2:
> https://patchwork.kernel.org/project/linux-amlogic/patch/20250331074420.3443748-1-christianshewitt@gmail.com/
...
More information about the linux-amlogic
mailing list