Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> writes: Hello Geert, [...] >> + if (!dst_pitch) { >> + bpp = drm_format_info_bpp(dst_fi, 0); >> + dst_pitch = DIV_ROUND_UP(drm_rect_width(clip) * bpp, 8); > > I know I'm a bit late to the party, but here's actually a helper for that: > > dst_pitch = drm_format_info_min_pitch(info, 0, drm_rect_width(clip)); > Nice, I didn't notice this. I'll send a follow-up patch. Thanks! -- Best regards, Javier Martinez Canillas Core Platforms Red Hat