mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH 1/2] Regenerate defconfig files
@ 2020-09-18 11:04 Uwe Kleine-König
  2020-09-18 11:04 ` [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550 Uwe Kleine-König
  2020-09-23  8:47 ` [PATCH 1/2] Regenerate defconfig files Sascha Hauer
  0 siblings, 2 replies; 9+ messages in thread
From: Uwe Kleine-König @ 2020-09-18 11:04 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

Done with the same script used to generate commit d952a0eeba37
("Regenerate defconfig files").

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/arm/configs/at91sam9263ek_defconfig         |  1 -
 arch/arm/configs/at91sam9x5ek_defconfig          |  1 -
 arch/arm/configs/freescale-mx21-ads_defconfig    |  1 -
 arch/arm/configs/imx_defconfig                   |  1 -
 arch/arm/configs/imx_v8_defconfig                |  2 --
 arch/arm/configs/kindle-mx50_defconfig           |  1 -
 arch/arm/configs/layerscape_defconfig            |  2 --
 arch/arm/configs/microchip_ksz9477_evb_defconfig |  1 -
 arch/arm/configs/socfpga-arria10_defconfig       |  1 -
 arch/arm/configs/virt2real_defconfig             |  1 -
 arch/arm/configs/zii_vf610_dev_defconfig         |  1 -
 arch/kvx/configs/generic_defconfig               | 12 +++++-------
 arch/mips/configs/bcm47xx_defconfig              |  1 -
 arch/mips/configs/qemu-malta_defconfig           |  1 -
 arch/x86/configs/efi_defconfig                   |  1 -
 15 files changed, 5 insertions(+), 23 deletions(-)

diff --git a/arch/arm/configs/at91sam9263ek_defconfig b/arch/arm/configs/at91sam9263ek_defconfig
index 45c6f79de40b..0cde5396af3c 100644
--- a/arch/arm/configs/at91sam9263ek_defconfig
+++ b/arch/arm/configs/at91sam9263ek_defconfig
@@ -1,4 +1,3 @@
-CONFIG_ARCH_AT91SAM9263=y
 CONFIG_AT91_MULTI_BOARDS=y
 CONFIG_MACH_AT91SAM9263EK=y
 CONFIG_BAREBOX_MAX_IMAGE_SIZE=0x40000
diff --git a/arch/arm/configs/at91sam9x5ek_defconfig b/arch/arm/configs/at91sam9x5ek_defconfig
index 11d1e4511e03..7b743abfaa17 100644
--- a/arch/arm/configs/at91sam9x5ek_defconfig
+++ b/arch/arm/configs/at91sam9x5ek_defconfig
@@ -1,4 +1,3 @@
-CONFIG_ARCH_AT91SAM9X5=y
 CONFIG_AT91_MULTI_BOARDS=y
 CONFIG_MACH_AT91SAM9X5EK=y
 CONFIG_BAREBOX_MAX_IMAGE_SIZE=0x40000
diff --git a/arch/arm/configs/freescale-mx21-ads_defconfig b/arch/arm/configs/freescale-mx21-ads_defconfig
index e1843db3e24c..b1d37f76a895 100644
--- a/arch/arm/configs/freescale-mx21-ads_defconfig
+++ b/arch/arm/configs/freescale-mx21-ads_defconfig
@@ -1,6 +1,5 @@
 CONFIG_TEXT_BASE=0xc3000000
 CONFIG_ARCH_IMX=y
-CONFIG_MACH_IMX21ADS=y
 CONFIG_ARM_OPTIMZED_STRING_FUNCTIONS=y
 CONFIG_MALLOC_SIZE=0x2000000
 CONFIG_CMDLINE_EDITING=y
diff --git a/arch/arm/configs/imx_defconfig b/arch/arm/configs/imx_defconfig
index 02e33e122ec8..e6333220b6c9 100644
--- a/arch/arm/configs/imx_defconfig
+++ b/arch/arm/configs/imx_defconfig
@@ -13,7 +13,6 @@ CONFIG_MMU=y
 CONFIG_MALLOC_SIZE=0x0
 CONFIG_MALLOC_TLSF=y
 CONFIG_KALLSYMS=y
-CONFIG_RELOCATABLE=y
 CONFIG_PANIC_HANG=y
 CONFIG_HUSH_FANCY_PROMPT=y
 CONFIG_CMDLINE_EDITING=y
diff --git a/arch/arm/configs/imx_v8_defconfig b/arch/arm/configs/imx_v8_defconfig
index 268770ec3f2e..06d79f594d5b 100644
--- a/arch/arm/configs/imx_v8_defconfig
+++ b/arch/arm/configs/imx_v8_defconfig
@@ -10,7 +10,6 @@ CONFIG_MMU=y
 CONFIG_MALLOC_SIZE=0x0
 CONFIG_MALLOC_TLSF=y
 CONFIG_KALLSYMS=y
-CONFIG_RELOCATABLE=y
 CONFIG_HUSH_FANCY_PROMPT=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_MENU=y
@@ -130,7 +129,6 @@ CONFIG_RTC_CLASS=y
 CONFIG_RTC_DRV_DS1307=y
 CONFIG_GENERIC_PHY=y
 CONFIG_USB_NOP_XCEIV=y
-CONFIG_PHY_FSL_IMX8MQ_USB=y
 CONFIG_FS_EXT4=y
 CONFIG_FS_TFTP=y
 CONFIG_FS_NFS=y
diff --git a/arch/arm/configs/kindle-mx50_defconfig b/arch/arm/configs/kindle-mx50_defconfig
index 552b2d6d331c..95fafd56e68c 100644
--- a/arch/arm/configs/kindle-mx50_defconfig
+++ b/arch/arm/configs/kindle-mx50_defconfig
@@ -9,7 +9,6 @@ CONFIG_MMU=y
 CONFIG_MALLOC_SIZE=0x0
 CONFIG_MALLOC_TLSF=y
 CONFIG_KALLSYMS=y
