[PATCH v27 05/11] soc: mediatek: refine code to use mtk_mmsys_update_bits API

Nancy Lin (林欣螢) Nancy.Lin at mediatek.com
Fri Nov 4 02:29:55 PDT 2022


Dear Nicolas,

Thanks for the review.

On Thu, 2022-11-03 at 16:32 -0400, Nícolas F. R. A. Prado wrote:
> On Thu, Nov 03, 2022 at 11:25:06AM +0800, Nancy.Lin wrote:
> [..]
> > --- a/drivers/soc/mediatek/mtk-mmsys.c
> > +++ b/drivers/soc/mediatek/mtk-mmsys.c
> 
> [..]
> > +static void mtk_mmsys_update_bits(struct mtk_mmsys *mmsys, u32
> > offset, u32 mask, u32 val)
> > +{
> > +	u32 tmp;
> > +
> > +	tmp = readl_relaxed(mmsys->regs + offset);
> > +	tmp = (tmp & ~mask) | val;
> 
> You should mask the value before writing to prevent bad values from
> overwriting
> bits outside the mask.
> 
> 	tmp = (tmp & ~mask) | (val & mask);
> 
> With that,
> 
> Reviewed-by: Nícolas F. R. A. Prado <nfraprado at collabora.com>
> 
> Thanks,
> Nícolas

OK, I will fix it in the next revision.

Thanks,
Nancy


More information about the Linux-mediatek mailing list