Re: [PATCH v2 0/6] Add support for Kalray VLIW family (kvx)

2020-04-14 Thread Sascha Hauer
Hi Clement, On Sat, Apr 04, 2020 at 10:29:04PM +0200, Clement Leger wrote: > Kalray kv3 core is embedded in Kalray Coolidge SoC. This core which is the > third of the KV family has the following features: > - 32/64 bits execution mode > - 6-issue VLIW architecture > - 64 x 64bits general purpo

[PATCH] ppc,riscv: set something to KBUILD_DEFCONFIG

2020-04-14 Thread Masahiro Yamada
'make ARCH=riscv defconfig' ends up with endless recursion. Users may get upset with it. In Linux, every architecture does something for 'make defconfig'. Set something to KBUILD_DEFCONFIG to make it work for riscv and ppc. There is only one file in arch/riscv/configs/, so there is no choice but

[PATCH] mfd: double-quote the menu prompt of MFD

2020-04-14 Thread Masahiro Yamada
Linux commit 801b27db4638 ("kconfig: drop T_WORD from the RHS of 'prompt' symbol") requires every prompt to be quoted even if it is a single word. This is the only place with no quoting. It must be surrounded by double-quotes in order to sync Kconfig to Linux 5.6-rc1 or later. While I was here, I

Re: [PATCH v2 7/7] added-zynq-fpga-manager

2020-04-14 Thread Michael Tretter
Hi Michael, On Tue, Apr 14, 2020 at 09:23:53PM +0200, Michael Graichen wrote: > Am 14.04.20 um 12:53 schrieb Michael Tretter: > > On Thu, Apr 09, 2020 at 02:44:00PM +, Michael Graichen wrote: > > > This adds support to programm the PL part of the Zynq SoC, > > > but only the non-secure way and

Re: [PATCH v2 7/7] added-zynq-fpga-manager

2020-04-14 Thread Michael Graichen
Hi Michael, Am 14.04.20 um 12:53 schrieb Michael Tretter: Hi Michael, On Thu, Apr 09, 2020 at 02:44:00PM +, Michael Graichen wrote: This adds support to programm the PL part of the Zynq SoC, but only the non-secure way and no partial reconfiguration. It adds the 'zynq_fpga_manager' so we c

[PATCH] fixup! of: port Linux of_get_compatible_child helper

2020-04-14 Thread Ahmad Fatoum
The barebox implementation of for_each_child_of_node doesn't set the iterator to NULL, when the list of children is empty. Fix of_get_compatible_child to deal with this. Suggested-by: Sascha Hauer Signed-off-by: Ahmad Fatoum --- drivers/of/base.c | 4 ++-- 1 file changed, 2 insertions(+), 2 de

Re: imx7d dual core boot

2020-04-14 Thread Giorgio
Hi Sascha, thank you for your help, your last suggestion was the last missing bit in my qspi setup. I had the following wrong parameter in my flash header file: loadaddr 0x8000 following your hint I changed it to: loadaddr 0x80001000 and now I don't need my evil hack with the branch opcode

Re: [PATCH v2 5/6] clocksource: kvx: Add kvx clocksource support

2020-04-14 Thread Sascha Hauer
On Sat, Apr 04, 2020 at 10:29:09PM +0200, Clement Leger wrote: > Add a clocksource for kvx architecture based on core performance > counter. This performance counter is configured to count cycles and as > such can be used to be a clocksource. > > Signed-off-by: Clement Leger > --- > arch/kvx/con

Re: [PATCH v2 7/7] added-zynq-fpga-manager

2020-04-14 Thread Michael Tretter
Hi Michael, On Thu, Apr 09, 2020 at 02:44:00PM +, Michael Graichen wrote: > This adds support to programm the PL part of the Zynq SoC, > but only the non-secure way and no partial reconfiguration. It adds the > 'zynq_fpga_manager' so we can use > > firmwareload -l > firmwareload -t zynq-fpga-

Re: [PATCH 4/5] spi: Add fsl-dspi driver

