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

From: Hadar Gat
Date: Mon Apr 27 2020 - 04:56:52 EST



> -----Original Message-----
> From: Arnd Bergmann <arnd@xxxxxxxx>
>
> On Mon, Apr 27, 2020 at 9:26 AM Hadar Gat <Hadar.Gat@xxxxxxx> wrote:
> > > -----Original Message-----
> > > From: Ard Biesheuvel <ardb@xxxxxxxxxx>
> > > > + 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