RE: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands.

From: Winkler, Tomas
Date: Tue Mar 06 2018 - 11:46:29 EST


>
> On Tue, 2018-03-06 at 08:06 +0000, Winkler, Tomas wrote:
> > >
> > >
> > > On Mon, Mar 05, 2018 at 01:09:09PM +0000, Winkler, Tomas wrote:
> > > >
> > > > Why you need cover letter?ÂÂWhat are u missing in the patch
> > > > description
> > >
> > > If you submit a *patch set* I *require* a cover letter, yes.
> >
> > It's good but it is not must, you are inventing your own rules.
>
> As long as the Maintainer is the gatekeeper, you're not going to get very far
> with this argument. ÂThe fact is that a lot of subsystems have varying rules;
> often undocumented, some of which are even in conflict, like alphabetic vs
> reverse christmas tree format for includes.

Usually I'm trying to stay in convention of the surouding code even if it's agains my personal taste.
I think this particular case was a bit different.

> A cover letter is actually one of the more uniform rules. ÂIt's referred to in
> submitting patches, but not actually documented there.

I'm all for cover letters, but for few code line fixes it's more comments then code.

What is most important I think I and Jarkko had found finally a common ground.

Thanks
Tomas