Re: [PATCH v1 1/4] xen/ns16550: solve compilation error on ARM with CONFIG_HAS_PCI enabled.

[Julien Grall <julien@xxxxxxx>]

Hi,

On 28/10/2020 10:41, Rahul Singh wrote:
> > On 28 Oct 2020, at 7:18 am, Jan Beulich <jbeulich@xxxxxxxx> wrote:
> >
> > On 28.10.2020 00:32, Stefano Stabellini wrote:
> > > On Mon, 26 Oct 2020, Rahul Singh wrote:
> > > > --- a/xen/drivers/char/Kconfig
> > > > +++ b/xen/drivers/char/Kconfig
> > > > @@ -4,6 +4,13 @@ config HAS_NS16550
> > > >         help
> > > >           This selects the 16550-series UART support. For most systems, say Y.
> > > >
> > > > +config HAS_NS16550_PCI
> > > > +       bool "NS16550 UART PCI support" if X86
> > > > +       default y
> > > > +       depends on X86 && HAS_NS16550 && HAS_PCI
> > > > +       help
> > > > +         This selects the 16550-series UART PCI support. For most systems, say 
> > > > Y.
> > > I think that this should be a silent option:
> > > if HAS_NS16550 && HAS_PCI && X86 -> automatically enable
> > > otherwise -> automatically disable
> > >
> > > No need to show it to the user.
> > I agree in principle, but I don't see why an X86 dependency gets
> > added here. HAS_PCI really should be all that's needed.
> Yes you are right . I will remove the X86 and make HAS_NS16550_PCI depend on "HAS_NS16550 
> && HAS_PCI"
There are quite a bit of work to make the PCI part of the implementation 
build on Arm because the code refer to x86 functions.
While in theory, an NS16550 PCI card could be used on Arm, there is only 
a slim chance for an actual users. So I am not convinced the effort is 
worth it here.
Cheers,

--
Julien Grall