From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751730AbdBHUBn (ORCPT ); Wed, 8 Feb 2017 15:01:43 -0500 Received: from gate2.alliedtelesis.co.nz ([202.36.163.20]:48722 "EHLO gate2.alliedtelesis.co.nz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751025AbdBHUBB (ORCPT ); Wed, 8 Feb 2017 15:01:01 -0500 From: Chris Packham To: Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" CC: 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 Thread-Topic: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support Thread-Index: AQHSfc9C28CILxYjnUCUVa3hUDUJ8Q== Date: Wed, 8 Feb 2017 20:00:15 +0000 Message-ID: References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> <9294933.cXSW0amFbE@wuerfel> Accept-Language: en-NZ, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [2001:df5:b000:22:e17b:e610:ff6f:7e72] Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v18K1qja013674 On 08/02/17 23:53, Arnd Bergmann wrote: > 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? > Correct. And that's what I sent out as v2. http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486467.html http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486471.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Packham Subject: Re: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support Date: Wed, 8 Feb 2017 20:00:15 +0000 Message-ID: References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> <9294933.cXSW0amFbE@wuerfel> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Language: en-US 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: Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" Cc: Mark Rutland , Andrew Lunn , Jason Cooper , "devicetree@vger.kernel.org" , Michael Turquette , Stephen Boyd , Russell King , "linux-kernel@vger.kernel.org" , Rob Herring , Gregory Clement , "linux-clk@vger.kernel.org" , Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org On 08/02/17 23:53, Arnd Bergmann wrote: > 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? > Correct. And that's what I sent out as v2. http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486467.html http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486471.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Chris Packham To: Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" CC: 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, 8 Feb 2017 20:00:15 +0000 Message-ID: References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> <9294933.cXSW0amFbE@wuerfel> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 List-ID: On 08/02/17 23:53, Arnd Bergmann wrote:=0A= > On Tuesday, February 7, 2017 3:07:37 AM CET Chris Packham wrote:=0A= >>>=0A= >>> Actually I wonder if I can try a bit harder to keep a system booting.= =0A= >>> The following might work=0A= >>> 1) add the compatible strings to the existing armada clock drivers.=0A= >>> 2) update the dts to use the new compatible strings.=0A= >>> 3) add the new driver and remove the compatible strings from the armada= =0A= >>> drivers.=0A= >>>=0A= >>> #1 would still upset checkpatch.pl because the documentation would only= =0A= >>> arrive in #2.=0A= >>=0A= >> Actually upon testing #1 is unnecessary. I lose some of the gated clocks= =0A= >> but nothing that prevents booting.=0A= >=0A= > Just to be sure: this means we can merge 2) and 3) independently and=0A= > having just one of them will not cause regressions over what we have=0A= > today?=0A= >=0A= =0A= Correct. And that's what I sent out as v2.=0A= =0A= http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486467.= html=0A= http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486471.= html=0A= =0A= From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris.Packham@alliedtelesis.co.nz (Chris Packham) Date: Wed, 8 Feb 2017 20:00:15 +0000 Subject: [PATCH 4/4] clk: mvebu: Expand mv98dx3236-core-clock support References: <20170203034012.29399-1-chris.packham@alliedtelesis.co.nz> <5a974d9d7260432087619486bf5d79ed@svr-chch-ex1.atlnz.lc> <6149d4126a214ac8b0b0184bd818bea6@svr-chch-ex1.atlnz.lc> <9294933.cXSW0amFbE@wuerfel> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 08/02/17 23:53, Arnd Bergmann wrote: > 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? > Correct. And that's what I sent out as v2. http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486467.html http://lists.infradead.org/pipermail/linux-arm-kernel/2017-February/486471.html