Re: [PATCH v2 1/1] serial: 8250_dw: Fix get_mctrl behaviour

From: Jonathan Richardson
Date: Mon Mar 09 2015 - 14:51:48 EST


On 15-03-09 11:40 AM, Dmitry Torokhov wrote:
> On Thu, Feb 26, 2015 at 4:35 PM, Jonathan Richardson
> <jonathar@xxxxxxxxxxxx> wrote:
>> @@ -334,6 +342,30 @@ static int dw8250_probe_of(struct uart_port *p,
>> if (id >= 0)
>> p->line = id;
>>
>> + if (of_property_read_bool(np, "dcd-override")) {
>> + /* Always report DCD as active */
>> + data->msr_mask_on |= UART_MSR_DCD;
>> + data->msr_mask_off |= UART_MSR_DDCD;
>> + }
>> +
>> + if (of_property_read_bool(np, "dsr-override")) {
>> + /* Always report DSR as active */
>> + data->msr_mask_on |= UART_MSR_DSR;
>> + data->msr_mask_off |= UART_MSR_DDSR;
>> + }
>> +
>> + if (of_property_read_bool(np, "cts-override")) {
>> + /* Always report DSR as active */
>> + data->msr_mask_on |= UART_MSR_DSR;
>> + data->msr_mask_off |= UART_MSR_DDSR;
>> + }
>> +
>> + if (of_property_read_bool(np, "ri-override")) {
>> + /* Always report Ring indicator as inactive */
>> + data->msr_mask_off |= UART_MSR_RI;
>> + data->msr_mask_off |= UART_MSR_TERI;
>
> This looks like a typo. Should the 1st line be data->msr_mask_on by any chance?

I think the code is correct. The signals other than ri need to be
reported as active (masked on) if specified, but ri needs to be inactive
(masked off) unlike the other 3. This is stated in the kernel docs
mentioned in the commit. ri behaves differently than the others.

>
> Thanks,
> Dmitry
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/