Messages in this thread | | | From | Arnd Bergmann <> | Date | Mon, 27 Apr 2020 10:19:28 +0200 | Subject | Re: [PATCH 2/3] hwrng: cctrng - change default to n |
| |
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.
Arnd
| |