ATM when receiving a packet the whole buffer is invalidated, this change
optimizes this behaviour.
Signed-off-by: NKarl Beldan <karl.beldan+oss@gmail.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Reviewed-by: NTom Rini <trini@konsulko.com>
Reviewed-by: NMugunthan V N <mugunthanvnm@ti.com>

check_cache_range() warns that the top boundaries are not properly
aligned when flushing or invalidating the buffers and make these
operations fail.

This gets rid of the remaining warnings:
CACHE: Misaligned operation at range
Signed-off-by: NKarl Beldan <karl.beldan+oss@gmail.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>
Reviewed-by: NTom Rini <trini@konsulko.com>
Reviewed-by: NMugunthan V N <mugunthanvnm@ti.com>

ATM the rx and tx descriptors are handled as cached memory while they
lie in a dedicated RAM of the SoCs, which is an uncached area.
Removing the said dcache ops, while optimizing the logic and clarifying
the code, also gets rid of most of the check_cache_range() incurred
warnings:
CACHE: Misaligned operation at range
Signed-off-by: NKarl Beldan <karl.beldan+oss@gmail.com>
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

We use an empty hostname, so remove all the "processing" of the
known-to-be-empty hostname and just write 0's where needed.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Instead of always allocating a huge temporary buffer on the stack and
then memcpy()ing the result into the transmit buffer, simply figure out
where in the transmit buffer the bytes will belong and write them there
directly as each message is built.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Much of the information is verbose and derived directly from the
environment. Only output in debug mode. This also saves about 300 bytes
from the code size.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Use the same name throughout the nfs code and use the same member of the
union to avoid casts.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

The buffer is of 32-bit elements, not bytes.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Instead of repeating the same large snippet for dealing with attributes
it should be shared with a helper function.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

Fix complaints from checkpatch.pl.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

The type of the buffer is uint32_t, but the parameter used to size it
is referring to bytes. Divide by the size of the array elements.

Strictly speaking, this shouldn't be needed at all... It could just be 1
just like the request.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

The v3 handles can be larger than v2, but that doesn't mean we need a
separate buffer. Reuse the same (larger) buffer for both.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

This patch enables NFSv3 support.
If NFSv2 is available use it as usual.
If NFSv2 is not available, but NFSv3 is available, use NFSv3.
If NFSv2 and NFSv3 are not available, print an error message since NFSv4 is not supported.

Tested on iMX6 sabrelite with 4 Linux NFS servers:
  * NFSv2 + NFSv3 + NFSv4 server: use NFSv2 protocol
  * NFSv2 + NFSv3 server: use NFSv2 protocol
  * NFSv3 + NFSv4 server: use NFSv3 protocol
  * NFSv3 server: use NFSv3 protocol
Signed-off-by: NGuillaume GARDET <guillaume.gardet@free.fr>
Cc: Tom Rini <trini@konsulko.com>
Cc: joe.hershberger@ni.com
Acked-by: NJoe Hershberger <joe.hershberger@ni.com>

Unreferenced, so remove the noise.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

There is no reason to store the default filename in a separate buffer
only to immediately copy it to the main name buffer. Just write it there
directly and remove the other buffer.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

At least on bfin, this "specimen" is actually allocated in the BSS and
wastes lots of memory in already tight memory conditions.

Also, with the introduction of NFSv3 support, this waste got
substantially larger.

Just remove it. If a board needs a specific different defragment size,
that board can override this setting.
Signed-off-by: NJoe Hershberger <joe.hershberger@ni.com>

