On Fri, May 24, 2024 at 09:49:08AM +0800, Shengjiu Wang wrote: > On Fri, May 24, 2024 at 4:36 AM Frank Li <Frank.li@xxxxxxx> wrote: > > > +static int clk_imx8mp_audiomix_reset_controller_register(struct device *dev, > > > + struct clk_imx8mp_audiomix_priv *priv) > > > +{ > > > + struct auxiliary_device __free(kfree) * adev = NULL; > > > > nit: *adev = NULL; > > Actually, the checkpatch.pl told me need to have space after '*'... Someone should fix checkpatch so that it knows that __free() is an attribute, not a function call. (The cleanup.h stuff is quite new). regards, dan carpenter