All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
@ 2016-01-09  0:17 tim.gardner
  2016-01-09 16:00 ` Jonathan Cameron
  0 siblings, 1 reply; 6+ messages in thread
From: tim.gardner @ 2016-01-09  0:17 UTC (permalink / raw)
  To: linux-iio, linux-kernel
  Cc: Tim Gardner, Jonathan Cameron, Hartmut Knaack,
	Lars-Peter Clausen, Peter Meerwald, Srinivas Pandruvada

From: Tim Gardner <tim.gardner@canonical.com>

drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset' may be used uninitialized in this function [-Wmaybe-uninitialized]
  data->def = &ak_def_array[chipset];

gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)

Cc: Jonathan Cameron <jic23@kernel.org>
Cc: Hartmut Knaack <knaack.h@gmx.de>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Peter Meerwald <pmeerw@pmeerw.net>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
---

This seems like a legitimate warning, though gcc should have complained
about an earlier use of chipset on line 782.

 drivers/iio/magnetometer/ak8975.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/magnetometer/ak8975.c b/drivers/iio/magnetometer/ak8975.c
index b13936d..80ec0ce 100644
--- a/drivers/iio/magnetometer/ak8975.c
+++ b/drivers/iio/magnetometer/ak8975.c
@@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client *client,
 	int eoc_gpio;
 	int err;
 	const char *name = NULL;
-	enum asahi_compass_chipset chipset;
+	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
 
 	/* Grab and set up the supplied GPIO. */
 	if (client->dev.platform_data)
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
  2016-01-09  0:17 [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning tim.gardner
@ 2016-01-09 16:00 ` Jonathan Cameron
  2016-01-09 16:17   ` Srinivas Pandruvada
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2016-01-09 16:00 UTC (permalink / raw)
  To: tim.gardner, linux-iio, linux-kernel
  Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, Srinivas Pandruvada

On 09/01/16 00:17, tim.gardner@canonical.com wrote:
> From: Tim Gardner <tim.gardner@canonical.com>
> 
> drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
> drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset' may be used uninitialized in this function [-Wmaybe-uninitialized]
>   data->def = &ak_def_array[chipset];
> 
> gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)
> 
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Hartmut Knaack <knaack.h@gmx.de>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Peter Meerwald <pmeerw@pmeerw.net>
> Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
Doesn't look to be an actual bug as we either end up with chipset being filled
based on the traditional match table in which case it'll be assigned
or based on the acpi match, which should succeed seeing as we've already
had to have matched one or the other for the probe to match in the first place.

So probably worth the change to make it easier to tell that it should be fine
and suppress the warning.  However, whilst we are here, I note that
*match_acpi_table has a path which returns NULL as the name and doesn't assign
the chipset.  We should be therefore checking if (!name) return -ENOSYS;
Though maybe another error code would be more appropriate.

Not sure that error path can actually happen either, but if we are going to
bother having the error path out of match_acpi_table then we ought to actually
handle it!

Don't suppose you'd mind fixing that one as well whilst here?

Jonathan 
> ---
> 
> This seems like a legitimate warning, though gcc should have complained
> about an earlier use of chipset on line 782.
> 
>  drivers/iio/magnetometer/ak8975.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/magnetometer/ak8975.c b/drivers/iio/magnetometer/ak8975.c
> index b13936d..80ec0ce 100644
> --- a/drivers/iio/magnetometer/ak8975.c
> +++ b/drivers/iio/magnetometer/ak8975.c
> @@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client *client,
>  	int eoc_gpio;
>  	int err;
>  	const char *name = NULL;
> -	enum asahi_compass_chipset chipset;
> +	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
>  
>  	/* Grab and set up the supplied GPIO. */
>  	if (client->dev.platform_data)
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
  2016-01-09 16:00 ` Jonathan Cameron
@ 2016-01-09 16:17   ` Srinivas Pandruvada
  2016-01-09 16:25     ` Jonathan Cameron
  0 siblings, 1 reply; 6+ messages in thread
From: Srinivas Pandruvada @ 2016-01-09 16:17 UTC (permalink / raw)
  To: Jonathan Cameron, tim.gardner, linux-iio, linux-kernel
  Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald

On Sat, 2016-01-09 at 16:00 +0000, Jonathan Cameron wrote:
> On 09/01/16 00:17, tim.gardner@canonical.com wrote:
> > From: Tim Gardner <tim.gardner@canonical.com>
> > 
> > drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
> > drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset' may be
> > used uninitialized in this function [-Wmaybe-uninitialized]
> >   data->def = &ak_def_array[chipset];
> > 
> > gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)
> > 
> > Cc: Jonathan Cameron <jic23@kernel.org>
> > Cc: Hartmut Knaack <knaack.h@gmx.de>
> > Cc: Lars-Peter Clausen <lars@metafoo.de>
> > Cc: Peter Meerwald <pmeerw@pmeerw.net>
> > Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> > Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> Doesn't look to be an actual bug as we either end up with chipset
> being filled
> based on the traditional match table in which case it'll be assigned
> or based on the acpi match, which should succeed seeing as we've
> already
> had to have matched one or the other for the probe to match in the
> first place.
> 
> So probably worth the change to make it easier to tell that it should
> be fine
> and suppress the warning.  However, whilst we are here, I note that
> *match_acpi_table has a path which returns NULL as the name and
> doesn't assign
> the chipset.  We should be therefore checking if (!name) return 
> -ENOSYS;
> Though maybe another error code would be more appropriate.
> 

Since in this case we are enumerated by ACPI using our match table, so
name can't be null. The "name" we provided in 
static const struct acpi_device_id ak_acpi_match[] = {..}
Same with the *chipset. Other than suppress warnings, I don't think it
will cause any real issue.

Thanks,
Srinivas 

