On 8/31/21 19:29, Peter Wang wrote:

REG_UFS_MTK_HW_VER is a read only mediatek dedicated register.
So, hw_ver will get a const value for mediatek to decide how to use
debug select. It only need read once, no need multi-threads protected.

Hi Peter,

The above can be concluded easily by a human but not by software. If 
this code is analyzed with KCSAN then KCSAN will probably complain. See 
also Documentation/dev-tools/kcsan.rst.

Anyway, I'm fine with this patch.

Thanks,

Bart.