On Wed, Oct 19, 2016 at 12:07 AM, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote: > ipu_cpmem_set_fmt is already prepared to handle DRM_FORMAT_YVU422. > > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> > --- > drivers/gpu/ipu-v3/ipu-cpmem.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c b/drivers/gpu/ipu-v3/ipu-cpmem.c > index fcb7dc8..e6b8bae 100644 > --- a/drivers/gpu/ipu-v3/ipu-cpmem.c > +++ b/drivers/gpu/ipu-v3/ipu-cpmem.c > @@ -440,6 +440,11 @@ void ipu_cpmem_set_yuv_planar(struct ipuv3_channel *ch, > u_offset = stride * height; > v_offset = u_offset + (uv_stride * height); > break; > + case DRM_FORMAT_YVU422: > + uv_stride = stride / 2; > + v_offset = stride * height; > + u_offset = v_offset + (uv_stride * height); > + break; It looks no one is using this function. How about removing it directly? Regards, Liu Ying > case DRM_FORMAT_NV12: > case DRM_FORMAT_NV16: > uv_stride = stride; > -- > 2.9.3 > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel