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

Jerome Brunet jbrunet at baylibre.com
Thu Aug 8 03:15:16 PDT 2024


On Thu 18 Jul 2024 at 12:29, Stephen Boyd <sboyd at kernel.org> 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



More information about the linux-amlogic mailing list