Re: [PATCH] misc: panel: move LCD_MINOR and KEYPAD_MINOR to include/linux/miscdevice.h

[Greg KH <gregkh@linuxfoundation.org>]

On Thu, Dec 15, 2016 at 08:43:52PM +0100, Corentin Labbe wrote:
> This patch move the define for LCD_MINOR and KEYPAD_MINOR to include/linux/miscdevice.h.
> It's better that all minor number definitions are in the same place.
> 
> Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> ---
>  drivers/misc/panel.c       | 3 ---
>  include/linux/miscdevice.h | 2 ++
>  2 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/misc/panel.c b/drivers/misc/panel.c
> index 6030ac5..b05c3c7 100644
> --- a/drivers/misc/panel.c
> +++ b/drivers/misc/panel.c
> @@ -61,9 +61,6 @@
>  #include <linux/io.h>
>  #include <linux/uaccess.h>
>  
> -#define LCD_MINOR		156
> -#define KEYPAD_MINOR		185
> -
>  #define PANEL_VERSION		"0.9.5"
>  
>  #define LCD_MAXBYTES		256	/* max burst write */
> diff --git a/include/linux/miscdevice.h b/include/linux/miscdevice.h
> index b1ba37aef..7efc045 100644
> --- a/include/linux/miscdevice.h
> +++ b/include/linux/miscdevice.h
> @@ -30,9 +30,11 @@
>  #define NVRAM_MINOR		144
>  #define SGI_MMTIMER		153
>  #define STORE_QUEUE_MINOR	155	/* unused */
> +#define LCD_MINOR		156
>  #define I2O_MINOR		166
>  #define HWRNG_MINOR		183
>  #define MICROCODE_MINOR		184
> +#define KEYPAD_MINOR		185
>  #define D7S_MINOR		193
>  #define VFIO_MINOR		196
>  #define TUN_MINOR		200

Doesn't apply to my tree :(