Thread (19 messages) 19 messages, 5 authors, 2019-07-29

Re: [PATCH v2 0/7] Add anx6345 DP/eDP bridge for Olimex Teres-I

From: Vasily Khoruzhick <anarsoul@gmail.com>
Date: 2019-07-22 18:47:45
Also in: dri-devel, linux-devicetree, lkml

On Mon, Jul 22, 2019 at 8:04 AM Torsten Duwe [off-list ref] wrote:
ANX6345 LVTTL->eDP video bridge, driver with device tree bindings.

Changes from v2:

* use SPDX-IDs throughout

* removed the panel output again, as it was not what Maxime had in mind.
  At least the Teres-I does very well without. No connector spec, no "quirks"[1],
  just plain EDID at work.
You still need a panel to put backlight in there. Otherwise backlight
will stay on when display is turned off.
* binding clarifications and cosmetic changes as suggested by Andrzej

Changes from v1:

* fixed up copyright information. Most code changes are only moves and thus
  retain copyright and module ownership. Even the new analogix-anx6345.c originates
  from the old 1495-line analogix-anx78xx.c, with 306 insertions and 987 deletions
  (ignoring the trivial anx78xx -> anx6345 replacements) 306 new vs. 508 old...

* fixed all minor formatting issues brought up

* merged previously separate new analogix_dp_i2c module into existing analogix_dp

* split additional defines into a preparatory patch

* renamed the factored-out common functions anx_aux_* -> anx_dp_aux_*, because
  anx_...aux_transfer was exported globally. Besides, it is now GPL-only exported.

* moved chip ID read into a separate function.

* keep the chip powered after a successful probe.
  (There's a good chance that this is the only display during boot!)

* updated the binding document: LVTTL input is now required, only the output side
  description is optional.

 Laurent: I have also looked into the drm_panel_bridge infrastructure,
 but it's not that trivial to convert these drivers to it.

Changes from the respective previous versions:

* the reset polarity is corrected in DT and the driver;
  things should be clearer now.

* as requested, add a panel (the known innolux,n116bge) and connect
  the ports.

* renamed dvdd?? to *-supply to match the established scheme

* trivial update to the #include list, to make it compile in 5.2

--------------
[1] I hesitate to associate information about a connected panel with that term.
    But should it be neccessary for maximum power saving (e.g. pinebooks),
    it would be good to have something here, regardless of nomenclature.

        Torsten
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help