Re: [PATCH] spi: rockchip: configure CTRLR1 according to size and data frame

2017-08-16 Thread Geert Uytterhoeven
Hi Huibin, On Wed, Aug 16, 2017 at 4:12 AM, Huibin Hong wrote: > CTRLR1 is number of data frames, when rx only. > When data frame is 8 bit, CTRLR1 is len-1. > When data frame is 16 bit, CTRLR1 is (len/2)-1. What does the last "else" case correspond to? >

Re: [PATCH] spi: rockchip: configure CTRLR1 according to size and data frame

2017-08-16 Thread Geert Uytterhoeven
Hi Huibin, On Wed, Aug 16, 2017 at 4:12 AM, Huibin Hong wrote: > CTRLR1 is number of data frames, when rx only. > When data frame is 8 bit, CTRLR1 is len-1. > When data frame is 16 bit, CTRLR1 is (len/2)-1. What does the last "else" case correspond to? > Signed-off-by: Huibin Hong > --- > >

[PATCH] spi: rockchip: configure CTRLR1 according to size and data frame

2017-08-15 Thread Huibin Hong
CTRLR1 is number of data frames, when rx only. When data frame is 8 bit, CTRLR1 is len-1. When data frame is 16 bit, CTRLR1 is (len/2)-1. Signed-off-by: Huibin Hong --- drivers/spi/spi-rockchip.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff

[PATCH] spi: rockchip: configure CTRLR1 according to size and data frame

2017-08-15 Thread Huibin Hong
CTRLR1 is number of data frames, when rx only. When data frame is 8 bit, CTRLR1 is len-1. When data frame is 16 bit, CTRLR1 is (len/2)-1. Signed-off-by: Huibin Hong --- drivers/spi/spi-rockchip.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git