linux-kernel - RE: [PATCH v2 resend 0/1] Input: add support HiDeep touchscreen.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <93AED97F53D7B941873C33A3FA30928855B9DA@mbox02.hideep.com>
Date:   Mon, 6 Nov 2017 04:00:02 +0000
From:   Anthony Kim <Anthony.Kim@...eep.com>
To:     'Dmitry Torokhov' <dmitry.torokhov@...il.com>
CC:     "'robh+dt@...nel.org'" <robh+dt@...nel.org>,
        "'linux-input@...r.kernel.org'" <linux-input@...r.kernel.org>,
        "'devicetree@...r.kernel.org'" <devicetree@...r.kernel.org>,
        "'linux-kernel@...r.kernel.org'" <linux-kernel@...r.kernel.org>,
        Dennis Hong <Dennis.Hong@...eep.com>
Subject: RE: [PATCH v2 resend 0/1] Input: add support HiDeep touchscreen.

Hi Dmitry,

Thanks for your mention.

>Hi Anthony,

>On Tue, Oct 31, 2017 at 07:17:56PM +0900, Anthony Kim wrote:
>> Hi,
>> 
>> I remake patch to base on dmitry's modified code.
>> Please refer as follow.
>> http://www.spinics.net/lists/linux-input/msg53724.html
>> 
>> That code is working well almost, but needed to some modified.
>> 
>> - v2
>>   - Our fw binary data is little endian, but our IC protocol
>>     for fw update use big endian. So it need to convert to big endian data
>>     and I added about it.

>I understand that _currently_ your firmware binary is little endian, however it is easily remedied on your side. I do not understand why you would want to encumber the kernel running on many devices to do that conversion on fly.

As your mention, I will don't convert to another endian again. I didn't know whether it affected. Just I hoped our IC work well.
And I will test again using big endian binary. If there need to modify, I will send modified patch.

Thanks! 

>Thanks.

>--
>Dmitry