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

From: Arnd Bergmann
Date: Mon Apr 27 2020 - 04:19:50 EST


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.

Arnd