Re: [PATCH 1/8] reset: amlogic: convert driver to regmap

From: Jerome Brunet
Date: Thu Aug 08 2024 - 06:15:45 EST


On Thu 18 Jul 2024 at 12:29, Stephen Boyd <sboyd@xxxxxxxxxx> wrote:

>>
>> Full analysis is following:
>> - using regmap_update_bits() instead of writel() is incorrect because
>> this changes the behavior of the driver
>> - regmap_update_bits() should not be used here because default value of
>> regs isn't taken into account and (_apparently_, the doc is terse) these
>> regs could be updated by hw itself.
>
> Maybe use regmap_write_bits() instead.

Actually regmap_write_bits() performs an update behind the scene.
You'd still get the undefined read value making a mess AFAICT.

I'll stick to the usual regmap_write().

--
Jerome