2020-04-14 Thread Sascha Hauer
On Sat, Apr 04, 2020 at 12:14:33PM -0700, Andrey Smirnov wrote: > On Wed, Mar 25, 2020 at 1:36 AM Sascha Hauer wrote: > > > > The fsl-dspi is found on Freescale Vybrid, Coldfire and Layerscape SoCs. > > This adds the driver based on the Linux driver as of v5.5. Coldfire > > support has been droppe

Re: [PATCH 2/2] ARM: sm.c: add the attribute 'volatile' to some asm() statements.

2020-04-14 Thread Sascha Hauer
On Tue, Apr 07, 2020 at 05:32:17PM +0200, Giorgio Dal Molin wrote: > Prevent the compiler to eventually cache register values read/written > to/from CP15. > > Signed-off-by: Giorgio Dal Molin > --- > arch/arm/cpu/sm.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) Applied, than

Re: [PATCH 1/2] ARM: i.MX: fixed enabling the MMU after switching in non secure mode.

2020-04-14 Thread Sascha Hauer
On Tue, Apr 07, 2020 at 05:31:11PM +0200, Giorgio Dal Molin wrote: > The Domain Access Control Register (DACR) in CP15 is banked between > secure and non secure mode: there a copy of the reg. in secure mode and a > second copy in non secure mode. > As barebox boots on the imx7 SOC it runs in secure

Re: [PATCH 3/3] driver: Call of_clk_set_defaults for each probed device

2020-04-14 Thread Sascha Hauer
On Mon, Apr 06, 2020 at 08:01:19AM +0200, Ahmad Fatoum wrote: > Hello Sascha, > > On 2/13/20 11:58 AM, Sascha Hauer wrote: > > So far we only honour the assigned-clocks and assigned-clock-rates > > device tree properties for the clock controller nodes. With this patch > > we also honour the proper

Re: [PATCH] commands: remove CONFIG_ prefix from Kconfig symbol

2020-04-14 Thread Sascha Hauer
On Mon, Apr 06, 2020 at 07:54:35AM +0200, Ahmad Fatoum wrote: > Kconfig symbols, unlike their preprocessor counterparts, don't have this > prefix. Drop it. > > Signed-off-by: Ahmad Fatoum > --- > commands/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied, thanks Sascha >

Re: [PATCH 07/21] of: port Linux of_get_compatible_child helper

2020-04-14 Thread Michael Tretter
Hello Ahmad, On Tue, Apr 14, 2020 at 09:29:36AM +0200, Ahmad Fatoum wrote: > Hello Sascha, > > On 4/14/20 8:46 AM, Sascha Hauer wrote: > > for_each_child_of_node() goes down to list_for_each_entry(). On an empty > > list child will point to the list head after leaving the loop. Return > > child e

Re: [PATCH V2] startup: introduce global.endianness variable

2020-04-14 Thread Sascha Hauer
On Mon, Apr 06, 2020 at 03:44:04PM +0300, Antony Pavlov wrote: > On Mon, 6 Apr 2020 15:13:25 +0300 > Antony Pavlov wrote: > > Hi Sascha, > > The global.version (and global.endianness) variable can > be easely modified by user. > Can we make it immutable? There's currently no way to make variab

Re: [PATCH] uimage: add define for RISC-V architecture

2020-04-14 Thread Sascha Hauer
On Mon, Apr 06, 2020 at 03:38:45PM +0300, Antony Pavlov wrote: > On Mon, 6 Apr 2020 15:23:13 +0300 > Antony Pavlov wrote: > > Hi Sascha, > > It looks like barebox IN_ARCH_* constants in image.h and the same constants > from U-Boot's image.h > are differ. > > > --- a/include/image.h > > +++ b/

Re: [PATCH] ARM: i.MX: phytec-som-imx6: Fix paths to SPI-NOR partition

2020-04-14 Thread Sascha Hauer
On Tue, Apr 07, 2020 at 11:52:56AM +0200, Yunus Bas wrote: > Due to a previously change of the partition namings, the kernel- and > oftree-paths in the SPI-bootscript have to be adapted to the changes. > > The previous change can be found here: > commit 72ba815bf6feaa0a98d2496f52784dd5b4258e73 >

Re: [PATCH] checkpatch.pl: Update DT vendor prefix check

