From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752624AbdK0Ohd (ORCPT ); Mon, 27 Nov 2017 09:37:33 -0500 Received: from mail-it0-f68.google.com ([209.85.214.68]:36388 "EHLO mail-it0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752238AbdK0Ohb (ORCPT ); Mon, 27 Nov 2017 09:37:31 -0500 X-Google-Smtp-Source: AGs4zMbuWuGfZPNFV1pn7BChTHegjpJGrgeaMSExUbvpKvuRbFUks0p5MpeKaMvUNqNn+LBr2iE+AQ== Date: Mon, 27 Nov 2017 09:37:27 -0500 From: Jeremy Lacomis To: Geert Uytterhoeven Cc: Dan Carpenter , Sudip Mukherjee , Teddy Wang , Greg KH , driverdevel , Linux Fbdev development list , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] staging: sm750fb: Fix coding style in ddk750_sii164.h Message-ID: <20171127143727.GA4270@gs17931.sp.cs.cmu.edu> References: <20171125182636.15245-1-j.lacomis@gmail.com> <20171127094622.phmx4ej2fljcytce@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Nov 27, 2017 at 11:45:00AM +0100, Geert Uytterhoeven wrote: > On Mon, Nov 27, 2017 at 10:46 AM, Dan Carpenter > wrote: > > On Sat, Nov 25, 2017 at 01:26:35PM -0500, Jeremy Lacomis wrote: > >> This patch to ddk750_sii164.h fixes line length warnings found by the > >> checkpatch.pl script and reformats comments uniformly. > >> > >> Signed-off-by: Jeremy Lacomis > >> --- > >> drivers/staging/sm750fb/ddk750_sii164.h | 57 +++++++++++++-------------------- > >> 1 file changed, 22 insertions(+), 35 deletions(-) > >> > >> diff --git a/drivers/staging/sm750fb/ddk750_sii164.h b/drivers/staging/sm750fb/ddk750_sii164.h > >> index 2e9a88cd6af3..393a3c5be3ae 100644 > >> --- a/drivers/staging/sm750fb/ddk750_sii164.h > >> +++ b/drivers/staging/sm750fb/ddk750_sii164.h > >> @@ -4,15 +4,20 @@ > >> > >> #define USE_DVICHIP > >> > >> -/* Hot Plug detection mode structure */ > >> +/* > >> + * Hot Plug detection mode structure: > >> + * Disable Hot Plug output bit (always high). > >> + * Use Monitor Detect Interrupt bit. > >> + * Use Receiver Sense detect bit. > >> + * Use Hot Plug detect bit. > >> + */ > >> enum sii164_hot_plug_mode { > >> - SII164_HOTPLUG_DISABLE = 0, /* Disable Hot Plug output bit (always high). */ > >> - SII164_HOTPLUG_USE_MDI, /* Use Monitor Detect Interrupt bit. */ > >> - SII164_HOTPLUG_USE_RSEN, /* Use Receiver Sense detect bit. */ > >> - SII164_HOTPLUG_USE_HTPLG /* Use Hot Plug detect bit. */ > >> + SII164_HOTPLUG_DISABLE = 0, > >> + SII164_HOTPLUG_USE_MDI, > >> + SII164_HOTPLUG_USE_RSEN, > >> + SII164_HOTPLUG_USE_HTPLG > >> }; > > > > I feel like this makes it less readable. The original was better. > > And more error prone. Thanks for taking the time to review my patch, I'm just getting into kernel development and am trying to do as little disservice as possible while learning the process. For something like this, would using the kernel-doc format make more sense, or should the original inline comments be kept? I found the original difficult to read because my terminal was just small enough that this didn't fit. Thanks, - Jeremy From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Lacomis Date: Mon, 27 Nov 2017 14:37:27 +0000 Subject: Re: [PATCH 1/2] staging: sm750fb: Fix coding style in ddk750_sii164.h Message-Id: <20171127143727.GA4270@gs17931.sp.cs.cmu.edu> List-Id: References: <20171125182636.15245-1-j.lacomis@gmail.com> <20171127094622.phmx4ej2fljcytce@mwanda> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Dan Carpenter , Sudip Mukherjee , Teddy Wang , Greg KH , driverdevel , Linux Fbdev development list , "linux-kernel@vger.kernel.org" On Mon, Nov 27, 2017 at 11:45:00AM +0100, Geert Uytterhoeven wrote: > On Mon, Nov 27, 2017 at 10:46 AM, Dan Carpenter > wrote: > > On Sat, Nov 25, 2017 at 01:26:35PM -0500, Jeremy Lacomis wrote: > >> This patch to ddk750_sii164.h fixes line length warnings found by the > >> checkpatch.pl script and reformats comments uniformly. > >> > >> Signed-off-by: Jeremy Lacomis > >> --- > >> drivers/staging/sm750fb/ddk750_sii164.h | 57 +++++++++++++-------------------- > >> 1 file changed, 22 insertions(+), 35 deletions(-) > >> > >> diff --git a/drivers/staging/sm750fb/ddk750_sii164.h b/drivers/staging/sm750fb/ddk750_sii164.h > >> index 2e9a88cd6af3..393a3c5be3ae 100644 > >> --- a/drivers/staging/sm750fb/ddk750_sii164.h > >> +++ b/drivers/staging/sm750fb/ddk750_sii164.h > >> @@ -4,15 +4,20 @@ > >> > >> #define USE_DVICHIP > >> > >> -/* Hot Plug detection mode structure */ > >> +/* > >> + * Hot Plug detection mode structure: > >> + * Disable Hot Plug output bit (always high). > >> + * Use Monitor Detect Interrupt bit. > >> + * Use Receiver Sense detect bit. > >> + * Use Hot Plug detect bit. > >> + */ > >> enum sii164_hot_plug_mode { > >> - SII164_HOTPLUG_DISABLE = 0, /* Disable Hot Plug output bit (always high). */ > >> - SII164_HOTPLUG_USE_MDI, /* Use Monitor Detect Interrupt bit. */ > >> - SII164_HOTPLUG_USE_RSEN, /* Use Receiver Sense detect bit. */ > >> - SII164_HOTPLUG_USE_HTPLG /* Use Hot Plug detect bit. */ > >> + SII164_HOTPLUG_DISABLE = 0, > >> + SII164_HOTPLUG_USE_MDI, > >> + SII164_HOTPLUG_USE_RSEN, > >> + SII164_HOTPLUG_USE_HTPLG > >> }; > > > > I feel like this makes it less readable. The original was better. > > And more error prone. Thanks for taking the time to review my patch, I'm just getting into kernel development and am trying to do as little disservice as possible while learning the process. For something like this, would using the kernel-doc format make more sense, or should the original inline comments be kept? I found the original difficult to read because my terminal was just small enough that this didn't fit. Thanks, - Jeremy