Re: [PATCH] device property: Make modifications of fwnode "flags" thread safe

From: Andy Shevchenko

Date: Tue Mar 17 2026 - 06:22:27 EST


On Tue, Mar 17, 2026 at 09:42:38AM +0100, Wolfram Sang wrote:
>
> > > What's the alignment for the u8 member in your SoC? 4 bytes or 8 bytes?
> > > (I assume it's 64-bit SoC.)
> >
> > FWIW, with the given change it will be still inside 64-byte data structure
> > which most likely occupies a single cache line (before this patch and after
> > as well).
>
> I consider this directon of the discussion irrelevant. If the number is
> (maybe? That's to be discussed!) needlessly bigger than 0, then it
> doesn't matter how big the number is.

That's why it was written 'FWIW', so it doesn't worth :-)

> Why don't you like the idea of taking the lock?

Like Danilo I am also not sure what lock protects fwnode accesses.

--
With Best Regards,
Andy Shevchenko