On Mon, Sep 20, 2021 at 11:52:11AM +0200, Horatiu Vultur wrote: > The lan966x switch SoC has a number of components that can be reset > indiviually, but at least the switch core needs to be in a well defined > state at power on, when any of the lan966x drivers starts to access the > switch core, this reset driver is available. > > The reset driver is loaded early via the postcore_initcall interface, and > will then be available for the other lan966x drivers (SGPIO, SwitchDev etc) > that are loaded next, and the first of them to be loaded can perform the > one-time switch core reset that is needed. A lot of this looks very similar to reset-microchip-sparx5.c. PROTECT_REG is 0x88 rather than 0x84, but actually using the value is the same. SOFT_RESET_REG is identical. So rather than adding a new driver, maybe you can generalize reset-microchip-sparx5.c, and add a second compatible string? Andrew