[linux-yocto] [PATCH 48/94] drivers/net: Added debug code.

[Paul Butler]

On Sat, Nov 9, 2013 at 9:49 AM, Bruce Ashfield <bruce.ashfield at windriver.com
> wrote:

> On 11/7/2013, 8:13 PM, Paul Butler wrote:
>
>> From: John Jacques <john.jacques at lsi.com>
>>
>> Signed-off-by: John Jacques <john.jacques at lsi.com>
>> ---
>>   drivers/net/ethernet/lsi/lsi_acp_mdio.c | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
>>
>> diff --git a/drivers/net/ethernet/lsi/lsi_acp_mdio.c
>> b/drivers/net/ethernet/lsi/lsi_acp_mdio.c
>> index 04c224c..18aaba1 100644
>> --- a/drivers/net/ethernet/lsi/lsi_acp_mdio.c
>> +++ b/drivers/net/ethernet/lsi/lsi_acp_mdio.c
>> @@ -98,6 +98,11 @@ acp_mdio_read(unsigned long address, unsigned long
>> offset,
>>         *value = (unsigned short)(command & 0xffff);
>>         spin_unlock_irqrestore(&mdio_lock, flags);
>>
>> +#if 0
>> +       printk("%s - Read 0x%x from 0x%x register 0x%x\n",
>> +              __FUNCTION__, *value, address, offset);
>> +#endif
>> +
>>         return 0;
>>   }
>>   EXPORT_SYMBOL(acp_mdio_read);
>> @@ -150,6 +155,11 @@ acp_mdio_write(unsigned long address, unsigned long
>> offset,
>>
>>         spin_unlock_irqrestore(&mdio_lock, flags);
>>
>> +#if 0
>>
>
> These *really* need to be controlled by a define, or Kconfig that controls
> a #define.
>
> Leaving #if 0 code around the tree is bad form.
>
> The #if 0 code was removed in a later patch in this series. Actually, this
file was touched again several times in this series. This is also true of
other patches with dead code removal issues.

In some cases I tried to squash the cleanup together with a change, but I
see other comments discouraging that approach. I think we are pretty stable
at the tip, but it's pretty ugly how we got there. What is preferred?
Squash later cleanup into this patch? (And other similar issues.)

Paul



> Bruce
>
>  +       printk("%s - Wrote 0x%x to 0x%x register 0x%x\n",
>> +              __FUNCTION__, value, address, offset);
>> +#endif
>> +
>>         return 0;
>>   }
>>   EXPORT_SYMBOL(acp_mdio_write);
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/linux-yocto/attachments/20131119/e68b82fd/attachment.html>