Linux-i3c Archive on lore.kernel.org
 help / Atom feed
* [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask
@ 2019-03-06 21:57 Vitor Soares
  2019-03-27 21:37 ` vitor
  0 siblings, 1 reply; 4+ messages in thread
From: Vitor Soares @ 2019-03-06 21:57 UTC (permalink / raw)
  To: bbrezillon; +Cc: linux-i3c, joao.pinto, Vitor Soares

The controller is disable by clear the DEV_CTRL_ENABLE bit rather
than set it. This is fixed by using the correct mask.

Fixes: 1dd728f5d4d4 ("i3c: master: Add driver for Synopsys DesignWare IP")
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);
 }
 
-- 
2.7.4


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask
  2019-03-06 21:57 [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask Vitor Soares
@ 2019-03-27 21:37 ` vitor
  2019-03-30  7:54   ` Boris Brezillon
  0 siblings, 1 reply; 4+ messages in thread
From: vitor @ 2019-03-27 21:37 UTC (permalink / raw)
  To: Vitor Soares, bbrezillon; +Cc: linux-i3c, joao.pinto

Hi Boris,


On 06/03/19 21:57, Vitor Soares wrote:
> The controller is disable by clear the DEV_CTRL_ENABLE bit rather
> than set it. This is fixed by using the correct mask.
>
> Fixes: 1dd728f5d4d4 ("i3c: master: Add driver for Synopsys DesignWare IP")
> 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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask
  2019-03-27 21:37 ` vitor
@ 2019-03-30  7:54   ` Boris Brezillon
  2019-04-01 16:58     ` vitor
  0 siblings, 1 reply; 4+ messages in thread
From: Boris Brezillon @ 2019-03-30  7:54 UTC (permalink / raw)
  To: vitor; +Cc: linux-i3c, joao.pinto, bbrezillon

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask
  2019-03-30  7:54   ` Boris Brezillon
@ 2019-04-01 16:58     ` vitor
  0 siblings, 0 replies; 4+ messages in thread
From: vitor @ 2019-04-01 16:58 UTC (permalink / raw)
  To: Boris Brezillon, vitor; +Cc: linux-i3c, joao.pinto, bbrezillon

Hi Boris,

Thanks for your feedback.

On 30/03/19 07:54, Boris Brezillon wrote:
> 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
>

Sure.


Best regards,
Vitor Soares

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-06 21:57 [PATCH] i3c: master:dw: Fix dw_i3c_master_disable controller by using correct mask Vitor Soares
2019-03-27 21:37 ` vitor
2019-03-30  7:54   ` Boris Brezillon
2019-04-01 16:58     ` vitor

Linux-i3c Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-i3c/0 linux-i3c/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-i3c linux-i3c/ https://lore.kernel.org/linux-i3c \
		linux-i3c@lists.infradead.org linux-i3c@archiver.kernel.org
	public-inbox-index linux-i3c


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.infradead.lists.linux-i3c


AGPL code for this site: git clone https://public-inbox.org/ public-inbox