On Wed, Dec 11, 2024 at 11:24:30AM +0000, Bryan O'Donoghue wrote: > On 11/12/2024 00:30, Richard Acayan wrote: > > > I think you sorted the regs, interrupts but forgot the clocks ? > > Do you mean the first three clocks placed first? The order was suggested > > by Vladimir in [1], but I can undo it. > > > > [1]https://lore.kernel.org/r/40cd7a52-1c60-40dc-aee6-730b5247b216@xxxxxxxxxx > > We've had a long, long discussion on ordering subsequent. > > Have a look at the sc7280 and align your ordering with that. Ok, it seems SC7280 renames the CSID clocks to change their alphabetical sorting so I'll rename them as well. csi0 -> vfe0_csid csi1 -> vfe1_csid csi2 -> vfe_lite_csid
