On Thu, 2016-02-04 at 14:56 +0100, Alessio Igor Bogani wrote: > Hi, > > Can we change this code (from mpc85xx_ds.c) > > if (of_flat_dt_is_compatible(root, "fsl,MPC8572DS-CAMP")) { > mpic = mpic_alloc(NULL, 0, > MPIC_NO_RESET | > MPIC_BIG_ENDIAN | > MPIC_SINGLE_DEST_CPU, > 0, 256, " OpenPIC "); > } else { > mpic = mpic_alloc(NULL, 0, > MPIC_BIG_ENDIAN | > MPIC_SINGLE_DEST_CPU, > 0, 256, " OpenPIC "); > } > > in this one > > mpic = mpic_alloc(NULL, 0, > MPIC_BIG_ENDIAN | > MPIC_SINGLE_DEST_CPU, > 0, 256, " OpenPIC "); > > using "pic-no-reset" in the device tree?
In theory that breaks existing device trees that don't specify pic-no-reset. I'm not sure how much it matters in this case as it's primarily meant as an example of how to do AMP rather than something that works out-of-the-box. BTW, Kyle, it looks like there was a meaningful difference between !MPIC_WANTS_RESET and MPIC_NO_RESET -- the former did not inhibit initialization of non-protected vectors and was the behavior of fsl,MPC8572DS -CAMP before your "powerpc/mpic: Remove duplicate MPIC_WANTS_RESET flag" patch. It's no longer possible to initialize all MPIC vectors except protected ones. Again, I'm not sure it matters much, but I'm also not sure how much continued value the protected-source code has. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev