* [PATCH 1/6] drm/bridge: use bus flags in bridge timings
@ 2018-09-05 18:00 Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 1/2] modeprint: use libutil to lookup strings Stefan Agner
` (4 more replies)
0 siblings, 5 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:00 UTC (permalink / raw)
To: dri-devel
The DRM bus flags convey additional information on pixel data on
the bus. All current available bus flags might be of interest for
a bridge. Remove the sampling_edge field and use bus_flags.
In the case at hand a dumb VGA bridge needs a specific data enable
polarity (DRM_BUS_FLAG_DE_LOW).
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
drivers/gpu/drm/bridge/dumb-vga-dac.c | 6 +++---
include/drm/drm_bridge.h | 11 +++++------
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c
index 9b706789a341..7a5c24967115 100644
--- a/drivers/gpu/drm/bridge/dumb-vga-dac.c
+++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c
@@ -234,7 +234,7 @@ static int dumb_vga_remove(struct platform_device *pdev)
*/
static const struct drm_bridge_timings default_dac_timings = {
/* Timing specifications, datasheet page 7 */
- .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
+ .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
.setup_time_ps = 500,
.hold_time_ps = 1500,
};
@@ -245,7 +245,7 @@ static const struct drm_bridge_timings default_dac_timings = {
*/
static const struct drm_bridge_timings ti_ths8134_dac_timings = {
/* From timing diagram, datasheet page 9 */
- .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
+ .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
/* From datasheet, page 12 */
.setup_time_ps = 3000,
/* I guess this means latched input */
@@ -258,7 +258,7 @@ static const struct drm_bridge_timings ti_ths8134_dac_timings = {
*/
static const struct drm_bridge_timings ti_ths8135_dac_timings = {
/* From timing diagram, datasheet page 14 */
- .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
+ .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
/* From datasheet, page 16 */
.setup_time_ps = 2000,
.hold_time_ps = 500,
diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
index bd850747ce54..85d4b51eae19 100644
--- a/include/drm/drm_bridge.h
+++ b/include/drm/drm_bridge.h
@@ -244,14 +244,13 @@ struct drm_bridge_funcs {
*/
struct drm_bridge_timings {
/**
- * @sampling_edge:
+ * @bus_flags:
*
- * Tells whether the bridge samples the digital input signal
- * from the display engine on the positive or negative edge of the
- * clock, this should reuse the DRM_BUS_FLAG_PIXDATA_[POS|NEG]EDGE
- * bitwise flags from the DRM connector (bit 2 and 3 valid).
+ * Tells what additional settings for the pixel data on the bus
+ * this bridge requires (like pixel signal polarity). See also
+ * &drm_display_info->bus_flags.
*/
- u32 sampling_edge;
+ u32 bus_flags;
/**
* @setup_time_ps:
*
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH libdrm 1/2] modeprint: use libutil to lookup strings
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
@ 2018-09-05 18:00 ` Stefan Agner
2018-10-04 13:28 ` Emil Velikov
2018-09-05 18:00 ` [PATCH 2/6] drm/bridge: allow to specify data-enable polarity Stefan Agner
` (3 subsequent siblings)
4 siblings, 1 reply; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:00 UTC (permalink / raw)
To: dri-devel
Use libutil to lookup connector type names and state. This also
makes sure that the latest connector type addition "DPI" gets
printed correctly.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
tests/modeprint/Makefile.am | 1 +
tests/modeprint/meson.build | 2 +-
tests/modeprint/modeprint.c | 43 ++++++-------------------------------
3 files changed, 9 insertions(+), 37 deletions(-)
diff --git a/tests/modeprint/Makefile.am b/tests/modeprint/Makefile.am
index 601dbc96..c8c5e797 100644
--- a/tests/modeprint/Makefile.am
+++ b/tests/modeprint/Makefile.am
@@ -15,4 +15,5 @@ endif
modeprint_SOURCES = \
modeprint.c
modeprint_LDADD = \
+ $(top_builddir)/tests/util/libutil.la \
$(top_builddir)/libdrm.la
diff --git a/tests/modeprint/meson.build b/tests/modeprint/meson.build
index 5f0eb24b..5e752798 100644
--- a/tests/modeprint/meson.build
+++ b/tests/modeprint/meson.build
@@ -23,7 +23,7 @@ modeprint = executable(
files('modeprint.c'),
c_args : warn_c_args,
include_directories : [inc_root, inc_tests, inc_drm],
- link_with : libdrm,
+ link_with : [libdrm, libutil],
dependencies : dep_threads,
install : with_install_tests,
)
diff --git a/tests/modeprint/modeprint.c b/tests/modeprint/modeprint.c
index c81dd91d..1d62270f 100644
--- a/tests/modeprint/modeprint.c
+++ b/tests/modeprint/modeprint.c
@@ -42,6 +42,7 @@
#include "xf86drmMode.h"
#include "util/common.h"
+#include "util/kms.h"
int current;
int connectors;
@@ -54,20 +55,6 @@ int crtcs;
int fbs;
char *module_name;
-static const char* getConnectionText(drmModeConnection conn)
-{
- switch (conn) {
- case DRM_MODE_CONNECTED:
- return "connected";
- case DRM_MODE_DISCONNECTED:
- return "disconnected";
- case DRM_MODE_UNKNOWNCONNECTION:
- default:
- return "unknown";
- }
-
-}
-
static int printMode(struct drm_mode_modeinfo *mode)
{
if (full_modes) {
@@ -141,40 +128,24 @@ static int printProperty(int fd, drmModeResPtr res, drmModePropertyPtr props, ui
return 0;
}
-static const char * const output_names[] = { "None",
- "VGA",
- "DVI-I",
- "DVI-D",
- "DVI-A",
- "Composite",
- "SVIDEO",
- "LVDS",
- "Component",
- "DIN",
- "DP",
- "HDMI-A",
- "HDMI-B",
- "TV",
- "eDP",
- "Virtual",
- "DSI",
-};
-
static int printConnector(int fd, drmModeResPtr res, drmModeConnectorPtr connector, uint32_t id)
{
int i = 0;
struct drm_mode_modeinfo *mode = NULL;
drmModePropertyPtr props;
+ const char *connector_type_name = NULL;
+
+ connector_type_name = util_lookup_connector_type_name(connector->connector_type);
- if (connector->connector_type < ARRAY_SIZE(output_names))
- printf("Connector: %s-%d\n", output_names[connector->connector_type],
+ if (connector_type_name)
+ printf("Connector: %s-%d\n", connector_type_name,
connector->connector_type_id);
else
printf("Connector: %d-%d\n", connector->connector_type,
connector->connector_type_id);
printf("\tid : %i\n", id);
printf("\tencoder id : %i\n", connector->encoder_id);
- printf("\tconn : %s\n", getConnectionText(connector->connection));
+ printf("\tconn : %s\n", util_lookup_connector_status_name(connector->connection));
printf("\tsize : %ix%i (mm)\n", connector->mmWidth, connector->mmHeight);
printf("\tcount_modes : %i\n", connector->count_modes);
printf("\tcount_props : %i\n", connector->count_props);
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 2/6] drm/bridge: allow to specify data-enable polarity
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 1/2] modeprint: use libutil to lookup strings Stefan Agner
@ 2018-09-05 18:00 ` Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 2/2] modeprint: print encoder type Stefan Agner
` (2 subsequent siblings)
4 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:00 UTC (permalink / raw)
To: dri-devel
Support boards with a passive RGB to VGA bridge which require a low
active data-enable polarity.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
Alternatively a new dt binding could be introduced for dumb VGA bridges
requiring low active data enable... However, also other polarities might
need a specific polarity, hence this generic approach might be better.
drivers/gpu/drm/bridge/dumb-vga-dac.c | 22 ++++++++++++++++++----
1 file changed, 18 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c b/drivers/gpu/drm/bridge/dumb-vga-dac.c
index 7a5c24967115..2b8c3d629f2f 100644
--- a/drivers/gpu/drm/bridge/dumb-vga-dac.c
+++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c
@@ -179,6 +179,7 @@ static struct i2c_adapter *dumb_vga_retrieve_ddc(struct device *dev)
static int dumb_vga_probe(struct platform_device *pdev)
{
struct dumb_vga *vga;
+ u32 de;
vga = devm_kzalloc(&pdev->dev, sizeof(*vga), GFP_KERNEL);
if (!vga)
@@ -194,6 +195,23 @@ static int dumb_vga_probe(struct platform_device *pdev)
dev_dbg(&pdev->dev, "No vdd regulator found: %d\n", ret);
}
+ vga->bridge.funcs = &dumb_vga_bridge_funcs;
+ vga->bridge.of_node = pdev->dev.of_node;
+ vga->bridge.timings = of_device_get_match_data(&pdev->dev);
+
+ if (!vga->bridge.timings &&
+ !of_property_read_u32(pdev->dev.of_node, "de-active", &de)) {
+ struct drm_bridge_timings *timings;
+
+ timings = devm_kzalloc(&pdev->dev, sizeof(*timings), GFP_KERNEL);
+ if (!timings)
+ return -ENOMEM;
+
+ timings->bus_flags = de ? DRM_BUS_FLAG_DE_HIGH :
+ DRM_BUS_FLAG_DE_LOW;
+ vga->bridge.timings = timings;
+ }
+
vga->ddc = dumb_vga_retrieve_ddc(&pdev->dev);
if (IS_ERR(vga->ddc)) {
if (PTR_ERR(vga->ddc) == -ENODEV) {
@@ -205,10 +223,6 @@ static int dumb_vga_probe(struct platform_device *pdev)
}
}
- vga->bridge.funcs = &dumb_vga_bridge_funcs;
- vga->bridge.of_node = pdev->dev.of_node;
- vga->bridge.timings = of_device_get_match_data(&pdev->dev);
-
drm_bridge_add(&vga->bridge);
return 0;
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH libdrm 2/2] modeprint: print encoder type
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 1/2] modeprint: use libutil to lookup strings Stefan Agner
2018-09-05 18:00 ` [PATCH 2/6] drm/bridge: allow to specify data-enable polarity Stefan Agner
@ 2018-09-05 18:00 ` Stefan Agner
2018-09-05 18:00 ` [PATCH 3/6] dt-bindings: display: add data-enable polarity property Stefan Agner
2018-09-05 18:03 ` [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
4 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:00 UTC (permalink / raw)
To: dri-devel
Print encoder types similar to connector types.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
tests/modeprint/modeprint.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/tests/modeprint/modeprint.c b/tests/modeprint/modeprint.c
index 1d62270f..ad727e12 100644
--- a/tests/modeprint/modeprint.c
+++ b/tests/modeprint/modeprint.c
@@ -186,7 +186,13 @@ static int printConnector(int fd, drmModeResPtr res, drmModeConnectorPtr connect
static int printEncoder(int fd, drmModeResPtr res, drmModeEncoderPtr encoder, uint32_t id)
{
- printf("Encoder\n");
+ const char *encoder_name;
+
+ encoder_name = util_lookup_encoder_type_name(encoder->encoder_type);
+ if (encoder_name)
+ printf("Encoder: %s\n", encoder_name);
+ else
+ printf("Encoder\n");
printf("\tid :%i\n", id);
printf("\tcrtc_id :%d\n", encoder->crtc_id);
printf("\ttype :%d\n", encoder->encoder_type);
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
` (2 preceding siblings ...)
2018-09-05 18:00 ` [PATCH libdrm 2/2] modeprint: print encoder type Stefan Agner
@ 2018-09-05 18:00 ` Stefan Agner
2018-09-05 18:03 ` [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
4 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:00 UTC (permalink / raw)
To: dri-devel
Allow to specify the data-enable polarity required by a dumb VGA
DAC converting parallel RGB to VGA.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
.../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
index 164cbb15f04c..adbd2ca0af2f 100644
--- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
+++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
@@ -18,6 +18,7 @@ graph bindings specified in Documentation/devicetree/bindings/graph.txt.
Optional properties:
- vdd-supply: Power supply for DAC
+- de-active: data-enable pulse is active low/high/ignored
Example
-------
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 1/6] drm/bridge: use bus flags in bridge timings
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
` (3 preceding siblings ...)
2018-09-05 18:00 ` [PATCH 3/6] dt-bindings: display: add data-enable polarity property Stefan Agner
@ 2018-09-05 18:03 ` Stefan Agner
4 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:03 UTC (permalink / raw)
To: dri-devel
On 05.09.2018 11:00, Stefan Agner wrote:
> The DRM bus flags convey additional information on pixel data on
> the bus. All current available bus flags might be of interest for
> a bridge. Remove the sampling_edge field and use bus_flags.
>
> In the case at hand a dumb VGA bridge needs a specific data enable
> polarity (DRM_BUS_FLAG_DE_LOW).
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
I meant to send out only the two patches with "PATCH libdrm" in subject.
Please disregard the other patches, they have been sent in error.
--
Stefan
> drivers/gpu/drm/bridge/dumb-vga-dac.c | 6 +++---
> include/drm/drm_bridge.h | 11 +++++------
> 2 files changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/dumb-vga-dac.c
> b/drivers/gpu/drm/bridge/dumb-vga-dac.c
> index 9b706789a341..7a5c24967115 100644
> --- a/drivers/gpu/drm/bridge/dumb-vga-dac.c
> +++ b/drivers/gpu/drm/bridge/dumb-vga-dac.c
> @@ -234,7 +234,7 @@ static int dumb_vga_remove(struct platform_device *pdev)
> */
> static const struct drm_bridge_timings default_dac_timings = {
> /* Timing specifications, datasheet page 7 */
> - .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> + .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> .setup_time_ps = 500,
> .hold_time_ps = 1500,
> };
> @@ -245,7 +245,7 @@ static const struct drm_bridge_timings
> default_dac_timings = {
> */
> static const struct drm_bridge_timings ti_ths8134_dac_timings = {
> /* From timing diagram, datasheet page 9 */
> - .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> + .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> /* From datasheet, page 12 */
> .setup_time_ps = 3000,
> /* I guess this means latched input */
> @@ -258,7 +258,7 @@ static const struct drm_bridge_timings
> ti_ths8134_dac_timings = {
> */
> static const struct drm_bridge_timings ti_ths8135_dac_timings = {
> /* From timing diagram, datasheet page 14 */
> - .sampling_edge = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> + .bus_flags = DRM_BUS_FLAG_PIXDATA_POSEDGE,
> /* From datasheet, page 16 */
> .setup_time_ps = 2000,
> .hold_time_ps = 500,
> diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
> index bd850747ce54..85d4b51eae19 100644
> --- a/include/drm/drm_bridge.h
> +++ b/include/drm/drm_bridge.h
> @@ -244,14 +244,13 @@ struct drm_bridge_funcs {
> */
> struct drm_bridge_timings {
> /**
> - * @sampling_edge:
> + * @bus_flags:
> *
> - * Tells whether the bridge samples the digital input signal
> - * from the display engine on the positive or negative edge of the
> - * clock, this should reuse the DRM_BUS_FLAG_PIXDATA_[POS|NEG]EDGE
> - * bitwise flags from the DRM connector (bit 2 and 3 valid).
> + * Tells what additional settings for the pixel data on the bus
> + * this bridge requires (like pixel signal polarity). See also
> + * &drm_display_info->bus_flags.
> */
> - u32 sampling_edge;
> + u32 bus_flags;
> /**
> * @setup_time_ps:
> *
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH libdrm 1/2] modeprint: use libutil to lookup strings
2018-09-05 18:00 ` [PATCH libdrm 1/2] modeprint: use libutil to lookup strings Stefan Agner
@ 2018-10-04 13:28 ` Emil Velikov
0 siblings, 0 replies; 19+ messages in thread
From: Emil Velikov @ 2018-10-04 13:28 UTC (permalink / raw)
To: Stefan Agner; +Cc: ML dri-devel
On Wed, 5 Sep 2018 at 19:01, Stefan Agner <stefan@agner.ch> wrote:
>
> Use libutil to lookup connector type names and state. This also
> makes sure that the latest connector type addition "DPI" gets
> printed correctly.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
For the series:
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Will do some sanity check and push in a moment.
Thanks
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
2018-09-05 18:10 ` Stefan Agner
(?)
@ 2018-09-05 20:50 ` Laurent Pinchart
-1 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 20:50 UTC (permalink / raw)
To: Stefan Agner
Cc: linus.walleij, airlied, robh+dt, mark.rutland, shawnguo, s.hauer,
p.zabel, kernel, fabio.estevam, linux-imx, architt, a.hajda,
gustavo, maarten.lankhorst, sean, marcel.ziswiler,
max.krummenacher, dri-devel, devicetree, linux-arm-kernel,
linux-kernel
Hi Stefan,
On Wednesday, 5 September 2018 21:10:08 EEST Stefan Agner wrote:
> On 05.09.2018 00:07, Laurent Pinchart wrote:
> > On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> >> Allow to specify the data-enable polarity required by a dumb VGA
> >> DAC converting parallel RGB to VGA.
> >>
> >> Signed-off-by: Stefan Agner <stefan@agner.ch>
> >> ---
> >>
> >> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> >> 164cbb15f04c..adbd2ca0af2f 100644
> >> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> @@ -18,6 +18,7 @@ graph bindings specified in
> >> Documentation/devicetree/bindings/graph.txt.
> >>
> >> Optional properties:
> >> - vdd-supply: Power supply for DAC
> >>
> >> +- de-active: data-enable pulse is active low/high/ignored
> >
> > Which value corresponds to low, high and ignored ?
>
> The wording is taken from
> Documentation/devicetree/bindings/display/panel/display-timing.txt. But
> I agree, not very useful.
>
> 0 is low active, 1 is high active, and none is using driver defaults.
>
> How about:
> - de-active: data-enable pulse is 0=active low/1=active high
The data enable signal isn't really a pulse. I would word this as
- de-active: Polarity of the data enable signal. 0 for active low, 1 for
active high, unset for system-specific defaults.
> >> Example
> >> -------
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 20:50 ` Laurent Pinchart
0 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 20:50 UTC (permalink / raw)
To: Stefan Agner
Cc: mark.rutland, devicetree, max.krummenacher, kernel,
marcel.ziswiler, airlied, linux-kernel, dri-devel, robh+dt,
linux-imx, fabio.estevam, sean, shawnguo, linux-arm-kernel
Hi Stefan,
On Wednesday, 5 September 2018 21:10:08 EEST Stefan Agner wrote:
> On 05.09.2018 00:07, Laurent Pinchart wrote:
> > On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> >> Allow to specify the data-enable polarity required by a dumb VGA
> >> DAC converting parallel RGB to VGA.
> >>
> >> Signed-off-by: Stefan Agner <stefan@agner.ch>
> >> ---
> >>
> >> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> >> 164cbb15f04c..adbd2ca0af2f 100644
> >> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> @@ -18,6 +18,7 @@ graph bindings specified in
> >> Documentation/devicetree/bindings/graph.txt.
> >>
> >> Optional properties:
> >> - vdd-supply: Power supply for DAC
> >>
> >> +- de-active: data-enable pulse is active low/high/ignored
> >
> > Which value corresponds to low, high and ignored ?
>
> The wording is taken from
> Documentation/devicetree/bindings/display/panel/display-timing.txt. But
> I agree, not very useful.
>
> 0 is low active, 1 is high active, and none is using driver defaults.
>
> How about:
> - de-active: data-enable pulse is 0=active low/1=active high
The data enable signal isn't really a pulse. I would word this as
- de-active: Polarity of the data enable signal. 0 for active low, 1 for
active high, unset for system-specific defaults.
> >> Example
> >> -------
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 20:50 ` Laurent Pinchart
0 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 20:50 UTC (permalink / raw)
To: linux-arm-kernel
Hi Stefan,
On Wednesday, 5 September 2018 21:10:08 EEST Stefan Agner wrote:
> On 05.09.2018 00:07, Laurent Pinchart wrote:
> > On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> >> Allow to specify the data-enable polarity required by a dumb VGA
> >> DAC converting parallel RGB to VGA.
> >>
> >> Signed-off-by: Stefan Agner <stefan@agner.ch>
> >> ---
> >>
> >> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> >> 164cbb15f04c..adbd2ca0af2f 100644
> >> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> >> @@ -18,6 +18,7 @@ graph bindings specified in
> >> Documentation/devicetree/bindings/graph.txt.
> >>
> >> Optional properties:
> >> - vdd-supply: Power supply for DAC
> >>
> >> +- de-active: data-enable pulse is active low/high/ignored
> >
> > Which value corresponds to low, high and ignored ?
>
> The wording is taken from
> Documentation/devicetree/bindings/display/panel/display-timing.txt. But
> I agree, not very useful.
>
> 0 is low active, 1 is high active, and none is using driver defaults.
>
> How about:
> - de-active: data-enable pulse is 0=active low/1=active high
The data enable signal isn't really a pulse. I would word this as
- de-active: Polarity of the data enable signal. 0 for active low, 1 for
active high, unset for system-specific defaults.
> >> Example
> >> -------
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
2018-09-05 7:07 ` Laurent Pinchart
(?)
@ 2018-09-05 18:10 ` Stefan Agner
-1 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:10 UTC (permalink / raw)
To: Laurent Pinchart
Cc: linus.walleij, airlied, robh+dt, mark.rutland, shawnguo, s.hauer,
p.zabel, kernel, fabio.estevam, linux-imx, architt, a.hajda,
gustavo, maarten.lankhorst, sean, marcel.ziswiler,
max.krummenacher, dri-devel, devicetree, linux-arm-kernel,
linux-kernel
On 05.09.2018 00:07, Laurent Pinchart wrote:
> Hi Stefan,
>
> Thank you for the patch.
>
> On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
>> Allow to specify the data-enable polarity required by a dumb VGA
>> DAC converting parallel RGB to VGA.
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>> ---
>> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
>> 164cbb15f04c..adbd2ca0af2f 100644
>> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> @@ -18,6 +18,7 @@ graph bindings specified in
>> Documentation/devicetree/bindings/graph.txt.
>>
>> Optional properties:
>> - vdd-supply: Power supply for DAC
>> +- de-active: data-enable pulse is active low/high/ignored
>
> Which value corresponds to low, high and ignored ?
>
The wording is taken from
Documentation/devicetree/bindings/display/panel/display-timing.txt. But
I agree, not very useful.
0 is low active, 1 is high active, and none is using driver defaults.
How about:
- de-active: data-enable pulse is 0=active low/1=active high
--
Stefan
>> Example
>> -------
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 18:10 ` Stefan Agner
0 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:10 UTC (permalink / raw)
To: Laurent Pinchart
Cc: mark.rutland, devicetree, max.krummenacher, kernel,
marcel.ziswiler, airlied, linux-kernel, dri-devel, robh+dt,
linux-imx, fabio.estevam, sean, shawnguo, linux-arm-kernel
On 05.09.2018 00:07, Laurent Pinchart wrote:
> Hi Stefan,
>
> Thank you for the patch.
>
> On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
>> Allow to specify the data-enable polarity required by a dumb VGA
>> DAC converting parallel RGB to VGA.
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>> ---
>> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
>> 164cbb15f04c..adbd2ca0af2f 100644
>> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> @@ -18,6 +18,7 @@ graph bindings specified in
>> Documentation/devicetree/bindings/graph.txt.
>>
>> Optional properties:
>> - vdd-supply: Power supply for DAC
>> +- de-active: data-enable pulse is active low/high/ignored
>
> Which value corresponds to low, high and ignored ?
>
The wording is taken from
Documentation/devicetree/bindings/display/panel/display-timing.txt. But
I agree, not very useful.
0 is low active, 1 is high active, and none is using driver defaults.
How about:
- de-active: data-enable pulse is 0=active low/1=active high
--
Stefan
>> Example
>> -------
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 18:10 ` Stefan Agner
0 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 18:10 UTC (permalink / raw)
To: linux-arm-kernel
On 05.09.2018 00:07, Laurent Pinchart wrote:
> Hi Stefan,
>
> Thank you for the patch.
>
> On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
>> Allow to specify the data-enable polarity required by a dumb VGA
>> DAC converting parallel RGB to VGA.
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>> ---
>> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
>> 164cbb15f04c..adbd2ca0af2f 100644
>> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
>> @@ -18,6 +18,7 @@ graph bindings specified in
>> Documentation/devicetree/bindings/graph.txt.
>>
>> Optional properties:
>> - vdd-supply: Power supply for DAC
>> +- de-active: data-enable pulse is active low/high/ignored
>
> Which value corresponds to low, high and ignored ?
>
The wording is taken from
Documentation/devicetree/bindings/display/panel/display-timing.txt. But
I agree, not very useful.
0 is low active, 1 is high active, and none is using driver defaults.
How about:
- de-active: data-enable pulse is 0=active low/1=active high
--
Stefan
>> Example
>> -------
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
2018-09-05 5:21 ` Stefan Agner
(?)
@ 2018-09-05 7:07 ` Laurent Pinchart
-1 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 7:07 UTC (permalink / raw)
To: Stefan Agner
Cc: linus.walleij, airlied, robh+dt, mark.rutland, shawnguo, s.hauer,
p.zabel, kernel, fabio.estevam, linux-imx, architt, a.hajda,
gustavo, maarten.lankhorst, sean, marcel.ziswiler,
max.krummenacher, dri-devel, devicetree, linux-arm-kernel,
linux-kernel
Hi Stefan,
Thank you for the patch.
On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> Allow to specify the data-enable polarity required by a dumb VGA
> DAC converting parallel RGB to VGA.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git
> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> 164cbb15f04c..adbd2ca0af2f 100644
> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> @@ -18,6 +18,7 @@ graph bindings specified in
> Documentation/devicetree/bindings/graph.txt.
>
> Optional properties:
> - vdd-supply: Power supply for DAC
> +- de-active: data-enable pulse is active low/high/ignored
Which value corresponds to low, high and ignored ?
> Example
> -------
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 7:07 ` Laurent Pinchart
0 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 7:07 UTC (permalink / raw)
To: Stefan Agner
Cc: mark.rutland, devicetree, max.krummenacher, kernel,
marcel.ziswiler, airlied, linux-kernel, dri-devel, robh+dt,
linux-imx, fabio.estevam, sean, shawnguo, linux-arm-kernel
Hi Stefan,
Thank you for the patch.
On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> Allow to specify the data-enable polarity required by a dumb VGA
> DAC converting parallel RGB to VGA.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git
> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> 164cbb15f04c..adbd2ca0af2f 100644
> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> @@ -18,6 +18,7 @@ graph bindings specified in
> Documentation/devicetree/bindings/graph.txt.
>
> Optional properties:
> - vdd-supply: Power supply for DAC
> +- de-active: data-enable pulse is active low/high/ignored
Which value corresponds to low, high and ignored ?
> Example
> -------
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 7:07 ` Laurent Pinchart
0 siblings, 0 replies; 19+ messages in thread
From: Laurent Pinchart @ 2018-09-05 7:07 UTC (permalink / raw)
To: linux-arm-kernel
Hi Stefan,
Thank you for the patch.
On Wednesday, 5 September 2018 08:21:10 EEST Stefan Agner wrote:
> Allow to specify the data-enable polarity required by a dumb VGA
> DAC converting parallel RGB to VGA.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
> .../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git
> a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt index
> 164cbb15f04c..adbd2ca0af2f 100644
> --- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> +++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
> @@ -18,6 +18,7 @@ graph bindings specified in
> Documentation/devicetree/bindings/graph.txt.
>
> Optional properties:
> - vdd-supply: Power supply for DAC
> +- de-active: data-enable pulse is active low/high/ignored
Which value corresponds to low, high and ignored ?
> Example
> -------
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
2018-09-05 5:21 Stefan Agner
2018-09-05 5:21 ` Stefan Agner
@ 2018-09-05 5:21 ` Stefan Agner
0 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 5:21 UTC (permalink / raw)
To: linus.walleij, airlied, robh+dt, mark.rutland, shawnguo, s.hauer,
p.zabel
Cc: kernel, fabio.estevam, linux-imx, architt, a.hajda,
Laurent.pinchart, gustavo, maarten.lankhorst, sean,
marcel.ziswiler, max.krummenacher, dri-devel, devicetree,
linux-arm-kernel, linux-kernel, Stefan Agner
Allow to specify the data-enable polarity required by a dumb VGA
DAC converting parallel RGB to VGA.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
.../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
index 164cbb15f04c..adbd2ca0af2f 100644
--- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
+++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
@@ -18,6 +18,7 @@ graph bindings specified in Documentation/devicetree/bindings/graph.txt.
Optional properties:
- vdd-supply: Power supply for DAC
+- de-active: data-enable pulse is active low/high/ignored
Example
-------
--
2.18.0
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 5:21 ` Stefan Agner
0 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 5:21 UTC (permalink / raw)
To: linus.walleij, airlied, robh+dt, mark.rutland, shawnguo, s.hauer,
p.zabel
Cc: devicetree, max.krummenacher, marcel.ziswiler, linux-kernel,
dri-devel, linux-imx, kernel, fabio.estevam, sean,
linux-arm-kernel, Laurent.pinchart
Allow to specify the data-enable polarity required by a dumb VGA
DAC converting parallel RGB to VGA.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
.../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
index 164cbb15f04c..adbd2ca0af2f 100644
--- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
+++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
@@ -18,6 +18,7 @@ graph bindings specified in Documentation/devicetree/bindings/graph.txt.
Optional properties:
- vdd-supply: Power supply for DAC
+- de-active: data-enable pulse is active low/high/ignored
Example
-------
--
2.18.0
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 3/6] dt-bindings: display: add data-enable polarity property
@ 2018-09-05 5:21 ` Stefan Agner
0 siblings, 0 replies; 19+ messages in thread
From: Stefan Agner @ 2018-09-05 5:21 UTC (permalink / raw)
To: linux-arm-kernel
Allow to specify the data-enable polarity required by a dumb VGA
DAC converting parallel RGB to VGA.
Signed-off-by: Stefan Agner <stefan@agner.ch>
---
.../devicetree/bindings/display/bridge/dumb-vga-dac.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
index 164cbb15f04c..adbd2ca0af2f 100644
--- a/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
+++ b/Documentation/devicetree/bindings/display/bridge/dumb-vga-dac.txt
@@ -18,6 +18,7 @@ graph bindings specified in Documentation/devicetree/bindings/graph.txt.
Optional properties:
- vdd-supply: Power supply for DAC
+- de-active: data-enable pulse is active low/high/ignored
Example
-------
--
2.18.0
^ permalink raw reply related [flat|nested] 19+ messages in thread
end of thread, other threads:[~2018-10-04 13:33 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-05 18:00 [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 1/2] modeprint: use libutil to lookup strings Stefan Agner
2018-10-04 13:28 ` Emil Velikov
2018-09-05 18:00 ` [PATCH 2/6] drm/bridge: allow to specify data-enable polarity Stefan Agner
2018-09-05 18:00 ` [PATCH libdrm 2/2] modeprint: print encoder type Stefan Agner
2018-09-05 18:00 ` [PATCH 3/6] dt-bindings: display: add data-enable polarity property Stefan Agner
2018-09-05 18:03 ` [PATCH 1/6] drm/bridge: use bus flags in bridge timings Stefan Agner
-- strict thread matches above, loose matches on Subject: below --
2018-09-05 5:21 Stefan Agner
2018-09-05 5:21 ` [PATCH 3/6] dt-bindings: display: add data-enable polarity property Stefan Agner
2018-09-05 5:21 ` Stefan Agner
2018-09-05 5:21 ` Stefan Agner
2018-09-05 7:07 ` Laurent Pinchart
2018-09-05 7:07 ` Laurent Pinchart
2018-09-05 7:07 ` Laurent Pinchart
2018-09-05 18:10 ` Stefan Agner
2018-09-05 18:10 ` Stefan Agner
2018-09-05 18:10 ` Stefan Agner
2018-09-05 20:50 ` Laurent Pinchart
2018-09-05 20:50 ` Laurent Pinchart
2018-09-05 20:50 ` Laurent Pinchart
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.