diff mbox

[01/08] devicetree: bindings: Remove sh7372 CMT binding

Message ID 20150621080915.10187.97464.sendpatchset@little-apple (mailing list archive)
State Changes Requested
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Magnus Damm June 21, 2015, 8:09 a.m. UTC
From: Magnus Damm <damm+renesas@opensource.se>

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 <damm+renesas@opensource.se>
---

 Documentation/devicetree/bindings/timer/renesas,cmt.txt |    2 --
 1 file changed, 2 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in

Comments

Geert Uytterhoeven June 22, 2015, 9:38 a.m. UTC | #1
Hi Magnus,

On Sun, Jun 21, 2015 at 10:09 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> 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 <damm+renesas@opensource.se>
> ---
>
>  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.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
Magnus Damm June 22, 2015, 9:41 a.m. UTC | #2
Hi Geert,

On Mon, Jun 22, 2015 at 6:38 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Magnus,
>
> On Sun, Jun 21, 2015 at 10:09 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> From: Magnus Damm <damm+renesas@opensource.se>
>>
>> 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 <damm+renesas@opensource.se>
>> ---
>>
>>  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
diff mbox

Patch

--- 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)
     - "renesas,cmt-48-sh73a0" for the sh73A0 48-bit CMT
 		(CMT1)
     - "renesas,cmt-48-r8a7740" for the r8a7740 48-bit CMT