Re: [PATCH 00/11] soc: ti: wkup_m3_ipc: support vtt toggle, io isolation & voltage scaling

From: Nishanth Menon <nm@ti.com>
To: Drew Fustini <dfustini@baylibre.com>
Cc: "Benoît Cousson" <bcousson@baylibre.com>,
	"Bjorn Andersson" <bjorn.andersson@linaro.org>,
	"Dave Gerlach" <d-gerlach@ti.com>,
	devicetree@vger.kernel.org, Keerthy <j-keerthy@ti.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-remoteproc@vger.kernel.org,
	"Mathieu Poirier" <mathieu.poirier@linaro.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Santosh Shilimkar" <ssantosh@kernel.org>,
	"Tony Lindgren" <tony@atomide.com>,
	s-anna@ti.com, khilman@baylibre.com
Subject: Re: [PATCH 00/11] soc: ti: wkup_m3_ipc: support vtt toggle, io isolation & voltage scaling
Date: Mon, 21 Feb 2022 06:58:17 -0600	[thread overview]
Message-ID: <20220221125817.eg2rf6ktrhiq7eck@overbook> (raw)
In-Reply-To: <20220219215328.485660-1-dfustini@baylibre.com>

On 13:53-20220219, Drew Fustini wrote:
> Improve the wkup_m3_ipc driver to better support the TI Wakeup Cortex M3 
> device found on TI AM33xx and AM43xx SoCs.
> 
> This series derives from these commits in the ti-linux-5.10.y branch of
> the TI Linux repo [1]:
> 
>   6ab4eff1034b ("remoteproc: move rproc_da_to_va declaration to remoteproc.h")
>   2a9be39a26f7 ("dt-bindings: wkup_m3_ipc: Add vtt toggling bindings")
>   c65263f9e12c ("wkup_m3_ipc: Add support for toggling VTT regulator")
>   5c6c821803e1 ("dt-bindings: wkup_m3_ipc: Add ti,io-isolation property")
>   196c46f7577d ("wkup_m3_ipc: Add support for IO Isolation")
>   c28acc847e5d ("soc: ti: wkup_m3_ipc: Add support for i2c voltage scaling")
>   a4f9ef4ab5ca ("ARM: dts: am437x-gp-evm: Enable wkup_m3 control of IO isolation")
>   94de756f1771 ("ARM: dts: am33xx: Add scale data fw to wkup_m3_ipc node")
>   b7ae4b063793 ("ARM: dts: am43xx: Add scale data fw to wkup_m3_ipc node")
>   451ec7871ae7 ("soc: ti: wkup_m3_ipc: Add debug option to halt m3 in suspend")
> 
> Minor changes have been made to some patches to resolve conflicts and 
> to split i2c voltage scaling dt-bindings into separate patch.
> 
> [1] git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git
> 
> Dave Gerlach (10):
>   dt-bindings: wkup_m3_ipc: Add vtt toggling bindings
>   soc: ti: wkup_m3_ipc: Add support for toggling VTT regulator
>   dt-bindings: wkup_m3_ipc: Add ti,io-isolation property
>   soc: ti: wkup_m3_ipc: Add support for IO Isolation
>   ARM: dts: am437x-gp-evm: Enable wkup_m3 control of IO isolation
>   dt-bindings: wkup_m3_ipc: Add scale-data-fw property
>   soc: ti: wkup_m3_ipc: Add support for i2c voltage scaling
>   ARM: dts: am33xx: Add scale data fw to wkup_m3_ipc node
>   ARM: dts: am43xx: Add scale data fw to wkup_m3_ipc node
>   soc: ti: wkup_m3_ipc: Add debug option to halt m3 in suspend
> 
> Suman Anna (1):
>   remoteproc: move rproc_da_to_va declaration to remoteproc.h

This series seems to needing to go to various maintainers here

Series should cleanup (example: txt files to yaml bindings),
prior to creating patch series for each maintainer. I'd suggest bindings
and drivers in 1 kernel rev and follow it up with dts updata for the
next rev.

> 
>  .../bindings/soc/ti/wkup_m3_ipc.txt           |  91 ++++++++
>  arch/arm/boot/dts/am335x-bone-common.dtsi     |   4 +
>  arch/arm/boot/dts/am335x-evm.dts              |   4 +
>  arch/arm/boot/dts/am335x-evmsk.dts            |   4 +
>  arch/arm/boot/dts/am437x-gp-evm.dts           |   5 +
>  arch/arm/boot/dts/am437x-sk-evm.dts           |   4 +
>  arch/arm/boot/dts/am43x-epos-evm.dts          |   4 +
>  drivers/remoteproc/remoteproc_internal.h      |   1 -
>  drivers/soc/ti/wkup_m3_ipc.c                  | 208 +++++++++++++++++-
>  include/linux/remoteproc.h                    |   1 +
>  include/linux/wkup_m3_ipc.h                   |  13 ++
>  11 files changed, 334 insertions(+), 5 deletions(-)
> 
> -- 
> 2.32.0
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D