On Thu, Jan 12, 2017 at 12:23:16PM +0200, Ville Syrjälä wrote: > On Wed, Jan 11, 2017 at 04:51:16PM -0800, Ben Widawsky wrote: ... > > + memcpy(plane->modifiers, format_modifiers, > > + format_modifier_count * sizeof(format_modifiers[0])); > > Looks all right since we do the same for formats anyway. But it did > occur to me (twice at least) that a kmemdup_array() might a nice thing > to have for things like this. But that's a separate topic. JFYI: https://lore.kernel.org/all/20231017052322.2636-2-kkartik@xxxxxxxxxx/ -- With Best Regards, Andy Shevchenko