linux-kernel - Re: [PATCH] Staging: comedi: fix spacing issues in ni

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date:	Sun, 21 Dec 2014 00:21:56 -0800
From:	Jeremiah Mahler <jmmahler@...il.com>
To:	Sage Vaillancourt <sagev9000@...il.com>
Cc:	Ian Abbott <abbotti@....co.uk>, hsweeten@...ionengravers.com,
	Greg KH <gregkh@...uxfoundation.org>,
	Chase Southwood <chase.southwood@...il.com>,
	Ebru Akagündüz <ebru.akagunduz@...il.com>,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Staging: comedi: fix spacing issues in ni_stc.h

Sage,

On Sun, Dec 21, 2014 at 02:49:29AM -0500, Sage Vaillancourt wrote:
> Jeremiah,
> 
> I will get this patching business eventually. When you say "gratuitous
> spacing" do you refer to this line?
> 
> > #define G_Gate_Polarity _bit13
> 
> Because it looks fine in the code, but there's a tab that's only one
> character wide, so when git adds its one character of formatting, the tab
> expands to its full 8 characters. Before my modification, a space was there
> instead, but I was under the impression that that was less-than-ideal
> practice. Would switching back to the space be preferable? And as far as
> width, should I add a \(enter) where necessary?
> 
The spacing in the code was fine.  I was referring to your log message
(below).

> And the blank line and two patch bit: understood, simple fix.
> 
> - Sage Vaillancourt
> 
> On Sun, Dec 21, 2014 at 2:12 AM, Jeremiah Mahler <jmmahler@...il.com> wrote:
> 
> > Sage,
> >
> > On Sat, Dec 20, 2014 at 09:01:54PM -0500, Sage Vaillancourt wrote:
> > > Patch to the ni_stc.h file that removes spaces before tabs, and adds
> > blank lines
> > >       after declarations.
     ^^^^^^^
This was the "gratuitous spacing" I was talking about.

> > > Signed-off-by: Sage Vaillancourt <sagev9000@...il.com>
> >
> > Format your log message so it is no more than 80 characters wide (I like
> > 72) and remove the gratuitous spacing on the second line.
> >
> > Put a blank line between your log message and your Signed-off-by:
> >
> > This patch is doing two things: adding blank lines and fixing
> > spaces/tabs in a line.  Break it up in to two patches.
> >
> > > ---
> > >  drivers/staging/comedi/drivers/ni_stc.h | 8 ++++++--
> > >  1 file changed, 6 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/staging/comedi/drivers/ni_stc.h
> > b/drivers/staging/comedi/drivers/ni_stc.h
> > > index bd69c3f..3b8b3c5 100644
> > > --- a/drivers/staging/comedi/drivers/ni_stc.h
> > > +++ b/drivers/staging/comedi/drivers/ni_stc.h
> > > @@ -166,7 +166,7 @@ enum Interrupt_B_Ack_Bits {
> > >  #define AI_SCAN_IN_PROG_Pulse                        _bit4
> > >  #define AI_EXTMUX_CLK_Pulse                  _bit3
> > >  #define AI_LOCALMUX_CLK_Pulse                        _bit2
> > > -#define AI_SC_TC_Pulse                               _bit1
> > > +#define AI_SC_TC_Pulse                               _bit1
> > >  #define AI_CONVERT_Pulse                     _bit0
> > >
> > >  #define AO_Command_1_Register                9
> > > @@ -326,6 +326,7 @@ static inline unsigned RTSI_Output_Bit(unsigned
> > channel, int is_mseries)
> > >  {
> > >       unsigned max_channel;
> > >       unsigned base_bit_shift;
> > > +
> > >       if (is_mseries) {
> > >               base_bit_shift = 8;
> > >               max_channel = 7;
> > > @@ -642,7 +643,7 @@ static unsigned AO_UPDATE_Output_Select(enum
> > ao_update_output_selection
> > >  #define G_Load_Source_Select         _bit7
> > >  #define G_Reload_Source_Switching    _bit15
> > >  #define G_Loading_On_Gate            _bit14
> > > -#define G_Gate_Polarity              _bit13
> > > +#define G_Gate_Polarity                      _bit13
> > >
> > >  #define G_Counting_Once(a)           (((a)&0x03)<<10)
> > >  #define G_Stop_Mode(a)                       (((a)&0x03)<<5)
> > > @@ -1142,6 +1143,7 @@ static inline unsigned
> > MSeries_AI_Config_Bank_Bits(enum ni_reg_type reg_type,
> > >                                                  unsigned channel)
> > >  {
> > >       unsigned bits = channel & 0x30;
> > > +
> > >       if (reg_type == ni_reg_622x) {
> > >               if (channel & 0x40)
> > >                       bits |= 0x400;
> > > @@ -1191,6 +1193,7 @@ enum MSeries_PLL_Control_Bits {
> > >  static inline unsigned MSeries_PLL_Divisor_Bits(unsigned divisor)
> > >  {
> > >       static const unsigned max_divisor = 0x10;
> > > +
> > >       if (divisor < 1 || divisor > max_divisor) {
> > >               pr_err("%s: bug, invalid divisor=%i\n", __func__, divisor);
> > >               return 0;
> > > @@ -1201,6 +1204,7 @@ static inline unsigned
> > MSeries_PLL_Divisor_Bits(unsigned divisor)
> > >  static inline unsigned MSeries_PLL_Multiplier_Bits(unsigned multiplier)
> > >  {
> > >       static const unsigned max_multiplier = 0x100;
> > > +
> > >       if (multiplier < 1 || multiplier > max_multiplier) {
> > >               pr_err("%s: bug, invalid multiplier=%i\n", __func__,
> > >                      multiplier);
> > > --
> > > 2.2.0
> > >
> > > --
> > > To unsubscribe from this list: send the line "unsubscribe linux-kernel"
> > in
> > > the body of a message to majordomo@...r.kernel.org
> > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > > Please read the FAQ at  http://www.tux.org/lkml/
> >
> > --
> > - Jeremiah Mahler
> >

-- 
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/