lkml.org 
[lkml]   [2017]   [May]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mfd: cros-ec: Add cros_ec_readmem() helpers for I2C/SPI based ECs
On Tue, May 09, 2017 at 09:34:56AM -0700, Moritz Fischer wrote:
> On Tue, May 09, 2017 at 01:56:14PM +0800, kbuild test robot wrote:
> > Hi Moritz,
> >
> > [auto build test WARNING on ljones-mfd/for-mfd-next]
> > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
> >
> > url: https://github.com/0day-ci/linux/commits/Moritz-Fischer/mfd-cros-ec-Add-cros_ec_readmem-helpers-for-I2C-SPI-based-ECs/20170509-040606
> > base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
> > config: tile-allmodconfig
> > compiler: tilegx-linux-gcc (GCC) 4.6.2
> > reproduce:
> > wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > chmod +x ~/bin/make.cross
> > make.cross ARCH=tile allmodconfig
> > make.cross ARCH=tile
>
> I just locally ran that and it wouldn't print a warning. False positive?
>
Good question. This is an unusual report - normally the actual warning ...

> >
> > All warnings (new ones prefixed by >>):
> >
... would be listed here, but is missing.

Guenter

> >
> > vim +279 drivers/mfd/cros_ec.c
> >
> > 263 return 0;
> > 264 }
> > 265 EXPORT_SYMBOL(cros_ec_resume);
> > 266
> > 267 #endif
> > 268
> > 269 int cros_ec_readmem(struct cros_ec_device *ec, unsigned int offset,
> > 270 unsigned int bytes, void *dest)
> > 271 {
> > 272 int ret;
> > 273 struct ec_params_read_memmap *params;
> > 274 struct cros_ec_command *msg;
> > 275
> > 276 if (offset >= EC_MEMMAP_SIZE - bytes)
> > 277 return -EINVAL;
> > 278
> > > 279 msg = kzalloc(sizeof(*msg) + max(sizeof(*params), bytes), GFP_KERNEL);
>
> What am I not seeing here?
> > 280 if (!msg)
> > 281 return -ENOMEM;
> > 282
> > 283 msg->version = 0;
> > 284 msg->command = EC_CMD_READ_MEMMAP;
> > 285 msg->insize = bytes;
> > 286 msg->outsize = sizeof(*params);
> > 287
> >
> > ---
> > 0-DAY kernel test infrastructure Open Source Technology Center
> > https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
> Thanks,
>
> Moritz


\
 
 \ /
  Last update: 2017-05-10 21:19    [W:0.062 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site