The duplicated parsing continued to bother me, so I've refactored things to avoid that for parsing the interrupt parent and args in the interrupt-map. It passes testing with unittests on QEMU virt platform, but I don't think that catches the problematic cases. So please test. v1: https://lore.kernel.org/all/20240528164132.2451685-1-maz@xxxxxxxxxx/ - Refactor existing interrupt-map parsing code and use it for fw_devlink Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx> --- Marc Zyngier (1): of: property: Fix fw_devlink handling of interrupt-map Rob Herring (Arm) (1): of/irq: Factor out parsing of interrupt-map parent phandle+args from of_irq_parse_raw() drivers/of/irq.c | 127 +++++++++++++++++++++++++++++------------------- drivers/of/of_private.h | 3 ++ drivers/of/property.c | 30 ++++-------- 3 files changed, 89 insertions(+), 71 deletions(-) --- base-commit: 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0 change-id: 20240529-dt-interrupt-map-fix-a37b9aff5ca0 Best regards, -- Rob Herring (Arm) <robh@xxxxxxxxxx>