On 5/11/21 5:30 PM, Tero Kristo wrote:
> From: Tero Kristo <t-kri...@ti.com>
> 
> Add support command for debugging K3 power domains. This is useful with
> the HSM rearch setup, where power domains are directly controlled by SPL
> instead of going through the TI SCI layer. The debugging support is only
> available in the u-boot codebase though, so the raw register access
> power domain layer must be enabled on u-boot side for this to work. By
> default, u-boot side uses the TI SCI layer, and R5 SPL only uses the
> direct access methods.
> 
> Signed-off-by: Tero Kristo <t-kri...@ti.com>
> Signed-off-by: Tero Kristo <kri...@kernel.org>

Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com>

Best Regards,
Jaehoon Chung

> ---
>  cmd/ti/Kconfig                         |   8 ++
>  cmd/ti/Makefile                        |   1 +
>  cmd/ti/pd.c                            | 185 +++++++++++++++++++++++++
>  drivers/power/domain/ti-power-domain.c |   6 +-
>  include/k3-dev.h                       |   9 ++
>  5 files changed, 206 insertions(+), 3 deletions(-)
>  create mode 100644 cmd/ti/pd.c
> 
> diff --git a/cmd/ti/Kconfig b/cmd/ti/Kconfig
> index efeff0d482..db557445a8 100644
> --- a/cmd/ti/Kconfig
> +++ b/cmd/ti/Kconfig
> @@ -7,4 +7,12 @@ config CMD_DDR3
>          supports memory verification, memory comapre and ecc
>          verification if supported.
>  
> +config CMD_PD
> +     bool "command for verifying power domains"
> +     depends on TI_POWER_DOMAIN
> +     help
> +        Debug command for K3 power domains. For this to work, the
> +        K3 power domain driver must be enabled for the u-boot; by
> +        default it is only enabled for SPL.
> +
>  endmenu
> diff --git a/cmd/ti/Makefile b/cmd/ti/Makefile
> index 16fbade9ed..045593396b 100644
> --- a/cmd/ti/Makefile
> +++ b/cmd/ti/Makefile
> @@ -5,4 +5,5 @@ obj- += dummy.o
>  
>  ifndef CONFIG_SPL_BUILD
>  obj-$(CONFIG_CMD_DDR3) += ddr3.o
> +obj-$(CONFIG_CMD_PD) += pd.o
>  endif
> diff --git a/cmd/ti/pd.c b/cmd/ti/pd.c
> new file mode 100644
> index 0000000000..a53ccdcc40
> --- /dev/null
> +++ b/cmd/ti/pd.c
> @@ -0,0 +1,185 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Power Domain test commands
> + *
> + * Copyright (C) 2020 Texas Instruments Incorporated, <www.ti.com>
> + */
> +
> +#include <common.h>
> +#include <command.h>
> +#include <dm.h>
> +#include <k3-dev.h>
> +
> +static const struct udevice_id ti_pd_of_match[] = {
> +     { .compatible = "ti,sci-pm-domain" },
> +     { /* sentinel */ }
> +};
> +
> +static struct ti_k3_pd_platdata *ti_pd_find_data(void)
> +{
> +     struct udevice *dev;
> +     int i = 0;
> +
> +     while (1) {
> +             uclass_get_device(UCLASS_POWER_DOMAIN, i++, &dev);
> +             if (!dev)
> +                     return NULL;
> +
> +             if (device_is_compatible(dev,
> +                                      ti_pd_of_match[0].compatible))
> +                     return  dev_get_priv(dev);
> +     }
> +
> +     return NULL;
> +}
> +
> +static void dump_lpsc(struct ti_k3_pd_platdata *data, struct ti_pd *pd)
> +{
> +     int i;
> +     struct ti_lpsc *lpsc;
> +     u8 state;
> +     static const char * const lpsc_states[] = {
> +             "swrstdis", "syncrst", "disable", "enable", "autosleep",
> +             "autowake", "unknown",
> +     };
> +
> +     for (i = 0; i < data->num_lpsc; i++) {
> +             lpsc = &data->lpsc[i];
> +             if (lpsc->pd != pd)
> +                     continue;
> +             state = lpsc_get_state(lpsc);
> +             if (state > ARRAY_SIZE(lpsc_states))
> +                     state = ARRAY_SIZE(lpsc_states) - 1;
> +             printf("    LPSC%d: state=%s, usecount=%d\n",
> +                    lpsc->id, lpsc_states[state], lpsc->usecount);
> +     }
> +}
> +
> +static void dump_pd(struct ti_k3_pd_platdata *data, struct ti_psc *psc)
> +{
> +     int i;
> +     struct ti_pd *pd;
> +     u8 state;
> +     static const char * const pd_states[] = {
> +             "off", "on", "unknown"
> +     };
> +
> +     for (i = 0; i < data->num_pd; i++) {
> +             pd = &data->pd[i];
> +             if (pd->psc != psc)
> +                     continue;
> +             state = ti_pd_state(pd);
> +             if (state > ARRAY_SIZE(pd_states))
> +                     state = ARRAY_SIZE(pd_states) - 1;
> +             printf("  PD%d: state=%s, usecount=%d:\n",
> +                    pd->id, pd_states[state], pd->usecount);
> +             dump_lpsc(data, pd);
> +     }
> +}
> +
> +static void dump_psc(struct ti_k3_pd_platdata *data)
> +{
> +     int i;
> +     struct ti_psc *psc;
> +
> +     for (i = 0; i < data->num_psc; i++) {
> +             psc = &data->psc[i];
> +             printf("PSC%d [%p]:\n", psc->id, psc->base);
> +             dump_pd(data, psc);
> +     }
> +}
> +
> +static int do_pd_dump(struct cmd_tbl *cmdtp, int flag, int argc,
> +                   char *const argv[])
> +{
> +     struct ti_k3_pd_platdata *data;
> +
> +     data = ti_pd_find_data();
> +     if (!data)
> +             return CMD_RET_FAILURE;
> +
> +     dump_psc(data);
> +
> +     return 0;
> +}
> +
> +static int do_pd_endis(int argc, char *const argv[], u8 state)
> +{
> +     u32 psc_id;
> +     u32 lpsc_id;
> +     int i;
> +     struct ti_k3_pd_platdata *data;
> +     struct ti_lpsc *lpsc;
> +     int ret;
> +
> +     if (argc < 3)
> +             return CMD_RET_FAILURE;
> +
> +     data = ti_pd_find_data();
> +     if (!data)
> +             return CMD_RET_FAILURE;
> +
> +     psc_id = simple_strtoul(argv[1], NULL, 10);
> +     lpsc_id = simple_strtoul(argv[2], NULL, 10);
> +
> +     for (i = 0; i < data->num_lpsc; i++) {
> +             lpsc = &data->lpsc[i];
> +             if (lpsc->pd->psc->id != psc_id)
> +                     continue;
> +             if (lpsc->id != lpsc_id)
> +                     continue;
> +             printf("%s pd [PSC:%d,LPSC:%d]...\n",
> +                    state == MDSTAT_STATE_ENABLE ? "Enabling" : "Disabling",
> +                    psc_id, lpsc_id);
> +             ret = ti_lpsc_transition(lpsc, state);
> +             if (ret)
> +                     return CMD_RET_FAILURE;
> +             else
> +                     return 0;
> +     }
> +
> +     printf("No matching psc/lpsc found.\n");
> +
> +     return CMD_RET_FAILURE;
> +}
> +
> +static int do_pd_enable(struct cmd_tbl *cmdtp, int flag, int argc,
> +                     char *const argv[])
> +{
> +     return do_pd_endis(argc, argv, MDSTAT_STATE_ENABLE);
> +}
> +
> +static int do_pd_disable(struct cmd_tbl *cmdtp, int flag, int argc,
> +                      char *const argv[])
> +{
> +     return do_pd_endis(argc, argv, MDSTAT_STATE_SWRSTDISABLE);
> +}
> +
> +static struct cmd_tbl cmd_pd[] = {
> +     U_BOOT_CMD_MKENT(dump, 1, 0, do_pd_dump, "", ""),
> +     U_BOOT_CMD_MKENT(enable, 3, 0, do_pd_enable, "", ""),
> +     U_BOOT_CMD_MKENT(disable, 3, 0, do_pd_disable, "", ""),
> +};
> +
> +static int ti_do_pd(struct cmd_tbl *cmdtp, int flag, int argc, char * const 
> argv[])
> +{
> +     struct cmd_tbl *c;
> +
> +     argc--;
> +     argv++;
> +
> +     c = find_cmd_tbl(argv[0], cmd_pd, ARRAY_SIZE(cmd_pd));
> +     if (c)
> +             return c->cmd(cmdtp, flag, argc, argv);
> +     else
> +             return CMD_RET_USAGE;
> +}
> +
> +U_BOOT_CMD(pd, 4, 1, ti_do_pd,
> +        "TI power domain control",
> +#ifdef CONFIG_SYS_LONGHELP
> +        "dump                 - show power domain status\n"
> +        "enable [psc] [lpsc]  - enable power domain\n"
> +        "disable [psc] [lpsc] - disable power domain\n"
> +#endif
> +);
> diff --git a/drivers/power/domain/ti-power-domain.c 
> b/drivers/power/domain/ti-power-domain.c
> index e418a7b996..567e7e1e8e 100644
> --- a/drivers/power/domain/ti-power-domain.c
> +++ b/drivers/power/domain/ti-power-domain.c
> @@ -132,7 +132,7 @@ static void ti_pd_transition(struct ti_pd *pd)
>       psc_write(BIT(pd->id), pd->psc, PSC_PTCMD);
>  }
>  
> -static u8 ti_pd_state(struct ti_pd *pd)
> +u8 ti_pd_state(struct ti_pd *pd)
>  {
>       return pd_read(pd, PSC_PDCTL) & PDCTL_STATE_MASK;
>  }
> @@ -227,12 +227,12 @@ static int ti_lpsc_wait(struct ti_lpsc *lpsc)
>       return ret;
>  }
>  
> -static u8 lpsc_get_state(struct ti_lpsc *lpsc)
> +u8 lpsc_get_state(struct ti_lpsc *lpsc)
>  {
>       return lpsc_read(lpsc, PSC_MDCTL) & MDSTAT_STATE_MASK;
>  }
>  
> -static int ti_lpsc_transition(struct ti_lpsc *lpsc, u8 state)
> +int ti_lpsc_transition(struct ti_lpsc *lpsc, u8 state)
>  {
>       struct ti_pd *psc_pd;
>       int ret;
> diff --git a/include/k3-dev.h b/include/k3-dev.h
> index de3a8bdf9e..55c5057db3 100644
> --- a/include/k3-dev.h
> +++ b/include/k3-dev.h
> @@ -22,6 +22,11 @@
>  #define PSC_PD_ALWAYSON         BIT(1)
>  #define PSC_PD_DEPENDS          BIT(2)
>  
> +#define MDSTAT_STATE_MASK            0x3f
> +#define MDSTAT_BUSY_MASK             0x30
> +#define MDSTAT_STATE_SWRSTDISABLE    0x0
> +#define MDSTAT_STATE_ENABLE          0x3
> +
>  struct ti_psc {
>       int id;
>       void __iomem *base;
> @@ -73,4 +78,8 @@ struct ti_k3_pd_platdata {
>  extern const struct ti_k3_pd_platdata j721e_pd_platdata;
>  extern const struct ti_k3_pd_platdata j7200_pd_platdata;
>  
> +u8 ti_pd_state(struct ti_pd *pd);
> +u8 lpsc_get_state(struct ti_lpsc *lpsc);
> +int ti_lpsc_transition(struct ti_lpsc *lpsc, u8 state);
> +
>  #endif
> 

Reply via email to