[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <32ff2c9b-1d34-4637-80ff-e8eefe253a95@loongson.cn>
Date: Mon, 12 Aug 2024 16:09:44 +0800
From: 郑豪威 <zhenghaowei@...ngson.cn>
To: Krzysztof Kozlowski <krzk@...nel.org>, gregkh@...uxfoundation.org,
jirislaby@...nel.org, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, chenhuacai@...nel.org, kernel@...0n.name,
p.zabel@...gutronix.de
Cc: linux-serial@...r.kernel.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, loongarch@...ts.linux.dev
Subject: Re: [PATCH v2 1/3] dt-bindings: serial: Add Loongson UART controller
在 2024/8/9 18:05, Krzysztof Kozlowski 写道:
> On 09/08/2024 11:55, 郑豪威 wrote:
>>>>>> + description: Enables fractional-N division. Currently,
>>>>>> + only LS2K1500 and LS2K2000 support this feature.
>>>>>> +
>>>>>> + rts-invert:
>>>>>> + description: Inverts the RTS value in the MCR register.
>>>>>> + This should be used on Loongson-3 series CPUs, Loongson-2K
>>>>>> + series CPUs, and Loongson LS7A bridge chips.
>>>>>> +
>>>>>> + dtr-invert:
>>>>>> + description: Inverts the DTR value in the MCR register.
>>>>>> + This should be used on Loongson-3 series CPUs, Loongson-2K
>>>>>> + series CPUs, and Loongson LS7A bridge chips.
>>>>>> +
>>>>>> + cts-invert:
>>>>>> + description: Inverts the CTS value in the MSR register.
>>>>>> + This should be used on Loongson-2K0500, Loongson-2K1000,
>>>>>> + and Loongson LS7A bridge chips.
>>>>>> +
>>>>>> + dsr-invert:
>>>>>> + description: Inverts the DSR value in the MSR register.
>>>>>> + This should be used on Loongson-2K0500, Loongson-2K1000,
>>>>>> + and Loongson LS7A bridge chips.
>>> Same questions for all these. Why choosing invert is a board level
>>> decision? If it "should be used" then why it is not used always?
>>>
>> Because these features are not applicable to all chips, such as
>> 'fractional-division',
> Hm?
>
>> which is currently supported only by 2K1500 and 2K2000, and for
>> Loongson-3 series
> These are SoCs. Compatible defines that. Please align with your
> colleagues, because *we talked about this* already.
>
> Best regards,
> Krzysztof
I consulted with my colleagues and would like to confirm with you. For
the five
properties provided, fractional-division is offered as a new feature,
supported by
2K1500 and 2K2000. As for the invert property, it is due to a bug in our
controller,
and its usage may vary across different chips. Should we add different
compatible
values to address these issues for different chips, whether they are new
features or
controller bugs?
Best regards,
Haowei Zheng
Powered by blists - more mailing lists