-CONFIG_RELOCATABLE=y
 CONFIG_HUSH_FANCY_PROMPT=y
 CONFIG_CMDLINE_EDITING=y
 CONFIG_AUTO_COMPLETE=y
diff --git a/arch/arm/configs/layerscape_defconfig b/arch/arm/configs/layerscape_defconfig
index b36f1944ecdc..394cd95c980a 100644
--- a/arch/arm/configs/layerscape_defconfig
+++ b/arch/arm/configs/layerscape_defconfig
@@ -78,7 +78,6 @@ CONFIG_CMD_OFTREE=y
 CONFIG_CMD_TIME=y
 CONFIG_NET=y
 CONFIG_NET_NETCONSOLE=y
-CONFIG_OFDEVICE=y
 CONFIG_OF_BAREBOX_DRIVERS=y
 CONFIG_DRIVER_SERIAL_NS16550=y
 CONFIG_DRIVER_NET_FSL_FMAN=y
@@ -105,7 +104,6 @@ CONFIG_EEPROM_AT24=y
 CONFIG_WATCHDOG=y
 CONFIG_WATCHDOG_IMX=y
 CONFIG_GPIO_PCA953X=y
-CONFIG_NVMEM=y
 CONFIG_REGULATOR=y
 CONFIG_REGULATOR_FIXED=y
 CONFIG_FS_EXT4=y
diff --git a/arch/arm/configs/microchip_ksz9477_evb_defconfig b/arch/arm/configs/microchip_ksz9477_evb_defconfig
index 4189b2c039d9..71304994902a 100644
--- a/arch/arm/configs/microchip_ksz9477_evb_defconfig
+++ b/arch/arm/configs/microchip_ksz9477_evb_defconfig
@@ -1,4 +1,3 @@
-CONFIG_ARCH_SAMA5D3=y
 CONFIG_AT91_MULTI_BOARDS=y
 CONFIG_MACH_MICROCHIP_KSZ9477_EVB=y
 CONFIG_AEABI=y
diff --git a/arch/arm/configs/socfpga-arria10_defconfig b/arch/arm/configs/socfpga-arria10_defconfig
index e47a0ab1836d..a37bae621729 100644
--- a/arch/arm/configs/socfpga-arria10_defconfig
+++ b/arch/arm/configs/socfpga-arria10_defconfig
@@ -18,7 +18,6 @@ CONFIG_BOOTM_INITRD=y
 CONFIG_BOOTM_OFTREE=y
 CONFIG_DEFAULT_COMPRESSION_LZO=y
 CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
-CONFIG_POLLER=y
 CONFIG_STATE=y
 CONFIG_LONGHELP=y
 CONFIG_CMD_IOMEM=y
diff --git a/arch/arm/configs/virt2real_defconfig b/arch/arm/configs/virt2real_defconfig
index 814fe69e42ce..62315b8cb3ec 100644
--- a/arch/arm/configs/virt2real_defconfig
+++ b/arch/arm/configs/virt2real_defconfig
@@ -36,7 +36,6 @@ CONFIG_CMD_LED=y
 CONFIG_CMD_OF_NODE=y
 CONFIG_CMD_OF_PROPERTY=y
 CONFIG_CMD_OFTREE=y
-CONFIG_OFDEVICE=y
 CONFIG_OF_BAREBOX_DRIVERS=y
 CONFIG_DRIVER_SERIAL_NS16550=y
 # CONFIG_SPI is not set
diff --git a/arch/arm/configs/zii_vf610_dev_defconfig b/arch/arm/configs/zii_vf610_dev_defconfig
index 45c24d6df471..3ed5d374586d 100644
--- a/arch/arm/configs/zii_vf610_dev_defconfig
+++ b/arch/arm/configs/zii_vf610_dev_defconfig
@@ -8,7 +8,6 @@ CONFIG_MMU=y
 CONFIG_MALLOC_SIZE=0x0
 CONFIG_MALLOC_TLSF=y
 CONFIG_KALLSYMS=y
-CONFIG_RELOCATABLE=y
 CONFIG_HUSH_FANCY_PROMPT=y
 CONFIG_AUTO_COMPLETE=y
 CONFIG_MENU=y
diff --git a/arch/kvx/configs/generic_defconfig b/arch/kvx/configs/generic_defconfig
index 816217174183..0d971ff3d5dd 100644
--- a/arch/kvx/configs/generic_defconfig
+++ b/arch/kvx/configs/generic_defconfig
@@ -1,15 +1,13 @@
 CONFIG_AUTO_COMPLETE=y
-CONFIG_BAUDRATE=115200
-CONFIG_CLOCKSOURCE_KVX=y
+CONFIG_CONSOLE_RATP=y
 CONFIG_CMD_BOOT=y
-CONFIG_CMD_BOOTM=y
+CONFIG_CMD_RESET=y
 CONFIG_CMD_CMP=y
-CONFIG_CMD_OF_DUMP=y
 CONFIG_CMD_POWEROFF=y
-CONFIG_CMD_RESET=y
 CONFIG_CMD_WD=y
-CONFIG_CONSOLE_RATP=y
+CONFIG_CMD_OF_DUMP=y
 CONFIG_DRIVER_SERIAL_NS16550=y
-CONFIG_PINCTRL_SINGLE=y
+CONFIG_CLOCKSOURCE_KVX=y
 CONFIG_WATCHDOG=y
 CONFIG_WATCHDOG_KVX=y
+CONFIG_PINCTRL_SINGLE=y
diff --git a/arch/mips/configs/bcm47xx_defconfig b/arch/mips/configs/bcm47xx_defconfig
index 0cd0e9e650e7..9cf172a70e57 100644
--- a/arch/mips/configs/bcm47xx_defconfig
+++ b/arch/mips/configs/bcm47xx_defconfig
@@ -9,7 +9,6 @@ CONFIG_MENU=y
 CONFIG_BOOTM_SHOW_TYPE=y
 CONFIG_PARTITION=y
 # CONFIG_DEFAULT_ENVIRONMENT is not set
