Chun-Hao Lin <h...@realtek.com> : > Change function name from "rtl_w1w0_phy" to "rtl_w0w1_phy". > And its behavior from "or first then mask" to "mask first then or".
- I don't mind if its changed for a reason but the commit message doesn't tell why. Is it related to the overlap of the masks below ? - rtl_w1w0_phy was supposed to abbreviate "write ones then write zeros" where parameters are taken in this order. You won't hit the 80 cols limit if you replace it with something like "rtl_set_clear_phy" (if you use something longer it won't fit for rtl_w1w0_eri). [...] > diff --git a/drivers/net/ethernet/realtek/r8169.c > b/drivers/net/ethernet/realtek/r8169.c > index 1bf6696..e68fe5e 100644 > --- a/drivers/net/ethernet/realtek/r8169.c > +++ b/drivers/net/ethernet/realtek/r8169.c [...] > @@ -3575,33 +3575,33 @@ static void rtl8168h_1_hw_phy_config(struct > rtl8169_private *tp) > /* CHN EST parameters adjust - giga master */ > rtl_writephy(tp, 0x1f, 0x0a43); > rtl_writephy(tp, 0x13, 0x809b); > - rtl_w1w0_phy(tp, 0x14, 0x8000, 0xf800); > + rtl_w0w1_phy(tp, 0x14, 0x8000, 0xf800); > rtl_writephy(tp, 0x13, 0x80a2); > - rtl_w1w0_phy(tp, 0x14, 0x8000, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x8000, 0xff00); > rtl_writephy(tp, 0x13, 0x80a4); > - rtl_w1w0_phy(tp, 0x14, 0x8500, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x8500, 0xff00); > rtl_writephy(tp, 0x13, 0x809c); > - rtl_w1w0_phy(tp, 0x14, 0xbd00, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0xbd00, 0xff00); > rtl_writephy(tp, 0x1f, 0x0000); > > /* CHN EST parameters adjust - giga slave */ > rtl_writephy(tp, 0x1f, 0x0a43); > rtl_writephy(tp, 0x13, 0x80ad); > - rtl_w1w0_phy(tp, 0x14, 0x7000, 0xf800); > + rtl_w0w1_phy(tp, 0x14, 0x7000, 0xf800); > rtl_writephy(tp, 0x13, 0x80b4); > - rtl_w1w0_phy(tp, 0x14, 0x5000, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x5000, 0xff00); > rtl_writephy(tp, 0x13, 0x80ac); > - rtl_w1w0_phy(tp, 0x14, 0x4000, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x4000, 0xff00); > rtl_writephy(tp, 0x1f, 0x0000); > > /* CHN EST parameters adjust - fnet */ > rtl_writephy(tp, 0x1f, 0x0a43); > rtl_writephy(tp, 0x13, 0x808e); > - rtl_w1w0_phy(tp, 0x14, 0x1200, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x1200, 0xff00); > rtl_writephy(tp, 0x13, 0x8090); > - rtl_w1w0_phy(tp, 0x14, 0xe500, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0xe500, 0xff00); > rtl_writephy(tp, 0x13, 0x8092); > - rtl_w1w0_phy(tp, 0x14, 0x9f00, 0xff00); > + rtl_w0w1_phy(tp, 0x14, 0x9f00, 0xff00); > rtl_writephy(tp, 0x1f, 0x0000); > > /* enable R-tune & PGA-retune function */ > @@ -3620,57 +3620,57 @@ static void rtl8168h_1_hw_phy_config(struct > rtl8169_private *tp) > dout_tapbin &= 0xf000; > rtl_writephy(tp, 0x1f, 0x0a43); > rtl_writephy(tp, 0x13, 0x827a); > - rtl_w1w0_phy(tp, 0x14, dout_tapbin, 0xf000); > + rtl_w0w1_phy(tp, 0x14, dout_tapbin, 0xf000); > rtl_writephy(tp, 0x13, 0x827b); > - rtl_w1w0_phy(tp, 0x14, dout_tapbin, 0xf000); > + rtl_w0w1_phy(tp, 0x14, dout_tapbin, 0xf000); > rtl_writephy(tp, 0x13, 0x827c); > - rtl_w1w0_phy(tp, 0x14, dout_tapbin, 0xf000); > + rtl_w0w1_phy(tp, 0x14, dout_tapbin, 0xf000); > rtl_writephy(tp, 0x13, 0x827d); > - rtl_w1w0_phy(tp, 0x14, dout_tapbin, 0xf000); > + rtl_w0w1_phy(tp, 0x14, dout_tapbin, 0xf000); [...] -- Ueimor -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
