On Sat, 2022-11-12 at 19:47 +0100, Thomas Gleixner wrote:
> No point in doing another lookup of irq_data, it's already provided as
> an argument.
>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> ---
> arch/s390/pci/pci_irq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/arch/s390/pci/pci_irq.c
> +++ b/arch/s390/pci/pci_irq.c
> @@ -132,7 +132,7 @@ static int zpci_clear_irq(struct zpci_de
> static int zpci_set_irq_affinity(struct irq_data *data, const struct cpumask *dest,
> bool force)
> {
> - struct msi_desc *entry = irq_get_msi_desc(data->irq);
> + struct msi_desc *entry = irq_data_get_msi_desc(data);
> struct msi_msg msg = entry->msg;
> int cpu_addr = smp_cpu_get_cpu_address(cpumask_first(dest));
>
Thanks for the patch, makes sense to me so have my:
Reviewed-by: Niklas Schnelle <schnelle@xxxxxxxxxxxxx>
That said it must be noted that this function is only called when using
directed PCI IRQs which is a hardware feature that has not made it to
any released hardware. Nevertheless no point int doing things more
complicated than necessary even for that case.
If there are no objections I'll apply this to our internal tree and it
will then go upstream via the s390 tree.
