On Mon, Jun 5, 2023 at 8:37 PM Xavier Drudis Ferran <[email protected]> wrote: > > This clock doesn't seem needed but appears in a phandle list used by > ehci-generic.c to bulk enable it. The phandle list comes from linux, > where it is needed for suspend/resume to work [1]. > > My tests give the same results with or without this patch, but Marek > Vasut found it weird to declare an empty clk_ops [2]. > > So I adapted the code from linux 6.1-rc8 so that it hopefully works > if it ever has some user. For now, without real use, it seems to > at least not give any errors when called. > > Link: [1] https://lkml.kernel.org/lkml/1731551.Q6cHK6n5ZM@phil/T/ > [2] > https://patchwork.ozlabs.org/project/uboot/patch/Y5IWpjYLB4aXMy9o@localhost/ > > Cc: Simon Glass <[email protected]> > Cc: Philipp Tomsich <[email protected]> > Cc: Kever Yang <[email protected]> > Cc: Lukasz Majewski <[email protected]> > Cc: Sean Anderson <[email protected]> > Cc: Marek Vasut <[email protected]> > Cc: Christoph Fritz <[email protected]> > Cc: Jagan Teki <[email protected]> > > Signed-off-by: Xavier Drudis Ferran <[email protected]> > ---
Reviewed-by: Jagan Teki <[email protected]> Tested-by: Jagan Teki <[email protected]> # rk3399, rk3328, rv1126
