From: "Gupta, Nipun" <Nipun.Gupta@amd.com> To: Greg KH <gregkh@linuxfoundation.org> Cc: "robh+dt@kernel.org" <robh+dt@kernel.org>, "krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>, "rafael@kernel.org" <rafael@kernel.org>, "eric.auger@redhat.com" <eric.auger@redhat.com>, "alex.williamson@redhat.com" <alex.williamson@redhat.com>, "cohuck@redhat.com" <cohuck@redhat.com>, "Gupta, Puneet (DCG-ENG)" <puneet.gupta@amd.com>, "song.bao.hua@hisilicon.com" <song.bao.hua@hisilicon.com>, "mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>, "maz@kernel.org" <maz@kernel.org>, "f.fainelli@gmail.com" <f.fainelli@gmail.com>, "jeffrey.l.hugo@gmail.com" <jeffrey.l.hugo@gmail.com>, "saravanak@google.com" <saravanak@google.com>, "Michael.Srba@seznam.cz" <Michael.Srba@seznam.cz>, "mani@kernel.org" <mani@kernel.org>, "yishaih@nvidia.com" <yishaih@nvidia.com>, "jgg@ziepe.ca" <jgg@ziepe.ca>, "jgg@nvidia.com" <jgg@nvidia.com>, "robin.murphy@arm.com" <robin.murphy@arm.com>, "will@kernel.org" <will@kernel.org>, "joro@8bytes.org" <joro@8bytes.org>, "masahiroy@kernel.org" <masahiroy@kernel.org>, "ndesaulniers@google.com" <ndesaulniers@google.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kbuild@vger.kernel.org" <linux-kbuild@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "kvm@vger.kernel.org" <kvm@vger.kernel.org>, "okaya@kernel.org" <okaya@kernel.org>, "Anand, Harpreet" <harpreet.anand@amd.com>, "Agarwal, Nikhil" <nikhil.agarwal@amd.com>, "Simek, Michal" <michal.simek@amd.com>, "Radovanovic, Aleksandar" <aleksandar.radovanovic@amd.com>, "git (AMD-Xilinx)" <git@amd.com> Subject: RE: [RFC PATCH v4 2/8] bus/cdx: add the cdx bus driver Date: Fri, 14 Oct 2022 08:20:31 +0000 [thread overview] Message-ID: <DM6PR12MB3082BB5B7E5756450BBC0DEBE8249@DM6PR12MB3082.namprd12.prod.outlook.com> (raw) In-Reply-To: <Y0kNPHOjKiQ3XfqI@kroah.com> [AMD Official Use Only - General] > -----Original Message----- > From: Greg KH <gregkh@linuxfoundation.org> > Sent: Friday, October 14, 2022 12:48 PM > To: Gupta, Nipun <Nipun.Gupta@amd.com> > Cc: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; rafael@kernel.org; > eric.auger@redhat.com; alex.williamson@redhat.com; cohuck@redhat.com; > Gupta, Puneet (DCG-ENG) <puneet.gupta@amd.com>; > song.bao.hua@hisilicon.com; mchehab+huawei@kernel.org; maz@kernel.org; > f.fainelli@gmail.com; jeffrey.l.hugo@gmail.com; saravanak@google.com; > Michael.Srba@seznam.cz; mani@kernel.org; yishaih@nvidia.com; > jgg@ziepe.ca; jgg@nvidia.com; robin.murphy@arm.com; will@kernel.org; > joro@8bytes.org; masahiroy@kernel.org; ndesaulniers@google.com; linux-arm- > kernel@lists.infradead.org; linux-kbuild@vger.kernel.org; linux- > kernel@vger.kernel.org; devicetree@vger.kernel.org; kvm@vger.kernel.org; > okaya@kernel.org; Anand, Harpreet <harpreet.anand@amd.com>; Agarwal, > Nikhil <nikhil.agarwal@amd.com>; Simek, Michal <michal.simek@amd.com>; > Radovanovic, Aleksandar <aleksandar.radovanovic@amd.com>; git (AMD-Xilinx) > <git@amd.com> > Subject: Re: [RFC PATCH v4 2/8] bus/cdx: add the cdx bus driver > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > On Fri, Oct 14, 2022 at 10:10:43AM +0530, Nipun Gupta wrote: > > --- a/drivers/bus/Makefile > > +++ b/drivers/bus/Makefile > > @@ -20,6 +20,9 @@ obj-$(CONFIG_INTEL_IXP4XX_EB) += intel-ixp4xx- > eb.o > > obj-$(CONFIG_MIPS_CDMM) += mips_cdmm.o > > obj-$(CONFIG_MVEBU_MBUS) += mvebu-mbus.o > > > > +#CDX bus > > No need for a comment like this :) Sure, will remove. > > > +obj-$(CONFIG_CDX_BUS) += cdx/ > > + > > # Interconnect bus driver for OMAP SoCs. > > obj-$(CONFIG_OMAP_INTERCONNECT) += omap_l3_smx.o omap_l3_noc.o > > > > diff --git a/drivers/bus/cdx/Kconfig b/drivers/bus/cdx/Kconfig > > new file mode 100644 > > index 000000000000..98ec05ad708d > > --- /dev/null > > +++ b/drivers/bus/cdx/Kconfig > > @@ -0,0 +1,13 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +# > > +# CDX bus configuration > > +# > > +# Copyright (C) 2022, Advanced Micro Devices, Inc. > > +# > > + > > +config CDX_BUS > > + bool "CDX Bus driver" > > + help > > + Driver to enable CDX Bus infrastructure. CDX bus uses > > + CDX controller and firmware to scan the FPGA based > > + devices. > > Why bool? Not as a module? That's broken. Okay. We see that most of other busses like amba, fslmc, pci are bool only. As we have MSI domain as part of this bus, we need to export multiple symbols to make it work as a module. > > And "FPGA based devices" means nothing to me, please expand on what this > all is as it is not descriptive at all. > > Also list the module name please. > > > > > diff --git a/drivers/bus/cdx/Makefile b/drivers/bus/cdx/Makefile > > new file mode 100644 > > index 000000000000..2e8f42611dfc > > --- /dev/null > > +++ b/drivers/bus/cdx/Makefile > > @@ -0,0 +1,8 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +# > > +# Makefile for CDX > > +# > > +# Copyright (C) 2022, Advanced Micro Devices, Inc. > > +# > > + > > +obj-$(CONFIG_CDX_BUS) += cdx.o > > diff --git a/drivers/bus/cdx/cdx.c b/drivers/bus/cdx/cdx.c > > new file mode 100644 > > index 000000000000..5a366f4ae69c > > --- /dev/null > > +++ b/drivers/bus/cdx/cdx.c > > @@ -0,0 +1,366 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * CDX bus driver. > > + * > > + * Copyright (C) 2022, Advanced Micro Devices, Inc. > > + */ > > + > > +/* > > + * Architecture Overview > > + * ===================== > > + * CDX is a Hardware Architecture designed for AMD FPGA devices. It > > + * consists of sophisticated mechanism for interaction between FPGA, > > + * Firmware and the APUs (Application CPUs). > > + * > > + * Firmware resides on RPU (Realtime CPUs) which interacts with > > + * the FPGA program manager and the APUs. The RPU provides memory- > mapped > > + * interface (RPU if) which is used to communicate with APUs. > > + * > > + * The diagram below shows an overview of the CDX architecture: > > + * > > + * +--------------------------------------+ > > + * | Application CPUs (APU) | > > + * | | > > + * | CDX device drivers| > > + * | Linux OS | | > > + * | CDX bus | > > + * | | | > > + * | CDX controller | > > + * | | | > > + * +-----------------------------|--------+ > > + * | (discover, config, > > + * | reset, rescan) > > + * | > > + * +------------------------| RPU if |----+ > > + * | | | > > + * | V | > > + * | Realtime CPUs (RPU) | > > + * | | > > + * +--------------------------------------+ > > + * | > > + * +---------------------|----------------+ > > + * | FPGA | | > > + * | +-----------------------+ | > > + * | | | | | > > + * | +-------+ +-------+ +-------+ | > > + * | | dev 1 | | dev 2 | | dev 3 | | > > + * | +-------+ +-------+ +-------+ | > > + * +--------------------------------------+ > > + * > > + * The RPU firmware extracts the device information from the loaded FPGA > > + * image and implements a mechanism that allows the APU drivers to > > + * enumerate such devices (device personality and resource details) via > > + * a dedicated communication channel. RPU mediates operations such as > > + * discover, reset and rescan of the FPGA devices for the APU. This is > > + * done using memory mapped interface provided by the RPU to APU. > > + */ > > + > > +#include <linux/init.h> > > +#include <linux/kernel.h> > > +#include <linux/slab.h> > > +#include <linux/cdx/cdx_bus.h> > > + > > +#include "cdx.h" > > + > > +/* > > + * Default DMA mask for devices on a CDX bus > > + */ > > +#define CDX_DEFAULT_DMA_MASK (~0ULL) > > + > > +static struct cdx_controller_t *cdx_controller; > > You don't get a static device, sorry, everything must be dynamic or your > design is broken. We will remove this static device here. > > And remove the crazy "_t" from your structure name, checkpatch should > have complained about that. Sure. Will remove from other places too. > > > > > + > > +static int reset_cdx_device(struct device *dev, void * __always_unused data) > > __always_unused is never needed. > > But the funny thing is, you added that variable, why are you not using > it? If it's not used, then don't have it. The 'data' parameter is here because the same reset_cdx_device() API is later used in device attributes patch where it is passed as an argument to bus_for_each_dev API, which needs the function to have these two arguments. Agree, that this parameter should be added as part of the device attribute patch, will be updated for next spin. Thanks, Nipun > > Try to get others at AMD to review your code first, before sending your > next batch out, this is really really odd... > > greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: "Gupta, Nipun" <Nipun.Gupta@amd.com> To: Greg KH <gregkh@linuxfoundation.org> Cc: "robh+dt@kernel.org" <robh+dt@kernel.org>, "krzysztof.kozlowski+dt@linaro.org" <krzysztof.kozlowski+dt@linaro.org>, "rafael@kernel.org" <rafael@kernel.org>, "eric.auger@redhat.com" <eric.auger@redhat.com>, "alex.williamson@redhat.com" <alex.williamson@redhat.com>, "cohuck@redhat.com" <cohuck@redhat.com>, "Gupta, Puneet (DCG-ENG)" <puneet.gupta@amd.com>, "song.bao.hua@hisilicon.com" <song.bao.hua@hisilicon.com>, "mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>, "maz@kernel.org" <maz@kernel.org>, "f.fainelli@gmail.com" <f.fainelli@gmail.com>, "jeffrey.l.hugo@gmail.com" <jeffrey.l.hugo@gmail.com>, "saravanak@google.com" <saravanak@google.com>, "Michael.Srba@seznam.cz" <Michael.Srba@seznam.cz>, "mani@kernel.org" <mani@kernel.org>, "yishaih@nvidia.com" <yishaih@nvidia.com>, "jgg@ziepe.ca" <jgg@ziepe.ca>, "jgg@nvidia.com" <jgg@nvidia.com>, "robin.murphy@arm.com" <robin.murphy@arm.com>, "will@kernel.org" <will@kernel.org>, "joro@8bytes.org" <joro@8bytes.org>, "masahiroy@kernel.org" <masahiroy@kernel.org>, "ndesaulniers@google.com" <ndesaulniers@google.com>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kbuild@vger.kernel.org" <linux-kbuild@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "kvm@vger.kernel.org" <kvm@vger.kernel.org>, "okaya@kernel.org" <okaya@kernel.org>, "Anand, Harpreet" <harpreet.anand@amd.com>, "Agarwal, Nikhil" <nikhil.agarwal@amd.com>, "Simek, Michal" <michal.simek@amd.com>, "Radovanovic, Aleksandar" <aleksandar.radovanovic@amd.com>, "git (AMD-Xilinx)" <git@amd.com> Subject: RE: [RFC PATCH v4 2/8] bus/cdx: add the cdx bus driver Date: Fri, 14 Oct 2022 08:20:31 +0000 [thread overview] Message-ID: <DM6PR12MB3082BB5B7E5756450BBC0DEBE8249@DM6PR12MB3082.namprd12.prod.outlook.com> (raw) In-Reply-To: <Y0kNPHOjKiQ3XfqI@kroah.com> [AMD Official Use Only - General] > -----Original Message----- > From: Greg KH <gregkh@linuxfoundation.org> > Sent: Friday, October 14, 2022 12:48 PM > To: Gupta, Nipun <Nipun.Gupta@amd.com> > Cc: robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; rafael@kernel.org; > eric.auger@redhat.com; alex.williamson@redhat.com; cohuck@redhat.com; > Gupta, Puneet (DCG-ENG) <puneet.gupta@amd.com>; > song.bao.hua@hisilicon.com; mchehab+huawei@kernel.org; maz@kernel.org; > f.fainelli@gmail.com; jeffrey.l.hugo@gmail.com; saravanak@google.com; > Michael.Srba@seznam.cz; mani@kernel.org; yishaih@nvidia.com; > jgg@ziepe.ca; jgg@nvidia.com; robin.murphy@arm.com; will@kernel.org; > joro@8bytes.org; masahiroy@kernel.org; ndesaulniers@google.com; linux-arm- > kernel@lists.infradead.org; linux-kbuild@vger.kernel.org; linux- > kernel@vger.kernel.org; devicetree@vger.kernel.org; kvm@vger.kernel.org; > okaya@kernel.org; Anand, Harpreet <harpreet.anand@amd.com>; Agarwal, > Nikhil <nikhil.agarwal@amd.com>; Simek, Michal <michal.simek@amd.com>; > Radovanovic, Aleksandar <aleksandar.radovanovic@amd.com>; git (AMD-Xilinx) > <git@amd.com> > Subject: Re: [RFC PATCH v4 2/8] bus/cdx: add the cdx bus driver > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > On Fri, Oct 14, 2022 at 10:10:43AM +0530, Nipun Gupta wrote: > > --- a/drivers/bus/Makefile > > +++ b/drivers/bus/Makefile > > @@ -20,6 +20,9 @@ obj-$(CONFIG_INTEL_IXP4XX_EB) += intel-ixp4xx- > eb.o > > obj-$(CONFIG_MIPS_CDMM) += mips_cdmm.o > > obj-$(CONFIG_MVEBU_MBUS) += mvebu-mbus.o > > > > +#CDX bus > > No need for a comment like this :) Sure, will remove. > > > +obj-$(CONFIG_CDX_BUS) += cdx/ > > + > > # Interconnect bus driver for OMAP SoCs. > > obj-$(CONFIG_OMAP_INTERCONNECT) += omap_l3_smx.o omap_l3_noc.o > > > > diff --git a/drivers/bus/cdx/Kconfig b/drivers/bus/cdx/Kconfig > > new file mode 100644 > > index 000000000000..98ec05ad708d > > --- /dev/null > > +++ b/drivers/bus/cdx/Kconfig > > @@ -0,0 +1,13 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +# > > +# CDX bus configuration > > +# > > +# Copyright (C) 2022, Advanced Micro Devices, Inc. > > +# > > + > > +config CDX_BUS > > + bool "CDX Bus driver" > > + help > > + Driver to enable CDX Bus infrastructure. CDX bus uses > > + CDX controller and firmware to scan the FPGA based > > + devices. > > Why bool? Not as a module? That's broken. Okay. We see that most of other busses like amba, fslmc, pci are bool only. As we have MSI domain as part of this bus, we need to export multiple symbols to make it work as a module. > > And "FPGA based devices" means nothing to me, please expand on what this > all is as it is not descriptive at all. > > Also list the module name please. > > > > > diff --git a/drivers/bus/cdx/Makefile b/drivers/bus/cdx/Makefile > > new file mode 100644 > > index 000000000000..2e8f42611dfc > > --- /dev/null > > +++ b/drivers/bus/cdx/Makefile > > @@ -0,0 +1,8 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +# > > +# Makefile for CDX > > +# > > +# Copyright (C) 2022, Advanced Micro Devices, Inc. > > +# > > + > > +obj-$(CONFIG_CDX_BUS) += cdx.o > > diff --git a/drivers/bus/cdx/cdx.c b/drivers/bus/cdx/cdx.c > > new file mode 100644 > > index 000000000000..5a366f4ae69c > > --- /dev/null > > +++ b/drivers/bus/cdx/cdx.c > > @@ -0,0 +1,366 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * CDX bus driver. > > + * > > + * Copyright (C) 2022, Advanced Micro Devices, Inc. > > + */ > > + > > +/* > > + * Architecture Overview > > + * ===================== > > + * CDX is a Hardware Architecture designed for AMD FPGA devices. It > > + * consists of sophisticated mechanism for interaction between FPGA, > > + * Firmware and the APUs (Application CPUs). > > + * > > + * Firmware resides on RPU (Realtime CPUs) which interacts with > > + * the FPGA program manager and the APUs. The RPU provides memory- > mapped > > + * interface (RPU if) which is used to communicate with APUs. > > + * > > + * The diagram below shows an overview of the CDX architecture: > > + * > > + * +--------------------------------------+ > > + * | Application CPUs (APU) | > > + * | | > > + * | CDX device drivers| > > + * | Linux OS | | > > + * | CDX bus | > > + * | | | > > + * | CDX controller | > > + * | | | > > + * +-----------------------------|--------+ > > + * | (discover, config, > > + * | reset, rescan) > > + * | > > + * +------------------------| RPU if |----+ > > + * | | | > > + * | V | > > + * | Realtime CPUs (RPU) | > > + * | | > > + * +--------------------------------------+ > > + * | > > + * +---------------------|----------------+ > > + * | FPGA | | > > + * | +-----------------------+ | > > + * | | | | | > > + * | +-------+ +-------+ +-------+ | > > + * | | dev 1 | | dev 2 | | dev 3 | | > > + * | +-------+ +-------+ +-------+ | > > + * +--------------------------------------+ > > + * > > + * The RPU firmware extracts the device information from the loaded FPGA > > + * image and implements a mechanism that allows the APU drivers to > > + * enumerate such devices (device personality and resource details) via > > + * a dedicated communication channel. RPU mediates operations such as > > + * discover, reset and rescan of the FPGA devices for the APU. This is > > + * done using memory mapped interface provided by the RPU to APU. > > + */ > > + > > +#include <linux/init.h> > > +#include <linux/kernel.h> > > +#include <linux/slab.h> > > +#include <linux/cdx/cdx_bus.h> > > + > > +#include "cdx.h" > > + > > +/* > > + * Default DMA mask for devices on a CDX bus > > + */ > > +#define CDX_DEFAULT_DMA_MASK (~0ULL) > > + > > +static struct cdx_controller_t *cdx_controller; > > You don't get a static device, sorry, everything must be dynamic or your > design is broken. We will remove this static device here. > > And remove the crazy "_t" from your structure name, checkpatch should > have complained about that. Sure. Will remove from other places too. > > > > > + > > +static int reset_cdx_device(struct device *dev, void * __always_unused data) > > __always_unused is never needed. > > But the funny thing is, you added that variable, why are you not using > it? If it's not used, then don't have it. The 'data' parameter is here because the same reset_cdx_device() API is later used in device attributes patch where it is passed as an argument to bus_for_each_dev API, which needs the function to have these two arguments. Agree, that this parameter should be added as part of the device attribute patch, will be updated for next spin. Thanks, Nipun > > Try to get others at AMD to review your code first, before sending your > next batch out, this is really really odd... > > greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-10-14 8:21 UTC|newest] Thread overview: 198+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-03 12:26 [RFC PATCH 0/2] add support for CDX bus MSI domain Nipun Gupta 2022-08-03 12:26 ` Nipun Gupta 2022-08-03 12:26 ` [RFC PATCH 1/2] irqchip: cdx-bus: add cdx-MSI domain with gic-its domain as parent Nipun Gupta 2022-08-03 12:26 ` Nipun Gupta 2022-08-03 12:33 ` Greg KH 2022-08-03 12:33 ` Greg KH 2022-08-03 12:37 ` Gupta, Nipun 2022-08-03 12:37 ` Gupta, Nipun 2022-08-04 8:49 ` Marc Zyngier 2022-08-04 8:49 ` Marc Zyngier 2022-08-04 9:18 ` Gupta, Nipun 2022-08-04 9:18 ` Gupta, Nipun 2022-08-04 10:38 ` Marc Zyngier 2022-08-04 12:11 ` Gupta, Nipun 2022-08-04 12:11 ` Gupta, Nipun 2022-08-03 12:26 ` [RFC PATCH 2/2] driver core: add compatible string in sysfs for platform devices Nipun Gupta 2022-08-03 12:26 ` Nipun Gupta 2022-08-03 12:31 ` Greg KH 2022-08-03 12:31 ` Greg KH 2022-08-03 12:46 ` Gupta, Nipun 2022-08-03 12:46 ` Gupta, Nipun 2022-08-03 14:16 ` [RFC PATCH 0/2] add support for CDX bus MSI domain Robin Murphy 2022-08-03 14:16 ` Robin Murphy 2022-08-04 4:23 ` Gupta, Nipun 2022-08-04 4:23 ` Gupta, Nipun 2022-08-17 16:00 ` Jason Gunthorpe 2022-08-17 16:00 ` Jason Gunthorpe 2022-08-17 15:05 ` [RFC PATCH v2 0/6] add support for CDX bus controller Nipun Gupta 2022-08-17 15:05 ` [RFC PATCH v2 1/6] Documentation: DT: Add entry for CDX controller Nipun Gupta 2022-08-18 9:54 ` Krzysztof Kozlowski 2022-08-18 9:59 ` Krzysztof Kozlowski 2022-09-05 14:05 ` Gupta, Nipun 2022-09-06 6:55 ` Krzysztof Kozlowski 2022-09-06 7:03 ` Gupta, Nipun 2022-09-06 7:20 ` Krzysztof Kozlowski 2022-08-17 15:05 ` [RFC PATCH v2 2/6] bus/cdx: add the cdx bus driver Nipun Gupta 2022-08-17 15:32 ` Greg KH 2022-08-17 15:46 ` Marc Zyngier 2022-08-22 13:21 ` Gupta, Nipun 2022-08-22 13:29 ` Greg KH 2022-08-24 8:50 ` Gupta, Nipun 2022-08-24 12:11 ` Greg KH 2022-08-24 23:31 ` Jason Gunthorpe 2022-08-25 18:38 ` Saravana Kannan 2022-08-25 19:57 ` Robin Murphy 2022-08-26 0:08 ` Jason Gunthorpe 2022-08-29 4:49 ` Gupta, Nipun 2022-08-29 4:49 ` Gupta, Nipun 2022-08-29 15:31 ` Jason Gunthorpe 2022-08-30 7:06 ` Gupta, Nipun 2022-08-30 11:25 ` Robin Murphy 2022-08-30 13:01 ` Jason Gunthorpe 2022-08-30 13:12 ` Gupta, Nipun 2022-08-17 15:05 ` [RFC PATCH v2 3/6] bus/cdx: add cdx-MSI domain with gic-its domain as parent Nipun Gupta 2022-08-17 15:33 ` Greg KH 2022-08-17 15:05 ` [RFC PATCH v2 4/6] bus/cdx: add rescan and reset support Nipun Gupta 2022-08-17 15:05 ` [RFC PATCH v2 5/6] vfio: platform: reset: add reset for cdx devices Nipun Gupta 2022-08-17 15:05 ` [RFC PATCH v2 6/6] driver core: add compatible string in sysfs for platform devices Nipun Gupta 2022-08-17 15:30 ` Greg KH 2022-08-17 16:04 ` Saravana Kannan 2022-09-05 13:54 ` Gupta, Nipun 2022-09-06 13:47 ` [RFC PATCH v3 0/7] add support for CDX bus Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-06 13:47 ` [RFC PATCH v3 1/7] dt-bindings: bus: add CDX bus device tree bindings Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-06 17:46 ` Rob Herring 2022-09-06 17:46 ` Rob Herring 2022-09-07 3:13 ` Gupta, Nipun 2022-09-07 3:13 ` Gupta, Nipun 2022-09-08 10:51 ` Krzysztof Kozlowski 2022-09-08 10:51 ` Krzysztof Kozlowski 2022-09-06 13:47 ` [RFC PATCH v3 2/7] bus/cdx: add the cdx bus driver Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-07 0:32 ` Saravana Kannan 2022-09-07 0:32 ` Saravana Kannan 2022-09-07 3:21 ` Gupta, Nipun 2022-09-07 3:21 ` Gupta, Nipun 2022-09-07 18:06 ` Saravana Kannan 2022-09-07 18:06 ` Saravana Kannan 2022-09-07 12:32 ` Greg KH 2022-09-07 12:32 ` Greg KH 2022-09-08 13:29 ` Gupta, Nipun 2022-09-08 13:29 ` Gupta, Nipun 2022-09-06 13:47 ` [RFC PATCH v3 3/7] iommu/arm-smmu-v3: support ops registration for CDX bus Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-07 0:10 ` Saravana Kannan 2022-09-07 0:10 ` Saravana Kannan 2022-09-07 3:17 ` Gupta, Nipun 2022-09-07 3:17 ` Gupta, Nipun 2022-09-07 8:27 ` Robin Murphy 2022-09-07 8:27 ` Robin Murphy 2022-09-07 18:24 ` Saravana Kannan 2022-09-07 18:24 ` Saravana Kannan 2022-09-07 20:40 ` Robin Murphy 2022-09-07 20:40 ` Robin Murphy 2022-09-08 0:14 ` Saravana Kannan 2022-09-08 0:14 ` Saravana Kannan 2022-09-06 13:47 ` [RFC PATCH v3 4/7] bus/cdx: add cdx-MSI domain with gic-its domain as parent Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-06 17:19 ` Jason Gunthorpe 2022-09-06 17:19 ` Jason Gunthorpe 2022-09-07 11:17 ` Marc Zyngier 2022-09-07 11:17 ` Marc Zyngier 2022-09-07 11:33 ` Robin Murphy 2022-09-07 11:33 ` Robin Murphy 2022-09-07 12:14 ` Marc Zyngier 2022-09-07 12:14 ` Marc Zyngier 2022-09-07 11:35 ` Radovanovic, Aleksandar 2022-09-07 11:35 ` Radovanovic, Aleksandar 2022-09-07 12:32 ` Marc Zyngier 2022-09-07 13:18 ` Radovanovic, Aleksandar 2022-09-07 13:18 ` Radovanovic, Aleksandar 2022-09-08 8:08 ` Marc Zyngier 2022-09-08 9:51 ` Radovanovic, Aleksandar 2022-09-08 9:51 ` Radovanovic, Aleksandar 2022-09-08 11:49 ` Robin Murphy 2022-09-08 11:49 ` Robin Murphy 2022-09-08 14:18 ` Marc Zyngier 2022-09-07 13:18 ` Marc Zyngier 2022-09-07 13:18 ` Marc Zyngier 2022-09-08 14:13 ` Gupta, Nipun 2022-09-08 14:13 ` Gupta, Nipun 2022-09-08 14:29 ` Marc Zyngier 2022-09-09 6:32 ` Gupta, Nipun 2022-09-09 6:32 ` Gupta, Nipun 2022-10-12 10:04 ` Gupta, Nipun 2022-10-12 10:04 ` Gupta, Nipun 2022-10-12 10:34 ` Radovanovic, Aleksandar 2022-10-12 10:34 ` Radovanovic, Aleksandar 2022-10-12 13:02 ` Jason Gunthorpe 2022-10-12 13:02 ` Jason Gunthorpe 2022-10-12 13:37 ` Radovanovic, Aleksandar 2022-10-12 13:37 ` Radovanovic, Aleksandar 2022-10-12 14:38 ` Jason Gunthorpe 2022-10-12 14:38 ` Jason Gunthorpe 2022-10-12 15:09 ` Radovanovic, Aleksandar 2022-10-12 15:09 ` Radovanovic, Aleksandar 2022-10-13 12:43 ` Jason Gunthorpe 2022-10-13 12:43 ` Jason Gunthorpe 2022-10-14 11:18 ` Radovanovic, Aleksandar 2022-10-14 11:18 ` Radovanovic, Aleksandar 2022-10-14 11:54 ` gregkh 2022-10-14 11:54 ` gregkh 2022-10-14 12:13 ` Radovanovic, Aleksandar 2022-10-14 12:13 ` Radovanovic, Aleksandar 2022-10-14 13:46 ` gregkh 2022-10-14 13:46 ` gregkh 2022-10-14 13:58 ` Jason Gunthorpe 2022-10-14 13:58 ` Jason Gunthorpe 2022-09-06 13:47 ` [RFC PATCH v3 5/7] bus/cdx: add bus and device attributes Nipun Gupta 2022-09-06 13:47 ` Nipun Gupta 2022-09-06 13:48 ` [RFC PATCH v3 6/7] vfio/cdx: add support for CDX bus Nipun Gupta 2022-09-06 13:48 ` Nipun Gupta 2022-09-06 17:20 ` Jason Gunthorpe 2022-09-06 17:20 ` Jason Gunthorpe 2022-09-06 17:23 ` Gupta, Nipun 2022-09-06 17:23 ` Gupta, Nipun 2022-09-06 13:48 ` [RFC PATCH v3 7/7] vfio/cdx: add interrupt support Nipun Gupta 2022-09-06 13:48 ` Nipun Gupta 2022-10-14 4:40 ` [RFC PATCH v4 0/8] add support for CDX bus Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 4:40 ` [RFC PATCH v4 1/8] dt-bindings: bus: add CDX bus device tree bindings Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 14:17 ` Rob Herring 2022-10-14 14:17 ` Rob Herring 2022-10-17 10:18 ` Gupta, Nipun 2022-10-17 10:18 ` Gupta, Nipun 2022-10-14 4:40 ` [RFC PATCH v4 2/8] bus/cdx: add the cdx bus driver Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 7:15 ` Greg KH 2022-10-14 7:15 ` Greg KH 2022-10-14 8:12 ` Gupta, Nipun 2022-10-14 8:12 ` Gupta, Nipun 2022-10-14 7:18 ` Greg KH 2022-10-14 7:18 ` Greg KH 2022-10-14 8:20 ` Gupta, Nipun [this message] 2022-10-14 8:20 ` Gupta, Nipun 2022-10-14 4:40 ` [RFC PATCH v4 3/8] iommu/arm-smmu-v3: support ops registration for CDX bus Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 4:51 ` Gupta, Nipun 2022-10-14 4:51 ` Gupta, Nipun 2022-10-14 4:40 ` [RFC PATCH v4 4/8] bux/cdx: support dma configuration for CDX devices Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 4:40 ` [RFC PATCH v4 5/8] bus/cdx: add bus and device attributes Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 4:40 ` [RFC PATCH v4 6/8] irq/msi: use implicit msi domain for alloc and free Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 4:40 ` [RFC PATCH v4 7/8] bus/cdx: add cdx-MSI domain with gic-its domain as parent Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-17 13:08 ` kernel test robot 2022-11-17 19:10 ` Thomas Gleixner 2022-11-17 19:10 ` Thomas Gleixner 2022-10-14 4:40 ` [RFC PATCH v4 8/8] bus/cdx: add cdx controller Nipun Gupta 2022-10-14 4:40 ` Nipun Gupta 2022-10-14 14:10 ` [RFC PATCH v4 0/8] add support for CDX bus Rob Herring 2022-10-14 14:10 ` Rob Herring 2022-10-17 10:08 ` Gupta, Nipun 2022-10-17 10:08 ` Gupta, Nipun
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=DM6PR12MB3082BB5B7E5756450BBC0DEBE8249@DM6PR12MB3082.namprd12.prod.outlook.com \ --to=nipun.gupta@amd.com \ --cc=Michael.Srba@seznam.cz \ --cc=aleksandar.radovanovic@amd.com \ --cc=alex.williamson@redhat.com \ --cc=cohuck@redhat.com \ --cc=devicetree@vger.kernel.org \ --cc=eric.auger@redhat.com \ --cc=f.fainelli@gmail.com \ --cc=git@amd.com \ --cc=gregkh@linuxfoundation.org \ --cc=harpreet.anand@amd.com \ --cc=jeffrey.l.hugo@gmail.com \ --cc=jgg@nvidia.com \ --cc=jgg@ziepe.ca \ --cc=joro@8bytes.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=kvm@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mani@kernel.org \ --cc=masahiroy@kernel.org \ --cc=maz@kernel.org \ --cc=mchehab+huawei@kernel.org \ --cc=michal.simek@amd.com \ --cc=ndesaulniers@google.com \ --cc=nikhil.agarwal@amd.com \ --cc=okaya@kernel.org \ --cc=puneet.gupta@amd.com \ --cc=rafael@kernel.org \ --cc=robh+dt@kernel.org \ --cc=robin.murphy@arm.com \ --cc=saravanak@google.com \ --cc=song.bao.hua@hisilicon.com \ --cc=will@kernel.org \ --cc=yishaih@nvidia.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.