power supply gating with ltc2978

power supply gating with ltc2978

[atull]

Hello,

I am interested in adding functionality to be able to gate power supplies 
going through a ltc2978.  I see that there is a hwmon driver already 
existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
have MFD's.  It looks like this ltc driver would need a MFD and a 
regulator driver added.  However I don't see other pmbus hwmon drivers
using MFD.

So I am asking for recommendations and reservations on how to proceed here 
before I get too far with this.

Thanks,
Alan Tull

[lm-sensors] power supply gating with ltc2978

[atull]

Hello,

I am interested in adding functionality to be able to gate power supplies 
going through a ltc2978.  I see that there is a hwmon driver already 
existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
have MFD's.  It looks like this ltc driver would need a MFD and a 
regulator driver added.  However I don't see other pmbus hwmon drivers
using MFD.

So I am asking for recommendations and reservations on how to proceed here 
before I get too far with this.

Thanks,
Alan Tull


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: power supply gating with ltc2978

[Guenter Roeck]

On 08/15/2014 02:34 PM, atull wrote:
> Hello,
>
> I am interested in adding functionality to be able to gate power supplies
> going through a ltc2978.  I see that there is a hwmon driver already
> existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers
> have MFD's.  It looks like this ltc driver would need a MFD and a
> regulator driver added.  However I don't see other pmbus hwmon drivers
> using MFD.
>
> So I am asking for recommendations and reservations on how to proceed here
> before I get too far with this.
>

I would suggest to add the regulator driver registration into the pmbus code.
I would have done this earlier myself, but there was no clear need for it
so I did not bother.

The functionality between regulator and limit/status reporting is heavily
intertwined in pmbus devices, and there is no clear functional separation
between power regulation and limit/status reporting related functionality.
Or, in other words, PMBus devices are not multi-function devices.

One of the challenges is that the values to write into the "operation" register
is device specific, so it is not possible to enable the functionality for all
PMBus chips. We'll need some device specific configuration data, such as on/off
values or maybe function callbacks into device specific code, to enable
regulator functionality.

Guenter

Re: [lm-sensors] power supply gating with ltc2978

[Guenter Roeck]

On 08/15/2014 02:34 PM, atull wrote:
> Hello,
>
> I am interested in adding functionality to be able to gate power supplies
> going through a ltc2978.  I see that there is a hwmon driver already
> existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers
> have MFD's.  It looks like this ltc driver would need a MFD and a
> regulator driver added.  However I don't see other pmbus hwmon drivers
> using MFD.
>
> So I am asking for recommendations and reservations on how to proceed here
> before I get too far with this.
>

I would suggest to add the regulator driver registration into the pmbus code.
I would have done this earlier myself, but there was no clear need for it
so I did not bother.

The functionality between regulator and limit/status reporting is heavily
intertwined in pmbus devices, and there is no clear functional separation
between power regulation and limit/status reporting related functionality.
Or, in other words, PMBus devices are not multi-function devices.

One of the challenges is that the values to write into the "operation" register
is device specific, so it is not possible to enable the functionality for all
PMBus chips. We'll need some device specific configuration data, such as on/off
values or maybe function callbacks into device specific code, to enable
regulator functionality.

Guenter


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: power supply gating with ltc2978

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:

> I am interested in adding functionality to be able to gate power supplies 
> going through a ltc2978.  I see that there is a hwmon driver already 
> existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> have MFD's.  It looks like this ltc driver would need a MFD and a 
> regulator driver added.  However I don't see other pmbus hwmon drivers
> using MFD.

> So I am asking for recommendations and reservations on how to proceed here 
> before I get too far with this.

Without knowing anything at all about pmbus or this particular hardware
it's hard to comment but what you're saying here sounds sensible (though
I do see that apparently splitting the drivers may not actually be
sensible from Guenter's followup).

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [lm-sensors] power supply gating with ltc2978

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 813 bytes --]

On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:

