On 09/19/2012 06:31 PM, Fengguang Wu wrote:
> Hi Lars-Peter,
>
> FYI, there are new coccinelle warnings show up in
>
> tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-next
> head: 5d06e3df280bd230e2eadc16372e62818c63e894
> commit: a0e545e0e75006a7de0e9bc5397f6b44c61990b2 [86/151] staging:iio:hwmon bridge: Use iio_read_channel_processed
Hi,
Thanks for info, patch looks good, but the warning is not introduced by the
commit you mention, though.
- Lars
>
> All coccinelle warnings:
>
> + drivers/staging/iio/iio_hwmon.c:96:21-27: ERROR: application of sizeof to pointer
>
> vim +96 drivers/staging/iio/iio_hwmon.c
> 86 st->channels = iio_channel_get_all(dev_name(&pdev->dev));
> 87 if (IS_ERR(st->channels)) {
> 88 ret = PTR_ERR(st->channels);
> 89 goto error_free_state;
> 90 }
> 91
> 92 /* count how many attributes we have */
> 93 while (st->channels[st->num_channels].indio_dev)
> 94 st->num_channels++;
> 95
> > 96 st->attrs = kzalloc(sizeof(st->attrs) * (st->num_channels + 1),
> 97 GFP_KERNEL);
> 98 if (st->attrs == NULL) {
> 99 ret = -ENOMEM;
> 100 goto error_release_channels;
> 101 }
> 102 for (i = 0; i < st->num_channels; i++) {
> 103 a = kzalloc(sizeof(*a), GFP_KERNEL);
> 104 if (a == NULL) {
> 105 ret = -ENOMEM;
> 106 goto error_free_attrs;
>
> Please consider folding the attached diff :-)
>
> ---
> 0-DAY kernel build testing backend Open Source Technology Centre
> Fengguang Wu, Yuanhan Liu Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
