Re: [PATCH v2] ARM: dts: samsung: exynos4210-i9100: Enable RTS/CTS in Bluetooth node
From: Paul Cercueil
Date: Thu Sep 03 2026 - 11:08:48 EST
Hi Marek,
Le mercredi 02 septembre 2026 à 10:36 +0200, Marek Szyprowski a écrit :
> On 29.08.2026 12:15, Paul Cercueil wrote:
> > The BCM4330 is connected using CTS/RTS too, so add the 'uart-has-
> > rtscts'
> > property.
> >
> > Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
>
>
> Indeed the CTS/RTS pins are used for BT on most of the ARM Exynos
> boards,
> as the "<&uartX_fctl>" property is assigned for pinctl-0. This is not
> really
> specific to i9100, so maybe this property should be added to other
> dts?
I can add it to all exynos4 boards that use the "uartX_fctl" property.
> However I wonder if Samsung UART driver actually use this "uart-has-
> rtscts"
> DT property? A quick "git grep uart-has-rtscts drivers/tty" shows
> nothing
> related to Samsung Exynos...
It looks like it does not. I still think the DT property should be here
but I won't fight over this.
Cheers,
-Paul
>
> > ---
> >
> > Notes:
> > Changes:
> >
> > V1 was: "ARM: dts: samsung: exynos4210-i9100: Fix Bluetooth
> > node"
> >
> > -v2: Drop max-speed property. It's only supposed to be used in
> > some
> > cases, which don't apply here. As this was the "fix" for
> > getting
> > Bluetooth to work on my device, which will be applied to
> > the
> > driver instead, the Fixes: tag was removed.
> >
> > arch/arm/boot/dts/samsung/exynos4210-i9100.dts | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> > b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> > index 8a635bee59fa..95964be7daa8 100644
> > --- a/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> > +++ b/arch/arm/boot/dts/samsung/exynos4210-i9100.dts
> > @@ -875,6 +875,7 @@ &serial_0 {
> > status = "okay";
> >
> > pinctrl-names = "default";
> > + uart-has-rtscts;
> > pinctrl-0 = <&bt_en>, <&bt_res>, <&uart0_data>,
> > <&uart0_fctl>;
> >
> > bluetooth {
>
> Best regards