From npitre at baylibre.com  Mon Feb  1 11:18:37 2021
From: npitre at baylibre.com (Nicolas Pitre)
Date: Mon, 1 Feb 2021 11:18:37 -0500 (EST)
Subject: [PATCH] i3c/master/mipi-i3c-hci: Specify HAS_IOMEM dependency
In-Reply-To: <20210127040636.1535722-1-davidgow@google.com>
References: <20210127040636.1535722-1-davidgow@google.com>
Message-ID: <9s6n963q-poro-6qnq-q510-83rr59785on@onlyvoer.pbz>

On Tue, 26 Jan 2021, David Gow wrote:

> The MIPI i3c HCI driver makes use of IOMEM functions like
> devm_platform_ioremap_resource(), which are only available if
> CONFIG_HAS_IOMEM is defined.
> 
> This causes the driver to be enabled under make ARCH=um allyesconfig,
> even though it won't build.
> 
> By adding a dependency on HAS_IOMEM, the driver will not be enabled on
> architectures which don't support it.
> 
> Fixes: 9ad9a52cce28 ("i3c/master: introduce the mipi-i3c-hci driver")
> Signed-off-by: David Gow <davidgow at google.com>

Acked-by: Nicolas Pitre <npitre at baylibre.com>


> ---
>  drivers/i3c/master/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/i3c/master/Kconfig b/drivers/i3c/master/Kconfig
> index e68f15f4b4d0..afff0e2320f7 100644
> --- a/drivers/i3c/master/Kconfig
> +++ b/drivers/i3c/master/Kconfig
> @@ -25,6 +25,7 @@ config DW_I3C_MASTER
>  config MIPI_I3C_HCI
>  	tristate "MIPI I3C Host Controller Interface driver (EXPERIMENTAL)"
>  	depends on I3C
> +	depends on HAS_IOMEM
>  	help
>  	  Support for hardware following the MIPI Aliance's I3C Host Controller
>  	  Interface specification.
> -- 
> 2.30.0.280.ga3ce27912f-goog
> 
> 
> -- 
> linux-i3c mailing list
> linux-i3c at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-i3c
> 


From alexandre.belloni at bootlin.com  Mon Feb  1 18:17:20 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Tue,  2 Feb 2021 00:17:20 +0100
Subject: [PATCH] i3c/master/mipi-i3c-hci: Specify HAS_IOMEM dependency
In-Reply-To: <20210127040636.1535722-1-davidgow@google.com>
References: <20210127040636.1535722-1-davidgow@google.com>
Message-ID: <161222143405.3432488.2995169601612926467.b4-ty@bootlin.com>

On Tue, 26 Jan 2021 20:06:37 -0800, David Gow wrote:
> The MIPI i3c HCI driver makes use of IOMEM functions like
> devm_platform_ioremap_resource(), which are only available if
> CONFIG_HAS_IOMEM is defined.
> 
> This causes the driver to be enabled under make ARCH=um allyesconfig,
> even though it won't build.
> 
> [...]

Applied, thanks!

[1/1] i3c/master/mipi-i3c-hci: Specify HAS_IOMEM dependency
      commit: 9d909f1b1e91b4aa7d016ed14b7b76dbf2675414

Best regards,
-- 
Alexandre Belloni <alexandre.belloni at bootlin.com>


From alexandre.belloni at bootlin.com  Mon Feb  1 18:17:21 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Tue,  2 Feb 2021 00:17:21 +0100
Subject: [PATCH 1/2] i3c: Handle drivers without probe or remove callback
In-Reply-To: <20210128091048.17006-1-u.kleine-koenig@pengutronix.de>
References: <20210128091048.17006-1-u.kleine-koenig@pengutronix.de>
Message-ID: <161222143405.3432488.2548904901699225293.b4-ty@bootlin.com>

On Thu, 28 Jan 2021 10:10:47 +0100, Uwe Kleine-K?nig wrote:
> A registered driver without a probe callback doesn't make sense, so
> refuse to register such a driver. (Otherwise i3c_device_probe() yields a
> NULL pointer exception.)
> 
> A driver without remove is possible, e.g. when all resources are freed
> using devm callbacks. So guard the call to driver->remove by a check
> for being non-NULL.
> 
> [...]

Applied, thanks!

[1/2] i3c: Handle drivers without probe or remove callback
      commit: 7456fea589c6ad3422b0c188871ae80e1d307737
[2/2] i3c: Make remove callback return void
      commit: dd9267034c0e364b49261c3e0070b863286d1242

Best regards,
-- 
Alexandre Belloni <alexandre.belloni at bootlin.com>


From robh at kernel.org  Fri Feb  5 16:31:38 2021
From: robh at kernel.org (Rob Herring)
Date: Fri, 5 Feb 2021 15:31:38 -0600
Subject: [PATCH v5 1/6] dt-bindings: i3c: Convert the bus description to
 yaml
In-Reply-To: <20210121101808.14654-2-miquel.raynal@bootlin.com>
References: <20210121101808.14654-1-miquel.raynal@bootlin.com>
 <20210121101808.14654-2-miquel.raynal@bootlin.com>
Message-ID: <20210205213138.GA3782695@robh.at.kernel.org>

On Thu, 21 Jan 2021 11:18:03 +0100, Miquel Raynal wrote:
> Attempting a conversion of the i3c.txt file to yaml schema with
> minimal content changes.
> 
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
>  Documentation/devicetree/bindings/i3c/i3c.txt | 140 --------------
>  .../devicetree/bindings/i3c/i3c.yaml          | 179 ++++++++++++++++++
>  2 files changed, 179 insertions(+), 140 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/i3c/i3c.txt
>  create mode 100644 Documentation/devicetree/bindings/i3c/i3c.yaml
> 

Reviewed-by: Rob Herring <robh at kernel.org>


From robh at kernel.org  Fri Feb  5 16:31:59 2021
From: robh at kernel.org (Rob Herring)
Date: Fri, 5 Feb 2021 15:31:59 -0600
Subject: [PATCH v5 2/6] dt-bindings: i3c: mipi-hci: Include the bus binding
In-Reply-To: <20210121101808.14654-3-miquel.raynal@bootlin.com>
References: <20210121101808.14654-1-miquel.raynal@bootlin.com>
 <20210121101808.14654-3-miquel.raynal@bootlin.com>
Message-ID: <20210205213159.GA3783349@robh.at.kernel.org>

On Thu, 21 Jan 2021 11:18:04 +0100, Miquel Raynal wrote:
> Update a little bit the content to match the bus binding, including:
> - the node title should have been named after the description done
>   in the historical i3c.txt file, ie: i3c-master@<address>
> - child nodes should be accepted even though the drivers do not currently
>   support it
> - #address-cells and #size-cells are also mandatory and have specific
>   values
> 
> Cc: Nicolas Pitre <nico at fluxnic.net>
> Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> ---
>  Documentation/devicetree/bindings/i3c/mipi-i3c-hci.yaml | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 

Reviewed-by: Rob Herring <robh at kernel.org>


From alexandre.belloni at bootlin.com  Fri Feb  5 18:44:29 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Sat,  6 Feb 2021 00:44:29 +0100
Subject: [PATCH v5 0/6] Silvaco I3C master driver
In-Reply-To: <20210121101808.14654-1-miquel.raynal@bootlin.com>
References: <20210121101808.14654-1-miquel.raynal@bootlin.com>
Message-ID: <161256866173.4836.10256863699232447925.b4-ty@bootlin.com>

On Thu, 21 Jan 2021 11:18:02 +0100, Miquel Raynal wrote:
> Here is the addition of a driver for the Silvaco I3C master IP.
> 
> Changes in v5:
> * Added Rob's R-by when relevant.
> * Fixed the robots warnings by pushing a little bit forward the changes in the
>   mipi-hci binding file.
> * Fixed the I3C master node name regex.
> * Removed the redundant $ref: entries when the hz suffix is used.
> * Simplified a little bit the regexes defining the child nodes.
> * Updated the reg propertie description to better describe each entry.
> 
> [...]

