state: Make an alias mandatory ...

This ports the following barebox commit:

| commit 4983191
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Thu May 4 12:27:21 2017 +0200
|
|     state: Make an alias mandatory
|
|     The userspace barebox-state utility gets confused when no alias exists.
|     Make the alias mandatory, so that people make it right^tm without having
|     to ask.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: warn when a state node will be overwritten ...

This ports the following barebox commit:

| commit b30047b
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Thu May 4 11:58:23 2017 +0200
|
|     state: warn when a state node will be overwritten
|
|     People do not seem to know that a state node in the kernel
|     device tree will be overwritten by barebox. Print a warning
|     when this happens so that people can get an idea why changes
|     in the kernel device tree state node do not have any effect.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: make parameter functions more consistent ...

This replaces a dummy version dev_add_param_int() by a dummy version of
dev_add_param_uint32() to make the compiler happy and ports the following
barebox commit:

| commit c5d95eb
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Fri Apr 7 11:57:08 2017 +0200
|
|     param: make parameter functions more consistent
|
|     This patch creates a consitent set of device parameter functions.
|
|     With this we have: dev_add_param_<type><access>
|
|     "type" is one of: int32, uint32, int64, uint64, string, mac, ipv4, enum,
|     bitmask
|     The improvement here is that we now can exactly specify the width of the
|     int type parameters and also correctly distinguish between signed and
|     unsigned variables which means that a variable no longer ends up with
|     INT_MAX when it's assigned -1.
|
|     "access" can be empty for regular read/write parameter, "_ro" for readonly
|     parameters which get their value from a variable pointer in the
|     background or "_fixed" for parameters which are set to a fixed value
|     (without a pointer in the background).
|
|     Some more exotic types are not (yet) implemented, like
|     dev_add_param_ip_ro.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: backend_bucket_circular: drop unused free_pattern constant ...

This ports the following barebox commit:

| commit b876c2b
| Author: Antony Pavlov <antonynpavlov@gmail.com>
| Date:   Wed May 3 14:41:18 2017 +0300
|
|     state: backend_bucket_circular: drop unused free_pattern constant
|
|     Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: Fix "pack" returning freed data for DTB backend ...

This ports the following barebox commit:

| commit ec01cbc
| Author: Ian Abbott <abbotti@mev.co.uk>
| Date:   Fri Apr 21 10:53:25 2017 +0100
|
|     state: Fix "pack" returning freed data for DTB backend
|
|     `state_backend_format_dtb_pack()` passes a buffer containing state
|     packed in flattened DTB format back to its caller via its `buf`
|     parameter.  It then frees the buffer before returning.  This means the
|     caller (`state_save()`) will be working on freed buffer contents before
|     freeing the buffer a second time itself.  Fix it by removing the
|     spurious call to `free()`.
|
|     This should fix a bug reported by Norbert Wiedmann in
|     <http://lists.infradead.org/pipermail/barebox/2017-April/029980.html>.
|
|     Cc: Norbert Wiedmann <info@n-wiedmann.de>
|     Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: Fix error return value ...

This ports the following barebox commit:

| commit d11584d
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Wed Apr 19 14:34:30 2017 +0200
|
|     state: Fix error return value
|
|     When the backend phandle cannot be resolved we jump to the error
|     return path without initializing ret. Fix this.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: backend_storage: Set needs_refresh back to 0 after refreshing ...

This ports the following barebox commit:

| commit 275f354
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Wed Apr 19 10:18:42 2017 +0200
|
|     state: backend_storage: Set needs_refresh back to 0 after refreshing
|
|     Set needs_refresh back to 0 after refreshing so that we do not refresh
|     it again without need. This would only happen when we read the state
|     from the storage multiple times, which normally is not the case.
|     However, it's more consistent like this.
|
|     Reported-by: Sam Ravnborg <sam@ravnborg.org>
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: backend_bucket_circular: Do not leak memory ...

This ports the following barebox commit:

| commit af18710
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Wed Apr 19 10:13:58 2017 +0200
|
|     state: backend_bucket_circular: Do not leak memory
|
|     buf was just allocated, free it before returning an error.
|
|     Reported-by: Sam Ravnborg <sam@ravnborg.org>
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: find device node from device path, not from device node path ...

