All of lore.kernel.org
 help / color / mirror / Atom feed
From: Brian Masney <masneyb@onstation.org>
To: bjorn.andersson@linaro.org, robh+dt@kernel.org,
	agross@kernel.org, a.hajda@samsung.com, narmstrong@baylibre.com,
	robdclark@gmail.com, sean@poorly.run
Cc: airlied@linux.ie, daniel@ffwll.ch, mark.rutland@arm.com,
	Laurent.pinchart@ideasonboard.com, jonas@kwiboo.se,
	jernej.skrabec@siol.net, linus.walleij@linaro.org,
	enric.balletbo@collabora.com, dri-devel@lists.freedesktop.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-msm@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	freedreno@lists.freedesktop.org
Subject: [PATCH 05/11] drm/bridge: analogix-anx78xx: correct value of TX_P0
Date: Wed, 14 Aug 2019 20:48:48 -0400	[thread overview]
Message-ID: <20190815004854.19860-6-masneyb@onstation.org> (raw)
In-Reply-To: <20190815004854.19860-1-masneyb@onstation.org>

When attempting to configure this driver on a Nexus 5 phone (msm8974),
setting up the dummy i2c bus for TX_P0 would fail due to an -EBUSY
error. The downstream MSM kernel sources [1] shows that the proper value
for TX_P0 is 0x78, not 0x70, so correct the value to allow device
probing to succeed.

[1] https://github.com/AICP/kernel_lge_hammerhead/blob/n7.1/drivers/video/slimport/slimport_tx_reg.h

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
 drivers/gpu/drm/bridge/analogix-anx78xx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/analogix-anx78xx.h b/drivers/gpu/drm/bridge/analogix-anx78xx.h
index 25e063bcecbc..bc511fc605c9 100644
--- a/drivers/gpu/drm/bridge/analogix-anx78xx.h
+++ b/drivers/gpu/drm/bridge/analogix-anx78xx.h
@@ -6,7 +6,7 @@
 #ifndef __ANX78xx_H
 #define __ANX78xx_H
 
-#define TX_P0				0x70
+#define TX_P0				0x78
 #define TX_P1				0x7a
 #define TX_P2				0x72
 
-- 
2.21.0


WARNING: multiple messages have this Message-ID (diff)
From: Brian Masney <masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
To: bjorn.andersson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	agross-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	a.hajda-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org,
	narmstrong-rdvid1DuHRBWk0Htik3J/w@public.gmane.org,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	sean-p7yTbzM4H96eqtR555YLDQ@public.gmane.org
Cc: mark.rutland-5wv7dgnIgG8@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	jernej.skrabec-gGgVlfcn5nU@public.gmane.org,
	jonas-uIzNG4q0ceqzQB+pC5nmwQ@public.gmane.org,
	airlied-cv59FeDIM0c@public.gmane.org,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	Laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org,
	daniel-/w4YWyX8dFk@public.gmane.org,
	enric.balletbo-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: [PATCH 05/11] drm/bridge: analogix-anx78xx: correct value of TX_P0
Date: Wed, 14 Aug 2019 20:48:48 -0400	[thread overview]
Message-ID: <20190815004854.19860-6-masneyb@onstation.org> (raw)
In-Reply-To: <20190815004854.19860-1-masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>

When attempting to configure this driver on a Nexus 5 phone (msm8974),
setting up the dummy i2c bus for TX_P0 would fail due to an -EBUSY
error. The downstream MSM kernel sources [1] shows that the proper value
for TX_P0 is 0x78, not 0x70, so correct the value to allow device
probing to succeed.

[1] https://github.com/AICP/kernel_lge_hammerhead/blob/n7.1/drivers/video/slimport/slimport_tx_reg.h

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
 drivers/gpu/drm/bridge/analogix-anx78xx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/analogix-anx78xx.h b/drivers/gpu/drm/bridge/analogix-anx78xx.h
index 25e063bcecbc..bc511fc605c9 100644
--- a/drivers/gpu/drm/bridge/analogix-anx78xx.h
+++ b/drivers/gpu/drm/bridge/analogix-anx78xx.h
@@ -6,7 +6,7 @@
 #ifndef __ANX78xx_H
 #define __ANX78xx_H
 
-#define TX_P0				0x70
+#define TX_P0				0x78
 #define TX_P1				0x7a
 #define TX_P2				0x72
 
-- 
2.21.0

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

WARNING: multiple messages have this Message-ID (diff)
From: Brian Masney <masneyb@onstation.org>
To: bjorn.andersson@linaro.org, robh+dt@kernel.org,
	agross@kernel.org, a.hajda@samsung.com, narmstrong@baylibre.com,
	robdclark@gmail.com, sean@poorly.run
Cc: mark.rutland@arm.com, devicetree@vger.kernel.org,
	jernej.skrabec@siol.net, jonas@kwiboo.se, airlied@linux.ie,
	linux-arm-msm@vger.kernel.org, linus.walleij@linaro.org,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Laurent.pinchart@ideasonboard.com, daniel@ffwll.ch,
	enric.balletbo@collabora.com, freedreno@lists.freedesktop.org,
	linux-arm-kernel@lists.infradead.org
Subject: [PATCH 05/11] drm/bridge: analogix-anx78xx: correct value of TX_P0
Date: Wed, 14 Aug 2019 20:48:48 -0400	[thread overview]
Message-ID: <20190815004854.19860-6-masneyb@onstation.org> (raw)
In-Reply-To: <20190815004854.19860-1-masneyb@onstation.org>

When attempting to configure this driver on a Nexus 5 phone (msm8974),
setting up the dummy i2c bus for TX_P0 would fail due to an -EBUSY
error. The downstream MSM kernel sources [1] shows that the proper value
for TX_P0 is 0x78, not 0x70, so correct the value to allow device
probing to succeed.

[1] https://github.com/AICP/kernel_lge_hammerhead/blob/n7.1/drivers/video/slimport/slimport_tx_reg.h

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
 drivers/gpu/drm/bridge/analogix-anx78xx.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/analogix-anx78xx.h b/drivers/gpu/drm/bridge/analogix-anx78xx.h
index 25e063bcecbc..bc511fc605c9 100644
--- a/drivers/gpu/drm/bridge/analogix-anx78xx.h
+++ b/drivers/gpu/drm/bridge/analogix-anx78xx.h
@@ -6,7 +6,7 @@
 #ifndef __ANX78xx_H
 #define __ANX78xx_H
 
-#define TX_P0				0x70
+#define TX_P0				0x78
 #define TX_P1				0x7a
 #define TX_P2				0x72
 
-- 
2.21.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2019-08-15  0:49 UTC|newest]

Thread overview: 100+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20190815004916epcas3p4d8a62e215eff5e227721d3449e6bfbd3@epcas3p4.samsung.com>
2019-08-15  0:48 ` [PATCH 00/11] ARM: dts: qcom: msm8974: add support for external display Brian Masney
2019-08-15  0:48   ` Brian Masney
2019-08-15  0:48   ` [PATCH 01/11] dt-bindings: drm/bridge: analogix-anx78xx: add new variants Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:17     ` Linus Walleij
2019-08-15  8:17       ` Linus Walleij
2019-08-15  8:17       ` Linus Walleij
2019-08-27 16:20     ` Rob Herring
2019-08-27 16:20       ` Rob Herring
2019-08-15  0:48   ` [PATCH 02/11] " Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:17     ` Linus Walleij
2019-08-15  8:17       ` Linus Walleij
2019-08-15  8:17       ` Linus Walleij
2019-08-15  0:48   ` [PATCH 03/11] drm/bridge: analogix-anx78xx: silence -EPROBE_DEFER warnings Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:35     ` Linus Walleij
2019-08-15  8:35       ` Linus Walleij
2019-08-15  8:35       ` Linus Walleij
2019-08-15  0:48   ` [PATCH 04/11] drm/bridge: analogix-anx78xx: convert to i2c_new_dummy_device Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:18     ` Linus Walleij
2019-08-15  8:18       ` Linus Walleij
2019-08-15  8:18       ` Linus Walleij
2019-08-15  0:48   ` Brian Masney [this message]
2019-08-15  0:48     ` [PATCH 05/11] drm/bridge: analogix-anx78xx: correct value of TX_P0 Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:24     ` Linus Walleij
2019-08-15  8:24       ` Linus Walleij
2019-08-15  8:24       ` Linus Walleij
2019-09-16 10:02     ` Andrzej Hajda
2019-09-16 10:02       ` Andrzej Hajda
2019-09-16 10:36       ` Brian Masney
2019-09-16 10:36         ` Brian Masney
2019-09-16 10:49         ` Laurent Pinchart
2019-09-16 10:49           ` Laurent Pinchart
2019-09-16 11:32           ` Enric Balletbo i Serra
2019-09-16 11:32             ` Enric Balletbo i Serra
2019-09-16 12:02             ` Brian Masney
2019-09-16 12:02               ` Brian Masney
2019-09-16 12:25               ` Andrzej Hajda
2019-09-16 12:25                 ` Andrzej Hajda
2019-09-17  7:28               ` Andrzej Hajda
2019-09-17  7:28                 ` Andrzej Hajda
2019-09-16 10:36       ` Enric Balletbo i Serra
2019-09-16 10:36         ` Enric Balletbo i Serra
2019-09-16 10:40         ` Brian Masney
2019-09-16 10:40           ` Brian Masney
2019-08-15  0:48   ` [PATCH RFC 06/11] drm/bridge: analogix-anx78xx: add support for avdd33 regulator Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:22     ` Linus Walleij
2019-08-15  8:22       ` Linus Walleij
2019-08-15  8:22       ` Linus Walleij
2019-08-15 22:51       ` Brian Masney
2019-08-15 22:51         ` Brian Masney
2019-08-15 22:51         ` Brian Masney
2019-08-15  0:48   ` [PATCH 07/11] ARM: qcom_defconfig: add CONFIG_DRM_ANALOGIX_ANX78XX Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:23     ` Linus Walleij
2019-08-15  8:23       ` Linus Walleij
2019-08-15  8:23       ` Linus Walleij
2019-08-15  0:48   ` [PATCH 08/11] drm/msm/hdmi: silence -EPROBE_DEFER warning Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:35     ` Linus Walleij
2019-08-15  8:35       ` Linus Walleij
2019-08-15  8:35       ` Linus Walleij
2019-08-15  0:48   ` [PATCH 09/11] ARM: dts: qcom: pm8941: add 5vs2 regulator node Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:34     ` Linus Walleij
2019-08-15  8:34       ` Linus Walleij
2019-08-15  8:34       ` Linus Walleij
2019-08-15 22:44       ` Brian Masney
2019-08-15 22:44         ` Brian Masney
2019-08-15 22:44         ` Brian Masney
2019-08-15  0:48   ` [PATCH RFC 10/11] ARM: dts: qcom: msm8974: add HDMI nodes Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:25     ` Linus Walleij
2019-08-15  8:25       ` Linus Walleij
2019-08-15  8:25       ` Linus Walleij
2019-08-15  0:48   ` [PATCH RFC 11/11] ARM: dts: qcom: msm8974-hammerhead: add support for external display Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  0:48     ` Brian Masney
2019-08-15  8:31     ` Linus Walleij
2019-08-15  8:31       ` Linus Walleij
2019-08-15  8:31       ` Linus Walleij
2019-09-16  8:13   ` [PATCH 00/11] ARM: dts: qcom: msm8974: " Andrzej Hajda
2019-09-16  8:13     ` Andrzej Hajda
2019-09-16  9:01     ` Brian Masney
2019-09-16  9:01       ` Brian Masney
2019-09-16 12:02       ` Andrzej Hajda
2019-09-16 12:02         ` Andrzej Hajda
2019-09-16 12:02         ` Andrzej Hajda

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190815004854.19860-6-masneyb@onstation.org \
    --to=masneyb@onstation.org \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=a.hajda@samsung.com \
    --cc=agross@kernel.org \
    --cc=airlied@linux.ie \
    --cc=bjorn.andersson@linaro.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=enric.balletbo@collabora.com \
    --cc=freedreno@lists.freedesktop.org \
    --cc=jernej.skrabec@siol.net \
    --cc=jonas@kwiboo.se \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=narmstrong@baylibre.com \
    --cc=robdclark@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=sean@poorly.run \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.