lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202211302210.Q80Fz0GM-lkp@intel.com>
Date:   Wed, 30 Nov 2022 22:34:30 +0800
From:   kernel test robot <lkp@...el.com>
To:     JaimeLiao <jaimeliao.tw@...il.com>, broonie@...nel.org,
        krzysztof.kozlowski@...aro.org, linux-spi@...r.kernel.org,
        linux-kernel@...r.kernel.org
Cc:     llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
        jaimeliao@...c.com.tw, JaimeLiao <jaimeliao.tw@...il.com>
Subject: Re: [PATCH v1 2/2] memory: renesas-rpc-if: Add Octal DTR mode support

Hi JaimeLiao,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on broonie-spi/for-next]
[also build test WARNING on krzk-mem-ctrl/for-next linus/master v6.1-rc7 next-20221130]
[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/JaimeLiao/Add-Octal-DTR-support-on-RPC-driver/20221124-154211
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next
patch link:    https://lore.kernel.org/r/20221124073957.22471-3-jaimeliao.tw%40gmail.com
patch subject: [PATCH v1 2/2] memory: renesas-rpc-if: Add Octal DTR mode support
config: hexagon-randconfig-r004-20221128
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 6e4cea55f0d1104408b26ac574566a0e4de48036)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/61bae8cb17ad5715386ee18a1d06549843775cd0
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review JaimeLiao/Add-Octal-DTR-support-on-RPC-driver/20221124-154211
        git checkout 61bae8cb17ad5715386ee18a1d06549843775cd0
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/memory/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   In file included from drivers/memory/renesas-rpc-if.c:11:
   In file included from include/linux/io.h:13:
   In file included from arch/hexagon/include/asm/io.h:334:
   include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           val = __raw_readb(PCI_IOBASE + addr);
                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           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'
   #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                     ^
   In file included from drivers/memory/renesas-rpc-if.c:11:
   In file included from include/linux/io.h:13:
   In file included from arch/hexagon/include/asm/io.h:334:
   include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           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'
   #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
                                                     ^
   In file included from drivers/memory/renesas-rpc-if.c:11:
   In file included from include/linux/io.h:13:
   In file included from arch/hexagon/include/asm/io.h:334:
   include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writeb(value, PCI_IOBASE + addr);
                               ~~~~~~~~~~ ^
   include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
   include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
                                                         ~~~~~~~~~~ ^
>> drivers/memory/renesas-rpc-if.c:484:30: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand]
                   if (op->ocmd.buswidth == 8 && RPCIF_SMENR_OCDE)
                                              ^  ~~~~~~~~~~~~~~~~
   drivers/memory/renesas-rpc-if.c:484:30: note: use '&' for a bitwise operation
                   if (op->ocmd.buswidth == 8 && RPCIF_SMENR_OCDE)
                                              ^~
                                              &
   drivers/memory/renesas-rpc-if.c:484:30: note: remove constant to silence this warning
                   if (op->ocmd.buswidth == 8 && RPCIF_SMENR_OCDE)
                                             ~^~~~~~~~~~~~~~~~~~~
   7 warnings generated.


