On Fri, 17 Jun 2022 15:02:07 +0100, Matt Ranostay <mranostay@xxxxxx> wrote: > > GICD region was overlapping with GICR causing the latter to not map > successfully, and in turn the gic-v3 driver would fail to initialize. > > This issue was hidden till commit 2b2cd74a06c3 ("irqchip/gic-v3: Claim iomem resources") > replaced of_iomap() calls with of_io_request_and_map() that internally > called request_mem_region(). Please correctly wrap long lines in the commit message. > > Respective console output before this patchset: > > [ 0.000000] GICv3: /bus@100000/interrupt-controller@1800000: couldn't map region 0 > > Fixes: 2b2cd74a06c3 ("irqchip/gic-v3: Claim iomem resources") No, that's the wrong target for this. Your patch fixes the *original* submission of the DT, and this should carry a: Fixes: b8545f9d3a54 ("arm64: dts: ti: Add initial support for J721S2 SoC") Please also consider adding a Cc: stable@xxxxxxxxxxxxxxx Thanks, M. -- Without deviation from the norm, progress is not possible.