Re: [PowerPC] [PASEMI] Issue with the identification of ATA drives after the of/irq updates 2024-05-29

Michael Ellerman
2024-07-04T11:53:37Z

Christian Zigotzky <chzigotzky@xenosoft.de> writes:
> On 02.07.24 18:54, Marc Zyngier wrote:
>> On Sun, 30 Jun 2024 11:21:55 +0100,
>> Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
>>> Hello,
>>>
>>> There is an issue with the identification of ATA drives with our
>>> P.A. Semi Nemo boards [1] after the
>>> commit "of/irq: Factor out parsing of interrupt-map parent
>>> phandle+args from of_irq_parse_raw()" [2].
...
>>
>> --- a/drivers/of/irq.c
>> +++ b/drivers/of/irq.c
>> @@ -282,8 +282,10 @@ int of_irq_parse_raw(const __be32 *addr, struct of_phandle_args *out_irq)
>>
>> oldimap = imap;
>> imap = of_irq_parse_imap_parent(oldimap, imaplen, out_irq);
>> - if (!imap)
>> - goto fail;
>> + if (!imap) {
>> + match = 0;
>> + break;
>> + }
>>
>> match &= of_device_is_available(out_irq->np);
>> if (match)
>>
>>
> We tested this patch yesterday and it solves the boot problem.

Hi Christian,

Instead of that patch, can you try the one below. AFAICS the device tree
fixups done in early boot mean the interrupt-map is not needed, and also
has the wrong content, so if we can remove it entirely that might avoid
the problems in the parsing code.

I don't know if your firmware actually implements those methods, I
couldn't find anything online to confirm or deny it. Seems the only
option is to test it.

cheers

diff --git a/arch/powerpc/kernel/prom_init.c b/arch/powerpc/kernel/prom_init.c
index fbb68fc28ed3..28fe082ede57 100644
--- a/arch/powerpc/kernel/prom_init.c
+++ b/arch/powerpc/kernel/prom_init.c
@@ -3138,6 +3138,14 @@ static void __init fixup_device_tree_pasemi(void)

prom_setprop(iob, name, "interrupt-controller", &val, 0);

+ prom_printf("nemo: deleting interrupt-map properties\n");
+ rc = call_prom("interpret", 1, 1,
+ " s\" /pxp@0,e0000000\" find-device"
+ " s\" interrupt-map\" delete-property"
+ " s\" interrupt-map-mask\" delete-property"
+ " device-end");
+ prom_printf("nemo: interpret returned %d\n", rc);
+
pci_name = "/pxp@0,e0000000/pci@11";
node = call_prom("finddevice", 1, 1, ADDR(pci_name));
parent = ADDR(iob);