[trini: Drop CMD_BOOTI as it's now on by default on ARM64]
Signed-off-by: NTom Rini <trini@konsulko.com>

'-' is never used in function names.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>

We were truncating the image offset within the target image to 16 bits
which again meant that we were potentially overwriting random memory
in the lower 16 bits of the image.

This patch casts the offset to a more reasonable 32bits.

With this applied, I can successfully see Shell.efi assert because it
can't find a protocol it expects to be available.
Signed-off-by: NAlexander Graf <agraf@suse.de>

make tags creates a symbolic link called tags to ctags. Remove this file
on make mrproper or make distclean.
Signed-off-by: NVignesh R <vigneshr@ti.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

addon 183923d3
MMC/SATA have no erase blocks, only blocks. Hence the warning
about erase block alignment might be confusing in such environment.
Signed-off-by: NAndreas Fenkart <andreas.fenkart@digitalstrom.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

For double buffering to work, the target buffer must always be big
enough to hold all data. This can only be ensured if buffers are of
equal size, otherwise one must be smaller and we risk data loss
when copying from the bigger to the smaller buffer.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NAndreas Fenkart <andreas.fenkart@digitalstrom.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

There is no reason to duplicate code for DRA7xx platforms as there
can be Rail grouping. The maximum voltage detection algorithm can still
be run on other platforms with no Rail grouping and does not harm as
it gives the same result.
Signed-off-by: NLokesh Vutla <lokeshvutla@ti.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

gpio_en field is introduced to detect if pmic is controlled by GPIO.
Make this field 0 on all TPS659* pmics available on DRA7/OMAP5 based platforms
and remove the #ifndefs.
Reviewed-by: NKeerthy <j-keerthy@ti.com>
Signed-off-by: NLokesh Vutla <lokeshvutla@ti.com>
Reviewed-by: NTom Rini <trini@konsulko.com>

This prevents capture of command output from terminating early on boards
that use a simple prompt (e.g. "=> ") that appears in the middle of
command output (e.g. crc32's "... ==> 2fa737e0").
Reported-by: NTom Rini <trini@konsulko.com>
Signed-off-by: NStephen Warren <swarren@wwwdotorg.org>
Tested-by: NTom Rini <trini@konsulko.com>
Reviewed-by: NHeiko Schocher <hs@denx.de>

This improves the cread_line() function so that it will correctly
process the 'Home', 'End', 'Delete' and arrow key escape sequences
produced by various terminal emulators. This makes command line editing
a more pleasant experience.

The previous code only supported the cursor keys and the 'Home' key, and
only for certain terminal emulator configurations. This adds support for
the 'End and 'Delete' keys, and recognises a wider range of escape
sequences. For example, the left arrow key can be 'ESC O D' instead of
'ESC [ D', and the 'Home' key can be 'ESC [ H', 'ESC O H', 'ESC 1 ~' or
'ESC 7 ~', depending on what terminal emulator you use and how it is
configured.
Signed-off-by: NJames Byrne <james.byrne@origamienergy.com>
Changes for v2
   - Explicitly initialize variable to avoid spurious compiler warning.
Changes for v3
   - Remove unnecessary setting of 'act' to ESC_REJECT (now its default
     value).

- move bcm23550_w1d to Kconfig
- move bcm28155_ap to Kconfig
Signed-off-by: NSteve Rae <steve.rae@raedomain.com>

implement Kconfig for the 'fastboot' feature set
Signed-off-by: NSteve Rae <steve.rae@raedomain.com>

When typing 'bootefi' from U-Boot shell, nothing outputs. Like other
commands, return CMD_RET_USAGE so that it can print help message.
Signed-off-by: NBin Meng <bmeng.cn@gmail.com>
Reviewed-by: NAlexander Graf <agraf@suse.de>

The 'bootz' command is really only for ARM32 Linux Kernel 'zImage' files
but has also been adapted for testing with sandbox.  Given that sandbox
is a test platform, don't add that logic under DISTRO_DEFAULTS.

Cc: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: NTom Rini <trini@konsulko.com>

This command is used to boot ARM64 Linux.

I made DISTRO_DEFAULTS select this option for ARM64 to respect
include/config_distro_defaults.h.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: NTom Rini <trini@konsulko.com>

The bootz and booti commands rely on common functionality that is found
in common/bootm.c and common/bootm_os.c.  They do not however rely on
the rest of cmd/bootm.c to be implemented so split them into their own
files.  Have various Makefiles include the required infrastructure for
CONFIG_CMD_BOOT[IZ] as well as CONFIG_CMD_BOOTM.  Move the declaration
of 'images' over to common/bootm.c.

Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: NTom Rini <trini@konsulko.com>

This adds a bunch of unit tests for the "fdt apply" command.

They've all been run successfully in the sandbox. However, as you still
require an out-of-tree dtc with overlay support, this is disabled by
default.
Acked-by: NSimon Glass <sjg@chromium.org>
Acked-by: NPantelis Antoniou <pantelis.antoniou@konsulko.com>
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

The device tree overlays are a good way to deal with user-modifyable
boards or boards with some kind of an expansion mechanism where we can
easily plug new board in (like the BBB or the raspberry pi).

However, so far, the usual mechanism to deal with it was to have in Linux
some driver detecting the expansion boards plugged in and then request
these overlays using the firmware interface.

That works in most cases, but in some cases, you might want to have the
overlays applied before the userspace comes in. Either because the new
board requires some kind of an early initialization, or because your root
filesystem is accessed through that expansion board.

The easiest solution in such a case is to simply have the component before
Linux applying that overlay, removing all these drawbacks.
Reviewed-by: NStefan Agner <stefan@agner.ch>
Acked-by: NPantelis Antoniou <pantelis.antoniou@konsulko.com>
Acked-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

The device tree overlays are a good way to deal with user-modifyable
boards or boards with some kind of an expansion mechanism where we can
easily plug new board in (like the BBB, the Raspberry Pi or the CHIP).

Add a new function to merge overlays with a base device tree.
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

Add a function to modify inplace only a portion of a property..

This is especially useful when the property is an array of values, and you
want to update one of them without changing the DT size.
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>
Acked-by: NSimon Glass <sjg@chromium.org>

Add a function to retrieve a writeable property only by the first
characters of its name.
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

Add a namelen variant of fdt_path_offset to retrieve the node offset using
only a fixed number of characters.
Reviewed-by: NSimon Glass <sjg@chromium.org>
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

The function fdt_path_next_seperator had an obvious mispell. Fix it.
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>

Add a function to retrieve the highest phandle in a given device tree.
Signed-off-by: NMaxime Ripard <maxime.ripard@free-electrons.com>
Reviewed-by: NStefan Agner <stefan@agner.ch>
Acked-by: NSimon Glass <sjg@chromium.org>