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=-15.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 C8E76C433B4 for ; Wed, 7 Apr 2021 05:37:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 9F206613AF for ; Wed, 7 Apr 2021 05:37:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232902AbhDGFhv (ORCPT ); Wed, 7 Apr 2021 01:37:51 -0400 Received: from mail.kernel.org ([198.145.29.99]:39906 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229558AbhDGFhu (ORCPT ); Wed, 7 Apr 2021 01:37:50 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 2D25A613AF; Wed, 7 Apr 2021 05:37:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1617773861; bh=nzwtY+L0kgmfWf4JSBCkm5fqQT9Wgv7V7OBgqfxP7iQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=xnCUb60RQc+dbIilRCXgHSUKVn1uSFZRGHhqBgIHEUtOafKvgpD8IHyGesMJ1jfXL 01RmSupd0IRlARujw4yym69PmeknlmD/m+ZPpiHBvIaUWDGCfrEvx0jMX/iGk0Yneh EqdAp4TF0KHnboEMVlH3NikrloWCoSpedSWQfP/8= Date: Wed, 7 Apr 2021 07:37:39 +0200 From: Greg KH To: Beatriz Martins de Carvalho Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, outreachy-kernel@googlegroups.com Subject: Re: [RESEND PATCH] staging: emxx_udc: Ending line with argument Message-ID: References: <20210406193409.96428-1-martinsdecarvalhobeatriz@gmail.com> <4c7df741-4e73-2ac4-f0d8-c9513ae29c88@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4c7df741-4e73-2ac4-f0d8-c9513ae29c88@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 06, 2021 at 09:00:07PM +0100, Beatriz Martins de Carvalho wrote: > > Em 06/04/21 20:36, Greg KH escreveu: > > On Tue, Apr 06, 2021 at 08:34:09PM +0100, Beatriz Martins de Carvalho wrote: > > > Cleans up check of "Lines should not end with a '('" > > > with argument present in next line in file emxx_udc.c > > > > > > Signed-off-by: Beatriz Martins de Carvalho > > > --- > > > drivers/staging/emxx_udc/emxx_udc.c | 11 ++++------- > > > 1 file changed, 4 insertions(+), 7 deletions(-) > > Why is this a [RESEND] ? > > > > What happened to the first version? > Sorry, I didn't receive your review, and in kernelnewbies tutorial, they say > if not receive a response, may have missed the patch, so I resent it. Do you have a pointer to your previous patch in the lore.kernel.org archives anywhere? I can't seem to find it. > > Also, your subject is odd, please look at the documentation for how to > > write good subject lines for patches. > > Yes, I know. It was my second patch, and I was learning, and when I resent > it, I didn't know if I can change the subject. Of course you can always fix up things you know you did incorrectly, don't make others review stuff that you know is wrong, that just wastes everyone's time. thanks, greg k-h