On Tue, 10 Sept 2024 at 03:12, Miquel Raynal <[email protected]> wrote: > > The VNBYTES() macro has been updated to silence possible warnings > regarding authorized (but unusual) uses of this macro, but the comment > was kept unchanged. A year has passed so let's fix the comment now to > avoid confusions. > > Fixes: cc05d352fbc ("video: Add parentheses around VNBYTES() macro") > Suggested-by: Tom Rini <[email protected]> > Link: https://lore.kernel.org/u-boot/20240906183432.GG3879073@bill-the-cat/ > Signed-off-by: Miquel Raynal <[email protected]> > --- > include/video.h | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-)
Reviewed-by: Simon Glass <[email protected]> > > diff --git a/include/video.h b/include/video.h > index 4d8df9baaad..59b56014874 100644 > --- a/include/video.h > +++ b/include/video.h > @@ -57,12 +57,8 @@ enum video_log2_bpp { > VIDEO_BPP32, > }; > > -/* > - * Convert enum video_log2_bpp to bytes and bits. Note we omit the outer > - * brackets to allow multiplication by fractional pixels. > - */ > +/* Convert enum video_log2_bpp to bytes and bits */ > #define VNBYTES(bpix) ((1 << (bpix)) / 8) > - > #define VNBITS(bpix) (1 << (bpix)) > > enum video_format { > -- > 2.43.0 >
