2023-02-10 17:07:33

by Christian Hewitt

[permalink] [raw]
Subject: [PATCH v2 0/3] arm64: dts: meson: gxbb-kii-pro: device-tree updates

First we sort nodes in the current dts and remove some blank lines. Then we
add the remaining bits needed for Bluetooth to work. And finally we add the
AIU audio card to have HDMI and S/PDIF audio output.

Changes since v1:

- Add reviews from Neil
- Drop unneded 'okay' from audio patch

Christian Hewitt (3):
arm64: dts: meson: gxbb-kii-pro: sort and tidy the dts
arm64: dts: meson: gxbb-kii-pro: complete the bluetooth node
arm64: dts: meson: gxbb-kii-pro: add initial audio support

.../boot/dts/amlogic/meson-gxbb-kii-pro.dts | 83 ++++++++++++++++---
1 file changed, 72 insertions(+), 11 deletions(-)

--
2.34.1



2023-02-10 17:07:36

by Christian Hewitt

[permalink] [raw]
Subject: [PATCH v2 1/3] arm64: dts: meson: gxbb-kii-pro: sort and tidy the dts

Alpha-sort the nodes, move the default line in the LED node to where it's
normally found, and remove excess spacing. No functional changes.

Signed-off-by: Christian Hewitt <[email protected]>
Tested-by: Ferass El Hafidi <[email protected]>
Reviewed-by: Neil Armstrong <[email protected]>
---
.../boot/dts/amlogic/meson-gxbb-kii-pro.dts | 32 ++++++++-----------
1 file changed, 14 insertions(+), 18 deletions(-)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
index 6d8cc00fedc7..9bfdb54d912a 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
@@ -6,21 +6,22 @@
/dts-v1/;

#include "meson-gxbb-p20x.dtsi"
-
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/input/input.h>
#include <dt-bindings/leds/common.h>
+
/ {
compatible = "videostrong,kii-pro", "amlogic,meson-gxbb";
model = "Videostrong KII Pro";

leds {
compatible = "gpio-leds";
+
status {
gpios = <&gpio_ao GPIOAO_13 GPIO_ACTIVE_LOW>;
- default-state = "off";
color = <LED_COLOR_ID_RED>;
function = LED_FUNCTION_STATUS;
+ default-state = "off";
};
};

@@ -34,24 +35,8 @@ button-reset {
gpios = <&gpio_ao GPIOAO_3 GPIO_ACTIVE_HIGH>;
};
};
-
-};
-
-
-
-&uart_A {
- status = "okay";
- pinctrl-0 = <&uart_a_pins>, <&uart_a_cts_rts_pins>;
- pinctrl-names = "default";
- uart-has-rtscts;
-
- bluetooth {
- compatible = "brcm,bcm4335a0";
- };
};

