2021-05-03 18:09:55

by Jerome Brunet

Subject: Re: [PATCHv1 0/9] Added Audio and HDMI power domain for Amlogic SoC

On Mon 03 May 2021 at 16:54, Anand Moon <[email protected]> wrote:

> Patch series add Audio and Hdmi power domain for Amlogic SoC.
>
> Tested on GXBB - Odroid C2
> SM1 - Odroid C4
> G12B - Odroid N2
> -Anand
>

AFAIK, the audio power domain is a no-op on the g12/sm1. At least this
is what we've been told by AML. Unless this solves something, I don't
think it should be done at all.

> Anand Moon (9):
> soc: amlogic: meson-ee-pwrc: Add audio power domain Meson g12a and
> g12b SoCs
> arm64: dts: amlogic: Add audio power domain for g12a and g12b
> soc: amlogic: meson-ee-pwrc: Add hdmi power domain Meson g12a SoCs
> arm64: dts: amlogic: Add hdmi power domain for g12a and g12b
> soc: amlogic: meson-ee-pwrc: Add hdmi power domain Meson sm1 SoCs
> arm64: dts: amlogic: Add audio power domain for sm1 SoC
> arm64: dts: amlogic: Add hdmi power domain for sm1 sbc
> soc: amlogic: meson-ee-pwrc: Add hdmi power domain Meson gxbb and gxl
> SoCs
> arm64: dts: amlogic: Add hdmi power domain for gxbb and gxl
>
> arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 2 ++
> arch/arm64/boot/dts/amlogic/meson-g12a-u200.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 2 ++
> arch/arm64/boot/dts/amlogic/meson-g12b-gsking-x.dts | 1 +
> .../arm64/boot/dts/amlogic/meson-g12b-gtking-pro.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts | 1 +
> .../arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dtsi | 2 ++
> arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-g12b-w400.dtsi | 1 +
> .../boot/dts/amlogic/meson-gx-libretech-pc.dtsi | 1 +
> arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxbb-nanopi-k2.dts | 1 +
> .../boot/dts/amlogic/meson-gxbb-nexbox-a95x.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxbb-p20x.dtsi | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxbb-wetek.dtsi | 1 +
> .../dts/amlogic/meson-gxl-s805x-libretech-ac.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxl-s805x-p241.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxl-s905d-p230.dts | 1 +
> .../boot/dts/amlogic/meson-gxl-s905x-khadas-vim.dts | 1 +
> .../dts/amlogic/meson-gxl-s905x-libretech-cc-v2.dts | 1 +
> .../dts/amlogic/meson-gxl-s905x-libretech-cc.dts | 1 +
> .../boot/dts/amlogic/meson-gxl-s905x-nexbox-a95x.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxl-s905x-p212.dts | 1 +
> .../arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts | 1 +
> arch/arm64/boot/dts/amlogic/meson-sm1-odroid.dtsi | 2 ++
> arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 2 ++
> drivers/soc/amlogic/meson-ee-pwrc.c | 12 ++++++++++++
> include/dt-bindings/power/meson-g12a-power.h | 2 ++
> include/dt-bindings/power/meson-gxbb-power.h | 1 +
> include/dt-bindings/power/meson-sm1-power.h | 1 +
> 33 files changed, 50 insertions(+)

2021-05-03 18:11:15

by Anand Moon

Subject: Re: [PATCHv1 0/9] Added Audio and HDMI power domain for Amlogic SoC

Hi Jerome,

On Mon, 3 May 2021 at 20:35, Jerome Brunet <[email protected]> wrote:
>
>
> On Mon 03 May 2021 at 16:54, Anand Moon <[email protected]> wrote:
>
> > Patch series add Audio and Hdmi power domain for Amlogic SoC.
> >
> > Tested on GXBB - Odroid C2
> > SM1 - Odroid C4
> > G12B - Odroid N2
> > -Anand
> >
>
> AFAIK, the audio power domain is a no-op on the g12/sm1. At least this
> is what we've been told by AML. Unless this solves something, I don't
> think it should be done at all.
>

Opps, I unitedly make some assumption,.I will drop these changes.

Thanks
-Anand