[PATCH v7 1/2] xen/char: Default HAS_NS16550 to y only for X86 and ARM

Connor Davis posted 2 patches 4 years, 8 months ago
There is a newer version of this series
[PATCH v7 1/2] xen/char: Default HAS_NS16550 to y only for X86 and ARM
Posted by Connor Davis 4 years, 8 months ago
Defaulting to yes only for X86 and ARM reduces the requirements
for a minimal build when porting new architectures.

Signed-off-by: Connor Davis <connojdavis@gmail.com>
Acked-by: Jan Beulich <jbeulich@suse.com>
---
 xen/drivers/char/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/xen/drivers/char/Kconfig b/xen/drivers/char/Kconfig
index b572305657..2ff5b288e2 100644
--- a/xen/drivers/char/Kconfig
+++ b/xen/drivers/char/Kconfig
@@ -1,5 +1,6 @@
 config HAS_NS16550
 	bool "NS16550 UART driver" if ARM
+	default n if RISCV
 	default y
 	help
 	  This selects the 16550-series UART support. For most systems, say Y.
-- 
2.31.1


Re: [PATCH v7 1/2] xen/char: Default HAS_NS16550 to y only for X86 and ARM
Posted by Alistair Francis 4 years, 8 months ago
On Thu, Jun 3, 2021 at 9:38 AM Connor Davis <connojdavis@gmail.com> wrote:
>
> Defaulting to yes only for X86 and ARM reduces the requirements
> for a minimal build when porting new architectures.
>
> Signed-off-by: Connor Davis <connojdavis@gmail.com>
> Acked-by: Jan Beulich <jbeulich@suse.com>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  xen/drivers/char/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/xen/drivers/char/Kconfig b/xen/drivers/char/Kconfig
> index b572305657..2ff5b288e2 100644
> --- a/xen/drivers/char/Kconfig
> +++ b/xen/drivers/char/Kconfig
> @@ -1,5 +1,6 @@
>  config HAS_NS16550
>         bool "NS16550 UART driver" if ARM
> +       default n if RISCV
>         default y
>         help
>           This selects the 16550-series UART support. For most systems, say Y.
> --
> 2.31.1
>

Re: [PATCH v7 1/2] xen/char: Default HAS_NS16550 to y only for X86 and ARM
Posted by Connor Davis 4 years, 8 months ago

On 6/3/21 5:27 PM, Alistair Francis wrote:
> On Thu, Jun 3, 2021 at 9:38 AM Connor Davis <connojdavis@gmail.com> wrote:
>> Defaulting to yes only for X86 and ARM reduces the requirements
>> for a minimal build when porting new architectures.
>>
>> Signed-off-by: Connor Davis <connojdavis@gmail.com>
>> Acked-by: Jan Beulich <jbeulich@suse.com>
> Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
>
> Alistair
Thanks, added

Connor