Lucas Stach <l.stach@pengutronix.de> writes:

> From: Christian Gmeiner <christian.gmeiner@gmail.com>

> +static bool etnaviv_validate_load_state(struct etnaviv_gpu *gpu, u32 *buf,
> +	unsigned int state, unsigned int num)
> +{
> +	return true;
> +	if (0x1200 - state < num * 4)
> +		return false;
> +	if (0x1228 - state < num * 4)
> +		return false;
> +	if (0x1238 - state < num * 4)
> +		return false;
> +	if (0x1284 - state < num * 4)
> +		return false;
> +	if (0x128c - state < num * 4)
> +		return false;
> +	if (0x1304 - state < num * 4)
> +		return false;
> +	if (0x1310 - state < num * 4)
> +		return false;
> +	if (0x1318 - state < num * 4)
> +		return false;
> +	if (0x1280c - state < num * 4 + 0x0c)
> +		return false;
> +	if (0x128ac - state < num * 4 + 0x0c)
> +		return false;
> +	if (0x128cc - state < num * 4 + 0x0c)
> +		return false;
> +	if (0x1297c - state < num * 4 + 0x0c)
> +		return false;
> +	return true;
> +}

I was browsing the code, and noticed that it looks like you've got a
debugging early return in your validation function here.