On Tue, 2013-08-13 at 10:48 +0800, kbuild test robot wrote:
> tree: git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git for-next
> head: 84963bad1fc0a07fd212de265e88432c3e0dc287
> commit: 2a1b60e2fb92e084548b319dba92ed25ea498f12 [1/3] target: replace strict_strto*() with kstrto*()
> config: make ARCH=i386 allyesconfig
>
> All warnings:
>
> drivers/target/tcm_fc/tfc_conf.c: In function 'ft_add_tpg':
> >> drivers/target/tcm_fc/tfc_conf.c:317:3: warning: return makes pointer from integer without a cast [enabled by default]
> return ret;
> ^
>
> vim +317 drivers/target/tcm_fc/tfc_conf.c
>
> 301 struct ft_lport_acl *lacl;
> 302 struct ft_tpg *tpg;
> 303 struct workqueue_struct *wq;
> 304 unsigned long index;
> 305 int ret;
> 306
> 307 pr_debug("tcm_fc: add tpg %s\n", name);
> 308
> 309 /*
> 310 * Name must be "tpgt_" followed by the index.
> 311 */
> 312 if (strstr(name, "tpgt_") != name)
> 313 return NULL;
> 314
> 315 ret = kstrtoul(name + 5, 10, &index);
> 316 if (ret)
> > 317 return ret;
Fixing this up now..
Thanks Fengguang!
--nab
> 318 if (index > UINT_MAX)
> 319 return NULL;
> 320
> 321 lacl = container_of(wwn, struct ft_lport_acl, fc_lport_wwn);
> 322 tpg = kzalloc(sizeof(*tpg), GFP_KERNEL);
> 323 if (!tpg)
> 324 return NULL;
> 325 tpg->index = index;
>
> ---
> 0-DAY kernel build testing backend Open Source Technology Center
> http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
