[PATCH 06/19] video: sunxi: Use DW-HDMI hpd function

["Jernej Škrabec" <jernej.skrabec@siol.net>]

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);