On Mon, 27 Dec 2010 12:55:38 -0800 (PST)
Tsozik <tsozik@xxxxxxxxx> wrote:
> +static void mct_u232_msr_to_icount(struct async_icount *icount,
> + unsigned char msr)
> +{
> + /* Translate Control Line states */
> + if (msr & MCT_U232_MSR_DDSR)
> + icount->dsr++;
> + if (msr & MCT_U232_MSR_DCTS)
> + icount->cts++;
> + if (msr & MCT_U232_MSR_DRI)
> + icount->rng++;
> + if (msr & MCT_U232_MSR_DCD)
> + icount->dcd++;
> +} /* mct_u232_msr_to_icount */
This looks good to me, but since it's a new hardware facility that
we did not use previously, I'd like to make sure this works for you.
> mct_u232_msr_to_icount uses delta nibble to track state changes.
> RM-60 testing showed the same levels as measured by PDM-2 in close
> proximity to RM-60.
Right, this is good. I have just one request for you: could you find
somewhere a computer with a built-in 8250-compatible serial port,
attach RM-60 to it, and run your counting application there?
I think your patch looks just fine, but I would like to make sure
that we (e.g. mct_u232+patch) are truly compatible now.
-- Pete
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
