On Thu, Dec 11, 2014 at 09:36:26AM +0100, Jean-Michel Hautbois wrote: > I started to add support for DT in this regulator, but I can't get a > booting kernel with it. I know it lacks regulator init in the parse_dt > function, but I don't really know what is the best way to do it. I > have used tps6507x as an example until know... Well, the best thing to do would be to backport the changes to add support for DT parsing in the core and use that. See "regulator: of: Provide simplified DT parsing method" and preceeding commits in mainline. Otherwise what you've got looks basically OK from a quick scan through. > I don't know for instance if a tps_board is needed (it would contain > only a struct regulator_init_data * so probably not very useful) ? Probably not, no. > Below is my current status (I also have created a tps65023.dtsi in > arch/arm/boot/dts which is only a copy of tps6507x.dtsi so didn't copy > it here). If you need to create a .dsti file for a regulator that's a warning sign - it should be empty. You'll want things in the particular board but that ought to be board specific things rather than something that applies to all boards using the regulator.
Attachment:
signature.asc
Description: Digital signature
