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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM0PR04MB4211EB200D335FCEC382BD0E800B0@AM0PR04MB4211.eurprd04.prod.outlook.com>
Date:   Mon, 27 Aug 2018 03:42:52 +0000
From:   "A.s. Dong" <aisheng.dong@....com>
To:     Alexandre Bailon <abailon@...libre.com>,
        "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
        "sboyd@...nel.org" <sboyd@...nel.org>
CC:     Jacky Bai <ping.bai@....com>, Anson Huang <anson.huang@....com>,
        "mturquette@...libre.com" <mturquette@...libre.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        dl-linux-imx <linux-imx@....com>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>
Subject: RE: [PATCH V4 0/9] clk: add imx7ulp clk support

Hi Alexandre,

> -----Original Message-----
> From: Alexandre Bailon [mailto:abailon@...libre.com]
> Sent: Wednesday, August 8, 2018 6:01 PM
> To: A.s. Dong <aisheng.dong@....com>; linux-clk@...r.kernel.org;
> sboyd@...nel.org
> Cc: Jacky Bai <ping.bai@....com>; Anson Huang <anson.huang@....com>;
> mturquette@...libre.com; linux-kernel@...r.kernel.org; dl-linux-imx
> <linux-imx@....com>; shawnguo@...nel.org;
> linux-arm-kernel@...ts.infradead.org
> 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