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 <[email protected]> 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 <[email protected]> > > --- > > > > 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); -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/3289785.2NrdWooNo1%40jernej-laptop.
