From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932442AbdBHLPe (ORCPT ); Wed, 8 Feb 2017 06:15:34 -0500 Received: from mout.kundenserver.de ([212.227.126.131]:58905 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753553AbdBHLO0 (ORCPT ); Wed, 8 Feb 2017 06:14:26 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Chris Packham , Stephen Boyd , "linux-clk@vger.kernel.org" , Michael Turquette , Rob Herring , Mark Rutland , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Russell King , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support Date: Wed, 08 Feb 2017 11:52:59 +0100 Message-ID: <9294933.cXSW0amFbE@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.8.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:iapsJyxLuu8oRAkm7fcUFhudXcFOfGcxkfU60XtDbxmDKoU+TNt yXSCvLPoC/meSMT8a8RDY3u/Vcj0bMx0DDbzmyQ17KWoOE9TaAjwU4VtDlybauPEcBr2NHr FfTrpKZSLblhhyrdS6pZMLz4J9sDfknw92kGFVEo6uIYkVYn3oZ8jP5oja6m6dtTBZ8LCPe Xgq8tDToWf355cLYko+oQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:/B46ByZU608=:sxIvT6I8SZwpeTimphrXdM Poj97+0WkG+pWi0HlOg8JG+8fzzSYoyCqnY8b9Lf+c+Idv1ub7kR0oZJTW8xs5gkgd3WvBR9N OB7ISgKNU/nuW8gEXyvURK25wRXRucoTkyeOGSJ5khPa7HYroloulP7qa03WX39s2xcegDJas tuAwGVOGskxxFH8EO3ZtgocXfIY3FxThSqOLy3WoHEgakqBHeXJVqHJBJXb1jg+fYGjegoKEB Ce3zqs2MxD4dc8Xfn3UEOKAIv5zlFC5zxv5wibysRPZyTvBKkuwr4tEtZybPdp0nyZQMQ5dcH BfPmw4y3S9NnlvIS5SBRXZKjTCeRIwsvYcQc5DJ1Akss4SHUZsah+Hf3SRC4+qq0LVGLgOUDY WYwPvONxpN/mtqbb5drwrjBg05KU86DZQFQB9aDn/EtuyE5/qON4sSLvY6OBJj/MZGXtDpseY Vf7sYz5oRGSY7zGCRSS49FhLTDKeJfojUIemRrp9uFAbGZZfJaeNt1XgaW8kmzsH+qsZ1PLLa 8Fikcex3budJIkTWfgR+YMe/QVN0Mzgvbqy7QNnT1nowzDsFLcrwUBZW2gobmvkDH+EiZPbl2 mPrED7vFymWeFZPEwz4kneiOgdLa23OkvZFYEfAB6U5/YKqWkW/lvHDWNC250mFxlL/+oqunl 3r8RBgcK9vlnqpdmxD3Kb+xEcdBCDKeeA44tXb7rh/NTd6ndlPqZCJkAslPq4eWZbk+NCD8G8 z6GWccdgqyMdZuBs Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday, February 7, 2017 3:07:37 AM CET Chris Packham wrote: > > > > Actually I wonder if I can try a bit harder to keep a system booting. > > The following might work > > 1) add the compatible strings to the existing armada clock drivers. > > 2) update the dts to use the new compatible strings. > > 3) add the new driver and remove the compatible strings from the armada > > drivers. > > > > #1 would still upset checkpatch.pl because the documentation would only > > arrive in #2. > > Actually upon testing #1 is unnecessary. I lose some of the gated clocks > but nothing that prevents booting. Just to be sure: this means we can merge 2) and 3) independently and having just one of them will not cause regressions over what we have today? Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support Date: Wed, 08 Feb 2017 11:52:59 +0100 Message-ID: <9294933.cXSW0amFbE@wuerfel> References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: linux-arm-kernel@lists.infradead.org Cc: Mark Rutland , Andrew Lunn , Jason Cooper , "devicetree@vger.kernel.org" , Michael Turquette , Stephen Boyd , Russell King , Rob Herring , "linux-kernel@vger.kernel.org" , Chris Packham , Gregory Clement , "linux-clk@vger.kernel.org" , Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org On Tuesday, February 7, 2017 3:07:37 AM CET Chris Packham wrote: > > > > Actually I wonder if I can try a bit harder to keep a system booting. > > The following might work > > 1) add the compatible strings to the existing armada clock drivers. > > 2) update the dts to use the new compatible strings. > > 3) add the new driver and remove the compatible strings from the armada > > drivers. > > > > #1 would still upset checkpatch.pl because the documentation would only > > arrive in #2. > > Actually upon testing #1 is unnecessary. I lose some of the gated clocks > but nothing that prevents booting. Just to be sure: this means we can merge 2) and 3) independently and having just one of them will not cause regressions over what we have today? Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Wed, 08 Feb 2017 11:52:59 +0100 Subject: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support In-Reply-To: <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> Message-ID: <9294933.cXSW0amFbE@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday, February 7, 2017 3:07:37 AM CET Chris Packham wrote: > > > > Actually I wonder if I can try a bit harder to keep a system booting. > > The following might work > > 1) add the compatible strings to the existing armada clock drivers. > > 2) update the dts to use the new compatible strings. > > 3) add the new driver and remove the compatible strings from the armada > > drivers. > > > > #1 would still upset checkpatch.pl because the documentation would only > > arrive in #2. > > Actually upon testing #1 is unnecessary. I lose some of the gated clocks > but nothing that prevents booting. Just to be sure: this means we can merge 2) and 3) independently and having just one of them will not cause regressions over what we have today? Arnd