Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Matthias Kaehlcke
On Tue, Jun 12, 2018 at 01:00:14PM -0700, Brian Norris wrote: > Hi, > > On Tue, Jun 12, 2018 at 10:11:40AM -0700, Matthias Kaehlcke wrote: > > On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > > > The purpose

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Matthias Kaehlcke
On Tue, Jun 12, 2018 at 01:00:14PM -0700, Brian Norris wrote: > Hi, > > On Tue, Jun 12, 2018 at 10:11:40AM -0700, Matthias Kaehlcke wrote: > > On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > > > The purpose

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Brian Norris
Hi, On Tue, Jun 12, 2018 at 10:11:40AM -0700, Matthias Kaehlcke wrote: > On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > > The purpose of the throttler is to provide support for non-thermal > > > throttling.

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Brian Norris
Hi, On Tue, Jun 12, 2018 at 10:11:40AM -0700, Matthias Kaehlcke wrote: > On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > > The purpose of the throttler is to provide support for non-thermal > > > throttling.

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Matthias Kaehlcke
Hi, On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > Hi! > > A few comments, but I didn't get to finish a thorough review yet. > > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > The purpose of the throttler is to provide support for non-thermal > >

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-12 Thread Matthias Kaehlcke
Hi, On Mon, Jun 11, 2018 at 06:49:13PM -0700, Brian Norris wrote: > Hi! > > A few comments, but I didn't get to finish a thorough review yet. > > > On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > > The purpose of the throttler is to provide support for non-thermal > >

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-11 Thread Brian Norris
Hi! A few comments, but I didn't get to finish a thorough review yet. On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > 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

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-11 Thread Brian Norris
Hi! A few comments, but I didn't get to finish a thorough review yet. On Thu, Jun 07, 2018 at 11:12:12AM -0700, Matthias Kaehlcke wrote: > 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

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-08 Thread kbuild test robot
Hi Matthias, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.17 next-20180608] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-08 Thread kbuild test robot
Hi Matthias, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.17 next-20180608] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-07 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 v2 09/11] misc: throttler: Add core support for non-thermal throttling

2018-06-07 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