u-boot.git/drivers/tpm, branch master

tpm: Correct warning on 32-bit build

2021-03-13T18:14:52+00:00

Fix the warning:

drivers/tpm/tpm2_tis_sandbox.c: In function ‘sandbox_tpm2_xfer’:
drivers/tpm/tpm2_tis_sandbox.c:288:48: warning: format ‘%ld’ expects
	argument of type ‘long int’, but argument 2 has type ‘size_t’
	{aka ‘unsigned int’} [-Wformat=]
   printf("TPM2: Unmatching length, received: %ld, expected: %d\n",
                                              ~~^
                                              %d
          send_size, length);
          ~~~~~~~~~

Signed-off-by: Simon Glass 
Reviewed-by: Heinrich Schuchardt

tpm: Use logging in the uclass

2021-03-02T20:53:37+00:00

Update this to use log_debug() instead of the old debug().

Signed-off-by: Simon Glass 
Reviewed-by: Ilias Apalodimas

tpm: Don't include cr50 in TPL/SPL

2021-03-02T20:53:37+00:00

At present the security chip is not used in these U-Boot phases. Update
the Makefile to exclude it.

Fix a few logging statements while we are here.

Signed-off-by: Simon Glass 
Reviewed-by: Ilias Apalodimas

dm: i2c: use CONFIG_IS_ENABLED macro for DM_I2C/DM_I2C_GPIO

2021-02-21T05:08:00+00:00

Use CONFIG_IS_ENABLED() macro, which provides more convenient
way to check $(SPL)DM_I2C/$(SPL)DM_I2C_GPIO configs
for both SPL and U-Boot proper.

CONFIG_IS_ENABLED(DM_I2C) expands to:
- 1 if CONFIG_SPL_BUILD is undefined and CONFIG_DM_I2C is set to 'y',
- 1 if CONFIG_SPL_BUILD is defined and CONFIG_SPL_DM_I2C is set to 'y',
- 0 otherwise.

All occurences were replaced automatically using these bash cmds:
$ find . -type f -exec sed -i
     's/ifndef CONFIG_DM_I2C/if !CONFIG_IS_ENABLED(DM_I2C)/g' {} +
$ find . -type f -exec sed -i
    's/ifdef CONFIG_DM_I2C/if CONFIG_IS_ENABLED(DM_I2C)/g' {} +
$ find . -type f -exec sed -i
    's/defined(CONFIG_DM_I2C)/CONFIG_IS_ENABLED(DM_I2C)/g' {} +
$ find . -type f -exec sed -i
    's/ifndef CONFIG_DM_I2C_GPIO/if !CONFIG_IS_ENABLED(DM_I2C_GPIO)/g' {} +
$ find . -type f -exec sed -i
    's/ifdef CONFIG_DM_I2C_GPIO/if CONFIG_IS_ENABLED(DM_I2C_GPIO)/g' {} +
$ find . -type f -exec sed -i
    's/defined(CONFIG_DM_I2C_GPIO)/CONFIG_IS_ENABLED(DM_I2C_GPIO)/g' {} +

Reviewed-by: Heiko Schocher 
Reviewed-by: Simon Glass 
Signed-off-by: Igor Opaniuk 
Reviewed-by: Tom Rini 
Reviewed-by: Priyanka Jain

tpm: cr50: Rename driver to work with of-platdata

2021-01-30T21:25:41+00:00

Update the driver name to match the compatible string, so it can work
with of-platdata.

Signed-off-by: Simon Glass

tpm: cr50: Add a better description and more debug

2021-01-30T21:25:41+00:00

Update the TPM description to include the interrupt mechanicm since this
is useful to know. Also add a warning if the TPM cannot be found and a
debug line if it succeeds.

Signed-off-by: Simon Glass

tpm: cr50: Check for valid locality

2021-01-30T21:25:41+00:00

When the Cr50 starts up it doesn't have a valid locality. The driver sets
it to -1 to indicate that. Tracking this allows cr50_i2c_cleanup() to
avoid releasing a locality that was not claimed.

However the helper functions that generate the flags use a u8 type which
cannot support -1, so they return a locality of 0xff.

Fix this by updating the type. With this, 'tpm startup TPM2_SU_CLEAR'
works as expected.

Signed-off-by: Simon Glass

dm: treewide: Rename ofdata_to_platdata() to of_to_plat()

2020-12-13T23:51:09+00:00

This name is far too long. Rename it to remove the 'data' bits. This makes
it consistent with the platdata->plat rename.

Signed-off-by: Simon Glass

dm: treewide: Rename dev_get_platdata() to dev_get_plat()

2020-12-13T23:51:09+00:00

Rename this to be consistent with the change from 'platdata'.

Signed-off-by: Simon Glass

dm: treewide: Rename 'platdata' variables to just 'plat'

2020-12-13T23:51:08+00:00

We use 'priv' for private data but often use 'platdata' for platform data.
We can't really use 'pdata' since that is ambiguous (it could mean private
or platform data).

Rename some of the latter variables to end with 'plat' for consistency.

Signed-off-by: Simon Glass