On Friday, March 07, 2014 9:25 AM, Arnd Bergmann wrote: > On Thursday 06 March 2014, Lucas Stach wrote: > > Am Donnerstag, den 06.03.2014, 10:36 -0700 schrieb Stephen Warren: > > > On 03/05/2014 06:25 AM, Lucas Stach wrote: > > > > This is the recommended method of doing the IRQ > > > > mapping. For old devicetrees we fall back to the > > > > previous practice. > > > > > > Tested-by: Stephen Warren <swarren@xxxxxxxxxx> > > > > > > I tested both with and without patch 1/6, and the PCIe-based NIC on > > > Beaver worked fine either way. Without patch 1/6, I do see: > > > > > > pci 0000:00:01.0: of_irq_parse_pci() failed with rc=-22 > > > > > > ... but that seems reasonable given that the DT that of_irq_parse_pci() > > > parses is missing, and did correctly trigger the fallback path, so > > > everything still worked. > > > > Yes, this should be normal. It spits this error for old DTs, but keeps > > doing the right thing. I'm not sure if we should downgrade this to info > > or dbg. > > No, I think printing an error like this is appropriate: it is an incentive > to update the dts files, but doesn't look too urgent as long as everything > still works. +1 I agree with Arnd's opinion. I think that all dts files should be updated properly. This error message can be the good way to do it. Thank you. Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html