Hi Bastian,
Please make sure you include the sunxi platform maintainers (as reported
by get_maintainers.pl) when sending your patch, so it gets in our inboxes.
Subject prefix should be "arm64: dts: allwinner: a64:" (with "dts:").
On 10/7/22 18:45, Bastian Germann wrote:
> From: Vasily Khoruzhick <anarsoul@xxxxxxxxx>
>
> Pinebook has an RTL8723CS WiFi + BT chip. BT is connected to UART1
> and uses PL5 as device wake GPIO and PL6 as host wake GPIO.
> The I2C controlling signals are connected to R_I2C bus.
Please remove this sentence; it doesn't make sense. The RTL8723CS does
not have any I2C connection.
> Enable it in the device tree.
>
> Signed-off-by: Vasily Khoruzhick <anarsoul@xxxxxxxxx>
> Signed-off-by: Bastian Germann <bage@xxxxxxxxxx>
> ---
> .../boot/dts/allwinner/sun50i-a64-pinebook.dts | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
> index 620cb3ef5f6c..19ba87c411ed 100644
> --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
> +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts
> @@ -406,6 +406,19 @@ &uart0 {
> status = "okay";
> };
>
> +&uart1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart1_pins>, <&uart1_rts_cts_pins>;
> + uart-has-rtscts;
> + status = "okay";
> +
> + bluetooth {
> + compatible = "realtek,rtl8723cs-bt";
> + device-wake-gpios = <&r_pio 0 5 GPIO_ACTIVE_LOW>; /* PL5 */
You also need:
enable-gpios = <&r_pio 0 4 GPIO_ACTIVE_HIGH>; /* PL4 */
(this is BT-RST-N in the schematic).
The rest looks good to me.
Regards,
Samuel
> + host-wake-gpios = <&r_pio 0 6 GPIO_ACTIVE_HIGH>; /* PL6 */
> + };
> +};
> +
> &usb_otg {
> dr_mode = "host";
> };