Applied, thanks!

[1/6] dt-bindings: i3c: Convert the bus description to yaml
      commit: 5e4cdca887fdb445f962b3dbc2a2514d7c025d9b
[2/6] dt-bindings: i3c: mipi-hci: Include the bus binding
      commit: de67276e66fcfcd404516eebfd6436239dd9882a
[3/6] dt-bindings: Add vendor prefix for Silvaco
      commit: 57f7c9ff1b3fdc2cccb377207e538bf5f3ab03cf
[4/6] dt-bindings: i3c: Describe Silvaco master binding
      commit: b8b0446f1f1afd58e5a9ba14ab2caa08797f3bb5
[5/6] i3c: master: svc: Add Silvaco I3C master driver
      commit: dd3c52846d5954acd43f0e771689302f27dadc28
[6/6] MAINTAINERS: Add Silvaco I3C master
      commit: f06a1af8e739cb573b79cd175fd88534e912bea0

Best regards,
-- 
Alexandre Belloni <alexandre.belloni at bootlin.com>


From alexandre.belloni at bootlin.com  Fri Feb  5 18:44:28 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Sat,  6 Feb 2021 00:44:28 +0100
Subject: [PATCH RESEND] i3c: master: dw: Drop redundant disec call
In-Reply-To: <20201228105501.6104-1-miquel.raynal@bootlin.com>
References: <20201228105501.6104-1-miquel.raynal@bootlin.com>
Message-ID: <161256866173.4836.3780146529249506859.b4-ty@bootlin.com>

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,
-- 
Alexandre Belloni <alexandre.belloni at bootlin.com>


From dan.carpenter at oracle.com  Tue Feb  9 05:57:28 2021
From: dan.carpenter at oracle.com (Dan Carpenter)
Date: Tue, 9 Feb 2021 13:57:28 +0300
Subject: [bug report] i3c: master: svc: Add Silvaco I3C master driver
Message-ID: <YCJqmKHvBWFVIudi@mwanda>

Hello Miquel Raynal,

The patch dd3c52846d59: "i3c: master: svc: Add Silvaco I3C master
driver" from Jan 21, 2021, leads to the following static checker
warning:

	drivers/i3c/master/svc-i3c-master.c:394 svc_i3c_master_ibi_work()
	error: uninitialized symbol 'dev'.

drivers/i3c/master/svc-i3c-master.c
   364          status = readl(master->regs + SVC_I3C_MSTATUS);
   365          ibitype = SVC_I3C_MSTATUS_IBITYPE(status);
   366          ibiaddr = SVC_I3C_MSTATUS_IBIADDR(status);
   367  
   368          /* Handle the critical responses to IBI's */
   369          switch (ibitype) {
   370          case SVC_I3C_MSTATUS_IBITYPE_IBI:
   371                  dev = svc_i3c_master_dev_from_addr(master, ibiaddr);
   372                  if (!dev)
   373                          svc_i3c_master_nack_ibi(master);
   374                  else
   375                          svc_i3c_master_handle_ibi(master, dev);

"dev" only valid on this path.

   376                  break;
   377          case SVC_I3C_MSTATUS_IBITYPE_HOT_JOIN:
   378                  svc_i3c_master_ack_ibi(master, false);
   379                  break;
   380          case SVC_I3C_MSTATUS_IBITYPE_MASTER_REQUEST:
   381                  svc_i3c_master_nack_ibi(master);
   382                  break;
   383          default:
   384                  break;
   385          }
   386  
   387          /*
   388           * If an error happened, we probably got interrupted and the exchange
   389           * timedout. In this case we just drop everything, emit a stop and wait
   390           * for the slave to interrupt again.
   391           */
   392          if (svc_i3c_master_error(master)) {
   393                  if (master->ibi.tbq_slot) {
   394                          data = i3c_dev_get_master_data(dev);
                                                               ^^^
Dereferenced here.

   395                          i3c_generic_ibi_recycle_slot(data->ibi_pool,
   396                                                       master->ibi.tbq_slot);
   397                          master->ibi.tbq_slot = NULL;
   398                  }
   399  
   400                  svc_i3c_master_emit_stop(master);
   401  
   402                  goto reenable_ibis;
   403          }
   404  
   405          /* Handle the non critical tasks */

