linux-kernel - Re: [RFC PATCH v3 2/2] staging: media: cedrus: Rename H265 to HEVC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 28 Mar 2022 19:09:11 +0200
From:   Jernej Škrabec <jernej.skrabec@...il.com>
To:     linux-media@...r.kernel.org,
        Sebastian Fricke <sebastian.fricke@...labora.com>
Cc:     acourbot@...omium.org, tfiga@...omium.org,
        hverkuil-cisco@...all.nl,
        Sebastian Fricke <sebastian.fricke@...labora.com>,
        Maxime Ripard <mripard@...nel.org>,
        Paul Kocialkowski <paul.kocialkowski@...tlin.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Chen-Yu Tsai <wens@...e.org>,
        "open list:STAGING SUBSYSTEM" <linux-staging@...ts.linux.dev>,
        "moderated list:ARM/Allwinner sunXi SoC support" 
        <linux-arm-kernel@...ts.infradead.org>,
        "open list:ARM/Allwinner sunXi SoC support" 
        <linux-sunxi@...ts.linux.dev>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH v3 2/2] staging: media: cedrus: Rename H265 to HEVC

Hi Sebastian!

Dne nedelja, 27. marec 2022 ob 10:43:10 CEST je Sebastian Fricke napisal(a):
> Rename all instances where the identifier H265 is used with HEVC.
> The codec has multiple identifiers H.265 (ITU-T), MPEG-H Part 2 (ISO),
> but the actual name of the codec is HEVC (High Efficiency Video Coding).
> Also nearly all other related drivers use the term HEVC instead of H265.
> 
> Signed-off-by: Sebastian Fricke <sebastian.fricke@...labora.com>

This patch is not constructive. I admit, it's unfortunate it's not using same 
name as other drivers, but anyone dealing with HEVC will know H265 is synonym 
for it. This patch doesn't improve anything and actually makes any fix harder 
to backport, if something cames up. According to diffstat, this rewrites more 
than 50% of cedrus_h265.c, including renaming the file. Additionally, it's easy 
to miss something to rename and we could play this game in the future too, for 
no real benefit.

So NAK.

Best regard,
Jernej

P.S. Please wait at least few days before resending patches, even if you find 
something. After all, RFC patches are meant to get some response, it's not 
expected they will be perfect.

> ---
>  drivers/staging/media/sunxi/cedrus/Makefile   |   2 +-
>  drivers/staging/media/sunxi/cedrus/cedrus.c   |  30 +-
>  drivers/staging/media/sunxi/cedrus/cedrus.h   |  14 +-
>  .../staging/media/sunxi/cedrus/cedrus_dec.c   |  10 +-
>  .../cedrus/{cedrus_h265.c => cedrus_hevc.c}   | 438 +++++++++---------
>  .../staging/media/sunxi/cedrus/cedrus_hw.c    |   4 +-
>  .../staging/media/sunxi/cedrus/cedrus_regs.h  | 394 ++++++++--------
>  .../staging/media/sunxi/cedrus/cedrus_video.c |   4 +-
>  8 files changed, 448 insertions(+), 448 deletions(-)
>  rename drivers/staging/media/sunxi/cedrus/{cedrus_h265.c => cedrus_hevc.c}
> (53%)