On 19-11-08 12:21:21, Martin Kepplinger wrote: > Hi Leonard, hi Abel, > > Thanks for having a look! To sum up this problem and not to get confused: > > We have the workaround that changes irq-imx-gpcv2 from this very email > thread, to be used with mainline ATF. when applying Abel's recent diff, > Linux 5.4 boots but I still don't have a cpuidle driver. > > When I enable CONFIG_ARM_PSCI_CPUIDLE, the kernel hangs during boot > (after probing mmc, but that doesn't tell much) > > What do I miss? > OK, please fetch the branches called "imx8mq-err11171" from both following github repos and give it a try: https://github.com/abelvesa/linux.git and https://github.com/abelvesa/arm-trusted-firmware.git I just tested it. Works with defconfig. > > Then (in parallel) we have NXP's ATF: > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.codeaurora.org%2Fexternal%2Fimx%2Fimx-atf%2Flog%2F%3Fh%3Dimx_4.19.35_1.0.0&data=02%7C01%7Cabel.vesa%40nxp.com%7C0a5a09f616c84932e06d08d7643dccaf%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637088088896134121&sdata=tq0aUGawG%2FhRRXZB9jdIi2xSNGHINhbWM1ZpDKPFrqU%3D&reserved=0 > that I test in parallel (and will actually want to have cpuidle right > now too). The workaround in Linux in that case looks like so: > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.codeaurora.org%2Fexternal%2Fimx%2Flinux-imx%2Fcommit%2F%3Fh%3Dimx_4.19.35_1.0.0%26id%3D26a59057f88997dfe48ab7f81898ddd6b6d3903e&data=02%7C01%7Cabel.vesa%40nxp.com%7C0a5a09f616c84932e06d08d7643dccaf%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637088088896134121&sdata=GqmtzpS8fB4bxmOXxTNQZrWNCV18lNu7XpX5dmcAEaY%3D&reserved=0 > which changes irq-gic-v3 only. > Forget about the NXP arm-trusted-firmware for now. It will never work with mainline + the workaround patches. > Since 5.4, also no cpu-sleep state anymore. What would need to change in > that "NXP" case, for 5.4 to have cpuidle again? > > When I enable ARM_PSCI_CPUIDLE here, right now Linux hangs during boot > (during probing sdhci but again that seems random). > > I'm still happy for hints :) > > Thanks, > > martin >
