Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-22 Thread Vasily Khoruzhick
After discussing it with Maxime in IRC I decided to wait till pinctrl driver for sunxi is ready. Please kindly disregard this series. Regards, Vasily On Fri, Sep 22, 2017 at 7:44 AM, Maxime Ripard wrote: > On Fri, Sep 22, 2017 at 04:42:24AM +, Vasily Khoruzhick wrote: >> >> >>> + lcdc_i

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-22 Thread Maxime Ripard
On Fri, Sep 22, 2017 at 04:42:24AM +, Vasily Khoruzhick wrote: > >> >>> + lcdc_init(lcdc); > >> >>> + sunxi_lcdc_config_pinmux(); > >> >> > >> >> This is already handled in sunxi_lcdc_tcon0_mode_set, why duplicate > >> >> it? > >> > > >> > Because the one that sunxi_lcdc_tcon0_mode_set(

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-21 Thread Vasily Khoruzhick
Hi Maxime, On Wed, Sep 20, 2017 at 11:51 PM, Maxime Ripard wrote: > On Thu, Sep 21, 2017 at 05:33:36AM +, Vasily Khoruzhick wrote: >> >>> + ret = uclass_find_device_by_name(UCLASS_DISPLAY, >> >>> + "sunxi_lcd", &disp); >> >>> + if (!ret) { >> >>> +

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-21 Thread icenowy
在 2017-09-21 14:51,Maxime Ripard 写道: On Thu, Sep 21, 2017 at 05:33:36AM +, Vasily Khoruzhick wrote: >>> + ret = uclass_find_device_by_name(UCLASS_DISPLAY, >>> + "sunxi_lcd", &disp); >>> + if (!ret) { >>> + int mux; >>> + >>> +

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-20 Thread Maxime Ripard
On Thu, Sep 21, 2017 at 05:33:36AM +, Vasily Khoruzhick wrote: > >>> + ret = uclass_find_device_by_name(UCLASS_DISPLAY, > >>> + "sunxi_lcd", &disp); > >>> + if (!ret) { > >>> + int mux; > >>> + > >>> + mux = 0; > >>> + > >>> +

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-20 Thread Vasily Khoruzhick
Hi, I did few tests, see results inline. On Tue, Sep 19, 2017 at 12:00 PM, Vasily Khoruzhick wrote: > On Tue, Sep 19, 2017 at 1:33 AM, Maxime Ripard > wrote: >> On Mon, Sep 18, 2017 at 10:04:21PM -0700, Vasily Khoruzhick wrote: >>> Extend DE2 driver with LCD support >> >> (All) your commit mess

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-19 Thread Icenowy Zheng
于 2017年9月20日 GMT+08:00 上午3:00:30, Vasily Khoruzhick 写到: >On Tue, Sep 19, 2017 at 1:33 AM, Maxime Ripard > wrote: >> On Mon, Sep 18, 2017 at 10:04:21PM -0700, Vasily Khoruzhick wrote: >>> Extend DE2 driver with LCD support >> >> (All) your commit messages could use a bit more details. > >OK, will

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-19 Thread Jernej Škrabec
Hi, Dne torek, 19. september 2017 ob 21:00:30 CEST je Vasily Khoruzhick napisal(a): > On Tue, Sep 19, 2017 at 1:33 AM, Maxime Ripard > > wrote: > > On Mon, Sep 18, 2017 at 10:04:21PM -0700, Vasily Khoruzhick wrote: > >> Extend DE2 driver with LCD support > > > > (All) your commit messages coul

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-19 Thread Vasily Khoruzhick
On Tue, Sep 19, 2017 at 1:33 AM, Maxime Ripard wrote: > On Mon, Sep 18, 2017 at 10:04:21PM -0700, Vasily Khoruzhick wrote: >> Extend DE2 driver with LCD support > > (All) your commit messages could use a bit more details. OK, will add in v2. > Here, for example, explaining the following things w

Re: [U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-19 Thread Maxime Ripard
On Mon, Sep 18, 2017 at 10:04:21PM -0700, Vasily Khoruzhick wrote: > Extend DE2 driver with LCD support (All) your commit messages could use a bit more details. Here, for example, explaining the following things would help: - Why are you creating yet another file - What is the situation with

[U-Boot] [RESEND PATCH 5/5] sunxi: video: add LCD support to DE2 driver

2017-09-18 Thread Vasily Khoruzhick
Extend DE2 driver with LCD support Signed-off-by: Vasily Khoruzhick --- arch/arm/mach-sunxi/Kconfig | 2 +- drivers/video/sunxi/Makefile| 2 +- drivers/video/sunxi/sunxi_de2.c | 17 + drivers/video/sunxi/sunxi_lcd.c | 142 4 files changed