[PATCH v5 2/5] clk: imx: clk-audiomix: Add reset controller

Shengjiu Wang shengjiu.wang at gmail.com
Sun May 26 20:43:28 PDT 2024


On Sat, May 25, 2024 at 3:27 PM Markus Elfring <Markus.Elfring at web.de> wrote:
>
>> > The reset controller is supported by the auxiliary device
> > framework.
>
> Would you like to add an imperative wording for an improved change description?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9#n94

ok, will update it.

>
>
>> > +++ b/drivers/clk/imx/clk-imx8mp-audiomix.c
>> > +static int clk_imx8mp_audiomix_reset_controller_register(struct device *dev,
> > +                                                      struct clk_imx8mp_audiomix_priv *priv)
> > +{
> > +     struct auxiliary_device __free(kfree) * adev = NULL;
> > +     int ret;
> > +
> > +     adev = kzalloc(sizeof(*adev), GFP_KERNEL);
>>
> May the following source code variant be applied here?
>
>         int ret;
>         struct auxiliary_device __free(kfree) *adev = kzalloc(sizeof(*adev), GFP_KERNEL);

ok, will update it.

best regards
Shengjiu Wang
>
>
> Regards,
> Markus



More information about the linux-arm-kernel mailing list