[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