On Thu, Dec 10, 2020 at 1:42 PM Thomas Gleixner <[email protected]> wrote: > > Going through a full irq descriptor lookup instead of just using the proper > helper function which provides direct access is suboptimal. > > In fact it _is_ wrong because the chip callback needs to get the chip data > which is relevant for the chip while using the irq descriptor variant > returns the irq chip data of the top level chip of a hierarchy. It does not > matter in this case because the chip is the top level chip, but that > doesn't make it more correct. > > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Karthikeyan Mitran <[email protected]> > Cc: Hou Zhiqiang <[email protected]> > Cc: Lorenzo Pieralisi <[email protected]> > Cc: Rob Herring <[email protected]> > Cc: Bjorn Helgaas <[email protected]> > Cc: [email protected] > --- > drivers/pci/controller/mobiveil/pcie-mobiveil-host.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-)
Reviewed-by: Rob Herring <[email protected]>
