Re: [Qemu-devel] [PATCH 1/8] hw/input/Kconfig: Rename the 8042 PS/2 Keyboard Controller

[BALATON Zoltan]

On Sun, 17 Mar 2019, Philippe Mathieu-Daudé wrote:
> The Intel 8042 chipset also includes some functions unrelated to
> the keyboard, such the A20-Gate. Rename it using the chipset name.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> default-configs/mips-softmmu-common.mak | 2 +-
> hw/alpha/Kconfig                        | 2 +-
> hw/i386/Kconfig                         | 2 +-
> hw/input/Kconfig                        | 2 +-
> hw/input/Makefile.objs                  | 2 +-
> hw/sparc64/Kconfig                      | 2 +-
> hw/unicore32/Kconfig                    | 2 +-
> 7 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/default-configs/mips-softmmu-common.mak 
> b/default-configs/mips-softmmu-common.mak
> index 0795d522db..bad7ee3360 100644
> --- a/default-configs/mips-softmmu-common.mak
> +++ b/default-configs/mips-softmmu-common.mak
> @@ -13,7 +13,7 @@ CONFIG_SERIAL_ISA=y
> CONFIG_PARALLEL=y
> CONFIG_I8254=y
> CONFIG_PCSPK=y
> -CONFIG_PCKBD=y
> +CONFIG_I8042=y
> CONFIG_FDC=y
> CONFIG_ACPI=y
> CONFIG_ACPI_X86=y
> diff --git a/hw/alpha/Kconfig b/hw/alpha/Kconfig
> index 22cefd9577..eb99765123 100644
> --- a/hw/alpha/Kconfig
> +++ b/hw/alpha/Kconfig
> @@ -8,5 +8,5 @@ config DP264
>     select IDE_CMD646
>     select MC146818RTC
>     select PCI
> -    select PCKBD
> +    select I8042
>     select SMC37C669
> diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig
> index 78fd70396a..7fa5395894 100644
> --- a/hw/i386/Kconfig
> +++ b/hw/i386/Kconfig
> @@ -20,7 +20,7 @@ config PC
>     select FDC
>     select I8259
>     select I8254
> -    select PCKBD
> +    select I8042
>     select PCSPK
>     select I82374
>     select I8257
> diff --git a/hw/input/Kconfig b/hw/input/Kconfig
> index e2e66f0858..14e76c5df4 100644
> --- a/hw/input/Kconfig
> +++ b/hw/input/Kconfig
> @@ -5,7 +5,7 @@ config LM832X
>     bool
>     depends on I2C
>
> -config PCKBD
> +config I8042
>     bool
>     default y
>     depends on ISA_BUS
> diff --git a/hw/input/Makefile.objs b/hw/input/Makefile.objs
> index c8b00f71ec..6e4c3389be 100644
> --- a/hw/input/Makefile.objs
> +++ b/hw/input/Makefile.objs
> @@ -1,7 +1,7 @@
> common-obj-$(CONFIG_ADB) += adb.o adb-mouse.o adb-kbd.o
> common-obj-y += hid.o
> common-obj-$(CONFIG_LM832X) += lm832x.o
> -common-obj-$(CONFIG_PCKBD) += pckbd.o
> +common-obj-$(CONFIG_I8042) += pckbd.o

This makes config option different from the filename so I wonder if it's 
worth changing it or better to keep it same as its file like the others? 
A20 gate is controlled by AT keyboard controller but that's probably a 
hack made by the designers of that machine, it's still used mainly as a 
keyboard controller so just to avoid changes we could live with the PCKBD 
name as well (which might even be simpler to understand for people not 
familiar with chip numbers).

Regards,
BALATON Zoltan

> common-obj-$(CONFIG_PL050) += pl050.o
> common-obj-y += ps2.o
> common-obj-$(CONFIG_STELLARIS_INPUT) += stellaris_input.o
> diff --git a/hw/sparc64/Kconfig b/hw/sparc64/Kconfig
> index 4a8166ebb7..24ccb11c87 100644
> --- a/hw/sparc64/Kconfig
> +++ b/hw/sparc64/Kconfig
> @@ -10,7 +10,7 @@ config SUN4U
>     select PCI_SABRE
>     select IDE_CMD646
>     select PARALLEL
> -    select PCKBD
> +    select I8042
>     select SIMBA
>
> config NIAGARA
> diff --git a/hw/unicore32/Kconfig b/hw/unicore32/Kconfig
> index 4443a29dd2..67f4c867c1 100644
> --- a/hw/unicore32/Kconfig
> +++ b/hw/unicore32/Kconfig
> @@ -1,5 +1,5 @@
> config PUV3
>     bool
>     select ISA_BUS
> -    select PCKBD
> +    select I8042
>     select PTIMER