On 12/22/23 2:35 PM, Claudiu wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > CSR.OPS bits specify the current operating mode and (according to > documentation) they are updated by HW when the operating mode change > request is processed. To comply with this check CSR.OPS before proceeding. > > Commit introduces ravb_set_opmode() that does all the necessities for > setting the operating mode (set DMA.CCC and wait for CSR.OPS) and call it What's DMA.CCC? Maybe CCC.OPC? > where needed. This should comply with all the HW manuals requirements as > different manual variants specify that different modes need to be checked > in CSR.OPS when setting DMA.CCC. > > Fixes: c156633f1353 ("Renesas Ethernet AVB driver proper") > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> [...] MBR, Sergey