Subject: Re: [PATCH v3] net: stmmac: Delete dead code for MDIO registration
To: Florian Fainelli <f.fainelli@gmail.com>,
        Giuseppe Cavallaro <peppe.cavallaro@st.com>,
        Alexandre Torgue <alexandre.torgue@st.com>,
        Andrew Lunn <andrew@lunn.ch>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20170823085058.23333-1-romain.perier@collabora.com>
 <d803f188-826b-706c-2d28-18ee87b8aa0d@gmail.com>
From: Romain Perier <romain.perier@collabora.com>
Message-ID: <10e0ebf9-b412-8547-d0d7-01a958edbcd9@collabora.com>
Date: Thu, 24 Aug 2017 08:49:18 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.2.1
MIME-Version: 1.0
In-Reply-To: <d803f188-826b-706c-2d28-18ee87b8aa0d@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1500
Lines: 47

Hello,


Le 23/08/2017 à 18:46, Florian Fainelli a écrit :
> On 08/23/2017 01:50 AM, Romain Perier wrote:
>> This code is no longer used, the logging function was changed by commit
>> fbca164776e4 ("net: stmmac: Use the right logging functi"). It was
>> previously showing information about the type if the IRQ, if it's polled,
>> ignored or a normal interrupt. As we don't want information loss, I have
>> moved this code to phy_attached_print().
>>
>> Fixes: fbca164776e4 ("net: stmmac: Use the right logging functi")
> For future submissions, do not truncate the commit subject that you are
> referencing.

Even if it exceeds 72 characters ?

>
>> Signed-off-by: Romain Perier <romain.perier@collabora.com>
> Since this is a fix you should indicate in the patch subject that this
> is targeting David's "net" tree, see
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/networking/netdev-FAQ.txt#n85

Ok, will do

>
>
>>  	const char *drv_name = phydev->drv ? phydev->drv->name : "unbound";
>> +	char *irq_str;
>> +	char irq_num[4];
>> +
>> +	switch(phydev->irq) {
>> +	case PHY_POLL:
>> +		irq_str = "POLL";
>> +		break;
>> +	case PHY_IGNORE_INTERRUPT:
>> +		irq_str = "IGNORE";
>> +		break;
>> +	default:
>> +		snprintf(irq_num, sizeof(irq_str), "%d", phydev->irq);
> sizeof(irq_str) = 4 or 8 depending on the architecture because it's a
> pointer, did not you mean sizeof(irq_num) here instead?

Ah good catch, yeah that's a stupid error

Regards,
Romain