lkml.org 
[lkml]   [2015]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/1] serial: 8250_dw: Fix get_mctrl behaviour
On 15-03-09 11:40 AM, Dmitry Torokhov wrote:
> On Thu, Feb 26, 2015 at 4:35 PM, Jonathan Richardson
> <jonathar@broadcom.com> 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@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>



\
 
 \ /
  Last update: 2015-03-09 20:01    [W:0.054 / U:0.996 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site