Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH v2 1/2] clk: intel: Add CGU clock driver for a new SoC
To:     Nathan Chancellor <natechancellor@gmail.com>
Cc:     mturquette@baylibre.com, sboyd@kernel.org, robh+dt@kernel.org,
        mark.rutland@arm.com, linux-clk@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        andriy.shevchenko@intel.com, yixin.zhu@linux.intel.com,
        qi-ming.wu@intel.com, rtanwar <rahul.tanwar@intel.com>,
        clang-built-linux@googlegroups.com
References: <cover.1576811332.git.rahul.tanwar@linux.intel.com>
 <ee8a8a0f0c882e22361895b2663870c8037c422f.1576811332.git.rahul.tanwar@linux.intel.com>
 <20191224052947.GA54145@ubuntu-m2-xlarge-x86>
From:   "Tanwar, Rahul" <rahul.tanwar@linux.intel.com>
Message-ID: <c61235a7-969f-f534-e25f-e3990b9c8d11@linux.intel.com>
Date:   Tue, 24 Dec 2019 13:49:05 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101
 Thunderbird/60.8.0
MIME-Version: 1.0
In-Reply-To: <20191224052947.GA54145@ubuntu-m2-xlarge-x86>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


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@intel.com>
>>
>> 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@linux.intel.com>
> 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