Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1702080 > unrolled thread

Re: [PATCH 0/5] tpm: tpm_ibmvtpm: - style fix

Started byJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
First post2017-08-02 14:40 +0200
Last post2017-08-02 14:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 0/5] tpm: tpm_ibmvtpm: - style fix Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-08-02 14:40 +0200

#1702080 — Re: [PATCH 0/5] tpm: tpm_ibmvtpm: - style fix

FromJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Date2017-08-02 14:40 +0200
SubjectRe: [PATCH 0/5] tpm: tpm_ibmvtpm: - style fix
Message-ID<ua16p-1sn-5@gated-at.bofh.it>
On Sat, Jul 29, 2017 at 03:24:28PM +0800, SZ Lin wrote:
> Fix styling WARNINGs and Errors of tpm_ibmvtpm.c driver by using checkpatch.pl

Changes are great but you should revise the patch series so that you
expain in each commit what goes wrong instead of copy paste of the
checkpatch output and why your changes fixes the problem.

> 
> SZ Lin (5):
>   Fix packed and aligned attribute warnings.
>   Fix "ERROR: code indent should use tabs where possible"
>   Fix 'void function return statements are not generally useful' warning
>   Remove unneccessary 'out of memory' message
>   Use __func__ instead of function name
> 
>  drivers/char/tpm/tpm_ibmvtpm.c | 23 +++++++++--------------
>  drivers/char/tpm/tpm_ibmvtpm.h |  2 +-
>  2 files changed, 10 insertions(+), 15 deletions(-)
> 
> --
> 2.13.3
> 

/Jarkko

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web