Re: [PATCH] ACPI: PMIC: Add comments with DSDT power opregion field names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 25, 2023 at 1:07 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
>
> The DSDTs of CHT devices using the Dollar Cove TI PMIC, all use
> LDO1 - LDO14 names for the DSDT power opregion field names.
>
> Add comments with these fields to make it easier to see which PMIC
> registers are being set by ACPI code using these.
>
> Note that LDO4 is missing and the mapped registers jump from 0x43
> to 0x45 to match. This matches with how the fields are declared
> in the DSDT where LDO3 is skipped to. Note there is no hole in

LDO4?

too

> the field addresses, LDO3 is simply just not defined on either side.

LDO4?

With commit message clarification (whatever is considered correct)
Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>

> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> ---
>  drivers/acpi/pmic/intel_pmic_chtdc_ti.c | 26 ++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/acpi/pmic/intel_pmic_chtdc_ti.c b/drivers/acpi/pmic/intel_pmic_chtdc_ti.c
> index 418eec523025..c84ef3d15181 100644
> --- a/drivers/acpi/pmic/intel_pmic_chtdc_ti.c
> +++ b/drivers/acpi/pmic/intel_pmic_chtdc_ti.c
> @@ -20,19 +20,19 @@
>  #define CHTDC_TI_GPADC         0x5a
>
>  static struct pmic_table chtdc_ti_power_table[] = {
> -       { .address = 0x00, .reg = 0x41 },
> -       { .address = 0x04, .reg = 0x42 },
> -       { .address = 0x08, .reg = 0x43 },
> -       { .address = 0x0c, .reg = 0x45 },
> -       { .address = 0x10, .reg = 0x46 },
> -       { .address = 0x14, .reg = 0x47 },
> -       { .address = 0x18, .reg = 0x48 },
> -       { .address = 0x1c, .reg = 0x49 },
> -       { .address = 0x20, .reg = 0x4a },
> -       { .address = 0x24, .reg = 0x4b },
> -       { .address = 0x28, .reg = 0x4c },
> -       { .address = 0x2c, .reg = 0x4d },
> -       { .address = 0x30, .reg = 0x4e },
> +       { .address = 0x00, .reg = 0x41 }, /* LDO1 */
> +       { .address = 0x04, .reg = 0x42 }, /* LDO2 */
> +       { .address = 0x08, .reg = 0x43 }, /* LDO3 */
> +       { .address = 0x0c, .reg = 0x45 }, /* LDO5 */
> +       { .address = 0x10, .reg = 0x46 }, /* LDO6 */
> +       { .address = 0x14, .reg = 0x47 }, /* LDO7 */
> +       { .address = 0x18, .reg = 0x48 }, /* LDO8 */
> +       { .address = 0x1c, .reg = 0x49 }, /* LDO9 */
> +       { .address = 0x20, .reg = 0x4a }, /* LD10 */
> +       { .address = 0x24, .reg = 0x4b }, /* LD11 */
> +       { .address = 0x28, .reg = 0x4c }, /* LD12 */
> +       { .address = 0x2c, .reg = 0x4d }, /* LD13 */
> +       { .address = 0x30, .reg = 0x4e }, /* LD14 */
>  };
>
>  static struct pmic_table chtdc_ti_thermal_table[] = {
> --
> 2.39.0
>


-- 
With Best Regards,
Andy Shevchenko



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux