On Thu, 16 Oct 2014, Guenter Roeck wrote: > On Wed, Oct 15, 2014 at 01:55:09PM -0500, atull@xxxxxxxxxxxxxxxxxxxxx wrote: > > From: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> > > > > Add support for simple on/off control of each channel. > > > > To add regulator support, the pmbus part driver needs to add > > regulator_desc information and number of regulators to its > > pmbus_driver_info struct. > > > > regulator_desc can be declared using default macro for a > > regulator (PMBUS_REGULATOR) that is in pmbus.h > > > > The regulator_init_data can be initialized from either > > platform data or the device tree. > > > > Signed-off-by: Alan Tull <atull@xxxxxxxxxxxxxxxxxxxxx> > > Reviewed-by: Mark Brown <broonie@xxxxxxxxxx> > > Cc: Guenter Roeck <linux@xxxxxxxxxxxx> > > Hi Alan, > > I am still seeing lots of the following: > > vout0: Failed to create debugfs directory > vout1: Failed to create debugfs directory > vout2: Failed to create debugfs directory > vout3: Failed to create debugfs directory > vout4: Failed to create debugfs directory > vout5: Failed to create debugfs directory > vout6: Failed to create debugfs directory > vout7: Failed to create debugfs directory > > I thought there was a problem in the regulator core, but after looking > into it concluded that the regulator core _should_ prepend the names > with the device name when creating the debugfs entries, unless no device > name is specified. So something must be missing. We'll need to sort > this out before I can accept the code. > > Thanks, > Guenter > Hi Guenter, OK, I will look into it. Alan -- 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
