Hi Sakari, Andy,
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 07, 2017 at 11:40:27PM +0300, Andy Shevchenko wrote:
> > On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus <sakari.ai...@iki.fi> wrote:
> > > O
Hi Sakari, Andy,
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 07, 2017 at 11:40:27PM +0300, Andy Shevchenko wrote:
> > On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus wrote:
> > > On Wed, Jun 07, 201
Hi Andy,
Thanks for the reviews and patience.
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 7, 2017 at 3:07 PM, Sakari Ailus <sakari.ai...@iki.fi> wrote:
>
> >> +static int ti_tps68470_pmic_get_power(
Hi Andy,
Thanks for the reviews and patience.
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 7, 2017 at 3:07 PM, Sakari Ailus wrote:
>
> >> +static int ti_tps68470_pmic_get_power(struct
Hi Sakari, Andy,
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 07, 2017 at 04:37:12PM +0300, Andy Shevchenko wrote:
> > >> +static acpi_status ti_pmic_com
Hi Sakari, Andy,
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 07, 2017 at 04:37:12PM +0300, Andy Shevchenko wrote:
> > >> +static acpi_status ti_pmic_com
gt; a...@vger.kernel.org; Lee Jones <lee.jo...@linaro.org>; Linus Walleij
> <linus.wall...@linaro.org>; Alexandre Courbot <gnu...@gmail.com>; Rafael J.
> Wysocki <r...@rjwysocki.net>; Len Brown <l...@kernel.org>
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC:
Lee Jones ; Linus Walleij
> ; Alexandre Courbot ; Rafael J.
> Wysocki ; Len Brown
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> Hi Rajmohan,
>
> Thanks for removing the redundant struct definition. A couple more comments
>
Hi Hans,
> >
> > As PMICs are typically linked to the kernel (vs. being modules),
> > there's no issue with the module name. I would suppose few if any
> > PMICs will be compiled as modules in general.
>
> Good point about the OpRegion driver usually being built-in, in my experience
> it
> MUST
Hi Hans,
> >
> > As PMICs are typically linked to the kernel (vs. being modules),
> > there's no issue with the module name. I would suppose few if any
> > PMICs will be compiled as modules in general.
>
> Good point about the OpRegion driver usually being built-in, in my experience
> it
> MUST
dego...@redhat.com>; linux-kernel@vger.kernel.org; linux-
> g...@vger.kernel.org; linux-a...@vger.kernel.org; Lee Jones
> <lee.jo...@linaro.org>; Linus Walleij <linus.wall...@linaro.org>; Alexandre
> Courbot <gnu...@gmail.com>; Rafael J. Wysocki <r...@rjwysocki.net>; Len
>
@vger.kernel.org; Lee Jones
> ; Linus Walleij ; Alexandre
> Courbot ; Rafael J. Wysocki ; Len
> Brown
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> On Wed, Jun 07, 2017 at 11:40:27PM +0300, Andy Shevchenko wrote:
> > On W
ux-g...@vger.kernel.org; linux-
> a...@vger.kernel.org; Lee Jones <lee.jo...@linaro.org>; Linus Walleij
> <linus.wall...@linaro.org>; Alexandre Courbot <gnu...@gmail.com>; Rafael J.
> Wysocki <r...@rjwysocki.net>; Len Brown <l...@kernel.org>
> Subject: Re: [P
Jones ; Linus Walleij
> ; Alexandre Courbot ; Rafael J.
> Wysocki ; Len Brown
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> Hi,
>
> On 06/06/2017 04:23 PM, Andy Shevchenko wrote:
> > +Cc Hans (that's why didn't delete anyth
: linux-kernel@vger.kernel.org; linux-g...@vger.kernel.org; linux-
> a...@vger.kernel.org; Lee Jones <lee.jo...@linaro.org>; Linus Walleij
> <linus.wall...@linaro.org>; Alexandre Courbot <gnu...@gmail.com>; Rafael J.
> Wysocki <r...@rjwysocki.net>; Len Brown <l...@
.kernel.org; linux-
> a...@vger.kernel.org; Lee Jones ; Linus Walleij
> ; Alexandre Courbot ; Rafael J.
> Wysocki ; Len Brown
> Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation
> region driver
>
> +Cc Hans (that's why didn't delete anything from original mail, just
&
Hi,
On 07-06-17 22:10, Sakari Ailus wrote:
Hi Andy,
On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
Follow the pattern, please, I
Hi,
On 07-06-17 22:10, Sakari Ailus wrote:
Hi Andy,
On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
Follow the pattern, please, I suppose
ti_pmic_tps68470.c
On Wed, Jun 07, 2017 at 11:40:27PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus wrote:
> > On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
> >> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> >> >
On Wed, Jun 07, 2017 at 11:40:27PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus wrote:
> > On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
> >> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> >> > On Tue, Jun 06, 2017 at 05:23:56PM +0300,
On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus wrote:
> On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
>> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
>> > On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
>> >>
On Wed, Jun 7, 2017 at 11:10 PM, Sakari Ailus wrote:
> On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
>> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
>> > On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
>> >> Follow the pattern, please, I suppose
>> >>
Hi Andy,
On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> > On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
> >> Follow the pattern, please, I suppose
> >> ti_pmic_tps68470.c
> >
> >
Hi Andy,
On Wed, Jun 07, 2017 at 04:40:13PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> > On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
> >> Follow the pattern, please, I suppose
> >> ti_pmic_tps68470.c
> >
> > This pattern is weird.
On Wed, Jun 07, 2017 at 04:37:12PM +0300, Andy Shevchenko wrote:
> >> +static acpi_status ti_pmic_common_handler(u32 function,
> > + acpi_physical_address address,
> > + u32 bits, u64 *value,
> > +
On Wed, Jun 07, 2017 at 04:37:12PM +0300, Andy Shevchenko wrote:
> >> +static acpi_status ti_pmic_common_handler(u32 function,
> > + acpi_physical_address address,
> > + u32 bits, u64 *value,
> > +
On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
>> Follow the pattern, please, I suppose
>> ti_pmic_tps68470.c
>
> This pattern is weird. "ti" in front of the file name is redundant, and in
> very few
On Wed, Jun 7, 2017 at 3:15 PM, Sakari Ailus wrote:
> On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
>> Follow the pattern, please, I suppose
>> ti_pmic_tps68470.c
>
> This pattern is weird. "ti" in front of the file name is redundant, and in
> very few places the vendor prefix
On Wed, Jun 7, 2017 at 3:07 PM, Sakari Ailus wrote:
>> +static int ti_tps68470_pmic_get_power(struct regmap *regmap, int reg,
>> +int bitmask, u64 *value)
>> +{
>> + int data;
>
> Shouldn't you use unsigned int here? Same in the
On Wed, Jun 7, 2017 at 3:07 PM, Sakari Ailus wrote:
>> +static int ti_tps68470_pmic_get_power(struct regmap *regmap, int reg,
>> +int bitmask, u64 *value)
>> +{
>> + int data;
>
> Shouldn't you use unsigned int here? Same in the functions below.
+1,
On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
> Follow the pattern, please, I suppose
> ti_pmic_tps68470.c
This pattern is weird. "ti" in front of the file name is redundant, and in
very few places the vendor prefix is used anyway. Especially when the chip
has a proper name ---
On Tue, Jun 06, 2017 at 05:23:56PM +0300, Andy Shevchenko wrote:
> Follow the pattern, please, I suppose
> ti_pmic_tps68470.c
This pattern is weird. "ti" in front of the file name is redundant, and in
very few places the vendor prefix is used anyway. Especially when the chip
has a proper name ---
Hi Rajmohan,
Thanks for removing the redundant struct definition. A couple more comments
below. Not really necessarily bugs but a few things to clean things up a
bit.
On Tue, Jun 06, 2017 at 04:55:18AM -0700, Rajmohan Mani wrote:
> The Kabylake platform coreboot (Chrome OS equivalent of
> BIOS)
Hi Rajmohan,
Thanks for removing the redundant struct definition. A couple more comments
below. Not really necessarily bugs but a few things to clean things up a
bit.
On Tue, Jun 06, 2017 at 04:55:18AM -0700, Rajmohan Mani wrote:
> The Kabylake platform coreboot (Chrome OS equivalent of
> BIOS)
Hi,
On 06/06/2017 04:23 PM, Andy Shevchenko wrote:
+Cc Hans (that's why didn't delete anything from original mail, just
adding my comments).
Hans, if you have few minutes it would be appreciated to glance on the
below for some issues if any since you did pass quite a good quest
with other PMIC
Hi,
On 06/06/2017 04:23 PM, Andy Shevchenko wrote:
+Cc Hans (that's why didn't delete anything from original mail, just
adding my comments).
Hans, if you have few minutes it would be appreciated to glance on the
below for some issues if any since you did pass quite a good quest
with other PMIC
+Cc Hans (that's why didn't delete anything from original mail, just
adding my comments).
Hans, if you have few minutes it would be appreciated to glance on the
below for some issues if any since you did pass quite a good quest
with other PMIC drivers.
On Tue, Jun 6, 2017 at 2:55 PM, Rajmohan
+Cc Hans (that's why didn't delete anything from original mail, just
adding my comments).
Hans, if you have few minutes it would be appreciated to glance on the
below for some issues if any since you did pass quite a good quest
with other PMIC drivers.
On Tue, Jun 6, 2017 at 2:55 PM, Rajmohan
38 matches
Mail list logo