Move the variable psci_method in .data section when EFI is not
activated and the psci driver safely access it before relocation.

Without this patch the variable is located in .bss section
and the psci probe requested before relocation corrupts the device
tree (probe is requested by board_f.c::print_resetinfo()).

When EFI_LOADER is activated, this variable in already located in the
.data.efi_runtime section by __efi_runtime_data.
Signed-off-by: NYann Gautier <yann.gautier@st.com>
Signed-off-by: NPatrick Delaunay <patrick.delaunay@st.com>

In the unlikely event that both blocks 10 and 11 are marked as bad (on a
32 bit machine), then the process of marking block 10 as bad stomps on
cached entry for block 11.  There are (of course) other examples.
Signed-off-by: NPatrick Doyle <pdoyle@irobot.com>
Reviewed-by: NRichard Weinberger <richard@nod.at>

This patch adds a feature for block device cloning similar to dd
command, this should be useful for boot-strapping a device where
usb gadget or networking is not available. For instance one can
clone a factory image into a blank emmc from an external sd card.
Signed-off-by: NJohn Chau <john@harmon.hk>

While using u-boot with qemu's virtio driver I stumbled across a
problem reading files less than sector size.  On the real hardware the
block reader seems ok with reading zero blocks, and while we could fix
the virtio host side of qemu to deal with a zero block read instead of
crashing, the u-boot fat driver should not be doing zero block reads
in the first place.  If you ask hardware to read zero blocks you are
just going to get zero data.  There may also be other hardware that
responds similarly to the virtio interface so this is worth fixing.

Without the patch I get the following and have to restart qemu because
it dies.
---------------------------------
=> fatls virtio 0:1
       30   cmdline.txt
=> fatload virtio 0:1 ${loadaddr} cmdline.txt
qemu-system-aarch64: virtio: zero sized buffers are not allowed
---------------------------------

With the patch I get the expected results.
---------------------------------
=> fatls virtio 0:1
       30   cmdline.txt
=> fatload virtio 0:1 ${loadaddr} cmdline.txt
30 bytes read in 11 ms (2 KiB/s)
=> md.b ${loadaddr} 0x1E
40080000: 64 77 63 5f 6f 74 67 2e 6c 70 6d 5f 65 6e 61 62    dwc_otg.lpm_enab
40080010: 6c 65 3d 30 20 72 6f 6f 74 77 61 69 74 0a          le=0 rootwait.

---------------------------------
Signed-off-by: NJason Wessel <jason.wessel@windriver.com>
Signed-off-by: NJason Wessel <jason.wessel@windriver.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

For 2020.10
-----------

- fixes for Toradex board
- fix warnings from previous PR
- HAB: reset instead of panic after failure
- new board:  MYiR Tech MYS-6ULX
- mx6cuboxi: use OF_PLATDATA
- further changes for DM

Travis: https://travis-ci.org/github/sbabic/u-boot-imx/builds/714513163

- Fix tbs2910 vidconsole environment
- Fix ipuv3 framebuffer output on i.MX6Q/D

- add DM based reset driver for SiFive SoC's.

- doc: fix qemu-mips build instructions
- MIPS: add GPIO, CLK and SPI drivers for Octeon MIPS64

- Further cleanup of common.h and dm.h usage in headers

These headers should not be included in other header files. Add a
checkpatch rule and test for this.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Many of the tests have the same comment and two have the same name. Fix
this.
Signed-off-by: NSimon Glass <sjg@chromium.org>

Header files generally should not include header files just for a struct,
since forward declarations work just as well and reduce overhead.

Add a warning for dm.h being included, since this has crept into U-Boot.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This file includes the header twice. Drop the second one.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration and un-inlining of dev_get_clk_ptr()
instead.

Fix up the kendryte header files to avoid build errors.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NSean Anderson <seanga2@gmail.com>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.

Also move the inline function out into a C file. We should not include C
code in headers.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.

Also drop asm/io.h
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.

Drop the common.h inclusion also.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
from each one and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

The layerscape header should not include dm.h so remove it.
Signed-off-by: NSimon Glass <sjg@chromium.org>

We only need to include this header once. Drop the duplicate.
Signed-off-by: NSimon Glass <sjg@chromium.org>

We only need to include this header once. Drop the duplicate.
Signed-off-by: NSimon Glass <sjg@chromium.org>

We only need to include this header once. Drop the duplicate.
Signed-off-by: NSimon Glass <sjg@chromium.org>

We only need to include this header once. Drop the duplicate.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This file should not include common.h and dm.h so remove them. Also move
the inclusion of this file to after the normal includes.
Signed-off-by: NSimon Glass <sjg@chromium.org>
Reviewed-by: NJaehoon Chung <jh80.chung@samsung.com>

Two files relies on efi_driver.h to include common.h and dm.h which is
incorrect. The former should always be included in a non-host C file and
the latter should be included if driver model is used.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use other headers and C inclusions instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

These header file should not be included in other header files. Remove
them and add to each individual file. Add test/test.h to test/ui.h since
that is a reasonable place.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and add it to the cmd file instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

These two defines are no-longer supported. Drop them.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.
Signed-off-by: NSimon Glass <sjg@chromium.org>

This header file should not be included in other header files. Remove it
and use a forward declaration instead.

Also remove the other headers that are not needed, since the inline code
was removed in a recent commit:

   b4d9452c ("watchdog: move initr_watchdog() to wdt-uclass.c")
Signed-off-by: NSimon Glass <sjg@chromium.org>

The -ENODEV error value in spi_nor_read_id() is incorrect since there
clearly is a device - it just cannot be supported. Use -ENOMEDIUM instead
which has the virtue of being less common.

Fix the return value in spi_nor_scan().

Also there are a few printf() statements which should be debug() since
they bloat the code with unused strings at present. Fix those while here.
Signed-off-by: NSimon Glass <sjg@chromium.org>

In some cases SPL needs to be able to erase but TPL just needs to read.
Allow these to have separate settings for SPI_FLASH_TINY.
Signed-off-by: NSimon Glass <sjg@chromium.org>