Hi Simon-san, > From: Simon Horman, Sent: Monday, September 2, 2019 5:06 PM > > From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> > > This patch removes the use of the undocumented registers > CDCR, LCCR, CERCR, CEECR and the undocumented BOC bit of the CCC register. > > Current documentation for EtherAVB (ravb) describes the offset of > what the driver uses as the BOC bit as reserved and that only a value of > 0 should be written. Furthermore, the offsets used for the undocumented > registers are also considered reserved nd should not be written to. > > After some internal investigation with Renesas it remains unclear > why this driver accesses these fields but regardless of what the historical > reasons are the current code is considered incorrect. > > Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@xxxxxxxxxxx> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> Thank you for the patch! Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> Best regards, Yoshihiro Shimoda
