Message ID | 20190815004854.19860-1-masneyb@onstation.org (mailing list archive) |
---|---|
Headers | show |
Series | ARM: dts: qcom: msm8974: add support for external display | expand |
On Thu, Aug 15, 2019 at 2:49 AM Brian Masney <masneyb@onstation.org> wrote: > Silence two warning messages that occur due to -EPROBE_DEFER errors to > help cleanup the system boot log. > > Signed-off-by: Brian Masney <masneyb@onstation.org> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
Hi Brian, On 15.08.2019 02:48, Brian Masney wrote: > This patch series begins to add support for the external display over > HDMI that is supported on msm8974 SoCs. I'm testing this series on the > Nexus 5, and I'm able to communicate with the HDMI bridge via the > analogix-anx78xx driver, however the external display is not working > yet. > > When I plug in the HDMI cable, the monitor detects that a device is > hooked up, but nothing is shown on the external monitor. The hot plug > detect GPIO (hpd-gpios) on the analogix-anx78xx bridge and MSM HDMI > drivers do not change state when the slimport adapter or HDMI cable is > plugged in or removed. I wonder if a regulator is not enabled somewhere? > I have a comment in patch 10 regarding 'hpd-gdsc-supply' that may > potentially be an issue. > > I'm still digging in on this, however I'd appreciate any feedback if > anyone has time. Most of these patches are ready now, so I marked the > ones that aren't ready with 'PATCH RFC'. > > I'm using an Analogix Semiconductor SP6001 SlimPort Micro-USB to 4K HDMI > Adapter to connect my phone to an external display via a standard HDMI > cable. This works just fine with the downstream MSM kernel using > Android. This patchset risks to be forgotten. To avoid it, at least partially, I can merge patches 1-5, is it OK for you? Regards Andrzej > > Brian Masney (11): > dt-bindings: drm/bridge: analogix-anx78xx: add new variants > drm/bridge: analogix-anx78xx: add new variants > drm/bridge: analogix-anx78xx: silence -EPROBE_DEFER warnings > drm/bridge: analogix-anx78xx: convert to i2c_new_dummy_device > drm/bridge: analogix-anx78xx: correct value of TX_P0 > drm/bridge: analogix-anx78xx: add support for avdd33 regulator > ARM: qcom_defconfig: add CONFIG_DRM_ANALOGIX_ANX78XX > drm/msm/hdmi: silence -EPROBE_DEFER warning > ARM: dts: qcom: pm8941: add 5vs2 regulator node > ARM: dts: qcom: msm8974: add HDMI nodes > ARM: dts: qcom: msm8974-hammerhead: add support for external display > > .../bindings/display/bridge/anx7814.txt | 6 +- > .../qcom-msm8974-lge-nexus5-hammerhead.dts | 140 ++++++++++++++++++ > arch/arm/boot/dts/qcom-msm8974.dtsi | 80 ++++++++++ > arch/arm/boot/dts/qcom-pm8941.dtsi | 10 ++ > arch/arm/configs/qcom_defconfig | 1 + > drivers/gpu/drm/bridge/analogix-anx78xx.c | 60 +++++++- > drivers/gpu/drm/bridge/analogix-anx78xx.h | 2 +- > drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 8 +- > 8 files changed, 295 insertions(+), 12 deletions(-) >
Hi Andrzej, On Mon, Sep 16, 2019 at 10:13:58AM +0200, Andrzej Hajda wrote: > Hi Brian, > > On 15.08.2019 02:48, Brian Masney wrote: > > This patch series begins to add support for the external display over > > HDMI that is supported on msm8974 SoCs. I'm testing this series on the > > Nexus 5, and I'm able to communicate with the HDMI bridge via the > > analogix-anx78xx driver, however the external display is not working > > yet. > > > > When I plug in the HDMI cable, the monitor detects that a device is > > hooked up, but nothing is shown on the external monitor. The hot plug > > detect GPIO (hpd-gpios) on the analogix-anx78xx bridge and MSM HDMI > > drivers do not change state when the slimport adapter or HDMI cable is > > plugged in or removed. I wonder if a regulator is not enabled somewhere? > > I have a comment in patch 10 regarding 'hpd-gdsc-supply' that may > > potentially be an issue. > > > > I'm still digging in on this, however I'd appreciate any feedback if > > anyone has time. Most of these patches are ready now, so I marked the > > ones that aren't ready with 'PATCH RFC'. > > > > I'm using an Analogix Semiconductor SP6001 SlimPort Micro-USB to 4K HDMI > > Adapter to connect my phone to an external display via a standard HDMI > > cable. This works just fine with the downstream MSM kernel using > > Android. > > > This patchset risks to be forgotten. To avoid it, at least partially, I > can merge patches 1-5, is it OK for you? That would be great if you could do that. Thanks, Brian
On 16.09.2019 11:01, Brian Masney wrote: > Hi Andrzej, > > On Mon, Sep 16, 2019 at 10:13:58AM +0200, Andrzej Hajda wrote: >> Hi Brian, >> >> On 15.08.2019 02:48, Brian Masney wrote: >>> This patch series begins to add support for the external display over >>> HDMI that is supported on msm8974 SoCs. I'm testing this series on the >>> Nexus 5, and I'm able to communicate with the HDMI bridge via the >>> analogix-anx78xx driver, however the external display is not working >>> yet. >>> >>> When I plug in the HDMI cable, the monitor detects that a device is >>> hooked up, but nothing is shown on the external monitor. The hot plug >>> detect GPIO (hpd-gpios) on the analogix-anx78xx bridge and MSM HDMI >>> drivers do not change state when the slimport adapter or HDMI cable is >>> plugged in or removed. I wonder if a regulator is not enabled somewhere? >>> I have a comment in patch 10 regarding 'hpd-gdsc-supply' that may >>> potentially be an issue. >>> >>> I'm still digging in on this, however I'd appreciate any feedback if >>> anyone has time. Most of these patches are ready now, so I marked the >>> ones that aren't ready with 'PATCH RFC'. >>> >>> I'm using an Analogix Semiconductor SP6001 SlimPort Micro-USB to 4K HDMI >>> Adapter to connect my phone to an external display via a standard HDMI >>> cable. This works just fine with the downstream MSM kernel using >>> Android. >> >> This patchset risks to be forgotten. To avoid it, at least partially, I >> can merge patches 1-5, is it OK for you? > That would be great if you could do that. I have queued 1-4 to drm-misc-next. 5th patch requires some discussion. Regards Andrzej > > Thanks, > > Brian > >