Re: [PATCH v2 1/2] clk: intel: Add CGU clock driver for a new SoC

From: Tanwar, Rahul
Date: Tue Dec 24 2019 - 00:49:23 EST




On 24/12/2019 1:29 PM, Nathan Chancellor wrote:
> On Fri, Dec 20, 2019 at 11:31:07AM +0800, Rahul Tanwar wrote:
>> From: rtanwar <rahul.tanwar@xxxxxxxxx>
>>
>> Clock Generation Unit(CGU) is a new clock controller IP of a forthcoming
>> Intel network processor SoC. It provides programming interfaces to control
>> & configure all CPU & peripheral clocks. Add common clock framework based
>> clock controller driver for CGU.
>>
>> Signed-off-by: Rahul Tanwar <rahul.tanwar@xxxxxxxxxxxxxxx>
> Hi Rahul,
>
> The 0day bot reported this warning with clang with your patch, mind
> taking a look at it since it seems like you will need to do a v2 based
> on other comments?
>
> It seems like the check either needs to be something different or the
> check should just be removed.
>
> Cheers,
> Nathan

Hi Nathan,

Yes sure, i will fix it in v3. I anyways need to post v3 to address review
comments received from few reviewers. Thanks.

Regards,
Rahul