[Letux-kernel] [PATCH] omap drm: fix compile errors where the conversion from omap_timing to videomode was not perfect

Peter Ujfalusi peter.ujfalusi at ti.com
Mon Jan 2 07:40:20 CET 2017


Hi,

On 12/26/2016 09:23 PM, H. Nikolaus Schaller wrote:
> Signed-off-by: H. Nikolaus Schaller <hns at goldelico.com>

I did compile and boot tests after each patch when doing the conversion,
but this certainly skipped because these were 'hidden' by
#ifdef PRINT_VERBOSE_VM_TIMINGS

which is not a Kconfig option and needs to be enabled in the dsi.c on
purpose.

Thank you for finding it and sorry for the inconvenience.

Acked-by: Peter Ujfalusi <peter.ujfalusi at ti.com>

> ---
>  drivers/gpu/drm/omapdrm/dss/dsi.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/omapdrm/dss/dsi.c
b/drivers/gpu/drm/omapdrm/dss/dsi.c
> index f060bda..f74615d 100644
> --- a/drivers/gpu/drm/omapdrm/dss/dsi.c
> +++ b/drivers/gpu/drm/omapdrm/dss/dsi.c
> @@ -4336,7 +4336,7 @@ static void print_dsi_vm(const char *str,
>
>  	wc = DIV_ROUND_UP(t->hact * t->bitspp, 8);
>  	pps = DIV_ROUND_UP(wc + 6, t->ndl); /* pixel packet size */
> -	bl = t->hss + t->hsa + t->hse + t->hbp + t->hfront_porch;
> +	bl = t->hss + t->hsa + t->hse + t->hbp + t->hfp;
>  	tot = bl + pps;
>
>  #define TO_DSI_T(x) ((u32)div64_u64((u64)x * 1000000000llu, byteclk))
> @@ -4345,14 +4345,14 @@ static void print_dsi_vm(const char *str,
>  			"%u/%u/%u/%u/%u/%u = %u + %u = %u\n",
>  			str,
>  			byteclk,
> -			t->hss, t->hsa, t->hse, t->hbp, pps, t->hfront_porch,
> +			t->hss, t->hsa, t->hse, t->hbp, pps, t->hfp,
>  			bl, pps, tot,
>  			TO_DSI_T(t->hss),
>  			TO_DSI_T(t->hsa),
>  			TO_DSI_T(t->hse),
>  			TO_DSI_T(t->hbp),
>  			TO_DSI_T(pps),
> -			TO_DSI_T(t->hfront_porch),
> +			TO_DSI_T(t->hfp),
>
>  			TO_DSI_T(bl),
>  			TO_DSI_T(pps),
> @@ -4367,7 +4367,7 @@ static void print_dispc_vm(const char *str,
const struct videomode *vm)
>  	int hact, bl, tot;
>
>  	hact = vm->hactive;
> -	bl = vm->hsync_len + vm->hbp + vm->hfront_porch;
> +	bl = vm->hsync_len + vm->hback_porch + vm->hfront_porch;
>  	tot = hact + bl;
>
>  #define TO_DISPC_T(x) ((u32)div64_u64((u64)x * 1000000000llu, pck))
> @@ -4376,10 +4376,10 @@ static void print_dispc_vm(const char *str,
const struct videomode *vm)
>  			"%u/%u/%u/%u = %u + %u = %u\n",
>  			str,
>  			pck,
> -			vm->hsync_len, vm->hbp, hact, vm->hfront_porch,
> +			vm->hsync_len, vm->hback_porch, hact, vm->hfront_porch,
>  			bl, hact, tot,
>  			TO_DISPC_T(vm->hsync_len),
> -			TO_DISPC_T(vm->hbp),
> +			TO_DISPC_T(vm->hback_porch),
>  			TO_DISPC_T(hact),
>  			TO_DISPC_T(vm->hfront_porch),
>  			TO_DISPC_T(bl),
> @@ -4401,12 +4401,12 @@ static void print_dsi_dispc_vm(const char *str,
>  	dsi_tput = (u64)byteclk * t->ndl * 8;
>  	pck = (u32)div64_u64(dsi_tput, t->bitspp);
>  	dsi_hact = DIV_ROUND_UP(DIV_ROUND_UP(t->hact * t->bitspp, 8) + 6,
t->ndl);
> -	dsi_htot = t->hss + t->hsa + t->hse + t->hbp + dsi_hact +
t->hfront_porch;
> +	dsi_htot = t->hss + t->hsa + t->hse + t->hbp + dsi_hact + t->hfp;
>
>  	vm.pixelclock = pck;
>  	vm.hsync_len = div64_u64((u64)(t->hsa + t->hse) * pck, byteclk);
> -	vm.hbp = div64_u64((u64)t->hbp * pck, byteclk);
> -	vm.hfront_porch = div64_u64((u64)t->hfront_porch * pck, byteclk);
> +	vm.hback_porch = div64_u64((u64)t->hbp * pck, byteclk);
> +	vm.hfront_porch = div64_u64((u64)t->hfp * pck, byteclk);
>  	vm.hactive = t->hact;
>
>  	print_dispc_vm(str, &vm);
>

-- 
Péter


More information about the Letux-kernel mailing list