[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPcc5Pj7Xo0ETPzpqgjnf9tA9AeNnqOFPiPFE7XXFYSinXRsXA@mail.gmail.com>
Date: Tue, 14 Apr 2015 17:34:40 +0300
From: Amir Vadai <amirv@...lanox.com>
To: Ben Hutchings <ben@...adent.org.uk>
Cc: Amir Vadai <amirv@...lanox.com>, netdev <netdev@...r.kernel.org>,
Or Gerlitz <ogerlitz@...lanox.com>,
Yevgeny Petrilin <yevgenyp@...lanox.com>,
Saeed Mahameed <saeedm@...lanox.com>,
Eyal Perry <eyalpe@...lanox.com>,
Eyal Grossman <eyalgr@...lanox.com>
Subject: Re: [PATCH ethtool 1/5] ethtool-copy.h: sync with net
On Sun, Apr 5, 2015 at 4:00 AM, Ben Hutchings <ben@...adent.org.uk> wrote:
> On Sun, 2015-01-25 at 15:51 +0200, Amir Vadai wrote:
>> From: Eyal Grossman <eyalgr@...lanox.com>
[...]
>
> You've copied from include/uapi/linux/ethtool.h, which is not quite
> right. The include/uapi directory contains definitions shared with
> userland, but the headers still need some processing (like removing the
> _UAPI prefix from header guards). You should use 'make headers_install'
> to generate the userland version in usr/include/linux/ethtool.h.
Sorry, didn't know.
> Anyway, I've done that myself based on today's net tree.
Is it in your git [1] ? Because I couldn't find the updated version there.
[1] - git://git.kernel.org/pub/scm/network/ethtool/ethtool.git
Thanks,
Amir
>
> Ben.
>
> --
> Ben Hutchings
> Quantity is no substitute for quality, but it's the only one we've got.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists