* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-12-19 12:12 Shirish S
2013-12-19 12:12 ` [PATCH 1/4] ARM: dts: smdk5250: Add hdmi phy settings Shirish S
` (2 more replies)
0 siblings, 3 replies; 13+ messages in thread
From: Shirish S @ 2013-12-19 12:12 UTC (permalink / raw)
To: dri-devel, inki.dae, devicetree, t.figa
Cc: kgene.kim, s.nawrocki, shirish, linux-samsung-soc, Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
V6:
V7: removed nr-configs form the dtsi files.
V8: Fixed build error
V9: rebased and incorporated Tomasz comments.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 34 ++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 74 ++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 74 ++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 ++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 89 ++++++++++++++++----
5 files changed, 327 insertions(+), 18 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 1/4] ARM: dts: smdk5250: Add hdmi phy settings
2013-12-19 12:12 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
@ 2013-12-19 12:12 ` Shirish S
[not found] ` <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-19 12:12 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2 siblings, 0 replies; 13+ messages in thread
From: Shirish S @ 2013-12-19 12:12 UTC (permalink / raw)
To: dri-devel, inki.dae, devicetree, t.figa
Cc: kgene.kim, s.nawrocki, shirish, linux-samsung-soc, Shirish S
This patch moves the hdmi phy setting to smdk5250
dts,as its more of a per board configuration and
also shall be easier for supporting future chipsets.
Signed-off-by: Shirish S <s.shirish@samsung.com>
---
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 +++++++++++++++++++++++++++++
1 file changed, 74 insertions(+)
diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts
index 2538b32..10da02f 100644
--- a/arch/arm/boot/dts/exynos5250-smdk5250.dts
+++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts
@@ -220,6 +220,80 @@
hdmi {
hpd-gpio = <&gpx3 7 0>;
+ hdmiphy-configs {
+ /*
+ * Eye diagram test passed for:
+ * Data de-emphasis: -0.7dB & Data Level: 880mV
+ * i.e., 0010 0110 = 0x26
+ * and Clock level of 515mV and diff 1030mV
+ * i.e., 0x66
+ */
+ config0: config0 {
+ samsung,pixel-clock = <25200000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config1: config1 {
+ samsung,pixel-clock = <27000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config2: config2 {
+ samsung,pixel-clock = <27027000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config3: config3 {
+ samsung,pixel-clock = <36000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config4: config4 {
+ samsung,pixel-clock = <40000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config5: config5 {
+ samsung,pixel-clock = <65000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config6: config6 {
+ samsung,pixel-clock = <74176000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config7: config7 {
+ samsung,pixel-clock = <74250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config8: config8 {
+ samsung,pixel-clock = <83500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config9: config9 {
+ samsung,pixel-clock = <106500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config10: config10 {
+ samsung,pixel-clock = <108000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config11: config11 {
+ samsung,pixel-clock = <146250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config12: config12 {
+ samsung,pixel-clock = <148500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ };
};
codec@11000000 {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 13+ messages in thread[parent not found: <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>]
* [PATCH 2/4] ARM: dts: arndale: Add hdmi phy settings
[not found] ` <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2013-12-19 12:12 ` Shirish S
2013-12-19 12:12 ` [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings Shirish S
1 sibling, 0 replies; 13+ messages in thread
From: Shirish S @ 2013-12-19 12:12 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, t.figa-Sze3O3UU22JBDgjK7y7TUQ
Cc: kgene.kim-Sze3O3UU22JBDgjK7y7TUQ,
s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ, shirish-F7+t8E8rja9g9hUCZPvPmw,
linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA, Shirish S
This patch moves the hdmi phy setting to arndale dts,
as its more of a per board configuration and also
shall be easier for supporting future chipsets.
Signed-off-by: Shirish S <s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
arch/arm/boot/dts/exynos5250-arndale.dts | 74 ++++++++++++++++++++++++++++++
1 file changed, 74 insertions(+)
diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts
index cee55fa..6bc562c 100644
--- a/arch/arm/boot/dts/exynos5250-arndale.dts
+++ b/arch/arm/boot/dts/exynos5250-arndale.dts
@@ -475,6 +475,80 @@
vdd_osc-supply = <&ldo10_reg>;
vdd_pll-supply = <&ldo8_reg>;
vdd-supply = <&ldo8_reg>;
+ hdmiphy-configs {
+ /*
+ * Eye diagram test passed for:
+ * Data de-emphasis: -0.7dB & Data Level: 880mV
+ * i.e., 0010 0110 = 0x26
+ * and Clock level of 515mV and diff 1030mV
+ * i.e., 0x66
+ */
+ config0: config0 {
+ samsung,pixel-clock = <25200000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config1: config1 {
+ samsung,pixel-clock = <27000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config2: config2 {
+ samsung,pixel-clock = <27027000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config3: config3 {
+ samsung,pixel-clock = <36000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config4: config4 {
+ samsung,pixel-clock = <40000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config5: config5 {
+ samsung,pixel-clock = <65000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config6: config6 {
+ samsung,pixel-clock = <74176000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config7: config7 {
+ samsung,pixel-clock = <74250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config8: config8 {
+ samsung,pixel-clock = <83500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config9: config9 {
+ samsung,pixel-clock = <106500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config10: config10 {
+ samsung,pixel-clock = <108000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config11: config11 {
+ samsung,pixel-clock = <146250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config12: config12 {
+ samsung,pixel-clock = <148500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ };
};
regulators {
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings
[not found] ` <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-19 12:12 ` [PATCH 2/4] ARM: dts: arndale: " Shirish S
@ 2013-12-19 12:12 ` Shirish S
2013-12-19 13:19 ` Tomasz Figa
1 sibling, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-12-19 12:12 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, t.figa-Sze3O3UU22JBDgjK7y7TUQ
Cc: kgene.kim-Sze3O3UU22JBDgjK7y7TUQ,
s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ, shirish-F7+t8E8rja9g9hUCZPvPmw,
linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA, Shirish S
This patch adds dt support to hdmiphy config settings
as it is board specific and depends on the signal pattern
of board.
Signed-off-by: Shirish S <s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
---
.../devicetree/bindings/video/exynos_hdmi.txt | 34 ++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 89 ++++++++++++++++----
2 files changed, 105 insertions(+), 18 deletions(-)
diff --git a/Documentation/devicetree/bindings/video/exynos_hdmi.txt b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
index 323983b..0766e6e 100644
--- a/Documentation/devicetree/bindings/video/exynos_hdmi.txt
+++ b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
@@ -13,6 +13,31 @@ Required properties:
b) pin number within the gpio controller.
c) optional flags and pull up/down.
+Optional-but-recommended properties:
+- hdmiphy-configs: following information about the hdmiphy config settings.
+ a) "config<N>: config<N>" specifies the phy configuration settings,
+ where 'N' denotes the number of configuration, since every
+ pixel clock can have its unique configuration.
+ "samsung,pixel-clock" specifies the pixel clock
+ "samsung,de-emphasis-level" provides fine control of TMDS data
+ pre emphasis, below shown is example for
+ data de-emphasis register at address 0x145D0040.
+ hdmiphy@38[16] for bits[3:0] permitted values are in
+ the range of 760 mVdiff to 1400 mVdiff at 20mVdiff
+ increments for every LSB
+ hdmiphy@38[16] for bits[7:4] permitted values are in
+ the range of 0dB to -7.45dB at increments of -0.45dB
+ for every LSB.
+ "samsung,clock-level" provides fine control of TMDS data
+ amplitude for each channel,
+ for example if 0x145D005C is the address of clock level
+ register then,
+ hdmiphy@38[23] for bits [1:0] permitted values are in
+ the range of 0 mVdiff & 60 mVdiff for each channel at
+ increments 20 mVdiff of amplitude levels for every LSB,
+ hdmiphy@38[23] for bits [7:3] permitted values are in
+ the range of 790 and 1430 mV at 20mV increments for
+ every LSB.
Example:
hdmi {
@@ -20,4 +45,13 @@ Example:
reg = <0x14530000 0x100000>;
interrupts = <0 95 0>;
hpd-gpio = <&gpx3 7 1>;
+ hdmiphy-configs {
+ config0: config0 {
+ samsung,pixel-clock = <25200000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+
+ /* ... */
+ }
};
diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c
index a0e10ae..2fa0074 100644
--- a/drivers/gpu/drm/exynos/exynos_hdmi.c
+++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
@@ -200,6 +200,9 @@ struct hdmi_context {
struct hdmi_resources res;
+ struct hdmiphy_config *confs;
+ int nr_confs;
+
int hpd_gpio;
enum hdmi_type type;
@@ -259,7 +262,7 @@ static const struct hdmiphy_config hdmiphy_v13_configs[] = {
},
};
-static const struct hdmiphy_config hdmiphy_v14_configs[] = {
+static struct hdmiphy_config hdmiphy_v14_configs[] = {
{
.pixel_clock = 25200000,
.conf = {
@@ -771,20 +774,10 @@ static struct edid *hdmi_get_edid(void *ctx, struct drm_connector *connector)
static int hdmi_find_phy_conf(struct hdmi_context *hdata, u32 pixel_clock)
{
- const struct hdmiphy_config *confs;
- int count, i;
-
- if (hdata->type == HDMI_TYPE13) {
- confs = hdmiphy_v13_configs;
- count = ARRAY_SIZE(hdmiphy_v13_configs);
- } else if (hdata->type == HDMI_TYPE14) {
- confs = hdmiphy_v14_configs;
- count = ARRAY_SIZE(hdmiphy_v14_configs);
- } else
- return -EINVAL;
+ int i;
- for (i = 0; i < count; i++)
- if (confs[i].pixel_clock == pixel_clock)
+ for (i = 0; i < hdata->nr_confs; i++)
+ if (hdata->confs[i].pixel_clock == pixel_clock)
return i;
DRM_DEBUG_KMS("Could not find phy config for %d\n", pixel_clock);
@@ -1363,10 +1356,7 @@ static void hdmiphy_conf_apply(struct hdmi_context *hdata)
return;
}
- if (hdata->type == HDMI_TYPE13)
- hdmiphy_data = hdmiphy_v13_configs[i].conf;
- else
- hdmiphy_data = hdmiphy_v14_configs[i].conf;
+ hdmiphy_data = hdata->confs[i].conf;
memcpy(buffer, hdmiphy_data, 32);
ret = i2c_master_send(hdata->hdmiphy_port, buffer, 32);
@@ -1858,6 +1848,62 @@ void hdmi_attach_hdmiphy_client(struct i2c_client *hdmiphy)
hdmi_hdmiphy = hdmiphy;
}
+static int drm_hdmi_dt_parse_phy_conf(struct platform_device *pdev,
+ struct hdmi_context *hdata)
+{
+ struct device *dev = &pdev->dev;
+ struct device_node *dev_np = dev->of_node;
+ struct device_node *phy_conf, *cfg_np;
+ int i, pixel_clock = 0;
+
+ /* Initialize with default config */
+ switch (hdata->type) {
+ case HDMI_TYPE13:
+ hdata->confs = hdmiphy_v13_configs;
+ hdata->nr_confs = ARRAY_SIZE(hdmiphy_v13_configs);
+ break;
+ case HDMI_TYPE14:
+ hdata->confs = hdmiphy_v14_configs;
+ hdata->nr_confs = ARRAY_SIZE(hdmiphy_v14_configs);
+ break;
+ default:
+ DRM_ERROR("Did not find valid HDMI version\n");
+ break;
+ }
+
+ phy_conf = of_get_child_by_name(dev_np, "hdmiphy-configs");
+ if (phy_conf == NULL) {
+ DRM_ERROR("Did not find hdmiphy-configs node\n");
+ return -ENODEV;
+ }
+
+ for_each_child_of_node(phy_conf, cfg_np) {
+ if (of_property_read_u32(cfg_np, "samsung,pixel-clock",
+ &pixel_clock))
+ continue;
+
+ for (i = 0; i < ARRAY_SIZE(hdata->nr_confs); i++) {
+ if (hdata->confs[i].pixel_clock == pixel_clock)
+ /* Update the data de-emphasis and data level */
+ if (of_property_read_u8_array(cfg_np,
+ "samsung,config-de-emphasis-level",
+ &hdata->confs[i].conf[16], 1)) {
+ DRM_ERROR("Failed to get conf\n");
+ return -EINVAL;
+ }
+ /* Update the clock level diff */
+ if (of_property_read_u8_array(cfg_np,
+ "samsung,config-clock-level",
+ &hdata->confs[i].conf[23], 1)) {
+ DRM_ERROR("Failed to get conf\n");
+ return -EINVAL;
+ }
+ }
+ }
+ return 0;
+
+}
+
static struct s5p_hdmi_platform_data *drm_hdmi_dt_parse_pdata
(struct device *dev)
{
@@ -1986,6 +2032,13 @@ static int hdmi_probe(struct platform_device *pdev)
goto err_hdmiphy;
}
+ /* assign hdmiphy configurations */
+ ret = drm_hdmi_dt_parse_phy_conf(pdev, hdata);
+ if (ret) {
+ DRM_ERROR("failed to get user defined config,will use
+ default configs, eye diagram tests may fail\n");
+ }
+
/* Attach HDMI Driver to common hdmi. */
exynos_hdmi_drv_attach(drm_hdmi_ctx);
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings
2013-12-19 12:12 ` [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings Shirish S
@ 2013-12-19 13:19 ` Tomasz Figa
2014-01-08 6:04 ` Shirish S
0 siblings, 1 reply; 13+ messages in thread
From: Tomasz Figa @ 2013-12-19 13:19 UTC (permalink / raw)
To: Shirish S
Cc: dri-devel, inki.dae, devicetree, kgene.kim, s.nawrocki, shirish,
linux-samsung-soc
On Thursday 19 of December 2013 17:42:28 Shirish S wrote:
> This patch adds dt support to hdmiphy config settings
> as it is board specific and depends on the signal pattern
> of board.
>
> Signed-off-by: Shirish S <s.shirish@samsung.com>
> ---
> .../devicetree/bindings/video/exynos_hdmi.txt | 34 ++++++++
> drivers/gpu/drm/exynos/exynos_hdmi.c | 89 ++++++++++++++++----
> 2 files changed, 105 insertions(+), 18 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/video/exynos_hdmi.txt b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> index 323983b..0766e6e 100644
> --- a/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> +++ b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
> @@ -13,6 +13,31 @@ Required properties:
> b) pin number within the gpio controller.
> c) optional flags and pull up/down.
>
> +Optional-but-recommended properties:
> +- hdmiphy-configs: following information about the hdmiphy config settings.
> + a) "config<N>: config<N>" specifies the phy configuration settings,
Why do you need this "config<N>: " part? (This is called "label" in DT
terminology by the way and can be used to reference the node from
properties of other nodes, by so called "phandle".)
> + where 'N' denotes the number of configuration, since every
> + pixel clock can have its unique configuration.
> + "samsung,pixel-clock" specifies the pixel clock
> + "samsung,de-emphasis-level" provides fine control of TMDS data
> + pre emphasis, below shown is example for
> + data de-emphasis register at address 0x145D0040.
> + hdmiphy@38[16] for bits[3:0] permitted values are in
> + the range of 760 mVdiff to 1400 mVdiff at 20mVdiff
> + increments for every LSB
> + hdmiphy@38[16] for bits[7:4] permitted values are in
> + the range of 0dB to -7.45dB at increments of -0.45dB
> + for every LSB.
> + "samsung,clock-level" provides fine control of TMDS data
> + amplitude for each channel,
> + for example if 0x145D005C is the address of clock level
> + register then,
> + hdmiphy@38[23] for bits [1:0] permitted values are in
> + the range of 0 mVdiff & 60 mVdiff for each channel at
> + increments 20 mVdiff of amplitude levels for every LSB,
> + hdmiphy@38[23] for bits [7:3] permitted values are in
> + the range of 790 and 1430 mV at 20mV increments for
> + every LSB.
> Example:
>
> hdmi {
> @@ -20,4 +45,13 @@ Example:
> reg = <0x14530000 0x100000>;
> interrupts = <0 95 0>;
> hpd-gpio = <&gpx3 7 1>;
> + hdmiphy-configs {
> + config0: config0 {
> + samsung,pixel-clock = <25200000>;
> + samsung,de-emphasis-level = /bits/ 8 <0x26>;
nit: Two spaces before "/bits/".
> + samsung,clock-level = /bits/ 8 < 0x66>;
nit: Two spaces before "/bits/" and incorrect space after "<".
Generally the list of configurations should look like below:
phy-configs {
#address-cells = <1>;
#size-cells = <0>;
config@0 {
reg = <0>;
/* other properties... */
};
config@1 {
reg = <1>;
/* other properties... */
};
/* ... */
};
This is how bus-like structures should be represented in device tree.
Also, since this is HDMI node, maybe it's enough to call the node simply
phy-configs. Please rework the patches to use this correct representation.
> +
> + /* ... */
> + }
> };
[snip]
> + for_each_child_of_node(phy_conf, cfg_np) {
> + if (of_property_read_u32(cfg_np, "samsung,pixel-clock",
> + &pixel_clock))
> + continue;
> +
> + for (i = 0; i < ARRAY_SIZE(hdata->nr_confs); i++) {
> + if (hdata->confs[i].pixel_clock == pixel_clock)
Can you have more than one config with the same pixel clock?
Even if not, the code could be made more readable if the code
below is moved outside the if and continue keyword is used instead.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings
2013-12-19 13:19 ` Tomasz Figa
@ 2014-01-08 6:04 ` Shirish S
0 siblings, 0 replies; 13+ messages in thread
From: Shirish S @ 2014-01-08 6:04 UTC (permalink / raw)
To: Tomasz Figa
Cc: Shirish S, dri-devel, InKi Dae, devicetree@vger.kernel.org,
kgene.kim, Sylwester Nawrocki, linux-samsung-soc
Hi Tomasz,
Thanks for the review comments, please find my replies inline.
On Thu, Dec 19, 2013 at 6:49 PM, Tomasz Figa <t.figa@samsung.com> wrote:
> On Thursday 19 of December 2013 17:42:28 Shirish S wrote:
>> This patch adds dt support to hdmiphy config settings
>> as it is board specific and depends on the signal pattern
>> of board.
>>
>> Signed-off-by: Shirish S <s.shirish@samsung.com>
>> ---
>> .../devicetree/bindings/video/exynos_hdmi.txt | 34 ++++++++
>> drivers/gpu/drm/exynos/exynos_hdmi.c | 89 ++++++++++++++++----
>> 2 files changed, 105 insertions(+), 18 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/video/exynos_hdmi.txt b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
>> index 323983b..0766e6e 100644
>> --- a/Documentation/devicetree/bindings/video/exynos_hdmi.txt
>> +++ b/Documentation/devicetree/bindings/video/exynos_hdmi.txt
>> @@ -13,6 +13,31 @@ Required properties:
>> b) pin number within the gpio controller.
>> c) optional flags and pull up/down.
>>
>> +Optional-but-recommended properties:
>> +- hdmiphy-configs: following information about the hdmiphy config settings.
>> + a) "config<N>: config<N>" specifies the phy configuration settings,
>
> Why do you need this "config<N>: " part? (This is called "label" in DT
> terminology by the way and can be used to reference the node from
> properties of other nodes, by so called "phandle".)
>
The config is required for every pixel clock that the IP supports,
since in the parsing i iterate through all pixel clocks, i have used 'N'.
However, if you proposed approach below is ok, then all of this shall
be removed.
>> + where 'N' denotes the number of configuration, since every
>> + pixel clock can have its unique configuration.
>> + "samsung,pixel-clock" specifies the pixel clock
>> + "samsung,de-emphasis-level" provides fine control of TMDS data
>> + pre emphasis, below shown is example for
>> + data de-emphasis register at address 0x145D0040.
>> + hdmiphy@38[16] for bits[3:0] permitted values are in
>> + the range of 760 mVdiff to 1400 mVdiff at 20mVdiff
>> + increments for every LSB
>> + hdmiphy@38[16] for bits[7:4] permitted values are in
>> + the range of 0dB to -7.45dB at increments of -0.45dB
>> + for every LSB.
>> + "samsung,clock-level" provides fine control of TMDS data
>> + amplitude for each channel,
>> + for example if 0x145D005C is the address of clock level
>> + register then,
>> + hdmiphy@38[23] for bits [1:0] permitted values are in
>> + the range of 0 mVdiff & 60 mVdiff for each channel at
>> + increments 20 mVdiff of amplitude levels for every LSB,
>> + hdmiphy@38[23] for bits [7:3] permitted values are in
>> + the range of 790 and 1430 mV at 20mV increments for
>> + every LSB.
>> Example:
>>
>> hdmi {
>> @@ -20,4 +45,13 @@ Example:
>> reg = <0x14530000 0x100000>;
>> interrupts = <0 95 0>;
>> hpd-gpio = <&gpx3 7 1>;
>> + hdmiphy-configs {
>> + config0: config0 {
>> + samsung,pixel-clock = <25200000>;
>> + samsung,de-emphasis-level = /bits/ 8 <0x26>;
>
> nit: Two spaces before "/bits/".
have corrected in the next patchset.
>
>> + samsung,clock-level = /bits/ 8 < 0x66>;
>
> nit: Two spaces before "/bits/" and incorrect space after "<".
have corrected in the next patchset.
>
> Generally the list of configurations should look like below:
>
> phy-configs {
> #address-cells = <1>;
> #size-cells = <0>;
>
> config@0 {
> reg = <0>;
> /* other properties... */
> };
>
> config@1 {
> reg = <1>;
> /* other properties... */
> };
>
> /* ... */
> };
>
> This is how bus-like structures should be represented in device tree.
> Also, since this is HDMI node, maybe it's enough to call the node simply
> phy-configs. Please rework the patches to use this correct representation.
>
I think there is slight misunderstanding here, the configs that i want
to mention are not physical entities,and
hence dont think would be a better way, i am planning to use the below
method, if you think its the right approach then i shall do the rework
and submit the patch:
phy-configs {
#pixel-clock = <1>;
#de-emphasis-level = <1>;
#clock-level = <1>;
phy-map = <25200000 0x26 0x66>,
<27000000 0x26 0x66>,
/*....so on....*/,
};>> +
>> + /* ... */
>> + }
>> };
> [snip]
>> + for_each_child_of_node(phy_conf, cfg_np) {
>> + if (of_property_read_u32(cfg_np, "samsung,pixel-clock",
>> + &pixel_clock))
>> + continue;
>> +
>> + for (i = 0; i < ARRAY_SIZE(hdata->nr_confs); i++) {
>> + if (hdata->confs[i].pixel_clock == pixel_clock)
>
> Can you have more than one config with the same pixel clock?
>
No, right now i intend to have one config parameters for every
corresponding pixel clock.
> Even if not, the code could be made more readable if the code
> below is moved outside the if and continue keyword is used instead.
>
The parsing mechanism shall be altered according to the new approach
proposed above.
Kindly let me know your thoughts about the approach.
> Best regards,
> Tomasz
>
Thanks & Regards,
Shirish S
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings
2013-12-19 12:12 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
2013-12-19 12:12 ` [PATCH 1/4] ARM: dts: smdk5250: Add hdmi phy settings Shirish S
[not found] ` <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
@ 2013-12-19 12:12 ` Shirish S
2 siblings, 0 replies; 13+ messages in thread
From: Shirish S @ 2013-12-19 12:12 UTC (permalink / raw)
To: dri-devel, inki.dae, devicetree, t.figa
Cc: kgene.kim, s.nawrocki, shirish, linux-samsung-soc, Shirish S
This patch moves the hdmi phy setting to arndale dts,
as its more of a per board configuration and also
shall be easier for supporting future chipsets.
Signed-off-by: Shirish S <s.shirish@samsung.com>
---
arch/arm/boot/dts/cros5250-common.dtsi | 74 ++++++++++++++++++++++++++++++++
1 file changed, 74 insertions(+)
diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
index dc259e8b..45c8583 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -301,6 +301,80 @@
hdmi {
hpd-gpio = <&gpx3 7 0>;
+ hdmiphy-configs {
+ /*
+ * Eye diagram test passed for:
+ * Data de-emphasis: -0.7dB & Data Level: 880mV
+ * i.e., 0010 0110 = 0x26
+ * and Clock level of 515mV and diff 1030mV
+ * i.e., 0x66
+ */
+ config0: config0 {
+ samsung,pixel-clock = <25200000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config1: config1 {
+ samsung,pixel-clock = <27000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config2: config2 {
+ samsung,pixel-clock = <27027000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config3: config3 {
+ samsung,pixel-clock = <36000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config4: config4 {
+ samsung,pixel-clock = <40000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config5: config5 {
+ samsung,pixel-clock = <65000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config6: config6 {
+ samsung,pixel-clock = <74176000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config7: config7 {
+ samsung,pixel-clock = <74250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config8: config8 {
+ samsung,pixel-clock = <83500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config9: config9 {
+ samsung,pixel-clock = <106500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config10: config10 {
+ samsung,pixel-clock = <108000000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config11: config11 {
+ samsung,pixel-clock = <146250000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ config12: config12 {
+ samsung,pixel-clock = <148500000>;
+ samsung,de-emphasis-level = /bits/ 8 <0x26>;
+ samsung,clock-level = /bits/ 8 < 0x66>;
+ };
+ };
};
gpio-keys {
--
1.7.9.5
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-12-04 4:44 Shirish S
[not found] ` <1386132300-29558-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-12-04 4:44 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, t.figa-Sze3O3UU22JBDgjK7y7TUQ
Cc: kgene.kim-Sze3O3UU22JBDgjK7y7TUQ,
s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ, shirish-F7+t8E8rja9g9hUCZPvPmw,
Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
V6:
V7: removed nr-configs form the dtsi files.
V8: Fixed build error
V9: rebased and incorporated Tomasz comments.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 34 ++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 74 ++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 74 ++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 ++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 89 ++++++++++++++++----
5 files changed, 327 insertions(+), 18 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-11-26 9:57 Shirish S
[not found] ` <1385459858-3972-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-11-26 9:57 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8
Cc: airlied-H+wXaHxf7aLQT0dZR+AlfA, shirish-F7+t8E8rja9g9hUCZPvPmw,
kgene.kim-Sze3O3UU22JBDgjK7y7TUQ, t.figa-Sze3O3UU22JBDgjK7y7TUQ,
s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ, Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
V6:
V7: removed nr-configs form the dtsi files.
V8: Fixed build error
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 31 ++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 +++++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 77 +++++++++++++++++++-
5 files changed, 326 insertions(+), 4 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-11-25 8:54 Shirish S
[not found] ` <1385369679-4337-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-11-25 8:54 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8
Cc: airlied-H+wXaHxf7aLQT0dZR+AlfA, shirish-F7+t8E8rja9g9hUCZPvPmw,
Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
V6:
V7: removed nr-configs form the dtsi files.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 31 ++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 +++++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 77 +++++++++++++++++++-
5 files changed, 326 insertions(+), 4 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-11-18 8:38 Shirish S
[not found] ` <1384763916-31868-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-11-18 8:38 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8
Cc: seanpaul-F7+t8E8rja9g9hUCZPvPmw,
sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ, shirish-F7+t8E8rja9g9hUCZPvPmw,
Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
V6: removed nr-configs form the dtsi files.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 33 +++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 74 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 74 +++++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 77 +++++++++++++++++++-
5 files changed, 328 insertions(+), 4 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-11-18 6:35 Shirish S
[not found] ` <1384756563-17097-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-11-18 6:35 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8
Cc: seanpaul-F7+t8E8rja9g9hUCZPvPmw,
sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ, shirish-F7+t8E8rja9g9hUCZPvPmw,
Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
V5: removed nr-configs feild and also the constraint
of having the exact number of configs in the dt file
as in the driver, the programmer can add only the pixel
clock that needs to be updated.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 33 +++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 75 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 75 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 75 +++++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 77 +++++++++++++++++++-
5 files changed, 331 insertions(+), 4 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread* [PATCH 0/4] Add dt support for exynos hdmiphy settings
@ 2013-10-29 8:12 Shirish S
[not found] ` <1383034352-15494-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Shirish S @ 2013-10-29 8:12 UTC (permalink / raw)
To: dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
inki.dae-Sze3O3UU22JBDgjK7y7TUQ,
devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: seanpaul-F7+t8E8rja9g9hUCZPvPmw,
sw0312.kim-Sze3O3UU22JBDgjK7y7TUQ, mark.rutland-5wv7dgnIgG8,
shirish-F7+t8E8rja9g9hUCZPvPmw, Shirish S
For various revisions of a chipset if the signal pattern is changed for every
revision, then the phy setting need to be updated correspondingly by measuring
the signal.
For getting correct signals the clock level and data de-emphasis
levels needs to be adjusted.
Since only these 2 values matter,we can move the same to dt,
wherein we can have different dt files for every revision.
This is an initial patchset towards achieving the same
for exynos 5250 and can be later extended to future chipsets.
V2: replaced moving of entire phy config structure with only
required and justifiable conf registers.
V3: Incorporated Mark Rutland's comments.
V4: Rebased and included cros5250-common.dtsi.
Shirish S (4):
ARM: dts: smdk5250: Add hdmi phy settings
ARM: dts: arndale: Add hdmi phy settings
ARM: exynos: dts: cros5250: Add hdmi phy settings
drm: exynos: hdmi: Add dt support for hdmiphy settings
.../devicetree/bindings/video/exynos_hdmi.txt | 34 +++++++++
arch/arm/boot/dts/cros5250-common.dtsi | 75 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-arndale.dts | 75 +++++++++++++++++++
arch/arm/boot/dts/exynos5250-smdk5250.dts | 75 +++++++++++++++++++
drivers/gpu/drm/exynos/exynos_hdmi.c | 79 +++++++++++++++++++-
5 files changed, 334 insertions(+), 4 deletions(-)
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2014-01-08 6:04 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-19 12:12 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
2013-12-19 12:12 ` [PATCH 1/4] ARM: dts: smdk5250: Add hdmi phy settings Shirish S
[not found] ` <1387455148-22999-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-19 12:12 ` [PATCH 2/4] ARM: dts: arndale: " Shirish S
2013-12-19 12:12 ` [PATCH 4/4] drm: exynos: hdmi: Add dt support for hdmiphy settings Shirish S
2013-12-19 13:19 ` Tomasz Figa
2014-01-08 6:04 ` Shirish S
2013-12-19 12:12 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
-- strict thread matches above, loose matches on Subject: below --
2013-12-04 4:44 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1386132300-29558-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-04 4:44 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2013-11-26 9:57 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1385459858-3972-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-11-26 9:57 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2013-11-25 8:54 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1385369679-4337-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-11-25 8:54 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2013-11-18 8:38 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1384763916-31868-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-11-18 8:38 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2013-11-18 6:35 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1384756563-17097-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-11-18 6:36 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
2013-10-29 8:12 [PATCH 0/4] Add dt support for exynos hdmiphy settings Shirish S
[not found] ` <1383034352-15494-1-git-send-email-s.shirish-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-10-29 8:12 ` [PATCH 3/4] ARM: exynos: dts: cros5250: Add hdmi phy settings Shirish S
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).