Re: [PATCH v2 03/22] MSI: Remove the redundant irq_set_chip_data()

From: Thierry Reding
Date: Thu Sep 25 2014 - 03:19:30 EST


On Thu, Sep 25, 2014 at 11:14:13AM +0800, Yijing Wang wrote:
> Currently, pcie-designware, pcie-rcar, pci-tegra drivers
> use irq chip_data to save the msi_chip pointer. They
> already call irq_set_chip_data() in their own MSI irq map
> functions. So irq_set_chip_data() in arch_setup_msi_irq()
> is useless.

Again, I think this should be the other way around. If drivers do
something that's already handled by the core, then the duplicate code
should be dropped from the drivers.

Thierry

Attachment: pgpMjEnCu9gh_.pgp
Description: PGP signature