From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933694AbcBQDQx (ORCPT ); Tue, 16 Feb 2016 22:16:53 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:58136 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755640AbcBQDQv (ORCPT ); Tue, 16 Feb 2016 22:16:51 -0500 Subject: Re: [RESEND PATCH v8 1/6] clk: hisilicon: add CRG driver for hi3519 soc To: Michael Turquette , , , , , , , , , , , , , , , References: <1454639472-17373-1-git-send-email-xuejiancheng@huawei.com> <1454639472-17373-2-git-send-email-xuejiancheng@huawei.com> <20160217004648.2278.64006@quark.deferred.io> CC: , , , , , , , , , , , , From: xuejiancheng Message-ID: <56C3E423.5080801@huawei.com> Date: Wed, 17 Feb 2016 11:08:19 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <20160217004648.2278.64006@quark.deferred.io> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.217.211] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090202.56C3E433.00C2,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: ae4b36957b1c798b06d3ff0ce11b32ee Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mike, Thank you very much for your comments. On 2016/2/17 8:46, Michael Turquette wrote: > Hello Jiancheng Xue, > > Quoting Jiancheng Xue (2016-02-04 18:31:07) >> diff --git a/drivers/clk/hisilicon/Makefile b/drivers/clk/hisilicon/Makefile >> index 74dba31..3f57b09 100644 >> --- a/drivers/clk/hisilicon/Makefile >> +++ b/drivers/clk/hisilicon/Makefile >> @@ -4,8 +4,10 @@ >> >> obj-y += clk.o clkgate-separated.o clkdivider-hi6220.o >> >> +obj-$(CONFIG_RESET_CONTROLLER) += reset.o > > Do you really want to build reset.o for all hisi SoCs? > This reset controller driver will be just used in some of hisilicon SOCs. I'll add a specific config item for it like CONFIG_RESET_HISI. The config item will be selected by default in SOCs needing this driver. I'll also fix other issues in next version. Thank you! Regards, Jiancheng.