On 04/17/2018 11:50 AM, Simon Horman wrote: > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> How about the description (or 2)? > Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > --- > drivers/net/ethernet/renesas/ravb.h | 5 ----- > drivers/net/ethernet/renesas/ravb_main.c | 15 --------------- > 2 files changed, 20 deletions(-) > > diff --git a/drivers/net/ethernet/renesas/ravb.h b/drivers/net/ethernet/renesas/ravb.h > index 57eea4a77826..fcd04dbc7dde 100644 > --- a/drivers/net/ethernet/renesas/ravb.h > +++ b/drivers/net/ethernet/renesas/ravb.h > @@ -197,15 +197,11 @@ enum ravb_reg { > MAHR = 0x05c0, > MALR = 0x05c8, > TROCR = 0x0700, /* Undocumented? */ > - CDCR = 0x0708, /* Undocumented? */ > - LCCR = 0x0710, /* Undocumented? */ > CEFCR = 0x0740, > FRECR = 0x0748, > TSFRCR = 0x0750, > TLFRCR = 0x0758, > RFCR = 0x0760, > - CERCR = 0x0768, /* Undocumented? */ > - CEECR = 0x0770, /* Undocumented? */ > MAFCR = 0x0778, > }; > I think this is a material of patch #1... > @@ -223,7 +219,6 @@ enum CCC_BIT { > CCC_CSEL_HPB = 0x00010000, > CCC_CSEL_ETH_TX = 0x00020000, > CCC_CSEL_GMII_REF = 0x00030000, > - CCC_BOC = 0x00100000, /* Undocumented? */ > CCC_LBME = 0x01000000, ... and this -- of patch #2. Or vice versa. :-) [...] MBR, Sergei