[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <h2td45a3acc1004181651xd70115dr5671210761035838@mail.gmail.com>
Date: Mon, 19 Apr 2010 09:51:01 +1000
From: andrew hendry <andrew.hendry@...il.com>
To: John Hughes <john@...va.com>
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH 1/6] X25: Use identifiers for X25 to device interface
Hi John,
Thanks, missed that, ill see if any of the driver areas have comments
and make a V2 patch with explicit values.
Your right, it would be nice if xotd/xoe could pickup the same
definition from user space.
include/net/x25device.h is not intended as a user space header.
include/linux/net/x25.h is the user space header for the x25 socket
layer interface, so I don't think these definitions should go there.
I think the right way to do it is add a new userspace header
include/linux/if_x25.h then include that from net/x25_device.h?
Regards,
Andrew.
On Sun, Apr 18, 2010 at 9:08 PM, John Hughes <john@...va.com> wrote:
> John Hughes wrote:
>>
>> Shouldn't you use explicit values here?
>>
>> enum {
>> X25_IFACE_DATA = 0x00, /* explicit value for ABI stability */
>> ...
>
> Oh, and is net/x25device.h suitable for inclusion from user space (xotd for
> example)?
>
>
--
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