> Not sure that error path can actually happen either, but if we are
> going to
> bother having the error path out of match_acpi_table then we ought to
> actually
> handle it!
> 
> Don't suppose you'd mind fixing that one as well whilst here?
> 
> Jonathan 
> > ---
> > 
> > This seems like a legitimate warning, though gcc should have
> > complained
> > about an earlier use of chipset on line 782.
> > 
> >  drivers/iio/magnetometer/ak8975.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/iio/magnetometer/ak8975.c
> > b/drivers/iio/magnetometer/ak8975.c
> > index b13936d..80ec0ce 100644
> > --- a/drivers/iio/magnetometer/ak8975.c
> > +++ b/drivers/iio/magnetometer/ak8975.c
> > @@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client
> > *client,
> >  	int eoc_gpio;
> >  	int err;
> >  	const char *name = NULL;
> > -	enum asahi_compass_chipset chipset;
> > +	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
> >  
> >  	/* Grab and set up the supplied GPIO. */
> >  	if (client->dev.platform_data)
> > 
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
  2016-01-09 16:17   ` Srinivas Pandruvada
@ 2016-01-09 16:25     ` Jonathan Cameron
  2016-01-09 16:51       ` Srinivas Pandruvada
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2016-01-09 16:25 UTC (permalink / raw)
  To: Srinivas Pandruvada, tim.gardner, linux-iio, linux-kernel
  Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald

On 09/01/16 16:17, Srinivas Pandruvada wrote:
> On Sat, 2016-01-09 at 16:00 +0000, Jonathan Cameron wrote:
>> On 09/01/16 00:17, tim.gardner@canonical.com wrote:
>>> From: Tim Gardner <tim.gardner@canonical.com>
>>>
>>> drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
>>> drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset' may be
>>> used uninitialized in this function [-Wmaybe-uninitialized]
>>>   data->def = &ak_def_array[chipset];
>>>
>>> gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)
>>>
>>> Cc: Jonathan Cameron <jic23@kernel.org>
>>> Cc: Hartmut Knaack <knaack.h@gmx.de>
>>> Cc: Lars-Peter Clausen <lars@metafoo.de>
>>> Cc: Peter Meerwald <pmeerw@pmeerw.net>
>>> Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
>>> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
>> Doesn't look to be an actual bug as we either end up with chipset
>> being filled
>> based on the traditional match table in which case it'll be assigned
>> or based on the acpi match, which should succeed seeing as we've
>> already
>> had to have matched one or the other for the probe to match in the
>> first place.
>>
>> So probably worth the change to make it easier to tell that it should
>> be fine
>> and suppress the warning.  However, whilst we are here, I note that
>> *match_acpi_table has a path which returns NULL as the name and
>> doesn't assign
>> the chipset.  We should be therefore checking if (!name) return 
>> -ENOSYS;
>> Though maybe another error code would be more appropriate.
>>
> 
> Since in this case we are enumerated by ACPI using our match table, so
> name can't be null. The "name" we provided in 
> static const struct acpi_device_id ak_acpi_match[] = {..}
> Same with the *chipset. Other than suppress warnings, I don't think it
> will cause any real issue.
True enough, in which case why are we checking the name?
I'd be included to drop that check and add a comment.
I haven't chased every path, but I think that might deal with the above
warning at it's root.
> 
> Thanks,
> Srinivas 
> 
>> Not sure that error path can actually happen either, but if we are
>> going to
>> bother having the error path out of match_acpi_table then we ought to
>> actually
>> handle it!
>>
>> Don't suppose you'd mind fixing that one as well whilst here?
>>
>> Jonathan 
>>> ---
>>>
>>> This seems like a legitimate warning, though gcc should have
>>> complained
>>> about an earlier use of chipset on line 782.
>>>
>>>  drivers/iio/magnetometer/ak8975.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/iio/magnetometer/ak8975.c
>>> b/drivers/iio/magnetometer/ak8975.c
>>> index b13936d..80ec0ce 100644
>>> --- a/drivers/iio/magnetometer/ak8975.c
>>> +++ b/drivers/iio/magnetometer/ak8975.c
>>> @@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client
>>> *client,
>>>  	int eoc_gpio;
>>>  	int err;
>>>  	const char *name = NULL;
>>> -	enum asahi_compass_chipset chipset;
>>> +	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
>>>  
>>>  	/* Grab and set up the supplied GPIO. */
>>>  	if (client->dev.platform_data)
>>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
  2016-01-09 16:25     ` Jonathan Cameron
@ 2016-01-09 16:51       ` Srinivas Pandruvada
  2016-01-09 16:59         ` Jonathan Cameron
  0 siblings, 1 reply; 6+ messages in thread
From: Srinivas Pandruvada @ 2016-01-09 16:51 UTC (permalink / raw)
  To: Jonathan Cameron, tim.gardner, linux-iio, linux-kernel
  Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald

On Sat, 2016-01-09 at 16:25 +0000, Jonathan Cameron wrote:
> 
On 09/01/16 16:17, Srinivas Pandruvada wrote:
> > On Sat, 2016-01-09 at 16:00 +0000, Jonathan Cameron wrote:
> > > On 09/01/16 00:17, tim.gardner@canonical.com wrote:
> > > > From: Tim Gardner <tim.gardner@canonical.com>
> > > > 
> > > > drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
> > > > drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset'
> > > > may be
> > > > used uninitialized in this function [-Wmaybe-uninitialized]
> > > >   data->def = &ak_def_array[chipset];
> > > > 
> > > > gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)
> > > > 
> > > > Cc: Jonathan Cameron <jic23@kernel.org>
> > > > Cc: Hartmut Knaack <knaack.h@gmx.de>
> > > > Cc: Lars-Peter Clausen <lars@metafoo.de>
> > > > Cc: Peter Meerwald <pmeerw@pmeerw.net>
> > > > Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> > > > Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> > > Doesn't look to be an actual bug as we either end up with chipset
> > > being filled
> > > based on the traditional match table in which case it'll be
> > > assigned
> > > or based on the acpi match, which should succeed seeing as we've
> > > already
> > > had to have matched one or the other for the probe to match in
> > > the
> > > first place.
> > > 
> > > So probably worth the change to make it easier to tell that it
> > > should
> > > be fine
> > > and suppress the warning.  However, whilst we are here, I note
> > > that
> > > *match_acpi_table has a path which returns NULL as the name and
> > > doesn't assign
> > > the chipset.  We should be therefore checking if (!name) return 
> > > -ENOSYS;
> > > Though maybe another error code would be more appropriate.
> > > 
> > 
> > Since in this case we are enumerated by ACPI using our match table,
> > so
> > name can't be null. The "name" we provided in 
> > static const struct acpi_device_id ak_acpi_match[] = {..}
> > Same with the *chipset. Other than suppress warnings, I don't think
> > it
> > will cause any real issue.
> True enough, in which case why are we checking the name?

