Hi Hn,
On Thu, May 21, 2015 at 03:43:08PM +0800, Hn Chen wrote:
> Hi, Dmitry,
>
> Thanks for your comments.
> Here are my replies for some comments.
>
> > + fw_id = ((fw_chunk_info.chunk_info.version_number >> 12) & 0xF);
> > + chip_id = (((dev_wdt87xx->sys_param.fw_id) >> 12) & 0xF);
> > +
> > + if (fw_id != chip_id) {
> > + wdt87xx_get_sysparam(client, dev_wdt87xx);
>
> >Why would it be different now?
> This driver is intend to support two controllers, WDT8753 & WDT8756.
> The chip_id and the the firmware are different for them, so we have to check
> if
> The firmware is suitable for this controller.
What I meant is as far as I understand dev_wdt87xx->sys_param.fw_id is
what we've read from the controller when we probed the device. What is
the point in trying to re-read it with wdt87xx_get_sysparam()? Do we
expect it to somehow change?
>
> > +
> > + return (unsigned short) y;
>
> >I think I might have asked this before, by anyway: are we sure that we don't
> >already have this checksum implementation in lib/? And if we don't maybe we
> >should?
> The MISR is a checksum function which the full name is "Multiple Input
> Signature Register".
> I checked the kernel library and not found any algorithm like this.
> You mean I have to submit the checksum function into kernel/lib ?
No, you do not have to. I just wanted to make sure we are not
duplicating something that already exists.
>
> > Can you tell me a bit more about the way device sends touch data? Does it
> > always send all 10 contacts regardless of data in
> > raw_buf[TOUCH_PK_OFFSET_FNGR_NUM]?
> Yes, it always send all 10 contacts data.
I see.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
