Hi Stefan, On Wed, May 27, 2015 at 09:12:48AM +0200, Stefan Wahren wrote: > Am 23.05.2015 um 19:41 schrieb Sebastian Reichel: > > On Wed, Apr 29, 2015 at 10:32:24PM +0000, Stefan Wahren wrote: > >> This patch adds a minimal driver for the Freescale i.MX23, i.MX28 > >> power subsystem. It's required to trigger the probing of the underlying > >> drivers like on-chip regulators. > > I'm not sure what you need the power supply class for. I see only > > one property (ac connected), which is constant. Maybe just drop > > the power supply stuff? > > those SoCs could be powered by 3 different sources: > - 5V wall > - USB > - battery > > I choose the first one as a base, because it's the simplest one. > Unfortunately i don't have the time to implement all of them. From what I can see the driver does not implement the 5V wall charger handling. It simply assumes, that the 5V wall charger is there. While you can skip implementation of battery connection check you should make sure, that AC is actually connected. > The Freescale downstream kernel (based on 2.6.35) [1] has a driver to > handle all MXS power sources. But i think a clean implementation from > the scratch would be necessary for mainline. > > > This would leave an almost empty driver, which just sets a single > > register at probe time and probes more drivers. That would probably > > be more fitting for the MFD subsystem. > > I don't have any problems with fitting the driver to MFD. You can keep in in drivers/power. From your code I assumed there would simply be a fixed AC connection and nothing else. > > -- Sebastian > > [1] - > http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/tree/drivers/power/mxs/linux.c?h=imx_2.6.35_maintain -- Sebastian
Attachment:
signature.asc
Description: Digital signature