[v8,5/7] PCI: mediatek-gen3: Add MSI support
Krzysztof Wilczyński
kw at linux.com
Wed Feb 24 09:31:20 EST 2021
Hi Jianjun,
[...]
> +static struct irq_chip mtk_msi_irq_chip = {
> + .name = "MSI",
> + .irq_enable = mtk_pcie_irq_unmask,
> + .irq_disable = mtk_pcie_irq_mask,
> + .irq_ack = irq_chip_ack_parent,
> + .irq_mask = mtk_pcie_irq_mask,
> + .irq_unmask = mtk_pcie_irq_unmask,
> +};
For consistency sake, what about aligning this like the
struct mtk_msi_bottom_irq_chip has been? See immediately below.
[...]
> +static struct irq_chip mtk_msi_bottom_irq_chip = {
> + .irq_ack = mtk_msi_bottom_irq_ack,
> + .irq_mask = mtk_msi_bottom_irq_mask,
> + .irq_unmask = mtk_msi_bottom_irq_unmask,
> + .irq_compose_msi_msg = mtk_compose_msi_msg,
> + .irq_set_affinity = mtk_pcie_set_affinity,
> + .name = "MSI",
> +};
Krzysztof
More information about the linux-arm-kernel
mailing list