From mboxrd@z Thu Jan 1 00:00:00 1970 From: Magnus Damm Date: Mon, 22 Jun 2015 09:41:51 +0000 Subject: Re: [PATCH 01/08] devicetree: bindings: Remove sh7372 CMT binding Message-Id: List-Id: References: <20150621080903.10187.20998.sendpatchset@little-apple> <20150621080915.10187.97464.sendpatchset@little-apple> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Laurent Pinchart , Geert Uytterhoeven , Linux-sh list , Daniel Lezcano , Simon Horman , Thomas Gleixner Hi Geert, On Mon, Jun 22, 2015 at 6:38 PM, Geert Uytterhoeven wrote: > Hi Magnus, > > On Sun, Jun 21, 2015 at 10:09 AM, Magnus Damm wrote: >> From: Magnus Damm >> >> Remove the sh7372 CMT compat string to reduce maintenance burden. >> >> It should be fine to break DT compatibility because: >> 1) The sh7372 SoC support has been removed from upstream >> 2) The sh7372 CMT DT binding was never part of upstream DTS >> 3) The CMT driver never matches on the sh7372 binding >> >> Signed-off-by: Magnus Damm >> --- >> >> Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 -- >> 1 file changed, 2 deletions(-) >> >> --- 0001/Documentation/devicetree/bindings/timer/renesas,cmt.txt >> +++ work/Documentation/devicetree/bindings/timer/renesas,cmt.txt 2015-06-20 13:47:59.922366518 +0900 >> @@ -32,8 +32,6 @@ Required Properties: >> (CMT[234] on sh7372, sh73a0 and r8a7740) >> This is a fallback for the above renesas,cmt-32-fast-* entries. >> >> - - "renesas,cmt-48-sh7372" for the sh7372 48-bit CMT >> - (CMT1) > > You forgot to remove the reference to sh7372 in the "renesas,cmt-48" > fallback section, and the "renesas,cmt-32-sh7372" and > "renesas,cmt-32-fast-sh7372" bindings. Thanks, yeah, I noticed that after sending out the series. I couldn't find any DT users of the "fast" group though, so perhaps I need to extend my pruning a bit. First it would be good to see if we could reach an agreement about how to handle DT compat strings for these kind of devices though. (devices where instances within one particular SoC are quite different). Cheers, / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933206AbbFVJmP (ORCPT ); Mon, 22 Jun 2015 05:42:15 -0400 Received: from mail-wg0-f54.google.com ([74.125.82.54]:32925 "EHLO mail-wg0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932965AbbFVJlx (ORCPT ); Mon, 22 Jun 2015 05:41:53 -0400 MIME-Version: 1.0 In-Reply-To: References: <20150621080903.10187.20998.sendpatchset@little-apple> <20150621080915.10187.97464.sendpatchset@little-apple> Date: Mon, 22 Jun 2015 18:41:51 +0900 Message-ID: Subject: Re: [PATCH 01/08] devicetree: bindings: Remove sh7372 CMT binding From: Magnus Damm To: Geert Uytterhoeven Cc: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Laurent Pinchart , Geert Uytterhoeven , Linux-sh list , Daniel Lezcano , Simon Horman , Thomas Gleixner Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Geert, On Mon, Jun 22, 2015 at 6:38 PM, Geert Uytterhoeven wrote: > Hi Magnus, > > On Sun, Jun 21, 2015 at 10:09 AM, Magnus Damm wrote: >> From: Magnus Damm >> >> Remove the sh7372 CMT compat string to reduce maintenance burden. >> >> It should be fine to break DT compatibility because: >> 1) The sh7372 SoC support has been removed from upstream >> 2) The sh7372 CMT DT binding was never part of upstream DTS >> 3) The CMT driver never matches on the sh7372 binding >> >> Signed-off-by: Magnus Damm >> --- >> >> Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 -- >> 1 file changed, 2 deletions(-) >> >> --- 0001/Documentation/devicetree/bindings/timer/renesas,cmt.txt >> +++ work/Documentation/devicetree/bindings/timer/renesas,cmt.txt 2015-06-20 13:47:59.922366518 +0900 >> @@ -32,8 +32,6 @@ Required Properties: >> (CMT[234] on sh7372, sh73a0 and r8a7740) >> This is a fallback for the above renesas,cmt-32-fast-* entries. >> >> - - "renesas,cmt-48-sh7372" for the sh7372 48-bit CMT >> - (CMT1) > > You forgot to remove the reference to sh7372 in the "renesas,cmt-48" > fallback section, and the "renesas,cmt-32-sh7372" and > "renesas,cmt-32-fast-sh7372" bindings. Thanks, yeah, I noticed that after sending out the series. I couldn't find any DT users of the "fast" group though, so perhaps I need to extend my pruning a bit. First it would be good to see if we could reach an agreement about how to handle DT compat strings for these kind of devices though. (devices where instances within one particular SoC are quite different). Cheers, / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/ From mboxrd@z Thu Jan 1 00:00:00 1970 From: Magnus Damm Subject: Re: [PATCH 01/08] devicetree: bindings: Remove sh7372 CMT binding Date: Mon, 22 Jun 2015 18:41:51 +0900 Message-ID: References: <20150621080903.10187.20998.sendpatchset@little-apple> <20150621080915.10187.97464.sendpatchset@little-apple> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-sh-owner@vger.kernel.org To: Geert Uytterhoeven Cc: "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Laurent Pinchart , Geert Uytterhoeven , Linux-sh list , Daniel Lezcano , Simon Horman , Thomas Gleixner List-Id: devicetree@vger.kernel.org Hi Geert, On Mon, Jun 22, 2015 at 6:38 PM, Geert Uytterhoeven wrote: > Hi Magnus, > > On Sun, Jun 21, 2015 at 10:09 AM, Magnus Damm wrote: >> From: Magnus Damm >> >> Remove the sh7372 CMT compat string to reduce maintenance burden. >> >> It should be fine to break DT compatibility because: >> 1) The sh7372 SoC support has been removed from upstream >> 2) The sh7372 CMT DT binding was never part of upstream DTS >> 3) The CMT driver never matches on the sh7372 binding >> >> Signed-off-by: Magnus Damm >> --- >> >> Documentation/devicetree/bindings/timer/renesas,cmt.txt | 2 -- >> 1 file changed, 2 deletions(-) >> >> --- 0001/Documentation/devicetree/bindings/timer/renesas,cmt.txt >> +++ work/Documentation/devicetree/bindings/timer/renesas,cmt.txt 2015-06-20 13:47:59.922366518 +0900 >> @@ -32,8 +32,6 @@ Required Properties: >> (CMT[234] on sh7372, sh73a0 and r8a7740) >> This is a fallback for the above renesas,cmt-32-fast-* entries. >> >> - - "renesas,cmt-48-sh7372" for the sh7372 48-bit CMT >> - (CMT1) > > You forgot to remove the reference to sh7372 in the "renesas,cmt-48" > fallback section, and the "renesas,cmt-32-sh7372" and > "renesas,cmt-32-fast-sh7372" bindings. Thanks, yeah, I noticed that after sending out the series. I couldn't find any DT users of the "fast" group though, so perhaps I need to extend my pruning a bit. First it would be good to see if we could reach an agreement about how to handle DT compat strings for these kind of devices though. (devices where instances within one particular SoC are quite different). Cheers, / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in