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