Convert the tpm_tis_lpc driver to use driver model and update boards which
use it.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Convert the tpm_tis_i2c driver to use driver model and update boards which
use it.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Add a TPM node to the various Chromebooks so that driver can be converted to
driver model.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Rather then crashing when there is no data, print an error. The error is
printed by the caller to parse_byte_string().
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Convert the sandbox TPM driver to use driver model. Add it to the device
tree so that it can be found on start-up.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

When a 'tpm' command fails, we set the return code but give no indication
of failure. This can be confusing.

Add an error message when any tpm command fails.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

I2C chips can support a register offset, with registers accessible by
sending this offset as the first part of any read or write transaction.
Most I2C chips have a single byte offset, thus the offset length is 1.
This provides access for up 256 registers.

However other offset lengths are supported, including 0.

Add a command to provide access to the offset length from the command
line. This allows the offset length to be read or written.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Add driver model support to the TPM command and the TPM library. Both
support only a single TPM at present.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Add a new uclass for TPMs which uses almost the same TIS (TPM Interface
Specification) as is currently implemented. Since init() is handled by the
normal driver model probe() method, we don't need to implement that. Also
rename the transfer method to xfer() which is a less clumbsy name.

Once all drivers and users are converted to driver model we can remove the
old code.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Use a _US suffix for microseconds and a _MS suffic for milliseconds. Move
all timeouts and delays into one place. Use mdelay() instead of udelay()
where appropriate.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Use the same prefix on each function for consistency.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Move all the init and uninit code into one place.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Some definitions are in the C file and some are in the header file. Move
everything into the header file for consistency and to reduce clutter.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

There are too many structures storing the same sort of information. Move the
fields from struct tpm into struct tpm_chip and remove the former struct.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

There are too many structures storing the same sort of information. Move the
fields from struct tpm_dev into struct tpm_chip and remove the former
struct.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

This function is misnamed since it only applies to a single driver. Merge
its fields into its parent.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

The function methods in struct tpm_vendor_specific just call local functions.
Change the code to use a direct call.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

The current Infineon I2C TPM driver is written in two parts, intended to
support use with other I2C devices. However we don't have any users and the
Atmel I2C TPM device does not use this file.

We should simplify this and remove the unused abstration. As a first step,
move the code into one file.

Also the name tpm_private.h suggests that the header file is generic to all
TPMs but it is not. Rename it indicate that it relates only to this driver
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Add an SPDX header to two drivers that don't have it.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Convert all TPM options to Kconfig and tidy up.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

Add new Kconfig options for TPMs in preparation for moving boards to use
Kconfig for TPM configuration.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: Christophe Ricard<christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

The address of the I2C TPM is now defined in the device tree so there is no
need for the CONFIG options.

Remove them from the README and board config to avoid confusion.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

This is not used anymore by any board so drop it.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Acked-by: NChristophe Ricard <christophe-h.ricard@st.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

If board uses downstream Chrome OS U-Boot as first stage
bootloader and upstream version is chained second stage,
1.1V is minimum voltage borderline.
Signed-off-by: NMisha Komarovskiy <zombah@gmail.com>
Acked-by: NSimon Glass <sjg@chromium.org>

This requires 'Series-cover_cc' at present which is incorrect. Fix it.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Sort different types of drivers in alphabetical order.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Right now PHYS_TO_BUS shows in the Kconfig main menu, move it.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Make "Generic Driver Options" menu show on the top in the Kconfig.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Add Kconfig entries for the simple-bus driver, both for U-Boot
and for SPL. The simple-bus is enabled by default in U-Boot and
disabled by default in SPL to preserve the original behavior.
Signed-off-by: NMarek Vasut <marex@denx.de>
Cc: Simon Glass <sjg@chromium.org>
Acked-by: NSimon Glass <sjg@chromium.org>
Modified to fit on top of Masahiro's $(SPL) setup:
Signed-off-by: NSimon Glass <sjg@chromium.org>

This is a convenient way for a driver to get the hardware address of a
device, when regmap or syscon are not being used. Change existing callers
to use it as an example to others.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NJoe Hershberger <joe.hershberger@ni.com>
Acked-by: NStephen Warren <swarren@wwwdotorg.org>

This can be simply written with list_for_each_entry(), maybe
this macro was not necessary in the first place.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Initial version of DTSI for ProXstream2 and PH1-LD6b and DTS for
PH1-LD6b reference board.

Import from Linux with some adjustments:
  - Use SPDX-License-Identifier
  - Add clock-frequency to serial nodes
  - Drop unusable nodes from -ref.dts

While I am here, sort Makefile entries alphabetically.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

Initial version of UniPhier PH1-Pro5 device tree.

(Imported from Linux with adjustment for SPDX License Identifier)
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

This commit imports device tree updates from Linux.  It eventually
adds pinctrl-related nodes and properties.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

This driver is enabled only for UniPhier SoCs and ARCH_UNIPHIER now
selects OF_CONTROL and SPL_OF_CONTROL.

This driver no longer needs to support platform data configuration.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Device Tree really improves code maintainability and is now
available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: NMarek Vasut <marex@denx.de>
Reviewed-by: NSimon Glass <sjg@chromium.org>

Now UniPhier SoCs highly depend on Driver Model for SPL, too.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

This macro is not referenced at all.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

The build error happens if CONFIG_DEBUG_LL and CONFIG_MACH_PH1_SLD3
are both enabled.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

Since commit 2580a2a7 ("mtd: nand: Increase max sizes of OOB and
Page size"), three boards (ph1_ld4, ph1_pro4, ph1_sld8) fail to build
with the following error message:
  arm-linux-gnueabi-ld.bfd: SPL image plus BSS too big

They compile drivers/mtd/nand/denali_spl.c and it has a page_buffer
as static data:

    static uint8_t page_buffer[NAND_MAX_PAGESIZE];

This buffer required 8KB in .bss section before that commit and now
it has been increased to 16KB.  Given limited code/memory size for SPL,
it is not a good idea to allocate a page buffer statically.  In the
first place, the load address 'dst' can be used as a page buffer.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>