On Tue, Jan 3, 2017 at 5:34 PM, Peter Senna Tschudin peter.senna@collabora.co.uk wrote:
Hi Rob,
Thank you for the review.
On 03 January, 2017 23:51 CET, Rob Herring robh@kernel.org wrote:
On Sun, Jan 01, 2017 at 09:24:29PM +0100, Peter Senna Tschudin wrote:
Devicetree bindings documentation for the GE B850v3 LVDS/DP++ display bridge.
Cc: Martyn Welch martyn.welch@collabora.co.uk Cc: Martin Donnelly martin.donnelly@ge.com Cc: Javier Martinez Canillas javier@dowhile0.org Cc: Enric Balletbo i Serra enric.balletbo@collabora.com Cc: Philipp Zabel p.zabel@pengutronix.de Cc: Rob Herring robh@kernel.org Cc: Fabio Estevam fabio.estevam@nxp.com Signed-off-by: Peter Senna Tschudin peter.senna@collabora.com
There was an Acked-by from Rob Herring robh@kernel.org for V6, but I changed the bindings to use i2c_new_secondary_device() so I removed it from the commit message.
.../devicetree/bindings/ge/b850v3-lvds-dp.txt | 39 ++++++++++++++++++++++
Generally, bindings are not organized by vendor. Put in bindings/display/bridge/... instead.
Will change that.
1 file changed, 39 insertions(+) create mode 100644 Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt
diff --git a/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt b/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt new file mode 100644 index 0000000..1bc6ebf --- /dev/null +++ b/Documentation/devicetree/bindings/ge/b850v3-lvds-dp.txt @@ -0,0 +1,39 @@ +Driver for GE B850v3 LVDS/DP++ display bridge
+Required properties:
- compatible : should be "ge,b850v3-lvds-dp".
Isn't '-lvds-dp' redundant? The part# should be enough.
b850v3 is the name of the product, this is why the proposed name. What about, b850v3-dp2 dp2 indicating the second DP output?
Humm, b850v3 is the board name? This node should be the name of the bridge chip.
Rob