On Wed, 2019-08-28 at 13:09 +0300, Felipe Balbi wrote: > Hi, > > Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> writes: > > > The split into multiple structures of the "ll" register bank is > > impractical. It makes it hard to add ll_lfps_timers_2 which is > > at offset 0x794, which is outside of the existing "lfps" structure > > and would require us to add yet another one. > > > > Instead, move all the "ll" registers into a single usb338x_ll_regs > > structure, and add ll_lfps_timers_2 while at it. It will be used > > in a subsequent patch. > > > > Signed-off-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> > > I tried applying your patches but it resulted in build break. Can you > collect all the dependencies and send a single series? I'm applying on > top of my testing/next branch. You mean the 2 net2280 patches ? Or something else ? What break did you get ? It's just one series of 2 patches.... I'll try rebasing them against your branch tomorrow. Cheers, Ben.