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?
Thanks,
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