vim +484 drivers/memory/renesas-rpc-if.c

   381	
   382	void rpcif_prepare(struct rpcif *rpc, const struct rpcif_op *op, u64 *offs,
   383			   size_t *len)
   384	{
   385		rpc->smcr = 0;
   386		rpc->smadr = 0;
   387		rpc->enable = 0;
   388		rpc->command = 0;
   389		rpc->option = 0;
   390		rpc->dummy = 0;
   391		rpc->ddr = 0;
   392		rpc->xferlen = 0;
   393	
   394		if (op->cmd.buswidth) {
   395			rpc->enable  = RPCIF_SMENR_CDE |
   396				RPCIF_SMENR_CDB(rpcif_bit_size(op->cmd.buswidth));
   397			rpc->command = RPCIF_SMCMR_CMD(op->cmd.opcode);
   398			if (op->cmd.ddr)
   399				rpc->ddr = RPCIF_SMDRENR_HYPE(0x5);
   400		}
   401		if (op->ocmd.buswidth) {
   402			rpc->enable  |= RPCIF_SMENR_OCDE |
   403				RPCIF_SMENR_OCDB(rpcif_bit_size(op->ocmd.buswidth));
   404			rpc->command |= RPCIF_SMCMR_OCMD(op->ocmd.opcode);
   405		}
   406	
   407		if (op->addr.buswidth) {
   408			rpc->enable |=
   409				RPCIF_SMENR_ADB(rpcif_bit_size(op->addr.buswidth));
   410			if (op->addr.nbytes == 4)
   411				rpc->enable |= RPCIF_SMENR_ADE(0xF);
   412			else
   413				rpc->enable |= RPCIF_SMENR_ADE(GENMASK(
   414							2, 3 - op->addr.nbytes));
   415			if (op->addr.ddr)
   416				rpc->ddr |= RPCIF_SMDRENR_ADDRE;
   417	
   418			if (offs && len)
   419				rpc->smadr = *offs;
   420			else
   421				rpc->smadr = op->addr.val;
   422		}
   423	
   424		if (op->dummy.buswidth) {
   425			rpc->enable |= RPCIF_SMENR_DME;
   426			rpc->dummy = RPCIF_SMDMCR_DMCYC(op->dummy.ncycles /
   427							op->dummy.buswidth);
   428		}
   429	
   430		if (op->option.buswidth) {
   431			rpc->enable |= RPCIF_SMENR_OPDE(
   432				rpcif_bits_set(rpc, op->option.nbytes)) |
   433				RPCIF_SMENR_OPDB(rpcif_bit_size(op->option.buswidth));
   434			if (op->option.ddr)
   435				rpc->ddr |= RPCIF_SMDRENR_OPDRE;
   436			rpc->option = op->option.val;
   437		}
   438	
   439		rpc->dir = op->data.dir;
   440		if (op->data.buswidth) {
   441			u32 nbytes;
   442	
   443			rpc->buffer = op->data.buf.in;
   444			switch (op->data.dir) {
   445			case RPCIF_DATA_IN:
   446				rpc->smcr = RPCIF_SMCR_SPIRE;
   447				break;
   448			case RPCIF_DATA_OUT:
   449				rpc->smcr = RPCIF_SMCR_SPIWE;
   450				break;
   451			default:
   452				break;
   453			}
   454			if (op->data.ddr)
   455				rpc->ddr |= RPCIF_SMDRENR_SPIDRE;
   456	
   457			if (offs && len)
   458				nbytes = *len;
   459			else
   460				nbytes = op->data.nbytes;
   461			rpc->xferlen = nbytes;
   462	
   463			rpc->enable |= RPCIF_SMENR_SPIDB(rpcif_bit_size(op->data.buswidth));
   464		}
   465	       /* Fixup in Octal DTR mode */
   466		if (op->cmd.buswidth == 8 && op->cmd.ddr) {
   467			rpc->bus_size = 2;
   468	
   469			regmap_update_bits(rpc->regmap, RPCIF_PHYCNT,
   470					   RPCIF_PHYCNT_OCTA(0x2) | RPCIF_PHYCNT_OCT |
   471					   RPCIF_PHYCNT_PHYMEM(0x1) ,
   472					   RPCIF_PHYCNT_OCTA(0x2) | RPCIF_PHYCNT_OCT |
   473					   RPCIF_PHYCNT_PHYMEM(0x1));
   474			regmap_update_bits(rpc->regmap, RPCIF_CMNCR, RPCIF_CMNCR_BSZ(1),
   475					   RPCIF_CMNCR_BSZ(1));
   476	
   477			regmap_update_bits(rpc->regmap, RPCIF_PHYOFFSET1,
   478					   RPCIF_PHYOFFSET1_DDRTMG(3),
   479					   RPCIF_PHYOFFSET1_DDRTMG(2));
   480			regmap_update_bits(rpc->regmap, RPCIF_PHYOFFSET2,
   481					   RPCIF_PHYOFFSET2_OCTTMG(7),
   482					   RPCIF_PHYOFFSET2_OCTTMG(3));
   483	
 > 484			if (op->ocmd.buswidth == 8 && RPCIF_SMENR_OCDE)
   485				rpc->enable &= ~RPCIF_SMENR_OCDE;
   486	
   487			if (op->addr.ddr)
   488				rpc->enable &= ~RPCIF_SMENR_ADE(0xF) |
   489						RPCIF_SMENR_ADE(0xc);
   490	
   491			if (op->dummy.buswidth == 8)
   492				rpc->dummy =
   493					RPCIF_SMDMCR_DMCYC(op->dummy.ncycles / 2);
   494	
   495			if (op->data.dir == RPCIF_DATA_IN)
   496				/* Set Extenal Address space Read mode */
   497				if (op->data.buswidth == 8 && op->data.ddr) {
   498					regmap_update_bits(rpc->regmap, RPCIF_PHYCNT,
   499						RPCIF_PHYCNT_EXDS, RPCIF_PHYCNT_EXDS);
   500				}
   501		}
   502	}
   503	EXPORT_SYMBOL(rpcif_prepare);
   504	

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

View attachment "config" of type "text/plain" (135063 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