Hello! On 09/02/2019 11:06 AM, Simon Horman wrote: > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> > > The field name is FBP rather than FPB. > > This field is unused and could equally be removed from the driver entirely. > But there seems no harm in leaving as documentation of the presence of the > field. > > Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> Acked-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> > --- > v0 - Kazuya Mizuguchi > > v1 - Simon Horman > * Extracted from larger patch I'd just claim the authorship in this case (and mentioned that it's based on Mizuguchi-san's large patch right in the change log). > * Wrote changelog > > diff --git a/drivers/net/ethernet/renesas/ravb.h b/drivers/net/ethernet/renesas/ravb.h > index ac9195add811..bdb051f04b0c 100644 > --- a/drivers/net/ethernet/renesas/ravb.h > +++ b/drivers/net/ethernet/renesas/ravb.h > @@ -317,7 +312,7 @@ enum UFCD_BIT { > > /* SFO */ > enum SFO_BIT { > - SFO_FPB = 0x0000003F, > + SFO_FBP = 0x0000003F, > }; > > /* RTC */ > --- This is where the actual patch starts, right? > drivers/net/ethernet/renesas/ravb.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/renesas/ravb.h b/drivers/net/ethernet/renesas/ravb.h > index ac9195add811..2596a95a4300 100644 > --- a/drivers/net/ethernet/renesas/ravb.h > +++ b/drivers/net/ethernet/renesas/ravb.h > @@ -317,7 +317,7 @@ enum UFCD_BIT { > > /* SFO */ > enum SFO_BIT { > - SFO_FPB = 0x0000003F, > + SFO_FBP = 0x0000003F, > }; > > /* RTC */ MBR, Sergei