| Message ID | 20241014144821.15094-3-afd@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Enable compile testing of TI MSGMGR/SPROXY driver | expand |
On Mon, Oct 14, 2024, at 14:48, Andrew Davis wrote: > The TI message manager driver can be compiled without ARCH_KEYSTONE > nor ARCH_K3 enabled. Allow it to be built under COMPILE_TEST. > > Signed-off-by: Andrew Davis <afd@ti.com> Acked-by: Arnd Bergmann <arnd@arndb.de>
Hi Andrew,
kernel test robot noticed the following build warnings:
[auto build test WARNING on soc/for-next]
[also build test WARNING on linus/master v6.12-rc3 next-20241015]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Andrew-Davis/mailbox-ti-msgmgr-Remove-use-of-of_match_ptr-helper/20241014-225353
base: https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git for-next
patch link: https://lore.kernel.org/r/20241014144821.15094-3-afd%40ti.com
patch subject: [PATCH 2/2] mailbox: ti-msgmgr: Allow building under COMPILE_TEST
config: hexagon-randconfig-r131-20241015 (https://download.01.org/0day-ci/archive/20241016/202410160435.WzzQa1p5-lkp@intel.com/config)
compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 70e0a7e7e6a8541bcc46908c592eed561850e416)
reproduce: (https://download.01.org/0day-ci/archive/20241016/202410160435.WzzQa1p5-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410160435.WzzQa1p5-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from drivers/firmware/ti_sci.c:14:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:548:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
548 | val = __raw_readb(PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:561:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
561 | val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
37 | #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
| ^
In file included from drivers/firmware/ti_sci.c:14:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:574:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
574 | val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
35 | #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
| ^
In file included from drivers/firmware/ti_sci.c:14:
In file included from include/linux/io.h:14:
In file included from arch/hexagon/include/asm/io.h:328:
include/asm-generic/io.h:585:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
585 | __raw_writeb(value, PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:595:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
595 | __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:605:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
605 | __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
drivers/firmware/ti_sci.c:200:20: warning: unused function 'ti_sci_debugfs_destroy' [-Wunused-function]
200 | static inline void ti_sci_debugfs_destroy(struct platform_device *dev,
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/firmware/ti_sci.c:3285:34: warning: unused variable 'ti_sci_of_match' [-Wunused-const-variable]
3285 | static const struct of_device_id ti_sci_of_match[] = {
| ^~~~~~~~~~~~~~~
8 warnings generated.
vim +/ti_sci_of_match +3285 drivers/firmware/ti_sci.c
aa276781a64a5f Nishanth Menon 2016-10-18 3284
aa276781a64a5f Nishanth Menon 2016-10-18 @3285 static const struct of_device_id ti_sci_of_match[] = {
aa276781a64a5f Nishanth Menon 2016-10-18 3286 {.compatible = "ti,k2g-sci", .data = &ti_sci_pmmc_k2g_desc},
754c9477ae7872 Peter Ujfalusi 2019-04-30 3287 {.compatible = "ti,am654-sci", .data = &ti_sci_pmmc_am654_desc},
aa276781a64a5f Nishanth Menon 2016-10-18 3288 { /* Sentinel */ },
aa276781a64a5f Nishanth Menon 2016-10-18 3289 };
aa276781a64a5f Nishanth Menon 2016-10-18 3290 MODULE_DEVICE_TABLE(of, ti_sci_of_match);
aa276781a64a5f Nishanth Menon 2016-10-18 3291
Hi Andrew, On Tue, Oct 15, 2024, at 20:46, kernel test robot wrote: > In file included from drivers/firmware/ti_sci.c:14: > In file included from include/linux/io.h:14: > In file included from arch/hexagon/include/asm/io.h:328: > include/asm-generic/io.h:548:31: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > 548 | val = __raw_readb(PCI_IOBASE + addr); > | ~~~~~~~~~~ ^ > include/asm-generic/io.h:561:61: warning: performing pointer > arithmetic on a null pointer has undefined behavior > [-Wnull-pointer-arithmetic] > 561 | val = __le16_to_cpu((__le16 > __force)__raw_readw(PCI_IOBASE + addr)); > | Hopefully we will be able to finally address these in 6.13 for all drivers, it's been a decade. Don't worry about the above. > platform_device *dev, > | ^~~~~~~~~~~~~~~~~~~~~~ >>> drivers/firmware/ti_sci.c:3285:34: warning: unused variable 'ti_sci_of_match' [-Wunused-const-variable] > 3285 | static const struct of_device_id ti_sci_of_match[] = { > | ^~~~~~~~~~~~~~~ > 8 warnings generated. > Needs this change, please fold into your patch. diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c index 160968301b1f..6a6bf7a2d134 100644 --- a/drivers/firmware/ti_sci.c +++ b/drivers/firmware/ti_sci.c @@ -3421,7 +3421,7 @@ static struct platform_driver ti_sci_driver = { .probe = ti_sci_probe, .driver = { .name = "ti-sci", - .of_match_table = of_match_ptr(ti_sci_of_match), + .of_match_table = ti_sci_of_match, .suppress_bind_attrs = true, }, };
On 10/15/24 3:54 PM, Arnd Bergmann wrote: > Hi Andrew, > > On Tue, Oct 15, 2024, at 20:46, kernel test robot wrote: >> In file included from drivers/firmware/ti_sci.c:14: >> In file included from include/linux/io.h:14: >> In file included from arch/hexagon/include/asm/io.h:328: >> include/asm-generic/io.h:548:31: warning: performing pointer >> arithmetic on a null pointer has undefined behavior >> [-Wnull-pointer-arithmetic] >> 548 | val = __raw_readb(PCI_IOBASE + addr); >> | ~~~~~~~~~~ ^ >> include/asm-generic/io.h:561:61: warning: performing pointer >> arithmetic on a null pointer has undefined behavior >> [-Wnull-pointer-arithmetic] >> 561 | val = __le16_to_cpu((__le16 >> __force)__raw_readw(PCI_IOBASE + addr)); >> | > > > Hopefully we will be able to finally address these in 6.13 > for all drivers, it's been a decade. Don't worry about > the above. > > >> platform_device *dev, >> | ^~~~~~~~~~~~~~~~~~~~~~ >>>> drivers/firmware/ti_sci.c:3285:34: warning: unused variable 'ti_sci_of_match' [-Wunused-const-variable] >> 3285 | static const struct of_device_id ti_sci_of_match[] = { >> | ^~~~~~~~~~~~~~~ >> 8 warnings generated. >> > > Needs this change, please fold into your patch. Was going to clean this up in several places in another series, but will add this one to this series, v2 on the way. Andrew > > diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c > index 160968301b1f..6a6bf7a2d134 100644 > --- a/drivers/firmware/ti_sci.c > +++ b/drivers/firmware/ti_sci.c > @@ -3421,7 +3421,7 @@ static struct platform_driver ti_sci_driver = { > .probe = ti_sci_probe, > .driver = { > .name = "ti-sci", > - .of_match_table = of_match_ptr(ti_sci_of_match), > + .of_match_table = ti_sci_of_match, > .suppress_bind_attrs = true, > }, > }; > >
diff --git a/drivers/mailbox/Kconfig b/drivers/mailbox/Kconfig index 6fb995778636a..af721d769fdf2 100644 --- a/drivers/mailbox/Kconfig +++ b/drivers/mailbox/Kconfig @@ -127,7 +127,7 @@ config STI_MBOX config TI_MESSAGE_MANAGER tristate "Texas Instruments Message Manager Driver" - depends on ARCH_KEYSTONE || ARCH_K3 + depends on ARCH_KEYSTONE || ARCH_K3 || COMPILE_TEST default ARCH_K3 help An implementation of Message Manager slave driver for Keystone
The TI message manager driver can be compiled without ARCH_KEYSTONE nor ARCH_K3 enabled. Allow it to be built under COMPILE_TEST. Signed-off-by: Andrew Davis <afd@ti.com> --- drivers/mailbox/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)