2020-04-14 Thread Sascha Hauer
On Tue, Apr 07, 2020 at 12:05:34AM +0300, Antony Pavlov wrote: > In commit 796af3473b82 ("dts: update to v5.2-rc1") > vendor-prefixes.txt has been converted to a DT schema. > Update the checkpatch.pl DT check to extract vendor prefixes from the new > vendor-prefixes.yaml file. > > Based on this li

Re: [PATCH] of: overlay: fix uninitialized read of variable

2020-04-14 Thread Sascha Hauer
On Tue, Apr 07, 2020 at 08:49:41PM +0200, Ahmad Fatoum wrote: > When there are no fragments, we return an uninitialized err. > Initialize err as zero, so it's returned in that case. > > Signed-off-by: Ahmad Fatoum > --- > drivers/of/overlay.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

Re: [PATCH] of: use of_get_child_by_name() to find child node

2020-04-14 Thread Sascha Hauer
On Tue, Apr 14, 2020 at 11:06:10AM +0200, Michael Tretter wrote: > After iterating the children of a node with for_each_child_of_node(), > the child node will never be NULL. If the node was not found, > overlay_child will always point to the first element in the list, which > might or might not be

Re: [PATCH] mci: arasan: fix missing select on sdhci-helpers

2020-04-14 Thread Ahmad Fatoum
Hello Michael, On 4/14/20 11:07 AM, Michael Tretter wrote: > The arasan driver uses the sdhci-helpers as well, but does not select > the sdhci-helpers. > > Add the missing select to fix the compilation of the arasan driver. There is another series on the mailing list that Sascha just accepted, w

[PATCH] mci: arasan: fix missing select on sdhci-helpers

2020-04-14 Thread Michael Tretter
The arasan driver uses the sdhci-helpers as well, but does not select the sdhci-helpers. Add the missing select to fix the compilation of the arasan driver. Signed-off-by: Michael Tretter --- drivers/mci/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mci/Kconfig b/drivers/m

[PATCH] of: use of_get_child_by_name() to find child node

2020-04-14 Thread Michael Tretter
After iterating the children of a node with for_each_child_of_node(), the child node will never be NULL. If the node was not found, overlay_child will always point to the first element in the list, which might or might not be the node that was searched. Use the of_get_child_by_name() helper functi

Re: [PATCH v2 1/7] add-MCI_SDHCI-dependency-to-Arasan-driver

2020-04-14 Thread Sascha Hauer
On Thu, Apr 09, 2020 at 02:36:55PM +, Michael Graichen wrote: > The Arasan driver depends on the SDHCI driver, without it we > will see undefined references while linking arasan-sdhci.c:283: undefined > reference to `sdhci_set_cmd_xfer_mode' arasan-sdhci.c:299: undefined > reference to `sdhci_r

Re: imx7d dual core boot

2020-04-14 Thread Sascha Hauer
On Tue, Apr 14, 2020 at 12:30:40AM +0200, Giorgio wrote: > Hi Ahmad, > > On 4/7/20 3:43 PM, Ahmad Fatoum wrote: > > Hello, > > > > On 4/7/20 2:28 PM, Giorgio wrote: > >>> Great. Even better than hardcoding the CLIENT_DOMAIN. > >>> > >> OK. > >> > >> To read the current value of DACR, in secure mo

Re: [PATCH 07/21] of: port Linux of_get_compatible_child helper

2020-04-14 Thread Ahmad Fatoum
Hello Sascha, On 4/14/20 8:46 AM, Sascha Hauer wrote: > for_each_child_of_node() goes down to list_for_each_entry(). On an empty > list child will point to the list head after leaving the loop. Return > child explicitly when it's compatible and NULL when leaving the loop > which is better readable

Re: [PATCH] fixup! regmap-mmio: Add big endian support

2020-04-14 Thread Sascha Hauer
On Mon, Apr 13, 2020 at 09:11:11AM +0200, Ahmad Fatoum wrote: > ("regmap-mmio: Add big endian support") required syscons to have a device_d > instantiated for them. This doesn't work for clock drivers though, > like the sama5d2's, because of_clk_init is called before > of_platform_populate allocate