Re: [PATCH v8 05/10] pmdomain: samsung: convert to using regmap
From: André Draszik
Date: Thu Mar 19 2026 - 07:58:31 EST
Hi Marek,
On Thu, 2026-03-19 at 11:29 +0100, Marek Szyprowski wrote:
> On 19.03.2026 11:13, Ulf Hansson wrote:
> > As a follow-up patch on top, please consider converting the open-coded
> > polling loop above into a readx_poll_timeout_atomic().
>
> This has been tried and it doesn't work in all cases required for power
> domain driver:
>
> https://lore.kernel.org/all/5c19e4ef-c4fd-4bf5-88b3-46c86751b14e@xxxxxxxxxxx/
>
> Probably a comment about that could be added directly to this code to
> avoid such conversion and breakage in the future.
I am planning to revisit this in the future and am hoping that we can
figure out what goes wrong when using regmap_read_poll_timeout().
Hopefully such a comment would only be short-lived, so maybe not really
worth it? I can add it, though, if you prefer.
Cheers,
Andre'