Messages in this thread | | | From | "Agarwal, Nikhil" <> | Subject | RE: [PATCH 1/2] cdx: Enable COMPILE_TEST | Date | Tue, 12 Dec 2023 04:40:29 +0000 |
| |
> -----Original Message----- > From: Rob Herring <robh@kernel.org> > Sent: Thursday, December 7, 2023 10:01 PM > To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Gupta, Nipun > <Nipun.Gupta@amd.com>; Agarwal, Nikhil <nikhil.agarwal@amd.com> > Cc: linux-kernel@vger.kernel.org > Subject: [PATCH 1/2] cdx: Enable COMPILE_TEST > > There is no reason CDX needs to depend on ARM64 other than limiting > visibility. So let's also enable building with COMPILE_TEST. > > The CONFIG_OF dependency is redundant as ARM64 always enables it and all > the DT functions have empty stubs. > > Signed-off-by: Rob Herring <robh@kernel.org> > --- > drivers/cdx/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cdx/Kconfig b/drivers/cdx/Kconfig index > a08958485e31..7cdb7c414453 100644 > --- a/drivers/cdx/Kconfig > +++ b/drivers/cdx/Kconfig > @@ -7,7 +7,7 @@ > > config CDX_BUS > bool "CDX Bus driver" > - depends on OF && ARM64 > + depends on ARM64 || COMPILE_TEST Hi Rob,
There is a CDX MSI support patch https://lore.kernel.org/lkml/20231116125609.245206-1-nipun.gupta@amd.com/ which is in review and is dependent on ARM64( msi_alloc_info_t definition differs on x86). So, the COMPILE_TEST would break once the MSI changes are added.
Regards Nikhil
| |