On Wed, Jun 20, 2012 at 01:36:07PM +0100, Rupesh Gujare wrote:
> ioctl numbers were not in order, rearranging them in ascending order.
> 
> Signed-off-by: Rupesh Gujare <[email protected]>
> ---
>  drivers/staging/ozwpan/ozappif.h |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/ozwpan/ozappif.h 
> b/drivers/staging/ozwpan/ozappif.h
> index 1b59b07..449a6ba 100644
> --- a/drivers/staging/ozwpan/ozappif.h
> +++ b/drivers/staging/ozwpan/ozappif.h
> @@ -30,9 +30,9 @@ struct oz_binding_info {
>  #define OZ_IOCTL_GET_PD_LIST _IOR(OZ_IOCTL_MAGIC, 0, struct oz_pd_list)
>  #define OZ_IOCTL_SET_ACTIVE_PD       _IOW(OZ_IOCTL_MAGIC, 1, struct 
> oz_mac_addr)
>  #define OZ_IOCTL_GET_ACTIVE_PD       _IOR(OZ_IOCTL_MAGIC, 2, struct 
> oz_mac_addr)
> -#define OZ_IOCTL_ADD_BINDING _IOW(OZ_IOCTL_MAGIC, 5, struct oz_binding_info)
> -#define OZ_IOCTL_REMOVE_BINDING      _IOW(OZ_IOCTL_MAGIC, 8, struct 
> oz_binding_info)
> -#define OZ_IOCTL_MAX         9
> +#define OZ_IOCTL_ADD_BINDING _IOW(OZ_IOCTL_MAGIC, 3, struct oz_binding_info)
> +#define OZ_IOCTL_REMOVE_BINDING      _IOW(OZ_IOCTL_MAGIC, 4, struct 
> oz_binding_info)
> +#define OZ_IOCTL_MAX         5

Really?  You just might have broken any userspace applications that were
compiled against the old header.

I'll go apply this, but you need to be aware that you just broke the
user/kernel api here...

greg k-h
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to