Unverified Commit 8c1bfd0c authored by Sandor Yu's avatar Sandor Yu Committed by Robert Foss
Browse files

drm: bridge: dw_hdmi: default enable workaround to clear the overflow 



i.MX8MPlus (v2.13a) has verified need the workaround to clear the
overflow with one iteration.
Only i.MX6Q(v1.30a) need the workaround with 4 iterations,
the others versions later than v1.3a have been identified as needing
the workaround with a single iteration.

Default enable the workaround with one iteration for all versions
later than v1.30a.

Signed-off-by: default avatarSandor Yu <Sandor.yu@nxp.com>
Acked-by: default avatarNeil Armstrong <narmstrong@baylibre.com>
Signed-off-by: default avatarRobert Foss <robert.foss@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/561951005a85574dcdd108e5d6a3a87df930ea3d.1649989179.git.Sandor.yu@nxp.com
parent 34263c1b

drivers/gpu/drm/bridge/synopsys/dw-hdmi.c

+7 −16
Original line number Diff line number Diff line
@@ -2086,30 +2086,21 @@ static void dw_hdmi_clear_overflow(struct dw_hdmi *hdmi) 
	 * then write one of the FC registers several times.

	 *

	 * The number of iterations matters and depends on the HDMI TX revision

	 * (and possibly on the platform). So far i.MX6Q (v1.30a), i.MX6DL

	 * (v1.31a) and multiple Allwinner SoCs (v1.32a) have been identified

	 * as needing the workaround, with 4 iterations for v1.30a and 1

	 * iteration for others.

	 * The Amlogic Meson GX SoCs (v2.01a) have been identified as needing

	 * the workaround with a single iteration.

	 * The Rockchip RK3288 SoC (v2.00a) and RK3328/RK3399 SoCs (v2.11a) have

	 * been identified as needing the workaround with a single iteration.

	 * (and possibly on the platform).

	 * 4 iterations for i.MX6Q(v1.30a) and 1 iteration for others.

	 * i.MX6DL (v1.31a), Allwinner SoCs (v1.32a), Rockchip RK3288 SoC (v2.00a),

	 * Amlogic Meson GX SoCs (v2.01a), RK3328/RK3399 SoCs (v2.11a)

	 * and i.MX8MPlus (v2.13a) have been identified as needing the workaround

	 * with a single iteration.

	 */



	switch (hdmi->version) {

	case 0x130a:

		count = 4;

		break;

	case 0x131a:

	case 0x132a:

	case 0x200a:

	case 0x201a:

	case 0x211a:

	case 0x212a:

	default:

		count = 1;

		break;

	default:

		return;

	}



	/* TMDS software reset */