Re: [PATCH net-next 02/10] r8169:add disable/enable RTL8168G pll function

From: David Miller
Date: Tue Sep 30 2014 - 16:43:38 EST


From: Chun-Hao Lin <hau@xxxxxxxxxxx>
Date: Tue, 30 Sep 2014 23:00:22 +0800

> @@ -4399,8 +4400,10 @@ static void r8168_pll_power_up(struct rtl8169_private *tp)
> break;
> case RTL_GIGA_MAC_VER_40:
> case RTL_GIGA_MAC_VER_41:
> + RTL_W8(PMCH, RTL_R8(PMCH) | 0xc0);
> rtl_w1w0_eri(tp, 0x1a8, ERIAR_MASK_1111, 0xfc000000,
> 0x00000000, ERIAR_EXGMAC);
> +
> break;

There is no reason to add an empty line before the break; statement.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/