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

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 LDO4 is skipped too. Note there is no hole in
the field addresses, LDO4 is simply just not defined on either side.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
Changes in v2:
- s/LDO3/LDO4/ in the commit message, because I somehow go this
  wrong initially (thank you for catching this Andy)
---
 drivers/acpi/pmic/intel_pmic_chtdc_ti.c | 26 ++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

On Wed, Jan 25, 2023 at 12:38 PM Hans de Goede <hdegoede@redhat.com> 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 LDO4 is skipped too. Note there is no hole in
> the field addresses, LDO4 is simply just not defined on either side.
>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> Changes in v2:
> - s/LDO3/LDO4/ in the commit message, because I somehow go this
>   wrong initially (thank you for catching this Andy)
> ---
>  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[] = {
> --

Applied as 6.3 material, thanks!