linux-i3c.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] i3c: add missing include to internal header
@ 2025-07-17 12:00 Wolfram Sang
  2025-07-17 12:04 ` Wolfram Sang
  2025-07-17 18:40 ` Frank Li
  0 siblings, 2 replies; 3+ messages in thread
From: Wolfram Sang @ 2025-07-17 12:00 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Wolfram Sang, kernel test robot, Alexandre Belloni, Frank Li,
	linux-i3c

LKP found a random config which failed to build because IO accessors
were not defined:

   In file included from drivers/i3c/master.c:21:
   drivers/i3c/internals.h: In function 'i3c_writel_fifo':
>> drivers/i3c/internals.h:35:9: error: implicit declaration of function 'writesl' [-Werror=implicit-function-declaration]

Add the proper header to where the IO accessors are used.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202507150208.BZDzzJ5E-lkp@intel.com/
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 drivers/i3c/internals.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/i3c/internals.h b/drivers/i3c/internals.h
index 67ed1c44f615..1bf6aecebcbb 100644
--- a/drivers/i3c/internals.h
+++ b/drivers/i3c/internals.h
@@ -9,6 +9,7 @@
 #define I3C_INTERNALS_H
 
 #include <linux/i3c/master.h>
+#include <linux/io.h>
 
 extern const struct device_type i3c_masterdev_type;
 
-- 
2.47.2


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

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

* Re: [PATCH] i3c: add missing include to internal header
  2025-07-17 12:00 [PATCH] i3c: add missing include to internal header Wolfram Sang
@ 2025-07-17 12:04 ` Wolfram Sang
  2025-07-17 18:40 ` Frank Li
  1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2025-07-17 12:04 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: kernel test robot, Alexandre Belloni, Frank Li, linux-i3c

On Thu, Jul 17, 2025 at 02:00:47PM +0200, Wolfram Sang wrote:
> LKP found a random config which failed to build because IO accessors
> were not defined:
> 
>    In file included from drivers/i3c/master.c:21:
>    drivers/i3c/internals.h: In function 'i3c_writel_fifo':
> >> drivers/i3c/internals.h:35:9: error: implicit declaration of function 'writesl' [-Werror=implicit-function-declaration]
> 
> Add the proper header to where the IO accessors are used.
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202507150208.BZDzzJ5E-lkp@intel.com/
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Fixes: b7371ff9bdf7 ("i3c: master: Add inline i3c_readl_fifo() and i3c_writel_fifo()")


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

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

* Re: [PATCH] i3c: add missing include to internal header
  2025-07-17 12:00 [PATCH] i3c: add missing include to internal header Wolfram Sang
  2025-07-17 12:04 ` Wolfram Sang
@ 2025-07-17 18:40 ` Frank Li
  1 sibling, 0 replies; 3+ messages in thread
From: Frank Li @ 2025-07-17 18:40 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-renesas-soc, kernel test robot, Alexandre Belloni,
	linux-i3c

On Thu, Jul 17, 2025 at 02:00:47PM +0200, Wolfram Sang wrote:
> LKP found a random config which failed to build because IO accessors
> were not defined:
>
>    In file included from drivers/i3c/master.c:21:
>    drivers/i3c/internals.h: In function 'i3c_writel_fifo':
> >> drivers/i3c/internals.h:35:9: error: implicit declaration of function 'writesl' [-Werror=implicit-function-declaration]
>
> Add the proper header to where the IO accessors are used.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202507150208.BZDzzJ5E-lkp@intel.com/
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Reviewed-by: Frank Li <Frank.Li@nxp.com>

> ---
>  drivers/i3c/internals.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/i3c/internals.h b/drivers/i3c/internals.h
> index 67ed1c44f615..1bf6aecebcbb 100644
> --- a/drivers/i3c/internals.h
> +++ b/drivers/i3c/internals.h
> @@ -9,6 +9,7 @@
>  #define I3C_INTERNALS_H
>
>  #include <linux/i3c/master.h>
> +#include <linux/io.h>
>
>  extern const struct device_type i3c_masterdev_type;
>
> --
> 2.47.2
>

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

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

end of thread, other threads:[~2025-07-17 18:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-17 12:00 [PATCH] i3c: add missing include to internal header Wolfram Sang
2025-07-17 12:04 ` Wolfram Sang
2025-07-17 18:40 ` Frank Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).