Re: [PATCH 1/5] video: simple_panel: add boe,nv140fhmn49 display

[Anatolij Gustschin]

On Mon, 20 Apr 2020 20:27:32 +0100
Peter Robinson pbrobin...@gmail.com wrote:

> add "boe,nv140fhmn49" display to compatible node.
> 
> Signed-off-by: Peter Robinson 
> Cc: Anatolij Gustschin 
> ---
>  drivers/video/simple_panel.c | 1 +
>  1 file changed, 1 insertion(+)

Applied to u-boot-video/master, thanks!

--
Anatolij

Re: [PATCH 1/5] video: simple_panel: add boe,nv140fhmn49 display

[Rtp]

[ fwiw, please note that I've not subscribed to the uboot ml ]

>> add "boe,nv140fhmn49" display to compatible node.
>
>I didn't see u-boot output on the eDP panel; not sure if that's
>expected. Once linux booted it did work just fine.

I was trying to get working the eDP of my PBP and managed to get it. You
need more patches in order to get it. The vopl mipi and eDP endpoints
are inverted in the rk3399.dtsi file and the rk_edp.c file needs
patching. I've put my current patches and uboot configuration at
http://people.hupstream.com/~rtp/pbp/ in case you want to try in.

In order to build the ohci support (it seems it's needed by the PBP
keyboard ?), you need to add to the
include/configs/pinebook_pro_rk3399.h file, but I'm not 100% what's the
correct setting for CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS.

#define CONFIG_SYS_USB_OHCI_MAX_ROOT_PORTS 1
#define CONFIG_USB_OHCI_NEW1

Please report success or failure.

Arnaud

Re: [PATCH 1/5] video: simple_panel: add boe,nv140fhmn49 display

[Vagrant Cascadian]

On 2020-04-20, Peter Robinson wrote:
> add "boe,nv140fhmn49" display to compatible node.

I didn't see u-boot output on the eDP panel; not sure if that's
expected. Once linux booted it did work just fine.

live well,
  vagrant

> Signed-off-by: Peter Robinson 
> Cc: Anatolij Gustschin 
> ---
>  drivers/video/simple_panel.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/video/simple_panel.c b/drivers/video/simple_panel.c
> index c3c0e84732..572287 100644
> --- a/drivers/video/simple_panel.c
> +++ b/drivers/video/simple_panel.c
> @@ -105,6 +105,7 @@ static const struct udevice_id simple_panel_ids[] = {
>   { .compatible = "auo,b133xtn01" },
>   { .compatible = "auo,b116xw03" },
>   { .compatible = "auo,b133htn01" },
> + { .compatible = "boe,nv140fhmn49" },
>   { .compatible = "lg,lb070wv8" },
>   { }
>  };
> -- 
> 2.26.1


signature.asc
Description: PGP signature

[PATCH 1/5] video: simple_panel: add boe,nv140fhmn49 display

[Peter Robinson]

add "boe,nv140fhmn49" display to compatible node.

Signed-off-by: Peter Robinson 
Cc: Anatolij Gustschin 
---
 drivers/video/simple_panel.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/video/simple_panel.c b/drivers/video/simple_panel.c
index c3c0e84732..572287 100644
--- a/drivers/video/simple_panel.c
+++ b/drivers/video/simple_panel.c
@@ -105,6 +105,7 @@ static const struct udevice_id simple_panel_ids[] = {
{ .compatible = "auo,b133xtn01" },
{ .compatible = "auo,b116xw03" },
{ .compatible = "auo,b133htn01" },
+   { .compatible = "boe,nv140fhmn49" },
{ .compatible = "lg,lb070wv8" },
{ }
 };
-- 
2.26.1