Re: [PATCH 1/5] tpm: unify tpm_gen_interrupt()

From: Jarkko Sakkinen
Date: Fri Jun 24 2016 - 16:26:07 EST


On Fri, Jun 24, 2016 at 01:30:24PM -0600, Jason Gunthorpe wrote:
> On Fri, Jun 17, 2016 at 11:10:43PM +0200, Jarkko Sakkinen wrote:
> > extern int tpm_get_timeouts(struct tpm_chip *);
> > -extern void tpm_gen_interrupt(struct tpm_chip *);
> > +void tpm_gen_interrupt(struct tpm_chip *);
>
> Dropping the argument name is not the kernel standard, if these lines
> are going to be churned they may as well be churned to the standard.
>
> FWIW, all the extern's should be dropped too.

Thanks for pointin these out. And yes, since this is mostly a cleanup
series, adding a patch that would clean up those unnecessary externs
would make sense. I'll send a revised version with your suggestions.

> Jason

/Jarkko