2020-04-27 08:57:51

by Hadar Gat

[permalink] [raw]
Subject: RE: [PATCH 2/3] hwrng: cctrng - change default to n


> -----Original Message-----
> From: Arnd Bergmann <[email protected]>
>
> On Mon, Apr 27, 2020 at 9:26 AM Hadar Gat <[email protected]> wrote:
> > > -----Original Message-----
> > > From: Ard Biesheuvel <[email protected]>
> > > > + 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