Hello.

On 04/07/2015 10:33 PM, David Miller wrote:

The explicitly inline functions belong to the header files, so move
cpu_to_edmac() and edmac_to_cpu() into the driver header.

While at it, make these functions return 'u32' instead of '__u32'.

Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>

Then... ummm... remove the inline keyword?

If it's not used anywhere else, that's the thing to do.

    Results in more code, as I noted in the patch posting that you skipped
    when replying. At least with gcc 4.7.

That's a compiler bug.

Even worse, these functions support the BE mode which never was used and never worked correctly. I'll just try to remove them altogether.

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to