Re: [PATCH] PCI: Fix PCI kconfig menu organization

From: Christoph Hellwig
Date: Tue Jan 15 2019 - 08:11:21 EST


On Mon, Jan 14, 2019 at 03:35:46PM -0600, Rob Herring wrote:
> After commit eb01d42a7778 ("PCI: consolidate PCI config entry in
> drivers/pci"), all the PCI kconfig options appear below "PCI support"
> rather than within a sub-menu. This is because menuconfig expects all
> kconfig entries to be enclosed in an if/endif section. Add the missing
> if/endif.
>
> With this, "depends on PCI" is redundant in the sub-menu entries and
> can be removed.

Thanks, this looks fine to me:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>