> I am interested in adding functionality to be able to gate power supplies 
> going through a ltc2978.  I see that there is a hwmon driver already 
> existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> have MFD's.  It looks like this ltc driver would need a MFD and a 
> regulator driver added.  However I don't see other pmbus hwmon drivers
> using MFD.

> So I am asking for recommendations and reservations on how to proceed here 
> before I get too far with this.

Without knowing anything at all about pmbus or this particular hardware
it's hard to comment but what you're saying here sounds sensible (though
I do see that apparently splitting the drivers may not actually be
sensible from Guenter's followup).

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 153 bytes --]

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: power supply gating with ltc2978

[Guenter Roeck]

On Sat, Aug 16, 2014 at 02:20:50PM +0100, Mark Brown wrote:
> On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:
> 
> > I am interested in adding functionality to be able to gate power supplies 
> > going through a ltc2978.  I see that there is a hwmon driver already 
> > existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> > have MFD's.  It looks like this ltc driver would need a MFD and a 
> > regulator driver added.  However I don't see other pmbus hwmon drivers
> > using MFD.
> 
> > So I am asking for recommendations and reservations on how to proceed here 
> > before I get too far with this.
> 
> Without knowing anything at all about pmbus or this particular hardware
> it's hard to comment but what you're saying here sounds sensible (though
> I do see that apparently splitting the drivers may not actually be
> sensible from Guenter's followup).

I had originally thought about converting the pmbus drivers to mfd with client
drivers, but I concluded that it would add a lot of complexity with little gain.
It makes sense to separate a driver into mfd and a number of client drivers
if a device has clear functional blocks for the different devices it supports.
With PMBus, this is not the case. Separating a PMBus driver would be a purely
artificial costruct, and there would be overlapping functionality. Separating
just a single driver out of the group of PMBus drivers, as seems to be suggested
above, makes even less sense as one simply can not separate the core PMBus
driver code from its front-end drivers.

On the other side, adding regulator support into the PMBus driver code would
make a lot of sense. It should also be quite straightforward.

Or anyway that is my opinion. If someone wants to spend the time and separate
the PMBus drivers into an MfD part and hwmon and regulator client drivers, I'll
be happy to look at the resulting patch set.

Guenter

Re: [lm-sensors] power supply gating with ltc2978

[Guenter Roeck]

On Sat, Aug 16, 2014 at 02:20:50PM +0100, Mark Brown wrote:
> On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:
> 
> > I am interested in adding functionality to be able to gate power supplies 
> > going through a ltc2978.  I see that there is a hwmon driver already 
> > existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> > have MFD's.  It looks like this ltc driver would need a MFD and a 
> > regulator driver added.  However I don't see other pmbus hwmon drivers
> > using MFD.
> 
> > So I am asking for recommendations and reservations on how to proceed here 
> > before I get too far with this.
> 
> Without knowing anything at all about pmbus or this particular hardware
> it's hard to comment but what you're saying here sounds sensible (though
> I do see that apparently splitting the drivers may not actually be
> sensible from Guenter's followup).

I had originally thought about converting the pmbus drivers to mfd with client
drivers, but I concluded that it would add a lot of complexity with little gain.
It makes sense to separate a driver into mfd and a number of client drivers
if a device has clear functional blocks for the different devices it supports.
With PMBus, this is not the case. Separating a PMBus driver would be a purely
artificial costruct, and there would be overlapping functionality. Separating
just a single driver out of the group of PMBus drivers, as seems to be suggested
above, makes even less sense as one simply can not separate the core PMBus
driver code from its front-end drivers.

On the other side, adding regulator support into the PMBus driver code would
make a lot of sense. It should also be quite straightforward.

Or anyway that is my opinion. If someone wants to spend the time and separate
the PMBus drivers into an MfD part and hwmon and regulator client drivers, I'll
be happy to look at the resulting patch set.

Guenter

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: power supply gating with ltc2978

[atull]

On Mon, 18 Aug 2014, Guenter Roeck wrote:

> On Sat, Aug 16, 2014 at 02:20:50PM +0100, Mark Brown wrote:
> > On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:
> > 
> > > I am interested in adding functionality to be able to gate power supplies 
> > > going through a ltc2978.  I see that there is a hwmon driver already 
> > > existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> > > have MFD's.  It looks like this ltc driver would need a MFD and a 
> > > regulator driver added.  However I don't see other pmbus hwmon drivers
> > > using MFD.
> > 
> > > So I am asking for recommendations and reservations on how to proceed here 
> > > before I get too far with this.
> > 
> > Without knowing anything at all about pmbus or this particular hardware
> > it's hard to comment but what you're saying here sounds sensible (though
> > I do see that apparently splitting the drivers may not actually be
> > sensible from Guenter's followup).
> 
> I had originally thought about converting the pmbus drivers to mfd with client
> drivers, but I concluded that it would add a lot of complexity with little gain.
> It makes sense to separate a driver into mfd and a number of client drivers
> if a device has clear functional blocks for the different devices it supports.
> With PMBus, this is not the case. Separating a PMBus driver would be a purely
> artificial costruct, and there would be overlapping functionality. Separating
> just a single driver out of the group of PMBus drivers, as seems to be suggested
> above, makes even less sense as one simply can not separate the core PMBus
> driver code from its front-end drivers.
> 
> On the other side, adding regulator support into the PMBus driver code would
> make a lot of sense. It should also be quite straightforward.
> 
> Or anyway that is my opinion. If someone wants to spend the time and separate
> the PMBus drivers into an MfD part and hwmon and regulator client drivers, I'll
> be happy to look at the resulting patch set.
> 
> Guenter
> 

Hi Guenter,

I'd really rather just add regulator support to the PMBus driver code and 
avoid all the mess of creating a MFD.  It seems more straigtforward. I'll 
let you know when I have something working to look at.

Alan

Re: [lm-sensors] power supply gating with ltc2978

[atull]

On Mon, 18 Aug 2014, Guenter Roeck wrote:

> On Sat, Aug 16, 2014 at 02:20:50PM +0100, Mark Brown wrote:
> > On Fri, Aug 15, 2014 at 04:34:49PM -0500, atull wrote:
> > 
> > > I am interested in adding functionality to be able to gate power supplies 
> > > going through a ltc2978.  I see that there is a hwmon driver already 
> > > existing (hwmon/pmbus/ltc2978.c).  I see some of the other hwmon drivers 
> > > have MFD's.  It looks like this ltc driver would need a MFD and a 
> > > regulator driver added.  However I don't see other pmbus hwmon drivers
> > > using MFD.
> > 
> > > So I am asking for recommendations and reservations on how to proceed here 
> > > before I get too far with this.
> > 
> > Without knowing anything at all about pmbus or this particular hardware
> > it's hard to comment but what you're saying here sounds sensible (though
> > I do see that apparently splitting the drivers may not actually be
> > sensible from Guenter's followup).
> 
> I had originally thought about converting the pmbus drivers to mfd with client
> drivers, but I concluded that it would add a lot of complexity with little gain.
> It makes sense to separate a driver into mfd and a number of client drivers
> if a device has clear functional blocks for the different devices it supports.
> With PMBus, this is not the case. Separating a PMBus driver would be a purely
> artificial costruct, and there would be overlapping functionality. Separating
> just a single driver out of the group of PMBus drivers, as seems to be suggested
> above, makes even less sense as one simply can not separate the core PMBus
> driver code from its front-end drivers.
> 
> On the other side, adding regulator support into the PMBus driver code would
> make a lot of sense. It should also be quite straightforward.
> 
> Or anyway that is my opinion. If someone wants to spend the time and separate
> the PMBus drivers into an MfD part and hwmon and regulator client drivers, I'll
> be happy to look at the resulting patch set.
> 
> Guenter
> 

Hi Guenter,

I'd really rather just add regulator support to the PMBus driver code and 
avoid all the mess of creating a MFD.  It seems more straigtforward. I'll 
let you know when I have something working to look at.

Alan

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors