Message ID | 1473328056-12555-2-git-send-email-yamada.masahiro@socionext.com |
---|---|
State | Accepted |
Commit | b98278be7bd64f8726b388416947118f65077b2b |
Delegated to: | Tom Rini |
Headers | show |
On 09/08/2016 11:47 AM, Masahiro Yamada wrote: > Creating multiple entries of "config FOO" often gives us bad > experiences. In this case, we should specify "default X86" > as platforms that want this keyboard by default. Yep, I like this patch: Acked-by: Marek Vasut <marex@denx.de> btw on some of my computers , the 8042 is physically removable ;-) > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > arch/x86/Kconfig | 3 --- > drivers/input/Kconfig | 1 + > 2 files changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 9207549..ac2d598 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -546,9 +546,6 @@ config I8254_TIMER > Intel 8254 timer contains three counters which have fixed uses. > Include this to have U-Boot set up the timer correctly. > > -config I8042_KEYB > - default y > - > config SEABIOS > bool "Support booting SeaBIOS" > help > diff --git a/drivers/input/Kconfig b/drivers/input/Kconfig > index d560328..b3873c1 100644 > --- a/drivers/input/Kconfig > +++ b/drivers/input/Kconfig > @@ -17,6 +17,7 @@ config CROS_EC_KEYB > config I8042_KEYB > bool "Enable Intel i8042 keyboard support" > depends on DM_KEYBOARD > + default X86 > help > This adds a driver for the i8042 keyboard controller, allowing the > keyboard to be used on devices which support this controller. The >
Hi Masahiro, On 8 September 2016 at 03:47, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > Creating multiple entries of "config FOO" often gives us bad > experiences. In this case, we should specify "default X86" > as platforms that want this keyboard by default. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > arch/x86/Kconfig | 3 --- > drivers/input/Kconfig | 1 + > 2 files changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 9207549..ac2d598 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -546,9 +546,6 @@ config I8254_TIMER > Intel 8254 timer contains three counters which have fixed uses. > Include this to have U-Boot set up the timer correctly. > > -config I8042_KEYB > - default y > - > config SEABIOS > bool "Support booting SeaBIOS" > help > diff --git a/drivers/input/Kconfig b/drivers/input/Kconfig > index d560328..b3873c1 100644 > --- a/drivers/input/Kconfig > +++ b/drivers/input/Kconfig > @@ -17,6 +17,7 @@ config CROS_EC_KEYB > config I8042_KEYB > bool "Enable Intel i8042 keyboard support" > depends on DM_KEYBOARD > + default X86 What does this mean? > help > This adds a driver for the i8042 keyboard controller, allowing the > keyboard to be used on devices which support this controller. The > -- > 1.9.1 > Regards, Simon
2016-09-19 9:57 GMT+09:00 Simon Glass <sjg@chromium.org>: > Hi Masahiro, > > On 8 September 2016 at 03:47, Masahiro Yamada > <yamada.masahiro@socionext.com> wrote: >> Creating multiple entries of "config FOO" often gives us bad >> experiences. In this case, we should specify "default X86" >> as platforms that want this keyboard by default. >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> >> --- >> >> arch/x86/Kconfig | 3 --- >> drivers/input/Kconfig | 1 + >> 2 files changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig >> index 9207549..ac2d598 100644 >> --- a/arch/x86/Kconfig >> +++ b/arch/x86/Kconfig >> @@ -546,9 +546,6 @@ config I8254_TIMER >> Intel 8254 timer contains three counters which have fixed uses. >> Include this to have U-Boot set up the timer correctly. >> >> -config I8042_KEYB >> - default y >> - >> config SEABIOS >> bool "Support booting SeaBIOS" >> help >> diff --git a/drivers/input/Kconfig b/drivers/input/Kconfig >> index d560328..b3873c1 100644 >> --- a/drivers/input/Kconfig >> +++ b/drivers/input/Kconfig >> @@ -17,6 +17,7 @@ config CROS_EC_KEYB >> config I8042_KEYB >> bool "Enable Intel i8042 keyboard support" >> depends on DM_KEYBOARD >> + default X86 > > What does this mean? The default value of I8042_KEYB is the value of X86. In another expression, default y if X86
On 19 September 2016 at 06:46, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > 2016-09-19 9:57 GMT+09:00 Simon Glass <sjg@chromium.org>: >> Hi Masahiro, >> >> On 8 September 2016 at 03:47, Masahiro Yamada >> <yamada.masahiro@socionext.com> wrote: >>> Creating multiple entries of "config FOO" often gives us bad >>> experiences. In this case, we should specify "default X86" >>> as platforms that want this keyboard by default. >>> >>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> >>> --- >>> >>> arch/x86/Kconfig | 3 --- >>> drivers/input/Kconfig | 1 + >>> 2 files changed, 1 insertion(+), 3 deletions(-) >>> >>> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig >>> index 9207549..ac2d598 100644 >>> --- a/arch/x86/Kconfig >>> +++ b/arch/x86/Kconfig >>> @@ -546,9 +546,6 @@ config I8254_TIMER >>> Intel 8254 timer contains three counters which have fixed uses. >>> Include this to have U-Boot set up the timer correctly. >>> >>> -config I8042_KEYB >>> - default y >>> - >>> config SEABIOS >>> bool "Support booting SeaBIOS" >>> help >>> diff --git a/drivers/input/Kconfig b/drivers/input/Kconfig >>> index d560328..b3873c1 100644 >>> --- a/drivers/input/Kconfig >>> +++ b/drivers/input/Kconfig >>> @@ -17,6 +17,7 @@ config CROS_EC_KEYB >>> config I8042_KEYB >>> bool "Enable Intel i8042 keyboard support" >>> depends on DM_KEYBOARD >>> + default X86 >> >> What does this mean? > > > The default value of I8042_KEYB is the > value of X86. > > > In another expression, > > default y if X86 Ah OK, thanks. Reviewed-by: Simon Glass <sjg@chromium.org> Regards, Simon
On Thu, Sep 08, 2016 at 06:47:36PM +0900, Masahiro Yamada wrote: > Creating multiple entries of "config FOO" often gives us bad > experiences. In this case, we should specify "default X86" > as platforms that want this keyboard by default. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Acked-by: Marek Vasut <marex@denx.de> > Reviewed-by: Simon Glass <sjg@chromium.org> Applied to u-boot/master, thanks!
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 9207549..ac2d598 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -546,9 +546,6 @@ config I8254_TIMER Intel 8254 timer contains three counters which have fixed uses. Include this to have U-Boot set up the timer correctly. -config I8042_KEYB - default y - config SEABIOS bool "Support booting SeaBIOS" help diff --git a/drivers/input/Kconfig b/drivers/input/Kconfig index d560328..b3873c1 100644 --- a/drivers/input/Kconfig +++ b/drivers/input/Kconfig @@ -17,6 +17,7 @@ config CROS_EC_KEYB config I8042_KEYB bool "Enable Intel i8042 keyboard support" depends on DM_KEYBOARD + default X86 help This adds a driver for the i8042 keyboard controller, allowing the keyboard to be used on devices which support this controller. The
Creating multiple entries of "config FOO" often gives us bad experiences. In this case, we should specify "default X86" as platforms that want this keyboard by default. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- arch/x86/Kconfig | 3 --- drivers/input/Kconfig | 1 + 2 files changed, 1 insertion(+), 3 deletions(-)