This adds a dummy version of of_find_node_by_devpath() to make the linker happy
and ports the following barebox commit:

| commit e627903
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Thu Mar 30 16:33:26 2017 +0200
|
|     state: find device node from device path, not from device node path
|
|     The device node path may change from the internal device tree to the
|     one Linux is started with, so using this path to fixup the tree is
|     not very robust. Instead, use of_find_node_by_devpath() which has
|     been created for exactly this purpose.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

state: Make pointing to the backend using a phandle the only supported method ...

This ports the following barebox commit and removes the now no longer used noop
definition of of_find_path():

| commit 99020ca
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Tue Mar 21 13:57:51 2017 +0100
|
|     state: Make pointing to the backend using a phandle the only supported method
|
|     All other methods are broken for some time already: When starting the
|     kernel the state code rewrites the state node in the device tree and
|     replaced the "backend" property with a phandle - even when the target
|     can't be described as a phandle. Since using phandles is the nicest way
|     to point to the storage device anyway remove the other methods.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

treewide: Use of_property_write_string() where appropriate ...

This ports the following barebox commit:

| commit b608918
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Thu Mar 30 10:38:19 2017 +0200
|
|     treewide: Use of_property_write_string() where appropriate
|
|     Replace users which use of_set_property() to set a property to a string
|     with of_property_write_string().
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

libdt: add of_property_write_string() ...

This ports the following barebox commits:

| commit 2d36433
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Thu Mar 30 10:36:58 2017 +0200
|
|     of: Add of_property_write_string()
|
|     Setting a property to a string is used many times. Create a convenience
|     function for it.
|
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

| commit 4da7cf0
| Author: Ulrich Ölmann <u.oelmann@pengutronix.de>
| Date:   Fri Oct 19 06:33:11 2018 +0200
|
|     of: base: fix typo
|
|     Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

backend_storage: remove fs.h ...

Starting with commit e790822a6676 ("state: simplify direct backend") the ioctl
BLKGETSIZE64 is no longer used so there is no need for fs.h anymore (included
since commit bbafbee71370 ("backend_storage: add fs.h for BLKGETSIZE64")).

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
Signed-off-by: Roland Hieber <rhi@pengutronix.de>

README: add note about subject prefix ...

Signed-off-by: Roland Hieber <rhi@pengutronix.de>

Release v2018.05.0 ...

Signed-off-by: Roland Hieber <rhi@pengutronix.de>

README: also use oss-tools@pengutronix.de here ...

Signed-off-by: Roland Hieber <r.hieber@pengutronix.de>

Fix compiler warnings about wrong type conversion in messages. ...

These warning were observed with gcc-6.3 on x86-64.

Signed-off-by: Philipp Rosenberger <p.rosenberger@linutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

src: fix compilation for glibc version 2.27.9000-36.fc29 and newer ...

As recent glibc versions (>= 2.27.9000-36.fc29) also define 'struct
statx' which is also defined in linux/stat.h, compilation fails with
error:

