Re: [PATCH v2] ARM: dts: samsung: exynos4210-i9100: Enable RTS/CTS in Bluetooth node

[Krzysztof Kozlowski]

On 07/09/2026 10:16, Marek Szyprowski wrote:
> On 03.09.2026 15:29, Paul Cercueil wrote:
>> 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.
> I also don't have strong opinion on this issue. Krzysztof: do You see
> any benefit from adding this "uart-has-rtscts" property, just to describe
> precisely the actual hardware, even if it won't be used by any driver?
> 

Yes, it makes sense, because drivers could be improved or DTS could be
used in other project, but it also does not really matter in such case.

Best regards,
Krzysztof