From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752070AbdB0PIy (ORCPT ); Mon, 27 Feb 2017 10:08:54 -0500 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:57415 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751422AbdB0PHi (ORCPT ); Mon, 27 Feb 2017 10:07:38 -0500 X-IronPort-AV: E=Sophos;i="5.35,215,1484002800"; d="scan'208";a="214881845" Date: Mon, 27 Feb 2017 16:07:27 +0100 (CET) From: Julia Lawall X-X-Sender: jll@hadrien To: Arushi Singhal cc: Greg KH , devel@driverdev.osuosl.org, outreachy-kernel@googlegroups.com, linux-kernel@vger.kernel.org Subject: Re: [Outreachy kernel] Re: [PATCH v2] staging: ks7010: remove code in comments. In-Reply-To: Message-ID: References: <20170226160820.GA20046@arushi-HP-Pavilion-Notebook> <20170227144852.GA24465@kroah.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="8323329-1698831945-1488208047=:3032" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --8323329-1698831945-1488208047=:3032 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT On Mon, 27 Feb 2017, Arushi Singhal wrote: > > > On Mon, Feb 27, 2017 at 8:18 PM, Greg KH wrote: > On Sun, Feb 26, 2017 at 09:38:20PM +0530, Arushi Singhal wrote: > > Commenting Code Is a Bad Idea. > > Comments are their to explain the code and how the code > achieves its > > goal and as codes in the comments  does not explain what the > code is > > doing so there is no use of commenting them. > > So in this patch codes in the comments are removed. > > > > Signed-off-by: Arushi Singhal > > > --- > >  changes in v2 > >  - subject lines is made short. > > > >  drivers/staging/ks7010/ks7010_sdio.c | 4 ---- > >  1 file changed, 4 deletions(-) > > This patch does not apply against my staging-testing branch :( > > > why? You should figure this out yourself. Pull Greg's tree again, and try to apply your patch. It's deterministic, so you should have the same information that he has. julia > Arushi > > -- > You received this message because you are subscribed to the Google Groups > "outreachy-kernel" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to outreachy-kernel+unsubscribe@googlegroups.com. > To post to this group, send email to outreachy-kernel@googlegroups.com. > To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF-7SfgGkcHGhraz2 > R%3D8qV4Vgqiky%2Bu%2BqSJNTMGr__JnTg%40mail.gmail.com. > For more options, visit https://groups.google.com/d/optout. > > --8323329-1698831945-1488208047=:3032--