Hello, On Sat, 21 Apr 2018 15:55:26 +0200, Miquel Raynal wrote: > The ICU DT nodes have now the 'syscon' compatible, we can switch to have now -> now have > regmap before splitting the code to support multiple platform devices to > be probed (one for the ICU, one per interrupt group). > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> As I explained in the review of PATCH 03/17, I think we could simply create the regmap in the ->probe() of the parent device, instead of using the "syscon" property, which is mainly useful when there is no parent device. The rest of the conversion to regmap looks good otherwise. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com -- 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