-CONFIG_POLLER=y
 CONFIG_LONGHELP=y
 CONFIG_CMD_IOMEM=y
 CONFIG_CMD_MEMINFO=y
diff --git a/arch/mips/configs/qemu-malta_defconfig b/arch/mips/configs/qemu-malta_defconfig
index 2465c0260d1d..ac0577c217e3 100644
--- a/arch/mips/configs/qemu-malta_defconfig
+++ b/arch/mips/configs/qemu-malta_defconfig
@@ -12,7 +12,6 @@ CONFIG_BOOTM_SHOW_TYPE=y
 CONFIG_CONSOLE_ALLOW_COLOR=y
 CONFIG_PARTITION=y
 CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
-CONFIG_POLLER=y
 CONFIG_CMD_DMESG=y
 CONFIG_LONGHELP=y
 CONFIG_CMD_IOMEM=y
diff --git a/arch/x86/configs/efi_defconfig b/arch/x86/configs/efi_defconfig
index 47842d10afdd..99bc39eaa6f1 100644
--- a/arch/x86/configs/efi_defconfig
+++ b/arch/x86/configs/efi_defconfig
@@ -14,7 +14,6 @@ CONFIG_BLSPEC=y
 CONFIG_CONSOLE_ACTIVATE_ALL=y
 CONFIG_PARTITION_DISK_EFI=y
 CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
-CONFIG_POLLER=y
 CONFIG_STATE=y
 CONFIG_DEBUG_LL=y
 CONFIG_LONGHELP=y
