On 01/31/2014 01:51 PM, Marc Kleine-Budde wrote: > On 01/31/2014 01:49 PM, Marc Kleine-Budde wrote: >> On 01/31/2014 01:45 PM, Florian Vaussard wrote: >>> Hello Marc, >>> >>> On 01/31/2014 01:33 PM, Marc Kleine-Budde wrote: >>>> On 01/31/2014 11:35 AM, Florian Vaussard wrote: >>>>> The OpenFirmware probe can be merged into the standard platform >>>>> probe to leverage common code. >>>> >>>> Good work, as we want to replace the existing driver, I'm quite picky on >>>> this patch, see more comments inline. >>>> >>>> Please don't delete of of_platform driver, yet. >>>> >>> >>> Do you have any reason for not deleting of_platform? After this patch, >>> we will have duplicated functionalities, this may be misleading for >>> other people. >> >> Okay. >> IIRC there _was_ a problem on sparc, it _should_ be fixed now, better >> get a Tested-by from Andreas (Cc'ed). >
So it is ok for you if I remove of_platform, right? > You have to use irq_of_parse_and_map() on sparc, but your patches does this. > My first implementation was not using irq_of_parse_and_map(), as I do not need it on ARM, but I suspected that other people might need it. This is why the final code is slightly more complicated to handle both cases. Regards, Florian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
