Hi Thomas, Gregory, On Mon, 30 Apr 2018 15:53:52 +0200, Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx> wrote: > 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. This is a much better idea than adding the 'syscon' compatible. I will work on it. > > The rest of the conversion to regmap looks good otherwise. > > Best regards, > > Thomas Thanks, Miquèl -- Miquel Raynal, 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
