Re: [RFC PATCH v2 1/3] drm/fourcc: Add 'bpp' field for formats with non-integer bytes-per-pixel

[Brian Starkey]

Hi Ayan,

On Mon, Sep 10, 2018 at 03:11:34PM +0100, Ayan Halder wrote:
> On Fri, Aug 31, 2018 at 10:17:30AM +0200, Daniel Vetter wrote:
> > On Thu, Aug 23, 2018 at 04:23:41PM +0100, Brian Starkey wrote:

[snip]

> > >  		unsigned int min_size;
> > > +		u8 bpp = drm_format_info_plane_bpp(fb->format, i);
> You might want to pass info here instead of fb->format because fb is
> NULL. ie drm_format_info_plane_bpp(info, i);

Thanks, good catch. I have to admit I only compile-tested these.

-Brian