Re: [PATCH 1/2] iio: Don't include extended name in shared attributes

[Jonathan Cameron <jic23@kernel.org>]

On 14/02/14 14:19, Lars-Peter Clausen wrote:
> The extended name is channel specific and should not be included in shared
> attributes.
>
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Applied to the togreg branch of iio.git

> ---
> It's a fix, but there are no drivers yet, which have channels with extended
> names and shared attributes, so no need to apply it to the fixes branch.
Thanks for making this clear - saves me checking ;)
> ---
>   drivers/iio/industrialio-core.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> index acc911a..fd96e8b 100644
> --- a/drivers/iio/industrialio-core.c
> +++ b/drivers/iio/industrialio-core.c
> @@ -558,7 +558,7 @@ int __iio_device_attr_init(struct device_attribute *dev_attr,
>   								    ->channel2],
>   						 postfix);
>   	} else {
> -		if (chan->extend_name == NULL)
> +		if (chan->extend_name == NULL || shared_by != IIO_SEPARATE)
>   			full_postfix = kstrdup(postfix, GFP_KERNEL);
>   		else
>   			full_postfix = kasprintf(GFP_KERNEL,
>