Re: [PATCH 5/5] ARM: i.MX: Drop unused usb-imx6.c

2019-05-21 Thread Andrey Smirnov
On Tue, May 21, 2019 at 2:50 AM Sascha Hauer wrote: > > On Tue, May 21, 2019 at 11:37:16AM +0200, Sascha Hauer wrote: > > On Mon, May 20, 2019 at 11:24:02AM -0700, Andrey Smirnov wrote: > > > Looks like usb-imx6.c was abandoned and there are no references to > > > either of the functions difined t

[PATCH v2 4/5] clk: imx6: Fix procedure to switch the parent of LDB_DI_CLK

2019-05-21 Thread Ahmad Fatoum
From: Fabio Estevam Due to incorrect placement of the clock gate cell in the ldb_di[x]_clk tree, the glitchy parent mux of ldb_di[x]_clk can cause a glitch to enter the ldb_di_ipu_div divider. If the divider gets locked up, no ldb_di[x]_clk is generated, and the LVDS display will hang when the ip

[PATCH v2 5/5] clk: imx6: define an enum for ldb mux inputs

2019-05-21 Thread Ahmad Fatoum
For better readability should this code be reviewed in future, replace the hardcoded input numbers with an enum. This is just a cosmetic change and was verified to not affect clk-imx6.o. Signed-off-by: Ahmad Fatoum --- drivers/clk/imx/clk-imx6.c | 39 +++--- 1 fi

[PATCH v2 3/5] clk: imx6: Mask mmdc_ch1 handshake for periph2_sel and mmdc_ch1_axi_podf

2019-05-21 Thread Ahmad Fatoum
From: Philipp Zabel MMDC CH1 is not used on i.MX6Q, so the handshake needed to change the parent of periph2_sel or the divider of mmdc_ch1_axi_podf will never succeed. Disable the handshake mechanism to allow changing the frequency of mmdc_ch1_axi, allowing to use it as a possible source for the

[PATCH v2 2/5] clk: imx6: Make the LDB_DI0 and LDB_DI1 clocks read-only

2019-05-21 Thread Ahmad Fatoum
From: Philipp Zabel Due to incorrect placement of the clock gate cell in the ldb_di[x]_clk tree, the glitchy parent mux of ldb_di[x]_clk can cause a glitch to enter the ldb_di_ipu_div divider. If the divider gets locked up, no ldb_di[x]_clk is generated, and the LVDS display will hang when the ip

[PATCH v2 0/5] clk: imx6: work around LDB hang caused by ERR009219

2019-05-21 Thread Ahmad Fatoum
Short version: barebox always reparents the ldb_di[01] clocks to pll5_video_div. Due to a hardware erratum, this can trigger a glitch that causes LDB lock up. Thus: 1) remove the automatic reparenting, like the kernel now does and mark the affected clocks read only, so assigned-clock-parents

[PATCH v2 1/5] clk: imx6: remove quirky clk_set_parent(LDB_diN_sel, pll5_video_div)

2019-05-21 Thread Ahmad Fatoum
barebox has inherited the clk_set_parent(ldb_diN_sel, pll5_video_div) from upstream kernel commit 32f3b8da22 ("ARM i.MX6q: set the LDB serial clock parent to the video PLL"), where it was enabled for all i.MX6Q revisions after 1.0. It was applied whenever CONFIG_DRIVER_VIDEO_IMX_IPUV3 was defined.

[PATCH] common: state: check length

2019-05-21 Thread Jan Remmet
if written_length is read from a partial written bucket it may be to big and xmalloc will panic barebox. Check if the value is sane. Signed-off-by: Jan Remmet --- common/state/backend_bucket_direct.c | 4 1 file changed, 4 insertions(+) diff --git a/common/state/backend_bucket_direct.c b

Re: [PATCH 5/5] ARM: i.MX: Drop unused usb-imx6.c

2019-05-21 Thread Sascha Hauer
On Tue, May 21, 2019 at 11:37:16AM +0200, Sascha Hauer wrote: > On Mon, May 20, 2019 at 11:24:02AM -0700, Andrey Smirnov wrote: > > Looks like usb-imx6.c was abandoned and there are no references to > > either of the functions difined there. Drop it. > > > > Signed-off-by: Andrey Smirnov > > ---

Re: [PATCH] ARM: boards: kindle mx50: adapt MCI name to DTS

2019-05-21 Thread Sascha Hauer
On Mon, May 20, 2019 at 08:11:31PM +, Alexander Kurz wrote: > From: Alexander Kurz > > The MCI instances got aliases in the DTS from linux upstream which changed > the eMMC devicename e.g. from disk0 to mmc2. Adapt to this. > > Signed-off-by: Alexander Kurz > --- > Documentation/boards/imx

Re: [PATCH] ARM: include: dma: Zero out DMA coherent memory in no-MMU case

2019-05-21 Thread Sascha Hauer
On Mon, May 20, 2019 at 11:33:25AM -0700, Andrey Smirnov wrote: > Add code to match the behavior of dma_alloc_coherent() when CONFIG_MMU > is selected. > > Signed-off-by: Andrey Smirnov > --- > arch/arm/include/asm/dma.h | 2 ++ > 1 file changed, 2 insertions(+) Applied, thanks Sascha -- Pe

Re: [PATCH 5/5] ARM: i.MX: Drop unused usb-imx6.c

2019-05-21 Thread Sascha Hauer
On Mon, May 20, 2019 at 11:24:02AM -0700, Andrey Smirnov wrote: > Looks like usb-imx6.c was abandoned and there are no references to > either of the functions difined there. Drop it. > > Signed-off-by: Andrey Smirnov > --- > arch/arm/mach-imx/Makefile | 2 +- > arch/arm/mach-imx/usb-imx6.c |

Re: [PATCH] gpio: add driver for PCF857x, PCA{85, 96}7x, and MAX732[89] I2C GPIO expanders

2019-05-21 Thread Sascha Hauer
On Sat, May 18, 2019 at 01:36:06AM +0200, Michael Grzeschik wrote: > +static int pcf857x_probe(struct device_d *dev) > +{ > + struct i2c_client *client = to_i2c_client(dev); > + struct pcf857x_platform_data*pdata = dev->platform_data; > + struct device_node *np = dev->d

Re: [PATCH v2] ARM: Initial OP-TEE support

2019-05-21 Thread Sascha Hauer
On Thu, May 16, 2019 at 04:12:47PM +0200, Rouven Czerwinski wrote: > diff --git a/arch/arm/cpu/start-kernel-optee.S > b/arch/arm/cpu/start-kernel-optee.S > new file mode 100644 > index 00..ce5ac178a4 > --- /dev/null > +++ b/arch/arm/cpu/start-kernel-optee.S > @@ -0,0 +1,14 @@ > +#include

Re: [PATCH v2 0/4] scripts/checkpatch.pl: rebase on top of upstream v5.1

2019-05-21 Thread Sascha Hauer
On Wed, May 15, 2019 at 11:36:40AM +0200, Ahmad Fatoum wrote: > "v1" is at <20190219141647.2537-3-a.fat...@pengutronix.de> > > Changes since v1: > - added dts/Bindings to compatible search patch (by Antony) > - split barebox-specific changes into a separate commit (by Sascha) > - split check