From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C3B00C433E0 for ; Wed, 27 Jan 2021 14:26:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7EF2D207C5 for ; Wed, 27 Jan 2021 14:26:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233278AbhA0O01 (ORCPT ); Wed, 27 Jan 2021 09:26:27 -0500 Received: from mail.kernel.org ([198.145.29.99]:43496 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231207AbhA0O0D (ORCPT ); Wed, 27 Jan 2021 09:26:03 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2E8C4206C2; Wed, 27 Jan 2021 14:25:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1611757523; bh=VRYlpb1rWDc5m2mMDWtLtAJrdDJQFb5s3vdwAXOzMac=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=sB4JMt1KVppwFIKslqsibX15IDWsSYoapKQwE0i64S63bzg584Z7bLafb+G5zCd1i 1CJKuSqtznWhpfmc/VXstm0fiolgYDWmd4Dr7yxkK6I302F07dLbrttV9BSAHf/sfN /k2kLr/E2vcc0sj4FCog7cO3QxQjm8owaEC9aE3U= Date: Wed, 27 Jan 2021 15:25:20 +0100 From: Greg KH To: carlis , Andy Whitcroft , Joe Perches Cc: devel@driverdev.osuosl.org, linux-fbdev@vger.kernel.org, mh12gx2825@gmail.com, oliver.graute@kococonnector.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, sbrivio@redhat.com, colin.king@canonical.com, zhangxuezhi1@yulong.com Subject: Re: [PATCH v10] staging: fbtft: add tearing signal detect Message-ID: References: <1611754972-151016-1-git-send-email-zhangxuezhi3@gmail.com> <20210127220809.000026fb@gmail.com> <20210127221708.00002568@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210127221708.00002568@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 27, 2021 at 10:17:08PM +0800, carlis wrote: > On Wed, 27 Jan 2021 15:13:05 +0100 > Greg KH wrote: > > > On Wed, Jan 27, 2021 at 10:08:09PM +0800, carlis wrote: > > > On Wed, 27 Jan 2021 14:51:55 +0100 > > > Greg KH wrote: > > > > > > > On Wed, Jan 27, 2021 at 09:42:52PM +0800, Carlis wrote: > > > > > From: zhangxuezhi > > > > > > > > > > For st7789v ic,when we need continuous full screen refresh, it > > > > > is best to wait for the TE signal arrive to avoid screen tearing > > > > > > > > > > Signed-off-by: zhangxuezhi > > > > > > > > Please slow down and wait at least a day between patch > > > > submissions, there is no rush here. > > > > > > > > And also, ALWAYS run scripts/checkpatch.pl on your submissions, so > > > > that you don't have a maintainer asking you about basic problems, > > > > like are in this current patch :( > > > > > > > > thanks, > > > > > > > > greg k-h > > > > > > hi, > > > This is my first patch contribution to Linux, so some of the rules > > > are not very clear .In addition, I can confirm that before sending > > > patch, I check it with checkPatch.py every time.Thank you very much > > > for your help > > > > Please read Documentation/SubmittingPatches which has a link to the > > checklist and other documentation you should read. > > > > And I doubt you are running checkpatch on your submission, as there is > > obvious coding style issues in it. If so, please provide the output > > as it must be broken :( > > > > thanks, > > > > greg k-h > hi, the patch v11 checkpatch.pl output is below: > > carlis@bf-rmsz-10:~/work/linux-kernel/linux$ ./scripts/checkpatch.pl > 0001-staging-fbtft-add-tearing-signal-detect.patch total: 0 errors, 0 > warnings, 0 checks, 176 lines checked > > 0001-staging-fbtft-add-tearing-signal-detect.patch has no obvious style > problems and is ready for submission. Wow, my apologies! Andy and Joe, there's something wrong here that is missing the fact that a line is being indented with spaces and not tabs in the patch at https://lore.kernel.org/r/1611754972-151016-1-git-send-email-zhangxuezhi3@gmail.com Any ideas what broke? thanks, greg k-h