Re: [yocto] Support UBI u-boot

2020-02-08 Thread JH
Hi Gabriele, Thanks for the help. On 2/8/20, Gabriele Zampieri wrote: > If you just want to run the menuconfig run > make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- menuconfig > and > make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- savedefconfig > In any environment. It does not seem the OE /

Re: [PATCH 1/1] net: designware: speed should be in a debug message

2020-02-08 Thread Heinrich Schuchardt
On 2/9/20 3:59 AM, Bin Meng wrote: On Sun, Feb 9, 2020 at 8:38 AM Heinrich Schuchardt wrote: The network connection speed is a debug information. So we should use debug() and not printf(). Signed-off-by: Heinrich Schuchardt --- drivers/net/designware.c | 6 +++--- 1 file changed, 3 insert

Re: [PATCH 1/1] net: designware: speed should be in a debug message

2020-02-08 Thread Bin Meng
On Sun, Feb 9, 2020 at 8:38 AM Heinrich Schuchardt wrote: > > The network connection speed is a debug information. So we should use > debug() and not printf(). > > Signed-off-by: Heinrich Schuchardt > --- > drivers/net/designware.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) >

Re: [PATCH 1/1] log: correct CONFIG_LOG_TEST prerequisites

2020-02-08 Thread Bin Meng
On Sun, Feb 9, 2020 at 6:17 AM Heinrich Schuchardt wrote: > > An error > > undefined reference to `do_log_test' > > occurs for CONFIG_CMD_LOG=y, CONFIG_LOG_TEST=y, CONGIG_UNIT_TEST=n > > Make CONFIG_UNIT_TEST a prerequisite. > > Signed-off-by: Heinrich Schuchardt > --- > common/Kconfig |

[PATCH 1/1] log: syslog driver

2020-02-08 Thread Heinrich Schuchardt
Provide a log driver that broadcasts RFC 3164 messages to syslog servers. rsyslog is one implementation of such a server. The messages are sent to the local broadcast address 255.255.255.255 on port 514. The environment variable log_hostname can be used to provide the HOSTNAME field for the messa

[PATCH 1/1] net: designware: speed should be in a debug message

2020-02-08 Thread Heinrich Schuchardt
The network connection speed is a debug information. So we should use debug() and not printf(). Signed-off-by: Heinrich Schuchardt --- drivers/net/designware.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/designware.c b/drivers/net/designware.c index 19fc

[PATCH 1/1] log: correct CONFIG_LOG_TEST prerequisites

2020-02-08 Thread Heinrich Schuchardt
An error undefined reference to `do_log_test' occurs for CONFIG_CMD_LOG=y, CONFIG_LOG_TEST=y, CONGIG_UNIT_TEST=n Make CONFIG_UNIT_TEST a prerequisite. Signed-off-by: Heinrich Schuchardt --- common/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/Kconf

Re: [PATCH 007/108] dm: pci: Allow disabling auto-config for a device

2020-02-08 Thread Bin Meng
Hi Simon, On Sun, Feb 9, 2020 at 12:07 AM Simon Glass wrote: > > Hi Bin, > > On Sat, 8 Feb 2020 at 08:54, Bin Meng wrote: > > > > Hi Simon, > > > > On Sat, Feb 8, 2020 at 11:51 PM Simon Glass wrote: > > > > > > Hi Bin, > > > > > > On Sat, 8 Feb 2020 at 08:15, Bin Meng wrote: > > > > > > > > Hi

Re: [PATCH 007/108] dm: pci: Allow disabling auto-config for a device

2020-02-08 Thread Simon Glass
Hi Bin, On Sat, 8 Feb 2020 at 08:54, Bin Meng wrote: > > Hi Simon, > > On Sat, Feb 8, 2020 at 11:51 PM Simon Glass wrote: > > > > Hi Bin, > > > > On Sat, 8 Feb 2020 at 08:15, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > > > > > > > Ad

Re: [PATCH 1/1] doc: fix AX25-AE350 RISC-V documentation

2020-02-08 Thread Bin Meng
Hi Heinrich, On Sat, Feb 8, 2020 at 5:37 AM Heinrich Schuchardt wrote: > > On 2/4/20 3:50 PM, Bin Meng wrote: > > On Sat, Dec 28, 2019 at 2:51 AM Heinrich Schuchardt > > wrote: > >> > >> Since commit 04883bf7acca ("doc: update AX25-AE350 RISC-V documentation") > >> `make htmldocs` produces a lo

Re: [PATCH 007/108] dm: pci: Allow disabling auto-config for a device

2020-02-08 Thread Bin Meng
Hi Simon, On Sat, Feb 8, 2020 at 11:51 PM Simon Glass wrote: > > Hi Bin, > > On Sat, 8 Feb 2020 at 08:15, Bin Meng wrote: > > > > Hi Simon, > > > > On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > > > > > Add a means to avoid configuring a device when needed. Add an explanation > > > of w

Re: [PATCH 007/108] dm: pci: Allow disabling auto-config for a device

2020-02-08 Thread Simon Glass
Hi Bin, On Sat, 8 Feb 2020 at 08:15, Bin Meng wrote: > > Hi Simon, > > On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > > > Add a means to avoid configuring a device when needed. Add an explanation > > of why this is useful to the binding file. > > > > Does disabling CONFIG_PCI_PNP not hel

Re: [PATCH v3][ 6/6] boards: tbs2910: add README

2020-02-08 Thread Bin Meng
Hi, On Sat, Feb 8, 2020 at 1:39 PM Denis 'GNUtoo' Carikli wrote: > > The installation documentation make use of the > "BOOT/UPDATE" mechanism that is documented in the TBS2910 > OS installation manual[1]. > > This is to make sure that they work on the different PCB > revisions of this board. > >

Re: [PATCH 015/108] x86: Move acpi_table header to main include/ directory

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > This file is potentially useful to other architectures saddled with ACPI > so move it into a common location. > > Signed-off-by: Simon Glass > --- > > arch/x86/cpu/baytrail/acpi.c | 2 +- > arch/x86/cpu/cpu.c| 2 +- >

Re: [PATCH 014/108] acpi: Add a simple sandbox test

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Add a sandbox test for the basic ACPI functionality we have so far. > > Signed-off-by: Simon Glass > --- > > arch/sandbox/dts/test.dts | 4 +++ > include/dm/uclass-id.h| 1 + > test/dm/Makefile | 1 + > test/dm/acpi.c

Re: [PATCH 011/108] x86: apl: Add Global NVS table header

2020-02-08 Thread Bin Meng
Hi Simon, On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Add the C version of this header. It includes a few Chrome OS bits which > are disabled for a normal build. > > Signed-off-by: Simon Glass > --- > > .../include/asm/arch-apollolake/global_nvs.h | 44 +++ > 1 file

Re: [PATCH 012/108] dm: core: Add basic ACPI support

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > ACPI (Advanced Configuration and Power Interface) is an Intel standard > for specifying information about a platform. It is a little like device > tree but considerably more complicated and with more backslashes. A > primary difference is that

Re: [PATCH v3][ 6/6] boards: tbs2910: add README

2020-02-08 Thread Soeren Moch
On 08.02.20 06:38, Denis 'GNUtoo' Carikli wrote: > The installation documentation make use of the > "BOOT/UPDATE" mechanism that is documented in the TBS2910 > OS installation manual[1]. > > This is to make sure that they work on the different PCB > revisions of this board. > > There is also a boot

Re: [PATCH v3][ 4/6] board: tbs2910: enable CONFIG_DISTRO_DEFAULTS

2020-02-08 Thread Soeren Moch
On 08.02.20 06:38, Denis 'GNUtoo' Carikli wrote: > The side effect is that it increase the size of the > resultimg image, which is already very close to the > size limit. > > With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola > GNU/Linux distribution, we have the following size > increase: > - te

Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-08 Thread Bin Meng
On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: > > Some compilers produce a warning about 'child' being used before init. > Silence this by setting to NULL at the start. Should be a compiler bug I think. Which compiler has such issue? > > Signed-off-by: Simon Glass > --- > > drivers/misc/p

Re: [PATCH 008/108] x86: Correct wording of coreboot source code

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Some files are taken or modified from coreboot, but the files are > no-longer part of the coreboot project. Fix the wording in a few places. > > Signed-off-by: Simon Glass > --- > > arch/x86/cpu/coreboot/timestamp.c | 4 ++-- >

Re: [PATCH 009/108] x86: apl: Move p2sb ofdata reading to the correct method

2020-02-08 Thread Bin Meng
+Wolfgang Wallner On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > With P2SB the initial BAR (base-address register) is set up by TPL and > this is used unchanged right through U-Boot. > > At present the reading of this address is split between the ofdata() and > probe() methods. There are

Re: [PATCH 010/108] pci: Adjust dm_pci_read_bar32() to return errors correctly

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > At present if reading a BAR returns 0x (e.g. the device is not > present) then the value is masked and a different value is returned. > This makes it harder to detect the problem when debugging. > > Update the function to avoid masking

Re: [PATCH 006/108] tpm: Don't cleanup unless an error happens

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > At present the cleanup() method is called on every transfer. It should > only be called on failing transfers. Fix this and tidy up the error > handling a little. > > Signed-off-by: Simon Glass > --- > > drivers/tpm/tpm-uclass.c | 13

Re: [PATCH 007/108] dm: pci: Allow disabling auto-config for a device

2020-02-08 Thread Bin Meng
Hi Simon, On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Add a means to avoid configuring a device when needed. Add an explanation > of why this is useful to the binding file. > Does disabling CONFIG_PCI_PNP not help? > Signed-off-by: Simon Glass > --- > > doc/device-tree-bindings/pci

Re: [PATCH 003/108] tpm: cr50: Release locality on exit

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > At present the cr50 driver claims the locality and does not release it for > Linux. This causes problems. Fix this by tracking what is claimed, and > adding a 'remove' method. > > Signed-off-by: Simon Glass > --- > > drivers/tpm/cr50_i2c.c |

Re: [PATCH 004/108] tpm: cr50: Tidy up various comments

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Add a comment for the private structure and drop the incorrect coreboot > reference in one function. > > Signed-off-by: Simon Glass > --- > > drivers/tpm/cr50_i2c.c | 13 +++-- > 1 file changed, 11 insertions(+), 2 deletions(-) > Re

Re: [PATCH 005/108] tpm: cr50: Use the correct GPIO binding

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > This device should use ready-gpios rather than ready-gpio. Fix it. Where does this requirement come from? Is this coming from the Linux kernel bindings? > > Signed-off-by: Simon Glass > --- > > arch/x86/dts/chromebook_coral.dts

Re: [PATCH 001/108] cpu: Support querying the address width

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > Different CPUs may support different address widths, meaning the amount of > memory they can address. Add a property for this to the cpu_info struct. > > Signed-off-by: Simon Glass > --- > > drivers/cpu/cpu_sandbox.c | 1 + > include/cpu.h

[PATCH] sandbox: p2sb: Silence compiler warning

2020-02-08 Thread Simon Glass
Some compilers produce a warning about 'child' being used before init. Silence this by setting to NULL at the start. Signed-off-by: Simon Glass --- drivers/misc/p2sb_emul.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/p2sb_emul.c b/drivers/misc/p2sb_emul.c in

Re: [PATCH 002/108] spi: Add SPI mode enums

2020-02-08 Thread Bin Meng
On Mon, Jan 27, 2020 at 1:08 PM Simon Glass wrote: > > With ACPI we need to describe the settings of the SPI bus. Add enums to > handle this. > > Signed-off-by: Simon Glass > --- > > include/spi.h | 33 + > 1 file changed, 33 insertions(+) > Reviewed-by: Bin Meng

Re: FYI: Please pull u-boot-dm

2020-02-08 Thread Simon Glass
Hi Stephen, On Thu, 6 Feb 2020 at 15:38, Simon Glass wrote: > > Hi Stephen, > > On Thu, 6 Feb 2020 at 15:32, Stephen Warren wrote: > > > > On 2/6/20 2:55 PM, Simon Glass wrote: > > > Hi Tom, > > > > > > This cannot be pulled yet since we need to update gitlab's docker > > > image to include SDL2

Re: [PATCH v3 0/6] Introduce DSA Ethernet switch class and Felix driver

2020-02-08 Thread Vladimir Oltean
On Tue, 3 Dec 2019 at 18:18, Alex Marginean wrote: > > DSA stands for Distributed Switch Architecture and it is a subsystem > introduced in the Linux kernel to support switches that: > - have an Ethernet link up to the CPU > - use some form of tagging to identify the source/destination port for >