diff mbox series

[RESEND] i3c: master: dw: Drop redundant disec call

Message ID 20201228105501.6104-1-miquel.raynal@bootlin.com (mailing list archive)
State Accepted
Commit 5c34b8e7e8bb605925b33e1aa7dc17966811219a
Headers show
Series [RESEND] i3c: master: dw: Drop redundant disec call | expand

Commit Message

Miquel Raynal Dec. 28, 2020, 10:55 a.m. UTC
Disabling all event calls is already handled by the core right before
starting the DAA process. Do not do it again when the DAA process
completes, it is redundant.

Fixes: 1dd728f5d4d4 ("i3c: master: Add driver for Synopsys DesignWare IP")
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---

AFAIK I did not receive any replies to this patch with is still valid,
so I'm just resending it as-is.

 drivers/i3c/master/dw-i3c-master.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Alexandre Belloni Feb. 5, 2021, 11:44 p.m. UTC | #1
On Mon, 28 Dec 2020 11:55:01 +0100, Miquel Raynal wrote:
> Disabling all event calls is already handled by the core right before
> starting the DAA process. Do not do it again when the DAA process
> completes, it is redundant.

Applied, thanks!

[1/1] i3c: master: dw: Drop redundant disec call
      commit: 5c34b8e7e8bb605925b33e1aa7dc17966811219a

Best regards,
diff mbox series

Patch

diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c
index 8513bd353c05..03a368da51b9 100644
--- a/drivers/i3c/master/dw-i3c-master.c
+++ b/drivers/i3c/master/dw-i3c-master.c
@@ -816,11 +816,6 @@  static int dw_i3c_master_daa(struct i3c_master_controller *m)
 
 	dw_i3c_master_free_xfer(xfer);
 
-	i3c_master_disec_locked(m, I3C_BROADCAST_ADDR,
-				I3C_CCC_EVENT_HJ |
-				I3C_CCC_EVENT_MR |
-				I3C_CCC_EVENT_SIR);
-
 	return 0;
 }