Hi Brian, On Wed, Dec 9, 2015 at 2:12 AM, Brian Norris <computersforpeace@xxxxxxxxx> wrote: > On Mon, Dec 07, 2015 at 09:58:35AM -0800, Brian Norris wrote: >> On Sat, Dec 05, 2015 at 12:45:36PM +0100, Jonas Gorski wrote: >> > On Fri, Dec 4, 2015 at 3:02 AM, Brian Norris >> > <computersforpeace@xxxxxxxxx> wrote: >> > > // proposed >> > > partitions { >> > > compatible = "partitions"; >> > >> > "partitions" sounds mode like a device_type thing than a compatible >> > name, maybe "fixed-partitions"? IMHO that would describe better what >> > these are, and doesn't invite to think using compatible = >> > "arm,arm-flash-structure", "partitions"; is a good idea. >> >> "fixed-partitions" sounds OK to me. If no objections, I'll apply these >> patches, with (approximately) a: >> >> s/"partitions"/"fixed-partitions"/ > > Pushed to linux-mtd.git with the above change. Aarghl, hadn't seen this patch before. This breaks the users that have already added the partitions subnodes (armada-xp-lenovo-ix4-300d.dts and a few shmobile). Will send patches to fix it... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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