Re: [PATCH v2 1/2] ASoC: fsl_mqs: Don't check clock is NULL before calling clk API

From: Shengjiu Wang
Date: Tue Jun 23 2020 - 04:32:16 EST


On Tue, Jun 23, 2020 at 3:38 PM Markus Elfring <Markus.Elfring@xxxxxx> wrote:
>
> > In-Reply-To: <cover.1592888591.git.shengjiu.wang@xxxxxxx>
>
> I guess that it should be sufficient to specify such a field once
> for the header information.

seems it's caused by my "git format-patch" command, I will update
it, hope it is better next time.

>
>
> > Because clk_prepare_enable and clk_disable_unprepare should
> > check input clock parameter is NULL or not internally,
>
> I find this change description unclear.

clk_prepare_enable and clk_disable_unprepare check the input
clock parameter in the beginning of the function, if the parameter
is NULL, clk_prepare_enable and clk_disable_unprepare will
return immediately.

So Don't need to check input clock parameters before calling clk
API.

Do you think this commit message is better?

best regards
wang shengjiu