[PATCH v10 2/7] iommu: Add iommu probe for CDX bus

[Gupta, Nipun]

[AMD Official Use Only - General]


> -----Original Message-----
> From: Greg KH <gregkh at linuxfoundation.org>
> Sent: Wednesday, March 29, 2023 3:56 PM
> To: Gupta, Nipun <Nipun.Gupta at amd.com>
> Cc: robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org;
> rafael at kernel.org; eric.auger at redhat.com; alex.williamson at redhat.com;
> cohuck at redhat.com; song.bao.hua at hisilicon.com;
> mchehab+huawei at kernel.org; maz at kernel.org; f.fainelli at gmail.com;
> jeffrey.l.hugo at gmail.com; saravanak at google.com;
> Michael.Srba at seznam.cz; mani at kernel.org; yishaih at nvidia.com;
> jgg at ziepe.ca; jgg at nvidia.com; robin.murphy at arm.com; will at kernel.org;
> joro at 8bytes.org; masahiroy at kernel.org; ndesaulniers at google.com;
> rdunlap at infradead.org; baolu.lu at linux.intel.com; linux-arm-
> kernel at lists.infradead.org; linux-kbuild at vger.kernel.org; linux-
> kernel at vger.kernel.org; devicetree at vger.kernel.org; okaya at kernel.org;
> Anand, Harpreet <harpreet.anand at amd.com>; Agarwal, Nikhil
> <nikhil.agarwal at amd.com>; Simek, Michal <michal.simek at amd.com>;
> Jansen Van Vuuren, Pieter <pieter.jansen-van-vuuren at amd.com>; Cascon,
> Pablo <pablo.cascon at amd.com>; git (AMD-Xilinx) <git at amd.com>
> Subject: Re: [PATCH v10 2/7] iommu: Add iommu probe for CDX bus
> 
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
> 
> 
> On Mon, Mar 13, 2023 at 06:56:31PM +0530, Nipun Gupta wrote:
> > Add CDX bus to iommu_buses so that IOMMU probe is called
> > for it.
> >
> > Signed-off-by: Nipun Gupta <nipun.gupta at amd.com>
> > Reviewed-by: Pieter Jansen van Vuuren <pieter.jansen-van-
> vuuren at amd.com>
> > Reviewed-by: Lu Baolu <baolu.lu at linux.intel.com>
> > Tested-by: Nikhil Agarwal <nikhil.agarwal at amd.com>
> > ---
> >  drivers/iommu/iommu.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
> > index 10db680acaed..0025a631b8e1 100644
> > --- a/drivers/iommu/iommu.c
> > +++ b/drivers/iommu/iommu.c
> > @@ -28,6 +28,7 @@
> >  #include <linux/fsl/mc.h>
> >  #include <linux/module.h>
> >  #include <linux/cc_platform.h>
> > +#include <linux/cdx/cdx_bus.h>
> >  #include <trace/events/iommu.h>
> >  #include <linux/sched/mm.h>
> >  #include <linux/msi.h>
> > @@ -129,6 +130,9 @@ static struct bus_type * const iommu_buses[] = {
> >  #ifdef CONFIG_TEGRA_HOST1X_CONTEXT_BUS
> >       &host1x_context_device_bus_type,
> >  #endif
> > +#ifdef CONFIG_CDX_BUS
> > +     &cdx_bus_type,
> > +#endif
> >  };
> 
> Note, this prevents the ability for this struct bus_type from being
> moved into read-only memory in the future.  But hopefully that will be
> fixed up eventually, you are in good company with some other busses
> here...
> 
> Other than that, this series looks good, I'll queue this up through my
> tree for now as I seem to collect new subsystems.  Feel free to send
> future patches through me as well until you get tired of that and want
> to send stuff to Linus directly.

Thanks, Greg, for the review and helping get the series in : )

Regards,
Nipun

> 
> thanks,
> 
> greg k-h