We can remove this check for !id
  id = acpi_match_device(dev->driver->acpi_match_table, dev);
- if (!id)
-	return NULL;
  *chipset = (int)id->driver_data;

Thanks,
Srinivas


> I'd be included to drop that check and add a comment.
> I haven't chased every path, but I think that might deal with the
> above
> warning at it's root.
> > Thanks,
> > Srinivas 
> > 
> > > Not sure that error path can actually happen either, but if we
> > > are
> > > going to
> > > bother having the error path out of match_acpi_table then we
> > > ought to
> > > actually
> > > handle it!
> > > 
> > > Don't suppose you'd mind fixing that one as well whilst here?
> > > 
> > > Jonathan 
> > > > ---
> > > > 
> > > > This seems like a legitimate warning, though gcc should have
> > > > complained
> > > > about an earlier use of chipset on line 782.
> > > > 
> > > >  drivers/iio/magnetometer/ak8975.c | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > 
> > > > diff --git a/drivers/iio/magnetometer/ak8975.c
> > > > b/drivers/iio/magnetometer/ak8975.c
> > > > index b13936d..80ec0ce 100644
> > > > --- a/drivers/iio/magnetometer/ak8975.c
> > > > +++ b/drivers/iio/magnetometer/ak8975.c
> > > > @@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client
> > > > *client,
> > > >  	int eoc_gpio;
> > > >  	int err;
> > > >  	const char *name = NULL;
> > > > -	enum asahi_compass_chipset chipset;
> > > > +	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
> > > >  
> > > >  	/* Grab and set up the supplied GPIO. */
> > > >  	if (client->dev.platform_data)
> > > > 
> > > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux
> > -iio" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning
  2016-01-09 16:51       ` Srinivas Pandruvada
@ 2016-01-09 16:59         ` Jonathan Cameron
  0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2016-01-09 16:59 UTC (permalink / raw)
  To: Srinivas Pandruvada, tim.gardner, linux-iio, linux-kernel
  Cc: Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald

On 09/01/16 16:51, Srinivas Pandruvada wrote:
> On Sat, 2016-01-09 at 16:25 +0000, Jonathan Cameron wrote:
>>
> On 09/01/16 16:17, Srinivas Pandruvada wrote:
>>> On Sat, 2016-01-09 at 16:00 +0000, Jonathan Cameron wrote:
>>>> On 09/01/16 00:17, tim.gardner@canonical.com wrote:
>>>>> From: Tim Gardner <tim.gardner@canonical.com>
>>>>>
>>>>> drivers/iio/magnetometer/ak8975.c: In function 'ak8975_probe':
>>>>> drivers/iio/magnetometer/ak8975.c:788:14: warning: 'chipset'
>>>>> may be
>>>>> used uninitialized in this function [-Wmaybe-uninitialized]
>>>>>   data->def = &ak_def_array[chipset];
>>>>>
>>>>> gcc version 5.3.1 20151219 (Ubuntu 5.3.1-4ubuntu1)
>>>>>
>>>>> Cc: Jonathan Cameron <jic23@kernel.org>
>>>>> Cc: Hartmut Knaack <knaack.h@gmx.de>
>>>>> Cc: Lars-Peter Clausen <lars@metafoo.de>
>>>>> Cc: Peter Meerwald <pmeerw@pmeerw.net>
>>>>> Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
>>>>> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
>>>> Doesn't look to be an actual bug as we either end up with chipset
>>>> being filled
>>>> based on the traditional match table in which case it'll be
>>>> assigned
>>>> or based on the acpi match, which should succeed seeing as we've
>>>> already
>>>> had to have matched one or the other for the probe to match in
>>>> the
>>>> first place.
>>>>
>>>> So probably worth the change to make it easier to tell that it
>>>> should
>>>> be fine
>>>> and suppress the warning.  However, whilst we are here, I note
>>>> that
>>>> *match_acpi_table has a path which returns NULL as the name and
>>>> doesn't assign
>>>> the chipset.  We should be therefore checking if (!name) return 
>>>> -ENOSYS;
>>>> Though maybe another error code would be more appropriate.
>>>>
>>>
>>> Since in this case we are enumerated by ACPI using our match table,
>>> so
>>> name can't be null. The "name" we provided in 
>>> static const struct acpi_device_id ak_acpi_match[] = {..}
>>> Same with the *chipset. Other than suppress warnings, I don't think
>>> it
>>> will cause any real issue.
>> True enough, in which case why are we checking the name?
> 
> We can remove this check for !id
>   id = acpi_match_device(dev->driver->acpi_match_table, dev);
> - if (!id)
> -	return NULL;
>   *chipset = (int)id->driver_data;
Yes, that's the one I meant rather than the name!

I'm not getting the warning Tim is seeing anyway so I'll leave
it to him to confirm if this clears that up as well (so we
don't need the other patch).

Tim, as you are working on this issue, do you want to try the
above and if it works post a patch making that change + adding
a note where the check is removed to say it cannot fail so there
is no need to check?

Thanks,

Jonathan
> 
> Thanks,
> Srinivas
> 
> 
>> I'd be included to drop that check and add a comment.
>> I haven't chased every path, but I think that might deal with the
>> above
>> warning at it's root.
>>> Thanks,
>>> Srinivas 
>>>
>>>> Not sure that error path can actually happen either, but if we
>>>> are
>>>> going to
>>>> bother having the error path out of match_acpi_table then we
>>>> ought to
>>>> actually
>>>> handle it!
>>>>
>>>> Don't suppose you'd mind fixing that one as well whilst here?
>>>>
>>>> Jonathan 
>>>>> ---
>>>>>
>>>>> This seems like a legitimate warning, though gcc should have
>>>>> complained
>>>>> about an earlier use of chipset on line 782.
>>>>>
>>>>>  drivers/iio/magnetometer/ak8975.c | 2 +-
>>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/iio/magnetometer/ak8975.c
>>>>> b/drivers/iio/magnetometer/ak8975.c
>>>>> index b13936d..80ec0ce 100644
>>>>> --- a/drivers/iio/magnetometer/ak8975.c
>>>>> +++ b/drivers/iio/magnetometer/ak8975.c
>>>>> @@ -732,7 +732,7 @@ static int ak8975_probe(struct i2c_client
>>>>> *client,
>>>>>  	int eoc_gpio;
>>>>>  	int err;
>>>>>  	const char *name = NULL;
>>>>> -	enum asahi_compass_chipset chipset;
>>>>> +	enum asahi_compass_chipset chipset = AK_MAX_TYPE;
>>>>>  
>>>>>  	/* Grab and set up the supplied GPIO. */
>>>>>  	if (client->dev.platform_data)
>>>>>
>>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux
>>> -iio" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-01-09 16:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-09  0:17 [PATCH v4.4-rc8] iio: magnetometer: ak8975: Silence 'may be used uninitialized' warning tim.gardner
2016-01-09 16:00 ` Jonathan Cameron
2016-01-09 16:17   ` Srinivas Pandruvada
2016-01-09 16:25     ` Jonathan Cameron
2016-01-09 16:51       ` Srinivas Pandruvada
2016-01-09 16:59         ` Jonathan Cameron

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.