regards,
dan carpenter


From alexandre.belloni at bootlin.com  Tue Feb  9 17:05:27 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Tue, 9 Feb 2021 23:05:27 +0100
Subject: [GIT PULL] I3C fixes for 5.11
Message-ID: <20210209220527.GA498175@piout.net>

Hello Linus,

Here is a single compilation warning fix for v5.11.

The following changes since commit 5c8fe583cce542aa0b84adc939ce85293de36e5e:

  Linux 5.11-rc1 (2020-12-27 15:30:22 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/i3c/linux.git tags/i3c/fixes-for-5.11

for you to fetch changes up to 291b5c9870fc546376d69cf792b7885cd0c9c1b3:

  i3c/master/mipi-i3c-hci: Fix position of __maybe_unused in i3c_hci_of_match (2020-12-31 18:41:37 +0100)

----------------------------------------------------------------
I3C fixes for 5.11

Drivers:
 - mipi-i3c-hci: fix compilation warning with Clang

----------------------------------------------------------------
Nathan Chancellor (1):
      i3c/master/mipi-i3c-hci: Fix position of __maybe_unused in i3c_hci_of_match

 drivers/i3c/master/mipi-i3c-hci/core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From nathan at kernel.org  Mon Feb  8 22:59:58 2021
From: nathan at kernel.org (Nathan Chancellor)
Date: Mon, 8 Feb 2021 20:59:58 -0700
Subject: [PATCH] i3c/master/mipi-i3c-hci: Fix position of __maybe_unused
 in i3c_hci_of_match
In-Reply-To: <160943653614.132002.12499200411415721355.b4-ty@bootlin.com>
References: <20201222025931.3043480-1-natechancellor@gmail.com>
 <160943653614.132002.12499200411415721355.b4-ty@bootlin.com>
Message-ID: <20210209035958.GA2059815@ubuntu-m3-large-x86>

On Thu, Dec 31, 2020 at 06:42:23PM +0100, Alexandre Belloni wrote:
> On Mon, 21 Dec 2020 19:59:31 -0700, Nathan Chancellor wrote:
> > Clang warns:
> > 
> >  ../drivers/i3c/master/mipi-i3c-hci/core.c:780:21: warning: attribute
> >  declaration must precede definition [-Wignored-attributes]
> >  static const struct __maybe_unused of_device_id i3c_hci_of_match[] = {
> >                      ^
> >  ../include/linux/compiler_attributes.h:267:56: note: expanded from macro
> >  '__maybe_unused'
> >  #define __maybe_unused                  __attribute__((__unused__))
> >                                                         ^
> >  ../include/linux/mod_devicetable.h:262:8: note: previous definition is
> >  here
> >  struct of_device_id {
> >         ^
> > 1 warning generated.
> > 
> > [...]
> 
> Applied, thanks!
> 
> [1/1] i3c/master/mipi-i3c-hci: Fix position of __maybe_unused in i3c_hci_of_match
>       commit: 291b5c9870fc546376d69cf792b7885cd0c9c1b3
> 
> Best regards,
> -- 
> Alexandre Belloni <alexandre.belloni at bootlin.com>
> 

Hi Alexandre,

I see as of v5.11-rc7 that this is still an issue even though it has
been applied to the fixes branch. Is it possible for this to make it in
to 5.11 final?

Cheers,
Nathan


From pr-tracker-bot at kernel.org  Wed Feb 10 15:19:31 2021
From: pr-tracker-bot at kernel.org (pr-tracker-bot at kernel.org)
Date: Wed, 10 Feb 2021 20:19:31 +0000
Subject: [GIT PULL] I3C fixes for 5.11
In-Reply-To: <20210209220527.GA498175@piout.net>
References: <20210209220527.GA498175@piout.net>
Message-ID: <161298837140.25163.5789660091863634580.pr-tracker-bot@kernel.org>

The pull request you sent on Tue, 9 Feb 2021 23:05:27 +0100:

> git://git.kernel.org/pub/scm/linux/kernel/git/i3c/linux.git tags/i3c/fixes-for-5.11

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/ef7d0b599938450c54a8dc0aa4b954d73d9a9370

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html


From miquel.raynal at bootlin.com  Tue Feb 16 10:21:01 2021
From: miquel.raynal at bootlin.com (Miquel Raynal)
Date: Tue, 16 Feb 2021 16:21:01 +0100
Subject: [bug report] i3c: master: svc: Add Silvaco I3C master driver
In-Reply-To: <YCJqmKHvBWFVIudi@mwanda>
References: <YCJqmKHvBWFVIudi@mwanda>
Message-ID: <20210216162101.4bc23c46@xps13>

Hi Dan,

Thanks for the report.

Dan Carpenter <dan.carpenter at oracle.com> wrote on Tue, 9 Feb 2021
13:57:28 +0300:

> Hello Miquel Raynal,
> 
> The patch dd3c52846d59: "i3c: master: svc: Add Silvaco I3C master
> driver" from Jan 21, 2021, leads to the following static checker
> warning:
> 
> 	drivers/i3c/master/svc-i3c-master.c:394 svc_i3c_master_ibi_work()
> 	error: uninitialized symbol 'dev'.
> 
> drivers/i3c/master/svc-i3c-master.c
>    364          status = readl(master->regs + SVC_I3C_MSTATUS);
>    365          ibitype = SVC_I3C_MSTATUS_IBITYPE(status);
>    366          ibiaddr = SVC_I3C_MSTATUS_IBIADDR(status);
>    367  
>    368          /* Handle the critical responses to IBI's */
>    369          switch (ibitype) {
>    370          case SVC_I3C_MSTATUS_IBITYPE_IBI:
>    371                  dev = svc_i3c_master_dev_from_addr(master, ibiaddr);
>    372                  if (!dev)
>    373                          svc_i3c_master_nack_ibi(master);
>    374                  else
>    375                          svc_i3c_master_handle_ibi(master, dev);
> 
> "dev" only valid on this path.
> 
>    376                  break;
>    377          case SVC_I3C_MSTATUS_IBITYPE_HOT_JOIN:
>    378                  svc_i3c_master_ack_ibi(master, false);
>    379                  break;
>    380          case SVC_I3C_MSTATUS_IBITYPE_MASTER_REQUEST:
>    381                  svc_i3c_master_nack_ibi(master);
>    382                  break;
>    383          default:
>    384                  break;
>    385          }
>    386  
>    387          /*
>    388           * If an error happened, we probably got interrupted and the exchange
>    389           * timedout. In this case we just drop everything, emit a stop and wait
>    390           * for the slave to interrupt again.
>    391           */
>    392          if (svc_i3c_master_error(master)) {
>    393                  if (master->ibi.tbq_slot) {
>    394                          data = i3c_dev_get_master_data(dev);
>                                                                ^^^
> Dereferenced here.

Only svc_i3c_master_handle_ibi() populates master->ibi.tbq_slot, so dev
will only be dereferenced if svc_i3c_master_handle_ibi() was executed,
which only happens if dev is valid.

I don't think there is a risk here, however if robots really are
unhappy we might add an extra sanity level by checking dev explicitly,
but this would be purely useless on a practical point of view.

Thanks,
Miqu?l


From dan.carpenter at oracle.com  Tue Feb 16 13:44:19 2021
From: dan.carpenter at oracle.com (Dan Carpenter)
Date: Tue, 16 Feb 2021 21:44:19 +0300
Subject: [bug report] i3c: master: svc: Add Silvaco I3C master driver
In-Reply-To: <20210216162101.4bc23c46@xps13>
References: <YCJqmKHvBWFVIudi@mwanda>
 <20210216162101.4bc23c46@xps13>
Message-ID: <20210216184419.GH2222@kadam>

On Tue, Feb 16, 2021 at 04:21:01PM +0100, Miquel Raynal wrote:
> Hi Dan,
> 
> Thanks for the report.
> 
> Dan Carpenter <dan.carpenter at oracle.com> wrote on Tue, 9 Feb 2021
> 13:57:28 +0300:
> 
> > Hello Miquel Raynal,
> > 
> > The patch dd3c52846d59: "i3c: master: svc: Add Silvaco I3C master
> > driver" from Jan 21, 2021, leads to the following static checker
> > warning:
> > 
> > 	drivers/i3c/master/svc-i3c-master.c:394 svc_i3c_master_ibi_work()
> > 	error: uninitialized symbol 'dev'.
> > 
> > drivers/i3c/master/svc-i3c-master.c
> >    364          status = readl(master->regs + SVC_I3C_MSTATUS);
> >    365          ibitype = SVC_I3C_MSTATUS_IBITYPE(status);
> >    366          ibiaddr = SVC_I3C_MSTATUS_IBIADDR(status);
> >    367  
> >    368          /* Handle the critical responses to IBI's */
> >    369          switch (ibitype) {
> >    370          case SVC_I3C_MSTATUS_IBITYPE_IBI:
> >    371                  dev = svc_i3c_master_dev_from_addr(master, ibiaddr);
> >    372                  if (!dev)
> >    373                          svc_i3c_master_nack_ibi(master);
> >    374                  else
> >    375                          svc_i3c_master_handle_ibi(master, dev);
> > 
> > "dev" only valid on this path.
> > 
> >    376                  break;
> >    377          case SVC_I3C_MSTATUS_IBITYPE_HOT_JOIN:
> >    378                  svc_i3c_master_ack_ibi(master, false);
> >    379                  break;
> >    380          case SVC_I3C_MSTATUS_IBITYPE_MASTER_REQUEST:
> >    381                  svc_i3c_master_nack_ibi(master);
> >    382                  break;
> >    383          default:
> >    384                  break;
> >    385          }
> >    386  
> >    387          /*
> >    388           * If an error happened, we probably got interrupted and the exchange
> >    389           * timedout. In this case we just drop everything, emit a stop and wait
> >    390           * for the slave to interrupt again.
> >    391           */
> >    392          if (svc_i3c_master_error(master)) {
> >    393                  if (master->ibi.tbq_slot) {
> >    394                          data = i3c_dev_get_master_data(dev);
> >                                                                ^^^
> > Dereferenced here.
> 
> Only svc_i3c_master_handle_ibi() populates master->ibi.tbq_slot, so dev
> will only be dereferenced if svc_i3c_master_handle_ibi() was executed,
> which only happens if dev is valid.
> 
> I don't think there is a risk here, however if robots really are
> unhappy we might add an extra sanity level by checking dev explicitly,
> but this would be purely useless on a practical point of view.

Nah.  I think this works.  It's fine.  Don't do things just to make the
robots happy.

regards,
dan carpenter



From alexandre.belloni at bootlin.com  Sun Feb 21 19:41:22 2021
From: alexandre.belloni at bootlin.com (Alexandre Belloni)
Date: Mon, 22 Feb 2021 01:41:22 +0100
Subject: [GIT PULL] I3C changes for 5.12
Message-ID: <YDL9slBuOqU83/RK@piout.net>

Hello Linus,

Here is the I3C pull request for v5.12 with a few improvements for the
core and a new driver.

The following changes since commit 5c8fe583cce542aa0b84adc939ce85293de36e5e:

  Linux 5.11-rc1 (2020-12-27 15:30:22 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/i3c/linux.git tags/i3c/for-5.12

for you to fetch changes up to 5c34b8e7e8bb605925b33e1aa7dc17966811219a:

  i3c: master: dw: Drop redundant disec call (2021-02-06 00:44:15 +0100)

----------------------------------------------------------------
I3C for 5.12

Subsystem:
 - Handle drivers without probe or remove callback
 - Remove callback now returns void
 - DT documentation is now in yaml

New driver:
 - Silvaco I3C master

----------------------------------------------------------------
David Gow (1):
      i3c/master/mipi-i3c-hci: Specify HAS_IOMEM dependency

Miquel Raynal (7):
      dt-bindings: i3c: Convert the bus description to yaml
      dt-bindings: i3c: mipi-hci: Include the bus binding
      dt-bindings: Add vendor prefix for Silvaco
      dt-bindings: i3c: Describe Silvaco master binding
      i3c: master: svc: Add Silvaco I3C master driver
      MAINTAINERS: Add Silvaco I3C master
      i3c: master: dw: Drop redundant disec call

Uwe Kleine-K?nig (2):
      i3c: Handle drivers without probe or remove callback
      i3c: Make remove callback return void

 Documentation/devicetree/bindings/i3c/i3c.txt      |  140 --
 Documentation/devicetree/bindings/i3c/i3c.yaml     |  179 +++
 .../devicetree/bindings/i3c/mipi-i3c-hci.yaml      |    9 +-
 .../bindings/i3c/silvaco,i3c-master.yaml           |   60 +
 .../devicetree/bindings/vendor-prefixes.yaml       |    2 +
 MAINTAINERS                                        |    8 +
 drivers/i3c/device.c                               |    5 +
 drivers/i3c/master.c                               |    8 +-
 drivers/i3c/master/Kconfig                         |    9 +
 drivers/i3c/master/Makefile                        |    1 +
 drivers/i3c/master/dw-i3c-master.c                 |    5 -
 drivers/i3c/master/svc-i3c-master.c                | 1478 ++++++++++++++++++++
 include/linux/i3c/device.h                         |    2 +-
 13 files changed, 1753 insertions(+), 153 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/i3c/i3c.txt
 create mode 100644 Documentation/devicetree/bindings/i3c/i3c.yaml
 create mode 100644 Documentation/devicetree/bindings/i3c/silvaco,i3c-master.yaml
 create mode 100644 drivers/i3c/master/svc-i3c-master.c

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


From pr-tracker-bot at kernel.org  Mon Feb 22 13:12:00 2021
From: pr-tracker-bot at kernel.org (pr-tracker-bot at kernel.org)
Date: Mon, 22 Feb 2021 18:12:00 +0000
Subject: [GIT PULL] I3C changes for 5.12
In-Reply-To: <YDL9slBuOqU83/RK@piout.net>
References: <YDL9slBuOqU83/RK@piout.net>
Message-ID: <161401752042.943.3286519972663636563.pr-tracker-bot@kernel.org>

The pull request you sent on Mon, 22 Feb 2021 01:41:22 +0100:

> git://git.kernel.org/pub/scm/linux/kernel/git/i3c/linux.git tags/i3c/for-5.12

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/36c1b20d15703662aa0f14a32a8bd19ab3a33076

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html


From colin.king at canonical.com  Wed Feb 24 10:13:49 2021
From: colin.king at canonical.com (Colin King)
Date: Wed, 24 Feb 2021 15:13:49 +0000
Subject: [PATCH][next] i3c: master: svc: remove redundant assignment to
 cmd->read_len
Message-ID: <20210224151349.202332-1-colin.king@canonical.com>

From: Colin Ian King <colin.king at canonical.com>

The assignment of xfer_len to cmd->read_len appears to be redundant
as the next statement re-assigns the value 0 to it.  Clean up the
code by removing the redundant first assignment.

Addresses-Coverity: ("Unused value")
Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 drivers/i3c/master/svc-i3c-master.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/i3c/master/svc-i3c-master.c b/drivers/i3c/master/svc-i3c-master.c
index 8d990696676e..1f6ba4221817 100644
--- a/drivers/i3c/master/svc-i3c-master.c
+++ b/drivers/i3c/master/svc-i3c-master.c
@@ -1124,7 +1124,6 @@ static int svc_i3c_master_send_direct_ccc_cmd(struct svc_i3c_master *master,
 	cmd->in = NULL;
 	cmd->out = &ccc->id;
 	cmd->len = 1;
-	cmd->read_len = xfer_len;
 	cmd->read_len = 0;
 	cmd->continued = true;
 
-- 
2.30.0



