Messages in this thread | | | From | Hadar Gat <> | Subject | RE: [PATCH 2/3] hwrng: cctrng - change default to n | Date | Mon, 27 Apr 2020 08:56:38 +0000 |
| |
> -----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
| |