From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Brugger Subject: Re: [PATCH v1 3/5] dt-bindings: clock: add clocks for MT2712 Date: Sun, 11 Mar 2018 23:27:41 +0100 Message-ID: References: <20180222054851.26096-1-weiyi.lu@mediatek.com> <20180222054851.26096-5-weiyi.lu@mediatek.com> <20180301224555.44pvl67l5oqibbkg@rob-hp-laptop> <1519957404.14513.5.camel@mtksdaap41> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1519957404.14513.5.camel@mtksdaap41> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Weiyi Lu , Rob Herring Cc: Stephen Boyd , Mike Turquette , James Liao , Fan Chen , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-clk@vger.kernel.org, srv_heupstream@mediatek.com List-Id: devicetree@vger.kernel.org On 03/02/2018 03:23 AM, Weiyi Lu wrote: > On Thu, 2018-03-01 at 16:45 -0600, Rob Herring wrote: >> On Thu, Feb 22, 2018 at 01:48:49PM +0800, Weiyi Lu wrote: >>> add new clocks according to ECO design change >>> >>> Signed-off-by: Weiyi Lu >>> --- >>> include/dt-bindings/clock/mt2712-clk.h | 294 +++++++++++++++++---------------- >>> 1 file changed, 151 insertions(+), 143 deletions(-) >> >> You can't just renumber your clocks. They are an ABI. Plus, for an ECO >> there can't have been that much change. >> > Hi Rob, > Got it! Is it correct if I append those new added clocks at the bottom > of each own subsystem? If the answer is yes, I'll resend a new version. Yes, that will work. Please do that and resubmit. Thanks, Matthias