Dne četrtek, 04. marec 2021 ob 02:42:08 CET je Andre Przywara napisal(a):
On Tue, 23 Feb 2021 21:46:18 +0100
Jernej Skrabec jernej.skrabec@siol.net wrote:
It turns out that even though A64, H3 and H5 have custom PHY, standard hot plug detection for DW-HDMI works just fine.
But the generic code looks for a different bit in a different register, doesn't it?
Yes.
I see SUNXI_HDMI_BASE + 0x10000 + 56, bit 19 before vs. SUNXI_HDMI_BASE + 0x3004, bit 2 in the new code.
Is that duplicated functionality in the AW PHY?
Yes.
Linux driver uses standard way and it works just fine, so there is no real reason not to use standard way here too.
Best regards, Jernej
Cheers, Andre
Remove custom hpd method to reduce amount of custom code.
Signed-off-by: Jernej Skrabec jernej.skrabec@siol.net
drivers/video/sunxi/sunxi_dw_hdmi.c | 34 +++++------------------------ 1 file changed, 6 insertions(+), 28 deletions(-)
diff --git a/drivers/video/sunxi/sunxi_dw_hdmi.c b/drivers/video/sunxi/sunxi_dw_hdmi.c index e3811a2ec15f..37e78ff24111 100644 --- a/drivers/video/sunxi/sunxi_dw_hdmi.c +++ b/drivers/video/sunxi/sunxi_dw_hdmi.c @@ -114,28 +114,6 @@ static void sunxi_dw_hdmi_phy_init(void)
writel(0x42494E47, &phy->unscramble);
}
-static int sunxi_dw_hdmi_get_plug_in_status(void) -{
- struct sunxi_hdmi_phy * const phy =
(struct sunxi_hdmi_phy *)(SUNXI_HDMI_BASE +
HDMI_PHY_OFFS);
- return !!(readl(&phy->status) & (1 << 19));
-}
-static int sunxi_dw_hdmi_wait_for_hpd(void) -{
- ulong start;
- start = get_timer(0);
- do {
if (sunxi_dw_hdmi_get_plug_in_status())return 0;udelay(100);- } while (get_timer(start) < 300);
- return -1;
-}
static void sunxi_dw_hdmi_phy_set(uint clock, int phy_div) {
struct sunxi_hdmi_phy * const phy =
@@ -370,12 +348,6 @@ static int sunxi_dw_hdmi_probe(struct udevice *dev)
sunxi_dw_hdmi_phy_init();
ret = sunxi_dw_hdmi_wait_for_hpd();
if (ret < 0) {
debug("hdmi can not get hpd signal\n");return -1;}
priv->hdmi.ioaddr = SUNXI_HDMI_BASE; priv->hdmi.i2c_clk_high = 0xd8; priv->hdmi.i2c_clk_low = 0xfe;
@@ -383,6 +355,12 @@ static int sunxi_dw_hdmi_probe(struct udevice *dev)
priv->hdmi.phy_set = sunxi_dw_hdmi_phy_cfg; priv->mux = uc_plat->source_id;
ret = dw_hdmi_phy_wait_for_hpd(&priv->hdmi);
if (ret < 0) {
debug("hdmi can not get hpd signal\n");return -1;}
uclass_get_device_by_phandle(UCLASS_I2C, dev, "ddc-i2c-bus",
&priv->hdmi.ddc_bus);