RE: [PATCH V4 0/9] clk: add imx7ulp clk support

From: A.s. Dong
Date: Sun Aug 26 2018 - 23:42:58 EST

Hi Alexandre,

> -----Original Message-----
> From: Alexandre Bailon [mailto:abailon@xxxxxxxxxxxx]
> Sent: Wednesday, August 8, 2018 6:01 PM
> To: A.s. Dong <aisheng.dong@xxxxxxx>; linux-clk@xxxxxxxxxxxxxxx;
> sboyd@xxxxxxxxxx
> Cc: Jacky Bai <ping.bai@xxxxxxx>; Anson Huang <anson.huang@xxxxxxx>;
> mturquette@xxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; dl-linux-imx
> <linux-imx@xxxxxxx>; shawnguo@xxxxxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH V4 0/9] clk: add imx7ulp clk support
> Hi Dong Aisheng,
> On 07/26/2018 03:50 AM, A.s. Dong wrote:
> > Hi Stephen,
> >
> > Do you have a chance to look at it?
> > This patch series has been pending for quite a long time without much
> comments.
> I'm not a kernel maintainer but I would not review a series that has checkpatch
> errors.
> Please run checkpatch, fix the errors and the warnings, and then resubmit a
> series.

Yes, I did it. There're some warnings. But unlike errors, normally warnings are judgable.
For this patch set, all known warnings seems to me not need the fix.
E.g. line over 80 or missing maintainer for new files....

BTW there's one special warning I don't see how to fix, seems reported wrongly.

WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
new file mode 100644

WARNING: function definition argument 'const char * const' should also have an identifier name
#145: FILE: drivers/clk/imx/clk.h:74:
+struct clk_hw *imx_clk_composite(const char *name,

total: 0 errors, 2 warnings, 104 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.

If you have better suggestions please let me know.

Dong Aisheng