diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-15 09:22:11 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-15 20:56:03 +0100 |
| commit | 52ab30b13b867c1f3cd5aa5d9eb3e24d430c49d5 (patch) | |
| tree | 85a5356335e918b55a8e8a0de1ed4f9ac761dfbb /src/drivers/i2c/tpm/cr50.c | |
| parent | 08824ec8d4d68ffa9edf1212e7f7e281da4d779f (diff) | |
drivers/i2c/tpm: Fix issues detected by checkpatch
Fix the following warnings detected by checkpatch.pl:
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
WARNING: braces {} are not necessary for single statement blocks
WARNING: Unnecessary parentheses - maybe == should be = ?
WARNING: line over 80 characters
WARNING: missing space after return type
TEST=Build and run on Galileo Gen2
Change-Id: I56f915f6c1975cce123fd38043bad2638717d88c
Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18832
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/drivers/i2c/tpm/cr50.c')
| -rw-r--r-- | src/drivers/i2c/tpm/cr50.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/drivers/i2c/tpm/cr50.c b/src/drivers/i2c/tpm/cr50.c index 0877de01ea..42c82d6fec 100644 --- a/src/drivers/i2c/tpm/cr50.c +++ b/src/drivers/i2c/tpm/cr50.c @@ -438,7 +438,7 @@ static void cr50_vendor_init(struct tpm_chip *chip) __func__, CR50_TIMEOUT_NOIRQ_MS); } -int tpm_vendor_probe(unsigned bus, uint32_t addr) +int tpm_vendor_probe(unsigned int bus, uint32_t addr) { struct tpm_inf_dev *tpm_dev = car_get_var_ptr(&g_tpm_dev); struct tpm_chip probe_chip; @@ -475,7 +475,7 @@ int tpm_vendor_probe(unsigned bus, uint32_t addr) return 0; } -int tpm_vendor_init(struct tpm_chip *chip, unsigned bus, uint32_t dev_addr) +int tpm_vendor_init(struct tpm_chip *chip, unsigned int bus, uint32_t dev_addr) { struct tpm_inf_dev *tpm_dev = car_get_var_ptr(&g_tpm_dev); uint32_t vendor; |