Re: [Freedreno] [PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-11-06 Thread Sam Ravnborg
Hi Jordan. > case COLOR_FMT_P010_UBWC: > - alignment = 256; > - stride = MSM_MEDIA_ALIGN(width * 2, alignment); > + stride = MSM_MEDIA_ALIGN(width * 2, 256); > break; > case COLOR_FMT_P010: > - alignment = 128; > -

Re: [Freedreno] [PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-11-06 Thread Sean Paul
On Mon, Nov 05, 2018 at 04:31:03PM -0700, Jordan Crouse wrote: > Do some cleanup in the static inline functions defined in > dpu_media_info.h by cleaning up gotos and unneeded local > variables. > > Signed-off-by: Jordan Crouse > --- > .../gpu/drm/msm/disp/dpu1/msm_media_info.h| 164 ++--

[Freedreno] [PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-11-05 Thread Jordan Crouse
Do some cleanup in the static inline functions defined in dpu_media_info.h by cleaning up gotos and unneeded local variables. Signed-off-by: Jordan Crouse --- .../gpu/drm/msm/disp/dpu1/msm_media_info.h| 164 ++ 1 file changed, 57 insertions(+), 107 deletions(-) diff --git a/

Re: [Freedreno] [PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-10-19 Thread Bruce Wang
On Thu, Oct 18, 2018 at 3:59 PM Jordan Crouse wrote: > > Do some cleanup in the static inline functions defined in > dpu_media_info.h by cleaning up gotos and unneeded local > variables. > > Signed-off-by: Jordan Crouse Reviewed-by: Bruce Wang > --- > .../gpu/drm/msm/disp/dpu1/msm_media_info.

[Freedreno] [PATCH 11/11] drm/msm/dpu: Clean up dpu_media_info.h static inline functions

2018-10-18 Thread Jordan Crouse
Do some cleanup in the static inline functions defined in dpu_media_info.h by cleaning up gotos and unneeded local variables. Signed-off-by: Jordan Crouse --- .../gpu/drm/msm/disp/dpu1/msm_media_info.h| 164 ++ 1 file changed, 57 insertions(+), 107 deletions(-) diff --git a/