Re: [PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-22 Thread Matthias Kaehlcke
On Thu, Jun 21, 2018 at 07:04:33PM -0700, Brian Norris wrote: > Hi, > > A few more things I noticed; probably my last thoughts on this > particular patch; and I think I reviewed the rest: > > On Wed, Jun 20, 2018 at 06:52:35PM -0700, Matthias Kaehlcke wrote: > > The purpose of the throttler is

Re: [PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-22 Thread Matthias Kaehlcke
On Thu, Jun 21, 2018 at 07:04:33PM -0700, Brian Norris wrote: > Hi, > > A few more things I noticed; probably my last thoughts on this > particular patch; and I think I reviewed the rest: > > On Wed, Jun 20, 2018 at 06:52:35PM -0700, Matthias Kaehlcke wrote: > > The purpose of the throttler is

Re: [PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-21 Thread Brian Norris
Hi, A few more things I noticed; probably my last thoughts on this particular patch; and I think I reviewed the rest: On Wed, Jun 20, 2018 at 06:52:35PM -0700, Matthias Kaehlcke wrote: > The purpose of the throttler is to provide support for non-thermal > throttling. Throttling is triggered by

Re: [PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-21 Thread Brian Norris
Hi, A few more things I noticed; probably my last thoughts on this particular patch; and I think I reviewed the rest: On Wed, Jun 20, 2018 at 06:52:35PM -0700, Matthias Kaehlcke wrote: > The purpose of the throttler is to provide support for non-thermal > throttling. Throttling is triggered by

[PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-20 Thread Matthias Kaehlcke
The purpose of the throttler is to provide support for non-thermal throttling. Throttling is triggered by external event, e.g. the detection of a high battery discharge current, close to the OCP limit of the battery. The throttler is only in charge of the throttling, not the monitoring, which is

[PATCH v4 10/12] misc: throttler: Add core support for non-thermal throttling

2018-06-20 Thread Matthias Kaehlcke
The purpose of the throttler is to provide support for non-thermal throttling. Throttling is triggered by external event, e.g. the detection of a high battery discharge current, close to the OCP limit of the battery. The throttler is only in charge of the throttling, not the monitoring, which is