lkml.org 
[lkml]   [2020]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 2/3] hwrng: cctrng - change default to n
Date

> -----Original Message-----
> From: Arnd Bergmann <arnd@arndb.de>
>
> On Mon, Apr 27, 2020 at 9:26 AM Hadar Gat <Hadar.Gat@arm.com> wrote:
> > > -----Original Message-----
> > > From: Ard Biesheuvel <ardb@kernel.org>
> > > > + default n
> > >
> > > 'default n' is the default so you can just remove the line
> >
> > Is this a guideline or just optional?
> > Personally I like things to be explicit and if allowed I prefer to keep this line.
>
> It's a common convention, and we have had patches in the past that mass-
> removed those lines. I'd also just leave it out. It is generally well understood
> that all options default to 'n' unless specified otherwise.

Ok, I'll remove it.
Thanks for the explanation.

>
> Arnd
\
 
 \ /
  Last update: 2020-04-27 10:57    [W:0.084 / U:7.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site