Hi Alexandre,
> -----Original Message-----
> From: Alexandre Bailon [mailto:[email protected]]
> Sent: Wednesday, August 8, 2018 6:01 PM
> To: A.s. Dong <[email protected]>; [email protected];
> [email protected]
> Cc: Jacky Bai <[email protected]>; Anson Huang <[email protected]>;
> [email protected]; [email protected]; dl-linux-imx
> <[email protected]>; [email protected];
> [email protected]
> 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.
0005-clk-imx-add-composite-clk-support.patch
--------------------------------------------
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#47:
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.
Regards
Dong Aisheng
