On Fri, Mar 21, 2025 at 08:09:37PM +0100, Théo Lebrun wrote: > Remove local variables clk_init and init. Those function pointers are > always equivalent to macb_config->clk_init and macb_config->init. > > Also remove NULL checks on macb_config as it is always valid: > - either it is its default value &default_gem_config, > - or it got overridden using match data. I probably would of made this two patches. But otherwise it looks O.K. Andrew