Date:   Mon, 29 Nov 2021 15:30:33 +0200
From:   Tony Lindgren <tony@atomide.com>
To:     Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     linux-omap@vger.kernel.org,
        =?utf-8?Q?Beno=C3=AEt?= Cousson <bcousson@baylibre.com>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        Thomas Gleixner <tglx@linutronix.de>,
        Rob Herring <robh+dt@kernel.org>,
        Jarkko Nikula <jarkko.nikula@bitmer.com>
Subject: Re: [PATCH] ARM: dts: Fix timer regression for beagleboard revision c
Message-ID: <YaTV+UJjbD2HLnR9@atomide.com>
References: <20211125144834.52457-1-tony@atomide.com>
 <6ce29c03-03ce-8e65-76e1-40fe2bf23caa@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <6ce29c03-03ce-8e65-76e1-40fe2bf23caa@linaro.org>
Precedence: bulk

Hi,

* Daniel Lezcano <daniel.lezcano@linaro.org> [211129 09:57]:
> On 25/11/2021 15:48, Tony Lindgren wrote:
> >  .../devicetree/bindings/arm/omap/omap.txt     |  3 ++
> >  arch/arm/boot/dts/Makefile                    |  1 +
> >  arch/arm/boot/dts/omap3-beagle-ab4.dts        | 47 +++++++++++++++++++
> >  arch/arm/boot/dts/omap3-beagle.dts            | 33 -------------
> >  drivers/clocksource/timer-ti-dm-systimer.c    |  2 +-
> >  5 files changed, 52 insertions(+), 34 deletions(-)
> >  create mode 100644 arch/arm/boot/dts/omap3-beagle-ab4.dts
> 
> Usually, bindings DT and driver changes are separate patches

Not always for fixes :) In this case we need to patch both the dts for
timer quirks, and also update the related driver quirk. The driver quirk
I originally added to deal with possble older dtb files complicating the
fix unncessarily..

If you have better ideas for a fix please let me know.

Regards,

Tony