| In file included from ../dt-utils-2018.05.0/src/crypto/digest.c:24:
| [..]/usr/include/linux/stat.h:56:8: error: redefinition of 'struct statx_timestamp'
|  struct statx_timestamp {
|         ^~~~~~~~~~~~~~~
| In file included from [..]/usr/include/sys/stat.h:446,
|                  from ../dt-utils-2018.05.0/src/dt/common.h:15,
|                  from ../dt-utils-2018.05.0/src/crypto/digest.c:19:
| [..]/usr/include/bits/statx.h:25:8: note: originally defined here
|  struct statx_timestamp
|         ^~~~~~~~~~~~~~~
| In file included from ../dt-utils-2018.05.0/src/crypto/digest.c:24:
| [..]/usr/include/linux/stat.h:99:8: error: redefinition of 'struct statx'
|  struct statx {
|         ^~~~~
| In file included from [..]/usr/include/sys/stat.h:446,
|                  from ../dt-utils-2018.05.0/src/dt/common.h:15,
|                  from ../dt-utils-2018.05.0/src/crypto/digest.c:19:
| [..]/usr/include/bits/statx.h:36:8: note: originally defined here
|  struct statx
|         ^~~~~

The linux/stat.h originates from the code that was copied from barebox
but is not explicitly required to be linux/stat.h instead of sys/stat.h
and we do not actually use struct statx.

Thus it is safe to simply replace occurrences of linux/stat.h by
sys/stat.h to fix compilation.

Signed-off-by: Enrico Joerns <ejo@pengutronix.de>

configure.ac: change project mail to oss-tools@pengutronix.de ...

Signed-off-by: Enrico Jorns <ejo@pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Release v2018.05.0 ...

Signed-off-by: Roland Hieber <r.hieber@pengutronix.de>

Makefile.am: add a reminder comment for library versioning ...

See https://www.gnu.org/software/libtool/manual/html_node/Updating-version-info.html

Signed-off-by: Roland Hieber <r.hieber@pengutronix.de>

barebox-state: block while waiting for file lock ...

In multi-processes environment, it can happen that barebox-state is
called concurrently. Since barebox-state usually only runs for at most a
few seconds, it is probably okay for the second process to block while
wait for the file lock, so the calling entity (user or script) doesn't
have to do error handling itself.

Signed-off-by: Roland Hieber <r.hieber@pengutronix.de>

barebox-state: Fail for unhandled arguments ...

When passing unhandled arguments as in

  % barebox-state foo

these will be silently ignored. This may lead to misunderstandings and
unintended behavior, thus the user should be informed and the programm
should exit with an error in these cases.

Signed-off-by: Enrico Jorns <ejo@pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

barebox-state: fix termination of long_options[] ...

From the getopt_long() documentation:

| The argument longopts must be an array of these structures, one for each
| long option. Terminate the array with an element containing all zeros.

Otherwise barebox-state will segfault for unknown options:

  % ./barebox-state --foo
  [1]    17946 segmentation fault (core dumped)  ./barebox-state --foo

Signed-off-by: Enrico Jorns <ejo@pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

common: Include sys/types.h header ...

Build with musl library fails with following error:

In file included from src/crypto/sha1.c:21:0:
./src/digest.h:95:10: error: unknown type name ‘ulong’
          ulong start, ulong size);
...

Fix that by including sys/types.h header in common.h, so ulong type
is defined with musl library.

Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

common: Rename strlcpy to DT_strlcpy ...

strlcpy function is defined in uClibc library, causing "static
declaration of ‘strlcpy’ follows non-static declaration" build
errors.

Rename internal strlcpy function to DT_strlcpy to avoid conflicts.

Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

barebox-state: backend_raw: init digest earlier ...

In backend_format_raw_pack() digest_length is used before it's
initialized in backend_raw_digest_init() which results in a too small
memory allocation for the raw backend. Fix this and prevent a memory
corruption.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

Release v2017.03.0 ...

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

libdt: support "partitions" subnode on mtd devices ...

In case there is an additional "partitions" subnode between the device
itself and its partitions like /soc/spi@?/mram@?/partitions/partition@?
also try the parent of the parent before giving up finding the device
node.

See kernel docs for reference:
  Documentation/devicetree/bindings/mtd/partition.txt

Signed-off-by: Bastian Stender <bst@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

state: Remove -EUCLEAN check from userspace tool ...

The state code is used for the userspace tool aswell, kept in sync
manually. This patch only introduces a change for the userspace
tool, not for barebox.

In Linux userspace there is no direct possibility to check for -EUCLEAN.
To indirectly check for -EUCLEAN the state tool reads the number of
corrected bits before and after reading a block. Unfortunately it does
not take the number of acceptable bitflips into account, but instead
returns -EUCLEAN even when only a single bitflip occurred on a whole
page. To be correct the algorithm must be more complicated: We would
have to read the bitflip_threshold from sysfs. This value is per ECC
step (often 512 byte), not per page. We would have to read the page
in ECC step size chunks, testing for bitflips lower than the threshold
for each chunk. Even if we would do that, there's still another issue.
The eccstats ioctl delivers the eccstats for the whole device, so a
concurrent reader would falsify the result.

Let's decide that this is not worth the hassle and assume that no
device has enough uptime that a cleanup in barebox is not sufficient.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>