Re: [PATCH v2 4/4] mmc: host: tmio: fill in response from auto cmd12

2017-02-15 Thread Ulf Hansson
On 15 February 2017 at 16:02, Wolfram Sang wrote: > >> > I see. Ulf, do you think it makes sense to extend the condition when to >> > call mmc_blk_cmd_recovery() with checking if stop.resp[0] has one of the >> > R1_* bits set which are marked with 'ex' (and probably 'erx',

[PATCH v4] spi: rspi: Fixes bogus received byte in qspi_transfer_in()

2017-02-15 Thread DongCV
In qspi_transfer_in(), when receiving the last n (or len) bytes of data, 1 bogus byte was written in the receive buffer. This code leads to a buffer overflow. "jffs2: jffs2_scan_eraseblock(): Magic bitmask 0x1985 not found at 0x03b4: 0x1900 instead jffs2: jffs2_scan_eraseblock(): Magic

[PATCH 2/2] spi: rspi: Replaces "n" by "len" in qspi_transfer_*()

2017-02-15 Thread DongCV
This patch replaced "n" by "len" bytes of data in qspi_transfer_in() and qspi_transfer_out() function. This will make improving readability. Signed-off-by: DongCV --- drivers/spi/spi-rspi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 0/2 v4] spi: rspi: Fixes bogus received byte and replaces "n" by "len"

2017-02-15 Thread DongCV
Hi Mark, Cc: Geert, Sergei Thanks for your comments and advices. I separated my patch into two patches as you've shown me. These patches fix: 3be09bec42a800d4 "spi: rspi: supports 32bytes buffer for DUAL and QUAD". One of them is to fix bogus received byte into qspi_transfer_in(), and other is

Re: [PATCH 2/2] spi: rspi: Replaces "n" by "len" in qspi_transfer_*()

2017-02-15 Thread Geert Uytterhoeven
On Wed, Feb 15, 2017 at 11:50 AM, DongCV wrote: > This patch replaced "n" by "len" bytes of data in qspi_transfer_in() and > qspi_transfer_out() function. This will make improving readability. > > Signed-off-by: DongCV Reviewed-by: Geert Uytterhoeven

Re: [PATCH v3] spi: rspi: Fix bogus received byte in qspi_transfer_in()

2017-02-15 Thread Geert Uytterhoeven
Hi Dong, Thanks for the update! On Wed, Feb 15, 2017 at 8:33 AM, DongCV wrote: > In qspi_transfer_in(), when receiving the last n (or len) bytes of data, > n(or len) bogus bytes were written in the receive buffer, 1 bogus byte was written And I think you should mention

Re: [PATCH v3 6/7] dt-bindings: media: Add Renesas R-Car DRIF binding

2017-02-15 Thread Rob Herring
On Tue, Feb 07, 2017 at 03:02:36PM +, Ramesh Shanmugasundaram wrote: > Add binding documentation for Renesas R-Car Digital Radio Interface > (DRIF) controller. > > Signed-off-by: Ramesh Shanmugasundaram > > --- >

Re: [PATCH/RFC] clk: renesas: r8a7795: Replace PLL3 multiplication setting

2017-02-15 Thread Yoshihiro Kaneko
Hi Geert-san, 2017-02-15 18:28 GMT+09:00 Geert Uytterhoeven : > Hi Kaneko-san, > > On Sun, Feb 12, 2017 at 6:35 PM, Yoshihiro Kaneko > wrote: >> From: Takeshi Kihara >> >> This patch replaces PLL3 multiplication

RE: [PATCH 2/3] dt-bindings: power: reset: add document for renesas-reset driver

2017-02-15 Thread Chris Brandt
Hi Geert, On Tuesday, February 14, 2017, Geert Uytterhoeven wrote: > > +Required properties: > > + - compatible: must be one or more of the following: > > +- "renesas,r7s72100-reset" for the r7s72100 SoC > > +- "renesas,wdt-reset" > > +This is a fallback for the above

[RFC] mmc: host: tmio: ensure end of DMA and SD access are in sync

2017-02-15 Thread Wolfram Sang
The current code assumes that DMA is finished before SD access end is flagged. Thus, it schedules the 'dma_complete' tasklet in the SD card interrupt routine when DATAEND is set. The assumption is not safe, though. Even by mounting an SD card, it can be seen that sometimes DMA complete is first,

Re: [PATCH v3 2/7] dt-bindings: media: Add MAX2175 binding description

2017-02-15 Thread Rob Herring
On Tue, Feb 07, 2017 at 03:02:32PM +, Ramesh Shanmugasundaram wrote: > Add device tree binding documentation for MAX2175 Rf to bits tuner > device. > > Signed-off-by: Ramesh Shanmugasundaram > > --- > .../devicetree/bindings/media/i2c/max2175.txt

Re: [PATCH v2 1/3] ARM: l2c: add pl310-no-sideband option

2017-02-15 Thread Rob Herring
On Tue, Feb 07, 2017 at 12:09:27PM -0500, Chris Brandt wrote: > If a PL310 is added to a system, but the sideband signals are not > connected, some Cortex A9 optimizations cannot be used. In particular, > enabling Full Line Zeros in the CA9 without sidebands connected will > crash the system since

[PULL REQUEST] renesas/topic/sdhi-dma-sync for renesas drivers

2017-02-15 Thread Wolfram Sang
Hi Geert, in case, we decide to include this patch despite not being tested for the original SDIO case, here is a formal pull request. Kind regards, Wolfram The following changes since commit 391a542fe891d6dac49fa7fc2422651e54af1d42: mmc: Adding AUTO_BKOPS_EN bit set for Auto BKOPS