All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kuldeep Singh <singh.kuldeep87k@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Daniel Lezcano <daniel.lezcano@linaro.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Rob Herring <robh+dt@kernel.org>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH 2/3] dt-bindings: timer: Document arm,cortex-a7-timer for arch timer
Date: Wed, 16 Mar 2022 23:13:06 +0530	[thread overview]
Message-ID: <20220316174306.GC21737@9a2d8922b8f1> (raw)
In-Reply-To: <662981fe-7cf4-b0e6-2d43-3f33a53a6abd@canonical.com>

On Wed, Mar 16, 2022 at 05:28:49PM +0100, Krzysztof Kozlowski wrote:
> On 16/03/2022 10:54, Kuldeep Singh wrote:
> > Renesas RZ/N1D platform uses compatible "arm,cortex-a7-timer" in
> > conjugation with "arm,armv7-timer". Since, initial entry is not
> > documented, it start raising dtbs_check warnings.
> > 
> > ['arm,cortex-a7-timer', 'arm,armv7-timer'] is too long
> > 'arm,cortex-a7-timer' is not one of ['arm,armv7-timer', 'arm,armv8-timer']
> > 'arm,cortex-a7-timer' is not one of ['arm,cortex-a15-timer']
> > 
> > In general, removing an existing entry is mostly devastating considering
> > backward compatibility. Therefore, document it.
> 
> How removing undocumented and unused compatible is devastating for
> backwards compatibility? I don't see it.
> 
> DTS is rather wrong, not bindings. Otherwise please explain it more, why
> DTS is correct.

Please see my comment on 3/3 patch.

- Kuldeep

WARNING: multiple messages have this Message-ID (diff)
From: Kuldeep Singh <singh.kuldeep87k@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Daniel Lezcano <daniel.lezcano@linaro.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Rob Herring <robh+dt@kernel.org>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH 2/3] dt-bindings: timer: Document arm, cortex-a7-timer for arch timer
Date: Wed, 16 Mar 2022 23:13:06 +0530	[thread overview]
Message-ID: <20220316174306.GC21737@9a2d8922b8f1> (raw)
In-Reply-To: <662981fe-7cf4-b0e6-2d43-3f33a53a6abd@canonical.com>

On Wed, Mar 16, 2022 at 05:28:49PM +0100, Krzysztof Kozlowski wrote:
> On 16/03/2022 10:54, Kuldeep Singh wrote:
> > Renesas RZ/N1D platform uses compatible "arm,cortex-a7-timer" in
> > conjugation with "arm,armv7-timer". Since, initial entry is not
> > documented, it start raising dtbs_check warnings.
> > 
> > ['arm,cortex-a7-timer', 'arm,armv7-timer'] is too long
> > 'arm,cortex-a7-timer' is not one of ['arm,armv7-timer', 'arm,armv8-timer']
> > 'arm,cortex-a7-timer' is not one of ['arm,cortex-a15-timer']
> > 
> > In general, removing an existing entry is mostly devastating considering
> > backward compatibility. Therefore, document it.
> 
> How removing undocumented and unused compatible is devastating for
> backwards compatibility? I don't see it.
> 
> DTS is rather wrong, not bindings. Otherwise please explain it more, why
> DTS is correct.

Please see my comment on 3/3 patch.

- Kuldeep

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-03-16 17:43 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-16  9:54 [PATCH 0/3] Fix dtbs warnings for arch timer Kuldeep Singh
2022-03-16  9:54 ` Kuldeep Singh
2022-03-16  9:54 ` [PATCH 1/3] dt-bindings: timer: Rearrange compatible entries of " Kuldeep Singh
2022-03-16  9:54   ` Kuldeep Singh
2022-03-16 16:29   ` Krzysztof Kozlowski
2022-03-16 16:29     ` Krzysztof Kozlowski
2022-03-17  7:02     ` Kuldeep Singh
2022-03-17  7:02       ` Kuldeep Singh
2022-03-16  9:54 ` [PATCH 2/3] dt-bindings: timer: Document arm,cortex-a7-timer for " Kuldeep Singh
2022-03-16  9:54   ` [PATCH 2/3] dt-bindings: timer: Document arm, cortex-a7-timer " Kuldeep Singh
2022-03-16 16:28   ` [PATCH 2/3] dt-bindings: timer: Document arm,cortex-a7-timer " Krzysztof Kozlowski
2022-03-16 16:28     ` [PATCH 2/3] dt-bindings: timer: Document arm, cortex-a7-timer " Krzysztof Kozlowski
2022-03-16 17:43     ` Kuldeep Singh [this message]
2022-03-16 17:43       ` Kuldeep Singh
2022-03-16  9:54 ` [PATCH 3/3] clocksource: arch_timer: Add arm,cortex-a7/15-timer in of_match list Kuldeep Singh
2022-03-16  9:54   ` [PATCH 3/3] clocksource: arch_timer: Add arm, cortex-a7/15-timer " Kuldeep Singh
2022-03-16 10:05   ` [PATCH 3/3] clocksource: arch_timer: Add arm,cortex-a7/15-timer " Marc Zyngier
2022-03-16 10:05     ` [PATCH 3/3] clocksource: arch_timer: Add arm, cortex-a7/15-timer " Marc Zyngier
2022-03-16 16:30   ` [PATCH 3/3] clocksource: arch_timer: Add arm,cortex-a7/15-timer " Krzysztof Kozlowski
2022-03-16 16:30     ` Krzysztof Kozlowski
2022-03-16 17:41     ` Kuldeep Singh
2022-03-16 17:41       ` Kuldeep Singh
2022-03-16 18:43       ` Marc Zyngier
2022-03-16 18:43         ` [PATCH 3/3] clocksource: arch_timer: Add arm, cortex-a7/15-timer " Marc Zyngier
2022-03-17  6:59         ` [PATCH 3/3] clocksource: arch_timer: Add arm,cortex-a7/15-timer " Kuldeep Singh
2022-03-17  6:59           ` Kuldeep Singh
2022-03-17  7:18           ` Krzysztof Kozlowski
2022-03-17  7:18             ` Krzysztof Kozlowski
2022-03-16 11:27 ` [PATCH 0/3] Fix dtbs warnings for arch timer Marc Zyngier
2022-03-16 11:27   ` Marc Zyngier
2022-03-16 17:20   ` Kuldeep Singh
2022-03-16 17:20     ` Kuldeep Singh
2022-03-16 18:47     ` Marc Zyngier
2022-03-16 18:47       ` Marc Zyngier
2022-03-23 19:38       ` Rob Herring
2022-03-23 19:38         ` Rob Herring

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220316174306.GC21737@9a2d8922b8f1 \
    --to=singh.kuldeep87k@gmail.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski@canonical.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.