Hi Stephen,
On 27 July 2016 at 15:24, Stephen Warren <swar...@wwwdotorg.org> wrote:
> From: Stephen Warren <swar...@nvidia.com>
>
> In Tegra186, SoC power domains are manipulated using IPC requests to
> the BPMP (Boot and Power Management Processor). This change implements a
> driver that does that.
>
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
> ---
> drivers/power/domain/Kconfig | 7 +++
> drivers/power/domain/Makefile | 1 +
> drivers/power/domain/tegra186-power-domain.c | 88
> ++++++++++++++++++++++++++++
> 3 files changed, 96 insertions(+)
> create mode 100644 drivers/power/domain/tegra186-power-domain.c
>
> diff --git a/drivers/power/domain/Kconfig b/drivers/power/domain/Kconfig
> index b90409743398..132e33250e8c 100644
> --- a/drivers/power/domain/Kconfig
> +++ b/drivers/power/domain/Kconfig
> @@ -17,4 +17,11 @@ config SANDBOX_POWER_DOMAIN
> simply accepts requests to power on/off various HW modules without
> actually doing anything beyond a little error checking.
>
> +config TEGRA186_POWER_DOMAIN
> + bool "Enable Tegra186 BPMP-based power domain driver"
> + depends on TEGRA186_BPMP
> + help
> + Enable support for manipulating Tegra's on-SoC power domains via IPC
> + requests to the BPMP (Boot and Power Management Processor).
> +
> endmenu
> diff --git a/drivers/power/domain/Makefile b/drivers/power/domain/Makefile
> index c18292f0ec88..2c3d92638fbe 100644
> --- a/drivers/power/domain/Makefile
> +++ b/drivers/power/domain/Makefile
> @@ -5,3 +5,4 @@
> obj-$(CONFIG_POWER_DOMAIN) += power-domain-uclass.o
> obj-$(CONFIG_SANDBOX_POWER_DOMAIN) += sandbox-power-domain.o
> obj-$(CONFIG_SANDBOX_POWER_DOMAIN) += sandbox-power-domain-test.o
> +obj-$(CONFIG_TEGRA186_POWER_DOMAIN) += tegra186-power-domain.o
> diff --git a/drivers/power/domain/tegra186-power-domain.c
> b/drivers/power/domain/tegra186-power-domain.c
> new file mode 100644
> index 000000000000..4ae4baabe57e
> --- /dev/null
> +++ b/drivers/power/domain/tegra186-power-domain.c
> @@ -0,0 +1,88 @@
> +/*
> + * Copyright (c) 2016, NVIDIA CORPORATION.
> + *
> + * SPDX-License-Identifier: GPL-2.0
> + */
> +
> +#include <common.h>
> +#include <dm.h>
> +#include <power-domain-uclass.h>
> +#include <asm/arch-tegra/bpmp_abi.h>
> +#include <asm/arch-tegra/tegra186_bpmp.h>
> +
> +#define UPDATE BIT(0)
> +#define ON BIT(1)
> +
> +static int tegra186_power_domain_request(struct power_domain *power_domain)
> +{
> + debug("%s(power_domain=%p) (dev=%p, id=%lu)\n", __func__,
> + power_domain, power_domain->dev, power_domain->id);
> +
> + return 0;
> +}
> +
> +static int tegra186_power_domain_free(struct power_domain *power_domain)
> +{
> + debug("%s(power_domain=%p) (dev=%p, id=%lu)\n", __func__,
> + power_domain, power_domain->dev, power_domain->id);
> +
> + return 0;
> +}
> +
> +static int tegra186_power_domain_common(struct power_domain *power_domain,
> + bool on)
> +{
> + struct mrq_pg_update_state_request req;
> + int on_state = on ? ON : 0;
> +
> + req.partition_id = power_domain->id;
> + req.logic_state = UPDATE | on_state;
> + req.sram_state = UPDATE | on_state;
> + /*
> + * Drivers manage their own clocks so they don't get out of sync, and
> + * since some power domains have many clocks, only a subset of which
> + * are actually needed depending on use-case.
> + */
> + req.clock_state = UPDATE;
> +
> + return tegra186_bpmp_call(power_domain->dev->parent,
> + MRQ_PG_UPDATE_STATE, &req, sizeof(req),
> NULL,
> + 0);
Again I wonder if this can use UCLASS_MISC?
> +}
> +
> +static int tegra186_power_domain_on(struct power_domain *power_domain)
> +{
> + debug("%s(power_domain=%p) (dev=%p, id=%lu)\n", __func__,
> + power_domain, power_domain->dev, power_domain->id);
> +
> + return tegra186_power_domain_common(power_domain, true);
> +}
> +
> +static int tegra186_power_domain_off(struct power_domain *power_domain)
> +{
> + debug("%s(power_domain=%p) (dev=%p, id=%lu)\n", __func__,
> + power_domain, power_domain->dev, power_domain->id);
> +
> + return tegra186_power_domain_common(power_domain, false);
> +}
> +
> +struct power_domain_ops tegra186_power_domain_ops = {
> + .request = tegra186_power_domain_request,
> + .free = tegra186_power_domain_free,
> + .on = tegra186_power_domain_on,
> + .off = tegra186_power_domain_off,
> +};
> +
> +static int tegra186_power_domain_probe(struct udevice *dev)
> +{
> + debug("%s(dev=%p)\n", __func__, dev);
> +
> + return 0;
> +}
> +
> +U_BOOT_DRIVER(tegra186_power_domain) = {
> + .name = "tegra186_power_domain",
> + .id = UCLASS_POWER_DOMAIN,
> + .probe = tegra186_power_domain_probe,
> + .ops = &tegra186_power_domain_ops,
> +};
> --
> 2.9.2
>
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