-- 
2.27.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-09-18 11:04 [PATCH 1/2] Regenerate defconfig files Uwe Kleine-König
@ 2020-09-18 11:04 ` Uwe Kleine-König
  2020-10-20  5:08   ` Ahmad Fatoum
  2020-09-23  8:47 ` [PATCH 1/2] Regenerate defconfig files Sascha Hauer
  1 sibling, 1 reply; 9+ messages in thread
From: Uwe Kleine-König @ 2020-09-18 11:04 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
both enabled results in the output appearing twice on the serial output.
So remove disable the lowlevel driver and rely on the EFI bios for serial.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 arch/x86/configs/efi_defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/x86/configs/efi_defconfig b/arch/x86/configs/efi_defconfig
index 99bc39eaa6f1..83794d7a0702 100644
--- a/arch/x86/configs/efi_defconfig
+++ b/arch/x86/configs/efi_defconfig
@@ -65,7 +65,6 @@ CONFIG_NET=y
 CONFIG_NET_NFS=y
 CONFIG_NET_NETCONSOLE=y
 CONFIG_DRIVER_SERIAL_EFI_STDIO=y
-CONFIG_DRIVER_SERIAL_NS16550=y
 CONFIG_DRIVER_NET_EFI_SNP=y
 # CONFIG_SPI is not set
 CONFIG_DISK=y
-- 
2.27.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/2] Regenerate defconfig files
  2020-09-18 11:04 [PATCH 1/2] Regenerate defconfig files Uwe Kleine-König
  2020-09-18 11:04 ` [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550 Uwe Kleine-König
@ 2020-09-23  8:47 ` Sascha Hauer
  1 sibling, 0 replies; 9+ messages in thread
From: Sascha Hauer @ 2020-09-23  8:47 UTC (permalink / raw)
  To: Uwe Kleine-König; +Cc: barebox, Ahmad Fatoum

On Fri, Sep 18, 2020 at 01:04:41PM +0200, Uwe Kleine-König wrote:
> Done with the same script used to generate commit d952a0eeba37
> ("Regenerate defconfig files").
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---

Applied, thanks

Sascha

>  arch/arm/configs/at91sam9263ek_defconfig         |  1 -
>  arch/arm/configs/at91sam9x5ek_defconfig          |  1 -
>  arch/arm/configs/freescale-mx21-ads_defconfig    |  1 -
>  arch/arm/configs/imx_defconfig                   |  1 -
>  arch/arm/configs/imx_v8_defconfig                |  2 --
>  arch/arm/configs/kindle-mx50_defconfig           |  1 -
>  arch/arm/configs/layerscape_defconfig            |  2 --
>  arch/arm/configs/microchip_ksz9477_evb_defconfig |  1 -
>  arch/arm/configs/socfpga-arria10_defconfig       |  1 -
>  arch/arm/configs/virt2real_defconfig             |  1 -
>  arch/arm/configs/zii_vf610_dev_defconfig         |  1 -
>  arch/kvx/configs/generic_defconfig               | 12 +++++-------
>  arch/mips/configs/bcm47xx_defconfig              |  1 -
>  arch/mips/configs/qemu-malta_defconfig           |  1 -
>  arch/x86/configs/efi_defconfig                   |  1 -
>  15 files changed, 5 insertions(+), 23 deletions(-)
> 
> diff --git a/arch/arm/configs/at91sam9263ek_defconfig b/arch/arm/configs/at91sam9263ek_defconfig
> index 45c6f79de40b..0cde5396af3c 100644
> --- a/arch/arm/configs/at91sam9263ek_defconfig
> +++ b/arch/arm/configs/at91sam9263ek_defconfig
> @@ -1,4 +1,3 @@
> -CONFIG_ARCH_AT91SAM9263=y
>  CONFIG_AT91_MULTI_BOARDS=y
>  CONFIG_MACH_AT91SAM9263EK=y
>  CONFIG_BAREBOX_MAX_IMAGE_SIZE=0x40000
> diff --git a/arch/arm/configs/at91sam9x5ek_defconfig b/arch/arm/configs/at91sam9x5ek_defconfig
> index 11d1e4511e03..7b743abfaa17 100644
> --- a/arch/arm/configs/at91sam9x5ek_defconfig
> +++ b/arch/arm/configs/at91sam9x5ek_defconfig
> @@ -1,4 +1,3 @@
> -CONFIG_ARCH_AT91SAM9X5=y
>  CONFIG_AT91_MULTI_BOARDS=y
>  CONFIG_MACH_AT91SAM9X5EK=y
>  CONFIG_BAREBOX_MAX_IMAGE_SIZE=0x40000
> diff --git a/arch/arm/configs/freescale-mx21-ads_defconfig b/arch/arm/configs/freescale-mx21-ads_defconfig
> index e1843db3e24c..b1d37f76a895 100644
> --- a/arch/arm/configs/freescale-mx21-ads_defconfig
> +++ b/arch/arm/configs/freescale-mx21-ads_defconfig
> @@ -1,6 +1,5 @@
>  CONFIG_TEXT_BASE=0xc3000000
>  CONFIG_ARCH_IMX=y
> -CONFIG_MACH_IMX21ADS=y
>  CONFIG_ARM_OPTIMZED_STRING_FUNCTIONS=y
>  CONFIG_MALLOC_SIZE=0x2000000
>  CONFIG_CMDLINE_EDITING=y
> diff --git a/arch/arm/configs/imx_defconfig b/arch/arm/configs/imx_defconfig
> index 02e33e122ec8..e6333220b6c9 100644
> --- a/arch/arm/configs/imx_defconfig
> +++ b/arch/arm/configs/imx_defconfig
> @@ -13,7 +13,6 @@ CONFIG_MMU=y
>  CONFIG_MALLOC_SIZE=0x0
>  CONFIG_MALLOC_TLSF=y
>  CONFIG_KALLSYMS=y
> -CONFIG_RELOCATABLE=y
>  CONFIG_PANIC_HANG=y
>  CONFIG_HUSH_FANCY_PROMPT=y
>  CONFIG_CMDLINE_EDITING=y
> diff --git a/arch/arm/configs/imx_v8_defconfig b/arch/arm/configs/imx_v8_defconfig
> index 268770ec3f2e..06d79f594d5b 100644
> --- a/arch/arm/configs/imx_v8_defconfig
> +++ b/arch/arm/configs/imx_v8_defconfig
> @@ -10,7 +10,6 @@ CONFIG_MMU=y
>  CONFIG_MALLOC_SIZE=0x0
>  CONFIG_MALLOC_TLSF=y
>  CONFIG_KALLSYMS=y
> -CONFIG_RELOCATABLE=y
>  CONFIG_HUSH_FANCY_PROMPT=y
>  CONFIG_AUTO_COMPLETE=y
>  CONFIG_MENU=y
> @@ -130,7 +129,6 @@ CONFIG_RTC_CLASS=y
>  CONFIG_RTC_DRV_DS1307=y
>  CONFIG_GENERIC_PHY=y
>  CONFIG_USB_NOP_XCEIV=y
> -CONFIG_PHY_FSL_IMX8MQ_USB=y
>  CONFIG_FS_EXT4=y
>  CONFIG_FS_TFTP=y
>  CONFIG_FS_NFS=y
> diff --git a/arch/arm/configs/kindle-mx50_defconfig b/arch/arm/configs/kindle-mx50_defconfig
> index 552b2d6d331c..95fafd56e68c 100644
> --- a/arch/arm/configs/kindle-mx50_defconfig
> +++ b/arch/arm/configs/kindle-mx50_defconfig
> @@ -9,7 +9,6 @@ CONFIG_MMU=y
>  CONFIG_MALLOC_SIZE=0x0
>  CONFIG_MALLOC_TLSF=y
>  CONFIG_KALLSYMS=y
> -CONFIG_RELOCATABLE=y
>  CONFIG_HUSH_FANCY_PROMPT=y
>  CONFIG_CMDLINE_EDITING=y
>  CONFIG_AUTO_COMPLETE=y
> diff --git a/arch/arm/configs/layerscape_defconfig b/arch/arm/configs/layerscape_defconfig
> index b36f1944ecdc..394cd95c980a 100644
> --- a/arch/arm/configs/layerscape_defconfig
> +++ b/arch/arm/configs/layerscape_defconfig
> @@ -78,7 +78,6 @@ CONFIG_CMD_OFTREE=y
>  CONFIG_CMD_TIME=y
>  CONFIG_NET=y
>  CONFIG_NET_NETCONSOLE=y
> -CONFIG_OFDEVICE=y
>  CONFIG_OF_BAREBOX_DRIVERS=y
>  CONFIG_DRIVER_SERIAL_NS16550=y
>  CONFIG_DRIVER_NET_FSL_FMAN=y
> @@ -105,7 +104,6 @@ CONFIG_EEPROM_AT24=y
>  CONFIG_WATCHDOG=y
>  CONFIG_WATCHDOG_IMX=y
>  CONFIG_GPIO_PCA953X=y
> -CONFIG_NVMEM=y
>  CONFIG_REGULATOR=y
>  CONFIG_REGULATOR_FIXED=y
>  CONFIG_FS_EXT4=y
> diff --git a/arch/arm/configs/microchip_ksz9477_evb_defconfig b/arch/arm/configs/microchip_ksz9477_evb_defconfig
> index 4189b2c039d9..71304994902a 100644
> --- a/arch/arm/configs/microchip_ksz9477_evb_defconfig
> +++ b/arch/arm/configs/microchip_ksz9477_evb_defconfig
> @@ -1,4 +1,3 @@
> -CONFIG_ARCH_SAMA5D3=y
>  CONFIG_AT91_MULTI_BOARDS=y
>  CONFIG_MACH_MICROCHIP_KSZ9477_EVB=y
>  CONFIG_AEABI=y
> diff --git a/arch/arm/configs/socfpga-arria10_defconfig b/arch/arm/configs/socfpga-arria10_defconfig
> index e47a0ab1836d..a37bae621729 100644
> --- a/arch/arm/configs/socfpga-arria10_defconfig
> +++ b/arch/arm/configs/socfpga-arria10_defconfig
> @@ -18,7 +18,6 @@ CONFIG_BOOTM_INITRD=y
>  CONFIG_BOOTM_OFTREE=y
>  CONFIG_DEFAULT_COMPRESSION_LZO=y
>  CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
> -CONFIG_POLLER=y
>  CONFIG_STATE=y
>  CONFIG_LONGHELP=y
>  CONFIG_CMD_IOMEM=y
> diff --git a/arch/arm/configs/virt2real_defconfig b/arch/arm/configs/virt2real_defconfig
> index 814fe69e42ce..62315b8cb3ec 100644
> --- a/arch/arm/configs/virt2real_defconfig
> +++ b/arch/arm/configs/virt2real_defconfig
> @@ -36,7 +36,6 @@ CONFIG_CMD_LED=y
>  CONFIG_CMD_OF_NODE=y
>  CONFIG_CMD_OF_PROPERTY=y
>  CONFIG_CMD_OFTREE=y
> -CONFIG_OFDEVICE=y
>  CONFIG_OF_BAREBOX_DRIVERS=y
>  CONFIG_DRIVER_SERIAL_NS16550=y
>  # CONFIG_SPI is not set
> diff --git a/arch/arm/configs/zii_vf610_dev_defconfig b/arch/arm/configs/zii_vf610_dev_defconfig
> index 45c24d6df471..3ed5d374586d 100644
> --- a/arch/arm/configs/zii_vf610_dev_defconfig
> +++ b/arch/arm/configs/zii_vf610_dev_defconfig
> @@ -8,7 +8,6 @@ CONFIG_MMU=y
>  CONFIG_MALLOC_SIZE=0x0
>  CONFIG_MALLOC_TLSF=y
>  CONFIG_KALLSYMS=y
> -CONFIG_RELOCATABLE=y
>  CONFIG_HUSH_FANCY_PROMPT=y
>  CONFIG_AUTO_COMPLETE=y
>  CONFIG_MENU=y
> diff --git a/arch/kvx/configs/generic_defconfig b/arch/kvx/configs/generic_defconfig
> index 816217174183..0d971ff3d5dd 100644
> --- a/arch/kvx/configs/generic_defconfig
> +++ b/arch/kvx/configs/generic_defconfig
> @@ -1,15 +1,13 @@
>  CONFIG_AUTO_COMPLETE=y
> -CONFIG_BAUDRATE=115200
> -CONFIG_CLOCKSOURCE_KVX=y
> +CONFIG_CONSOLE_RATP=y
>  CONFIG_CMD_BOOT=y
> -CONFIG_CMD_BOOTM=y
> +CONFIG_CMD_RESET=y
>  CONFIG_CMD_CMP=y
> -CONFIG_CMD_OF_DUMP=y
>  CONFIG_CMD_POWEROFF=y
> -CONFIG_CMD_RESET=y
>  CONFIG_CMD_WD=y
> -CONFIG_CONSOLE_RATP=y
> +CONFIG_CMD_OF_DUMP=y
>  CONFIG_DRIVER_SERIAL_NS16550=y
> -CONFIG_PINCTRL_SINGLE=y
> +CONFIG_CLOCKSOURCE_KVX=y
>  CONFIG_WATCHDOG=y
>  CONFIG_WATCHDOG_KVX=y
> +CONFIG_PINCTRL_SINGLE=y
> diff --git a/arch/mips/configs/bcm47xx_defconfig b/arch/mips/configs/bcm47xx_defconfig
> index 0cd0e9e650e7..9cf172a70e57 100644
> --- a/arch/mips/configs/bcm47xx_defconfig
> +++ b/arch/mips/configs/bcm47xx_defconfig
> @@ -9,7 +9,6 @@ CONFIG_MENU=y
>  CONFIG_BOOTM_SHOW_TYPE=y
>  CONFIG_PARTITION=y
>  # CONFIG_DEFAULT_ENVIRONMENT is not set
> -CONFIG_POLLER=y
>  CONFIG_LONGHELP=y
>  CONFIG_CMD_IOMEM=y
>  CONFIG_CMD_MEMINFO=y
> diff --git a/arch/mips/configs/qemu-malta_defconfig b/arch/mips/configs/qemu-malta_defconfig
> index 2465c0260d1d..ac0577c217e3 100644
> --- a/arch/mips/configs/qemu-malta_defconfig
> +++ b/arch/mips/configs/qemu-malta_defconfig
> @@ -12,7 +12,6 @@ CONFIG_BOOTM_SHOW_TYPE=y
>  CONFIG_CONSOLE_ALLOW_COLOR=y
>  CONFIG_PARTITION=y
>  CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
> -CONFIG_POLLER=y
>  CONFIG_CMD_DMESG=y
>  CONFIG_LONGHELP=y
>  CONFIG_CMD_IOMEM=y
> diff --git a/arch/x86/configs/efi_defconfig b/arch/x86/configs/efi_defconfig
> index 47842d10afdd..99bc39eaa6f1 100644
> --- a/arch/x86/configs/efi_defconfig
> +++ b/arch/x86/configs/efi_defconfig
> @@ -14,7 +14,6 @@ CONFIG_BLSPEC=y
>  CONFIG_CONSOLE_ACTIVATE_ALL=y
>  CONFIG_PARTITION_DISK_EFI=y
>  CONFIG_DEFAULT_ENVIRONMENT_GENERIC_NEW=y
> -CONFIG_POLLER=y
>  CONFIG_STATE=y
>  CONFIG_DEBUG_LL=y
>  CONFIG_LONGHELP=y
> -- 
> 2.27.0
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-09-18 11:04 ` [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550 Uwe Kleine-König
@ 2020-10-20  5:08   ` Ahmad Fatoum
  2020-10-20  7:26     ` Uwe Kleine-König
  0 siblings, 1 reply; 9+ messages in thread
From: Ahmad Fatoum @ 2020-10-20  5:08 UTC (permalink / raw)
  To: Uwe Kleine-König, barebox

Hello Uwe,

On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
> For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
> both enabled results in the output appearing twice on the serial output.
> So remove disable the lowlevel driver and rely on the EFI bios for serial.

On the flip-side, some systems now won't output anything over serial with
the defconfig, like the one I just ran into. Maybe we should keep both
enabled, but have a quirk table, which boards to disable the 16550 for?

My board is
  EFI v2.40 by American Megatrends v327691". What's yours?
  barebox@barebox EFI payload:/ devinfo acpi-FACP0
  [...]
  Revision: 5
  OemId: ALASKA
  OemTableId: A M I
  OemRevision: 17244169
  CreatorId: 0x20494d41
  CreatorRevision: 65555

What's yours?
 
Cheers,
Ahmad

 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
>  arch/x86/configs/efi_defconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/x86/configs/efi_defconfig b/arch/x86/configs/efi_defconfig
> index 99bc39eaa6f1..83794d7a0702 100644
> --- a/arch/x86/configs/efi_defconfig
> +++ b/arch/x86/configs/efi_defconfig
> @@ -65,7 +65,6 @@ CONFIG_NET=y
>  CONFIG_NET_NFS=y
>  CONFIG_NET_NETCONSOLE=y
>  CONFIG_DRIVER_SERIAL_EFI_STDIO=y
> -CONFIG_DRIVER_SERIAL_NS16550=y
>  CONFIG_DRIVER_NET_EFI_SNP=y
>  # CONFIG_SPI is not set
>  CONFIG_DISK=y
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-10-20  5:08   ` Ahmad Fatoum
@ 2020-10-20  7:26     ` Uwe Kleine-König
  2020-10-20  7:35       ` Ahmad Fatoum
  0 siblings, 1 reply; 9+ messages in thread
From: Uwe Kleine-König @ 2020-10-20  7:26 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox


[-- Attachment #1.1: Type: text/plain, Size: 2339 bytes --]

On Tue, Oct 20, 2020 at 07:08:47AM +0200, Ahmad Fatoum wrote:
> Hello Uwe,
> 
> On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
> > For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
> > both enabled results in the output appearing twice on the serial output.
> > So remove disable the lowlevel driver and rely on the EFI bios for serial.

How embarrassing, s/remove //.

> On the flip-side, some systems now won't output anything over serial with
> the defconfig, like the one I just ran into. Maybe we should keep both
> enabled, but have a quirk table, which boards to disable the 16550 for?

Expecting that most systems are happy with EFI_STDIO I would suggest a
quirk table for those systems which are not?!

> My board is
>   EFI v2.40 by American Megatrends v327691". What's yours?
>   barebox@barebox EFI payload:/ devinfo acpi-FACP0
>   [...]
>   Revision: 5
>   OemId: ALASKA
>   OemTableId: A M I
>   OemRevision: 17244169
>   CreatorId: 0x20494d41
>   CreatorRevision: 65555
> 
> What's yours?

I saw the doubled output on my Laptop (Lenovo T460p).

dmidecode has:

	BIOS Information
		Vendor: LENOVO
		Version: R07ET81W (2.21 )
		Release Date: 12/18/2017
		Address: 0xE0000
		Runtime Size: 128 kB
		ROM Size: 16 MB
		Characteristics:
			PCI is supported
			PNP is supported
			BIOS is upgradeable
			BIOS shadowing is allowed
			Boot from CD is supported
			Selectable boot is supported
			EDD is supported
			3.5"/720 kB floppy services are supported (int 13h)
			Print screen service is supported (int 5h)
			8042 keyboard services are supported (int 9h)
			Serial services are supported (int 14h)
			Printer services are supported (int 17h)
			CGA/mono video services are supported (int 10h)
			ACPI is supported
			USB legacy is supported
			BIOS boot specification is supported
			Targeted content distribution is supported
			UEFI is supported
		BIOS Revision: 2.21
		Firmware Revision: 1.4

(This is the system I write this mail on, I don't feel like rebooting to check
devinfo in barebox, take a picture and then type the stuff into a mail
by hand.)

Best regards
Uwe


-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 149 bytes --]

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-10-20  7:26     ` Uwe Kleine-König
@ 2020-10-20  7:35       ` Ahmad Fatoum
  2020-10-26 18:06         ` Michael Olbrich
  0 siblings, 1 reply; 9+ messages in thread
From: Ahmad Fatoum @ 2020-10-20  7:35 UTC (permalink / raw)
  To: Uwe Kleine-König; +Cc: barebox

Hi,

On 10/20/20 9:26 AM, Uwe Kleine-König wrote:
> On Tue, Oct 20, 2020 at 07:08:47AM +0200, Ahmad Fatoum wrote:
>> Hello Uwe,
>>
>> On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
>>> For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
>>> both enabled results in the output appearing twice on the serial output.
>>> So remove disable the lowlevel driver and rely on the EFI bios for serial.
> 
> How embarrassing, s/remove //.
> 
>> On the flip-side, some systems now won't output anything over serial with
>> the defconfig, like the one I just ran into. Maybe we should keep both
>> enabled, but have a quirk table, which boards to disable the 16550 for?
> 
> Expecting that most systems are happy with EFI_STDIO I would suggest a
> quirk table for those systems which are not?!

Doubled output is better than no output, so a defconfig that
by default displays anything if the BIOS is not in the quirk
table sounds like it's the better choice?

> 
>> My board is
>>   EFI v2.40 by American Megatrends v327691". What's yours?
>>   barebox@barebox EFI payload:/ devinfo acpi-FACP0
>>   [...]
>>   Revision: 5
>>   OemId: ALASKA
>>   OemTableId: A M I
>>   OemRevision: 17244169
>>   CreatorId: 0x20494d41
>>   CreatorRevision: 65555
>>
>> What's yours?
> 
> I saw the doubled output on my Laptop (Lenovo T460p).

You have serial output on your laptop?

> 
> dmidecode has:
> 
> 	BIOS Information
> 		Vendor: LENOVO
> 		Version: R07ET81W (2.21 )
> 		Release Date: 12/18/2017
> 		Address: 0xE0000
> 		Runtime Size: 128 kB
> 		ROM Size: 16 MB
> 		Characteristics:
> 			PCI is supported
> 			PNP is supported
> 			BIOS is upgradeable
> 			BIOS shadowing is allowed
> 			Boot from CD is supported
> 			Selectable boot is supported
> 			EDD is supported
> 			3.5"/720 kB floppy services are supported (int 13h)
> 			Print screen service is supported (int 5h)
> 			8042 keyboard services are supported (int 9h)
> 			Serial services are supported (int 14h)
> 			Printer services are supported (int 17h)
> 			CGA/mono video services are supported (int 10h)
> 			ACPI is supported
> 			USB legacy is supported
> 			BIOS boot specification is supported
> 			Targeted content distribution is supported
> 			UEFI is supported
> 		BIOS Revision: 2.21
> 		Firmware Revision: 1.4
> 
> (This is the system I write this mail on, I don't feel like rebooting to check
> devinfo in barebox, take a picture and then type the stuff into a mail
> by hand.)
> 
> Best regards
> Uwe
> 
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-10-20  7:35       ` Ahmad Fatoum
@ 2020-10-26 18:06         ` Michael Olbrich
  2020-10-26 21:07           ` Ahmad Fatoum
  0 siblings, 1 reply; 9+ messages in thread
From: Michael Olbrich @ 2020-10-26 18:06 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox, Uwe Kleine-König

On Tue, Oct 20, 2020 at 09:35:30AM +0200, Ahmad Fatoum wrote:
> On 10/20/20 9:26 AM, Uwe Kleine-König wrote:
> > On Tue, Oct 20, 2020 at 07:08:47AM +0200, Ahmad Fatoum wrote:
> >> Hello Uwe,
> >>
> >> On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
> >>> For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
> >>> both enabled results in the output appearing twice on the serial output.
> >>> So remove disable the lowlevel driver and rely on the EFI bios for serial.
> > 
> > How embarrassing, s/remove //.
> > 
> >> On the flip-side, some systems now won't output anything over serial with
> >> the defconfig, like the one I just ran into. Maybe we should keep both
> >> enabled, but have a quirk table, which boards to disable the 16550 for?
> > 
> > Expecting that most systems are happy with EFI_STDIO I would suggest a
> > quirk table for those systems which are not?!
> 
> Doubled output is better than no output, so a defconfig that
> by default displays anything if the BIOS is not in the quirk
> table sounds like it's the better choice?

Maybe disable CONSOLE_ACTIVATE_ALL instead? Then you should only get the
output from one driver.

Michael

> > 
> >> My board is
> >>   EFI v2.40 by American Megatrends v327691". What's yours?
> >>   barebox@barebox EFI payload:/ devinfo acpi-FACP0
> >>   [...]
> >>   Revision: 5
> >>   OemId: ALASKA
> >>   OemTableId: A M I
> >>   OemRevision: 17244169
> >>   CreatorId: 0x20494d41
> >>   CreatorRevision: 65555
> >>
> >> What's yours?
> > 
> > I saw the doubled output on my Laptop (Lenovo T460p).
> 
> You have serial output on your laptop?
> 
> > 
> > dmidecode has:
> > 
> > 	BIOS Information
> > 		Vendor: LENOVO
> > 		Version: R07ET81W (2.21 )
> > 		Release Date: 12/18/2017
> > 		Address: 0xE0000
> > 		Runtime Size: 128 kB
> > 		ROM Size: 16 MB
> > 		Characteristics:
> > 			PCI is supported
> > 			PNP is supported
> > 			BIOS is upgradeable
> > 			BIOS shadowing is allowed
> > 			Boot from CD is supported
> > 			Selectable boot is supported
> > 			EDD is supported
> > 			3.5"/720 kB floppy services are supported (int 13h)
> > 			Print screen service is supported (int 5h)
> > 			8042 keyboard services are supported (int 9h)
> > 			Serial services are supported (int 14h)
> > 			Printer services are supported (int 17h)
> > 			CGA/mono video services are supported (int 10h)
> > 			ACPI is supported
> > 			USB legacy is supported
> > 			BIOS boot specification is supported
> > 			Targeted content distribution is supported
> > 			UEFI is supported
> > 		BIOS Revision: 2.21
> > 		Firmware Revision: 1.4
> > 
> > (This is the system I write this mail on, I don't feel like rebooting to check
> > devinfo in barebox, take a picture and then type the stuff into a mail
> > by hand.)
> > 
> > Best regards
> > Uwe
> > 
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-10-26 18:06         ` Michael Olbrich
@ 2020-10-26 21:07           ` Ahmad Fatoum
  2020-10-28  8:38             ` Sascha Hauer
  0 siblings, 1 reply; 9+ messages in thread
From: Ahmad Fatoum @ 2020-10-26 21:07 UTC (permalink / raw)
  To: Uwe Kleine-König, barebox



On 10/26/20 7:06 PM, Michael Olbrich wrote:
> On Tue, Oct 20, 2020 at 09:35:30AM +0200, Ahmad Fatoum wrote:
>> On 10/20/20 9:26 AM, Uwe Kleine-König wrote:
>>> On Tue, Oct 20, 2020 at 07:08:47AM +0200, Ahmad Fatoum wrote:
>>>> Hello Uwe,
>>>>
>>>> On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
>>>>> For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
>>>>> both enabled results in the output appearing twice on the serial output.
>>>>> So remove disable the lowlevel driver and rely on the EFI bios for serial.
>>>
>>> How embarrassing, s/remove //.
>>>
>>>> On the flip-side, some systems now won't output anything over serial with
>>>> the defconfig, like the one I just ran into. Maybe we should keep both
>>>> enabled, but have a quirk table, which boards to disable the 16550 for?
>>>
>>> Expecting that most systems are happy with EFI_STDIO I would suggest a
>>> quirk table for those systems which are not?!
>>
>> Doubled output is better than no output, so a defconfig that
>> by default displays anything if the BIOS is not in the quirk
>> table sounds like it's the better choice?
> 
> Maybe disable CONSOLE_ACTIVATE_ALL instead? Then you should only get the
> output from one driver.

And which one should be default? You'll get more compile time coverage
with the defconfig but the user experience problem of having a defconfig
that does print nothing at all on some devices is still there.

> 
> Michael
> 
>>>
>>>> My board is
>>>>   EFI v2.40 by American Megatrends v327691". What's yours?
>>>>   barebox@barebox EFI payload:/ devinfo acpi-FACP0
>>>>   [...]
>>>>   Revision: 5
>>>>   OemId: ALASKA
>>>>   OemTableId: A M I
>>>>   OemRevision: 17244169
>>>>   CreatorId: 0x20494d41
>>>>   CreatorRevision: 65555
>>>>
>>>> What's yours?
>>>
>>> I saw the doubled output on my Laptop (Lenovo T460p).
>>
>> You have serial output on your laptop?
>>
>>>
>>> dmidecode has:
>>>
>>> 	BIOS Information
>>> 		Vendor: LENOVO
>>> 		Version: R07ET81W (2.21 )
>>> 		Release Date: 12/18/2017
>>> 		Address: 0xE0000
>>> 		Runtime Size: 128 kB
>>> 		ROM Size: 16 MB
>>> 		Characteristics:
>>> 			PCI is supported
>>> 			PNP is supported
>>> 			BIOS is upgradeable
>>> 			BIOS shadowing is allowed
>>> 			Boot from CD is supported
>>> 			Selectable boot is supported
>>> 			EDD is supported
>>> 			3.5"/720 kB floppy services are supported (int 13h)
>>> 			Print screen service is supported (int 5h)
>>> 			8042 keyboard services are supported (int 9h)
>>> 			Serial services are supported (int 14h)
>>> 			Printer services are supported (int 17h)
>>> 			CGA/mono video services are supported (int 10h)
>>> 			ACPI is supported
>>> 			USB legacy is supported
>>> 			BIOS boot specification is supported
>>> 			Targeted content distribution is supported
>>> 			UEFI is supported
>>> 		BIOS Revision: 2.21
>>> 		Firmware Revision: 1.4
>>>
>>> (This is the system I write this mail on, I don't feel like rebooting to check
>>> devinfo in barebox, take a picture and then type the stuff into a mail
>>> by hand.)
>>>
>>> Best regards
>>> Uwe
>>>
>>>
>>
>> -- 
>> Pengutronix e.K.                           |                             |
>> Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
>> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
>> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
>>
>> _______________________________________________
>> barebox mailing list
>> barebox@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/barebox
>>
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550
  2020-10-26 21:07           ` Ahmad Fatoum
@ 2020-10-28  8:38             ` Sascha Hauer
  0 siblings, 0 replies; 9+ messages in thread
From: Sascha Hauer @ 2020-10-28  8:38 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox, Uwe Kleine-König

On Mon, Oct 26, 2020 at 10:07:19PM +0100, Ahmad Fatoum wrote:
> 
> 
> On 10/26/20 7:06 PM, Michael Olbrich wrote:
> > On Tue, Oct 20, 2020 at 09:35:30AM +0200, Ahmad Fatoum wrote:
> >> On 10/20/20 9:26 AM, Uwe Kleine-König wrote:
> >>> On Tue, Oct 20, 2020 at 07:08:47AM +0200, Ahmad Fatoum wrote:
> >>>> Hello Uwe,
> >>>>
> >>>> On 9/18/20 1:04 PM, Uwe Kleine-König wrote:
> >>>>> For most machines having DRIVER_SERIAL_NS16550 and DRIVER_SERIAL_EFI_STDIO
> >>>>> both enabled results in the output appearing twice on the serial output.
> >>>>> So remove disable the lowlevel driver and rely on the EFI bios for serial.
> >>>
> >>> How embarrassing, s/remove //.
> >>>
> >>>> On the flip-side, some systems now won't output anything over serial with
> >>>> the defconfig, like the one I just ran into. Maybe we should keep both
> >>>> enabled, but have a quirk table, which boards to disable the 16550 for?
> >>>
> >>> Expecting that most systems are happy with EFI_STDIO I would suggest a
> >>> quirk table for those systems which are not?!
> >>
> >> Doubled output is better than no output, so a defconfig that
> >> by default displays anything if the BIOS is not in the quirk
> >> table sounds like it's the better choice?
> > 
> > Maybe disable CONSOLE_ACTIVATE_ALL instead? Then you should only get the
> > output from one driver.
> 
> And which one should be default? You'll get more compile time coverage
> with the defconfig but the user experience problem of having a defconfig
> that does print nothing at all on some devices is still there.

Maybe there is some way to find out where DRIVER_SERIAL_EFI_STDIO
directs it output to?

Sascha

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-10-28  8:38 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-18 11:04 [PATCH 1/2] Regenerate defconfig files Uwe Kleine-König
2020-09-18 11:04 ` [PATCH 2/2] x86/config/efi: drop DRIVER_SERIAL_NS16550 Uwe Kleine-König
2020-10-20  5:08   ` Ahmad Fatoum
2020-10-20  7:26     ` Uwe Kleine-König
2020-10-20  7:35       ` Ahmad Fatoum
2020-10-26 18:06         ` Michael Olbrich
2020-10-26 21:07           ` Ahmad Fatoum
2020-10-28  8:38             ` Sascha Hauer
2020-09-23  8:47 ` [PATCH 1/2] Regenerate defconfig files Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox