lkml.org 
[lkml]   [2020]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/7] media: sun6i-csi: Fix the bpp for 10-bit bayer formats
    On Fri, Aug 21, 2020 at 04:59:29PM +0200, Kévin L'hôpital wrote:
    > 10-bit bayer formats are aligned to 16 bits in memory, so this is what
    > needs to be used as bpp for calculating the size of the buffers to
    > allocate.
    >
    > Signed-off-by: Kévin L'hôpital <kevin.lhopital@bootlin.com>

    Generally speaking, you should also explain why it's not an issue for
    the callers. Depending on what that function is supposed to be doing
    (returning the padded bits or the padded bits per pixel), your patch
    could be either right or wrong.

    Since all the callers are using it to generate the number of bytes per
    line, your patch is indeed correct. But it should be mentionned in the
    commit log.

    Maxime
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2020-08-25 15:44    [W:4.279 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site