[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <e2bb53eb-70ef-23e0-53ae-6dbfcb54639d@linaro.org>
Date: Fri, 18 Aug 2023 12:16:35 +0200
From: Daniel Lezcano <daniel.lezcano@...aro.org>
To: Mans Rullgard <mans@...sr.com>, Chen-Yu Tsai <wens@...e.org>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Samuel Holland <samuel@...lland.org>,
Maxime Ripard <mripard@...nel.org>
Cc: Thomas Gleixner <tglx@...utronix.de>, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev
Subject: Re: [PATCH 0/3] clocksource: sun5i: clean up and convert to platform
driver
On 30/06/2023 22:01, Mans Rullgard wrote:
> The first two of these patches remove a bunch of duplicated code/data in
> the sun5i hstimer driver. To keep the diff clearer, the first patch
> introduces an otherwise pointless struct which is then removed in the
> second.
>
> The third patch converts the driver to a platform_device driver. This
> is to make it work again on A20 and A31 (I think) where it broke when
> the ccu driver was changed to a platform driver.
>
> I have only tested this on an A20 where the hstimer is registered as
> expected and can be selected through sysfs. I hope it doesn't break
> something else.
>
> Some additional discussion here:
> https://lore.kernel.org/linux-clk/20211119033338.25486-4-samuel@sholland.org/
>
> Mans Rullgard (3):
> clocksource: sun5i: remove duplication of code and data
> clocksource: sun5i: remove pointless struct
> clocksource: sun5i: convert to platform device driver
>
> drivers/clocksource/timer-sun5i.c | 288 +++++++++++++-----------------
> 1 file changed, 123 insertions(+), 165 deletions(-)
Applied, thanks
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
Powered by blists - more mailing lists