From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wanlong Gao Subject: Re: [PATCH 4/6] Input: elantech - work around EC buffer Date: Thu, 18 Aug 2011 14:54:56 +0800 Message-ID: <4E4CB740.5020301@cn.fujitsu.com> References: <1313632629-23603-1-git-send-email-jj_ding@emc.com.tw> <1313632629-23603-5-git-send-email-jj_ding@emc.com.tw> <4E4C81EB.7040303@cn.fujitsu.com> <8739gz43lp.fsf@emc.com.tw> Reply-To: gaowanlong@cn.fujitsu.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from cn.fujitsu.com ([222.73.24.84]:59401 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751665Ab1HRG4B (ORCPT ); Thu, 18 Aug 2011 02:56:01 -0400 In-Reply-To: <8739gz43lp.fsf@emc.com.tw> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: JJ Ding Cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Seth Forshee , Dmitry Torokhov , Aaron Huang , Tom Lin , Eric Piel , Daniel Kurtz , Chase Douglas , Henrik Rydberg , Alessandro Rubini On 08/18/2011 02:48 PM, JJ Ding wrote: > Hi Wanlong Gao, > > On Thu, 18 Aug 2011 11:07:23 +0800, Wanlong Gao wrote: >> On 08/18/2011 09:57 AM, JJ Ding wrote: >>> >>> return PSMOUSE_FULL_PACKET; >>> + >>> + bad_packet: >> You may introduce space here. > Sorry, I am following you here. > Do you want me to add one more line below bad_packet:, > or you want me to remove the space before bad_packet? > > I keep that space to make the goto tags look consistent with other > places in this file. > > Should I remove all starting spaces in goto tags in the file? > > Thanks, > > jj aha, I see that some with space but some without. Now, I'm not sure....;) Thanks -Wanlong Gao > >>> + memmove(psmouse->packet, psmouse->packet + 1, psmouse->pktsize - 1); >>> + psmouse->pktcnt--; >>> + return PSMOUSE_GOOD_DATA; >>> } >>> >>> /* >> >> >> -- >> Thanks >> Wanlong Gao > -- > To unsubscribe from this list: send the line "unsubscribe linux-input" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >