Hi Greg,
On 9/11/2023 7:22 PM, Nipun Gupta wrote:
> Add CDX-MSI domain per CDX controller with gic-its domain as
> a parent, to support MSI for CDX devices. CDX devices allocate
> MSIs from the CDX domain. Also, introduce APIs to alloc and free
> IRQs for CDX domain.
>
> In CDX subsystem firmware is a controller for all devices and
> their configuration. CDX bus controller sends all the write_msi_msg
> commands to firmware running on RPU and the firmware interfaces with
> actual devices to pass this information to devices
>
> Since, CDX controller is the only way to communicate with the Firmware
> for MSI write info, CDX domain per controller required in contrast to
> having a CDX domain per device.
>
> Co-developed-by: Nikhil Agarwal <[email protected]>
> Signed-off-by: Nikhil Agarwal <[email protected]>
> Co-developed-by: Abhijit Gangurde <[email protected]>
> Signed-off-by: Abhijit Gangurde <[email protected]>
> Signed-off-by: Nipun Gupta <[email protected]>
> Reviewed-by: Pieter Jansen van Vuuren <[email protected]>
> Tested-by: Nikhil Agarwal <[email protected]>
Please help in making progress on this patch. Is there anything that
needs to be updated in this patch for CDX bus?
Thanks,
Nipun
On Tue, Sep 26, 2023 at 03:18:58PM +0530, Gupta, Nipun wrote:
> Hi Greg,
>
> On 9/11/2023 7:22 PM, Nipun Gupta wrote:
> > Add CDX-MSI domain per CDX controller with gic-its domain as
> > a parent, to support MSI for CDX devices. CDX devices allocate
> > MSIs from the CDX domain. Also, introduce APIs to alloc and free
> > IRQs for CDX domain.
> >
> > In CDX subsystem firmware is a controller for all devices and
> > their configuration. CDX bus controller sends all the write_msi_msg
> > commands to firmware running on RPU and the firmware interfaces with
> > actual devices to pass this information to devices
> >
> > Since, CDX controller is the only way to communicate with the Firmware
> > for MSI write info, CDX domain per controller required in contrast to
> > having a CDX domain per device.
> >
> > Co-developed-by: Nikhil Agarwal <[email protected]>
> > Signed-off-by: Nikhil Agarwal <[email protected]>
> > Co-developed-by: Abhijit Gangurde <[email protected]>
> > Signed-off-by: Abhijit Gangurde <[email protected]>
> > Signed-off-by: Nipun Gupta <[email protected]>
> > Reviewed-by: Pieter Jansen van Vuuren <[email protected]>
> > Tested-by: Nikhil Agarwal <[email protected]>
>
> Please help in making progress on this patch. Is there anything that needs
> to be updated in this patch for CDX bus?
$ mdfrm -c ~/mail/todo/
2031 messages in /home/gregkh/mail/todo/
So perhaps help in reviewing other pending patches for other subsystems?
It's in my queue, but have been traveling for 2 weeks for conferences,
will be catching up next week when I get a chance.
Also, you need to get the msi/interrupt developers to agree with this,
why not get their review and acceptance first?
thanks,
greg k-h