Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933341AbbFJOvi (ORCPT ); Wed, 10 Jun 2015 10:51:38 -0400 Received: from ml01.weidahitech.com ([61.222.87.235]:2391 "EHLO ml01.weidahitech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754208AbbFJOv2 (ORCPT ); Wed, 10 Jun 2015 10:51:28 -0400 X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Subject: RE: [PATCH v5] Fix the resolution issue in ChromeOS Date: Wed, 10 Jun 2015 22:51:03 +0800 Message-ID: <1A63A1423E772B419A9D2E882CBCF6C4020229A9@mail> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v5] Fix the resolution issue in ChromeOS Thread-Index: AdCcl0hu4/7d7M7EQUO+Ud9tJKPKmgG9EPOA References: <1432873649-32484-1-git-send-email-hn.chen@weidahitech.com> <1A63A1423E772B419A9D2E882CBCF6C401F3DB61@mail> <20150601181829.GB4469@bugger.home> From: "Hn Chen" To: "Frans Klaver" Cc: , , Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t5AEpf2O003859 Content-Length: 2360 Lines: 50 Hi, Frans, > Alright, I was just wondering. Seems like a waste to be waiting for something that's already finished ;-). > There's of course a risk that times may fluctuate between firmware versions. Did you take that into account in the code? > Or is there a hard maximum time for these operations defined for the firmware? Thanks for your reminding. After I check with the firmware guy, I will change the value of delay. What we do here is to read the data from flash and calculate their checksum and will cost about 6ms for 1024 bytes. So there is a delay for 10 ms per 1024 bytes. But in some situation, the controller will change it's running frequency(like do noise immunity), 10ms could be too margin. Hn.chen -----Original Message----- From: Frans Klaver [mailto:fransklaver@gmail.com] Sent: Tuesday, June 02, 2015 2:18 AM To: Hn Chen Cc: linux-input@vger.kernel.org; linux-kernel@vger.kernel.org; dmitry.torokhov@gmail.com Subject: Re: [PATCH v5] Fix the resolution issue in ChromeOS On Tue, Jun 02, 2015 at 12:39:13AM +0800, Hn Chen wrote: > Hi, Klaver, > > Sorry for replying late and thanks for your opinion ! > > About the patch descrition, I will follow your suggestion and Ok. More on this is in Documentation/SubmittingPatches. > maybe add more commemts between codes to be easy to read. Only where really necessary. If you do, explain _why_ you do stuff, rather than how. The how is already in the code. If how isn't clear enough, clear up the code instead. > >Are these (and other) delay times based on datasheet values? > The time consuming is about the computing power of WDT87xx's controller. > The value is from the algorithm/firmware engineer of wdt87xx. > They think it is reasonable value to wait the controller to finish the computing. Alright, I was just wondering. Seems like a waste to be waiting for something that's already finished ;-). There's of course a risk that times may fluctuate between firmware versions. Did you take that into account in the code? Or is there a hard maximum time for these operations defined for the firmware? > For the rest parts, I'll just follow your opinion to modify them. > > Best Regards, > hn.chen > Thanks, Frans ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?