-
-
&ethmac {
status = "okay";
pinctrl-0 = <&eth_rmii_pins>;
@@ -78,3 +63,14 @@ eth_phy0: ethernet-phy@0 {
&ir {
linux,rc-map-name = "rc-videostrong-kii-pro";
};
+
+&uart_A {
+ status = "okay";
+ pinctrl-0 = <&uart_a_pins>, <&uart_a_cts_rts_pins>;
+ pinctrl-names = "default";
+ uart-has-rtscts;
+
+ bluetooth {
+ compatible = "brcm,bcm4335a0";
+ };
+};
--
2.34.1


2023-02-10 17:07:43

by Christian Hewitt

[permalink] [raw]
Subject: [PATCH v2 2/3] arm64: dts: meson: gxbb-kii-pro: complete the bluetooth node

Add missing content to the bluetooth node to align it with the content
used in (all) other GXBB dts files.

Signed-off-by: Christian Hewitt <[email protected]>
Tested-by: Ferass El Hafidi <[email protected]>
Reviewed-by: Neil Armstrong <[email protected]>
---
arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
index 9bfdb54d912a..435ae2aa404e 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
@@ -72,5 +72,10 @@ &uart_A {

bluetooth {
compatible = "brcm,bcm4335a0";
+ shutdown-gpios = <&gpio GPIOX_20 GPIO_ACTIVE_HIGH>;
+ host-wakeup-gpios = <&gpio GPIOX_21 GPIO_ACTIVE_HIGH>;
+ max-speed = <2000000>;
+ clocks = <&wifi32k>;
+ clock-names = "lpo";
};
};
--
2.34.1


2023-02-10 17:07:47

by Christian Hewitt

[permalink] [raw]
Subject: [PATCH v2 3/3] arm64: dts: meson: gxbb-kii-pro: add initial audio support

Add initial support for HDMI and S/PDIF audio output.

Signed-off-by: Christian Hewitt <[email protected]>
Tested-by: Ferass El Hafidi <[email protected]>
Reviewed-by: Neil Armstrong <[email protected]>
---
.../boot/dts/amlogic/meson-gxbb-kii-pro.dts | 60 +++++++++++++++++++
1 file changed, 60 insertions(+)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
index 435ae2aa404e..a09340e9f73a 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts
@@ -9,11 +9,19 @@
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/input/input.h>
#include <dt-bindings/leds/common.h>
+#include <dt-bindings/sound/meson-aiu.h>

/ {
compatible = "videostrong,kii-pro", "amlogic,meson-gxbb";
model = "Videostrong KII Pro";

+ spdif_dit: audio-codec-0 {
+ #sound-dai-cells = <0>;
+ compatible = "linux,spdif-dit";
+ status = "okay";
+ sound-name-prefix = "DIT";
+ };
+
leds {
compatible = "gpio-leds";

@@ -35,6 +43,58 @@ button-reset {
gpios = <&gpio_ao GPIOAO_3 GPIO_ACTIVE_HIGH>;
};
};
+
+ sound {
+ compatible = "amlogic,gx-sound-card";
+ model = "KII-PRO";
+ assigned-clocks = <&clkc CLKID_MPLL0>,
+ <&clkc CLKID_MPLL1>,
+ <&clkc CLKID_MPLL2>;
+ assigned-clock-parents = <0>, <0>, <0>;
+ assigned-clock-rates = <294912000>,
+ <270950400>,
+ <393216000>;
+
+ dai-link-0 {
+ sound-dai = <&aiu AIU_CPU CPU_I2S_FIFO>;
+ };
+
+ dai-link-1 {
+ sound-dai = <&aiu AIU_CPU CPU_SPDIF_FIFO>;
+ };
+
+ dai-link-2 {
+ sound-dai = <&aiu AIU_CPU CPU_I2S_ENCODER>;
+ dai-format = "i2s";
+ mclk-fs = <256>;
+
+ codec-0 {
+ sound-dai = <&aiu AIU_HDMI CTRL_I2S>;
+ };
+ };
+
+ dai-link-3 {
+ sound-dai = <&aiu AIU_CPU CPU_SPDIF_ENCODER>;
+
+ codec-0 {
+ sound-dai = <&spdif_dit>;
+ };
+ };
+
+ dai-link-4 {
+ sound-dai = <&aiu AIU_HDMI CTRL_OUT>;
+
+ codec-0 {
+ sound-dai = <&hdmi_tx>;
+ };
+ };
+ };
+};
+
+&aiu {
+ status = "okay";
+ pinctrl-0 = <&spdif_out_y_pins>;
+ pinctrl-names = "default";
};

&ethmac {
--
2.34.1


2023-02-11 20:07:30

by Martin Blumenstingl

[permalink] [raw]
Subject: Re: [PATCH v2 1/3] arm64: dts: meson: gxbb-kii-pro: sort and tidy the dts

On Fri, Feb 10, 2023 at 6:07 PM Christian Hewitt
<[email protected]> wrote:
>
> Alpha-sort the nodes, move the default line in the LED node to where it's
> normally found, and remove excess spacing. No functional changes.
>
> Signed-off-by: Christian Hewitt <[email protected]>
> Tested-by: Ferass El Hafidi <[email protected]>
> Reviewed-by: Neil Armstrong <[email protected]>
Reviewed-by: Martin Blumenstingl <[email protected]>

2023-02-11 20:07:59

by Martin Blumenstingl

[permalink] [raw]
Subject: Re: [PATCH v2 2/3] arm64: dts: meson: gxbb-kii-pro: complete the bluetooth node

On Fri, Feb 10, 2023 at 6:07 PM Christian Hewitt
<[email protected]> wrote:
>
> Add missing content to the bluetooth node to align it with the content
> used in (all) other GXBB dts files.
>
> Signed-off-by: Christian Hewitt <[email protected]>
> Tested-by: Ferass El Hafidi <[email protected]>
> Reviewed-by: Neil Armstrong <[email protected]>
Reviewed-by: Martin Blumenstingl <[email protected]>

2023-02-11 20:08:55

by Martin Blumenstingl

[permalink] [raw]
Subject: Re: [PATCH v2 3/3] arm64: dts: meson: gxbb-kii-pro: add initial audio support

On Fri, Feb 10, 2023 at 6:07 PM Christian Hewitt
<[email protected]> wrote:
>
> Add initial support for HDMI and S/PDIF audio output.
>
> Signed-off-by: Christian Hewitt <[email protected]>
> Tested-by: Ferass El Hafidi <[email protected]>
> Reviewed-by: Neil Armstrong <[email protected]>
Reviewed-by: Martin Blumenstingl <[email protected]>

2023-03-06 08:40:32

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH v2 0/3] arm64: dts: meson: gxbb-kii-pro: device-tree updates

Hi !

On 10/02/2023 18:07, Christian Hewitt wrote:
> First we sort nodes in the current dts and remove some blank lines. Then we
> add the remaining bits needed for Bluetooth to work. And finally we add the
> AIU audio card to have HDMI and S/PDIF audio output.
>
> Changes since v1:
>
> - Add reviews from Neil
> - Drop unneded 'okay' from audio patch
>
> Christian Hewitt (3):
> arm64: dts: meson: gxbb-kii-pro: sort and tidy the dts
> arm64: dts: meson: gxbb-kii-pro: complete the bluetooth node
> arm64: dts: meson: gxbb-kii-pro: add initial audio support
>
> .../boot/dts/amlogic/meson-gxbb-kii-pro.dts | 83 ++++++++++++++++---
> 1 file changed, 72 insertions(+), 11 deletions(-)
>

This doesn't apply anymore on v6.3-rc1, can you rebase ?
Thanks!

Neil

2023-03-06 12:16:42

by Christian Hewitt

[permalink] [raw]
Subject: Re: [PATCH v2 0/3] arm64: dts: meson: gxbb-kii-pro: device-tree updates


> On 6 Mar 2023, at 12:40 pm, Neil Armstrong <[email protected]> wrote:
>
> Hi !
>
> On 10/02/2023 18:07, Christian Hewitt wrote:
>> First we sort nodes in the current dts and remove some blank lines. Then we
>> add the remaining bits needed for Bluetooth to work. And finally we add the
>> AIU audio card to have HDMI and S/PDIF audio output.
>> Changes since v1:
>> - Add reviews from Neil
>> - Drop unneded 'okay' from audio patch
>> Christian Hewitt (3):
>> arm64: dts: meson: gxbb-kii-pro: sort and tidy the dts
>> arm64: dts: meson: gxbb-kii-pro: complete the bluetooth node
>> arm64: dts: meson: gxbb-kii-pro: add initial audio support
>> .../boot/dts/amlogic/meson-gxbb-kii-pro.dts | 83 ++++++++++++++++---
>> 1 file changed, 72 insertions(+), 11 deletions(-)
>
> This doesn't apply anymore on v6.3-rc1, can you rebase ?
> Thanks!

v3 rebased on for-next sent.

Christian