On 06/18/2014 03:25 PM, Mark Rutland wrote:
> On Wed, Jun 18, 2014 at 10:26:16AM +0100, Varka Bhadram wrote:
>> +static int cc2520_probe(struct spi_device *spi)
>> +{
>> + struct cc2520_private *priv;
>> + struct pinctrl *pinctrl;
>> + struct cc2520_platform_data *pdata;
>> + struct device_node __maybe_unused *np = spi->dev.of_node;
>>
>> This looks to be definitely unused rather than __maybe_unused. There are
>> no other occuences of "np" in this function.
>>
>> This macro defined like this.
>> #define __maybe_unused __attribute__((unused))
> Sure, but that's irrelevant. There's absolutely no point in this
> variable existing at all, and it can simply disappear.
>
> Mark.
If we don't use that macro the compiler triggers warnings saying that:
warning: unused variable ‘np’ [-Wunused-variable]
Varka Bhadram
------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel
