Re: [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask

[Boris Brezillon <boris.brezillon@collabora.com>]

On Wed, 27 Mar 2019 21:37:59 +0000
vitor <vitor.soares@synopsys.com> wrote:

> Hi Boris,
> 
> 
> On 06/03/19 21:57, Vitor Soares wrote:
> > The controller is disable by clear the DEV_CTRL_ENABLE bit rather

		      ^disabled by clearing

> > than set it. This is fixed by using the correct mask.

	 ^setting


The commit message is not accurate. You were not setting the
DEV_CTRL_ENABLE bit, you were actually masking all other bits and
keeping the DEV_CTRL_ENABLE one unchanged.

> >
> > Fixes: 1dd728f5d4d4 ("i3c: master: Add driver for Synopsys DesignWare IP")

Missing

Cc: <stable@vger.kernel.org>

line.

Can you send a new version with this fixed?

Thanks,

Boris

> > Signed-off-by: Vitor Soares <vitor.soares@synopsys.com>
> > ---
> >  drivers/i3c/master/dw-i3c-master.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c
> > index bb03079..ec385fb 100644
> > --- a/drivers/i3c/master/dw-i3c-master.c
> > +++ b/drivers/i3c/master/dw-i3c-master.c
> > @@ -300,7 +300,7 @@ to_dw_i3c_master(struct i3c_master_controller *master)
> >  
> >  static void dw_i3c_master_disable(struct dw_i3c_master *master)
> >  {
> > -	writel(readl(master->regs + DEVICE_CTRL) & DEV_CTRL_ENABLE,
> > +	writel(readl(master->regs + DEVICE_CTRL) & ~DEV_CTRL_ENABLE,
> >  	       master->regs + DEVICE_CTRL);
> >  }
> >    
> 
> Friendly ping.
> 
> 
> Best regards,
> Vitor Soares
> 
> _______________________________________________
> linux-i3c mailing list
> linux-i3c@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-i3c


_______________________________________________
linux-i3c mailing list
linux-i3c@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-i3c