linux-kernel - Re: [PATCH v3 00/14] drm/tidss: dispc: Convert to FIELD

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] [day] [month] [year] [list]

Message-ID: <9d587099-9212-4107-83cb-be52f3bfe1f7@ideasonboard.com>
Date: Mon, 1 Sep 2025 10:42:27 +0300
From: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>
To: Maxime Ripard <mripard@...nel.org>, Jyri Sarha <jyri.sarha@....fi>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>,
 Simona Vetter <simona@...ll.ch>
Cc: dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 00/14] drm/tidss: dispc: Convert to FIELD_* API

Hi Maxime,

On 27/08/2025 18:12, Maxime Ripard wrote:
> Hi,
> 
> The tidss driver rolls its own API equivalent to the FIELD_* API already
> provided the kernel.
> 
> Since it's an ad-hoc implementation, it also is less convenient and
> doesn't provide some useful features like being able to share the field
> definitions that will come handy in the future.
> 
> Thus, this series converts the driver to that API and drops its own
> version.

This looks good. I'll push to drm-misc-next.

 Tomi

> 
> Let me know what you think,
> Maxime
> 
> Signed-off-by: Maxime Ripard <mripard@...nel.org>
> ---
> Changes in v3:
> - Rebase on top of latest linux-next, fix merge conflicts
> - Link to v2: https://lore.kernel.org/r/20250820-drm-tidss-field-api-v2-0-43cab671c648@kernel.org
> 
> Changes in v2:
> - Switch to macros to prevent a gcc error
> - Link to v1: https://lore.kernel.org/r/20250730-drm-tidss-field-api-v1-0-a71ae8dd2782@kernel.org
> 
> ---
> Maxime Ripard (14):
>       drm/tidss: dispc: Remove unused OVR_REG_GET
>       drm/tidss: dispc: Convert accessors to macros
>       drm/tidss: dispc: Switch to GENMASK instead of FLD_MASK
>       drm/tidss: dispc: Get rid of FLD_VAL
>       drm/tidss: dispc: Get rid of FLD_GET
>       drm/tidss: dispc: Get rid of FLD_MOD
>       drm/tidss: dispc: Switch REG_GET to using a mask
>       drm/tidss: dispc: Switch REG_FLD_MOD to using a mask
>       drm/tidss: dispc: Switch VID_REG_GET to using a mask
>       drm/tidss: dispc: Switch VID_REG_FLD_MOD to using a mask
>       drm/tidss: dispc: Switch VP_REG_GET to using a mask
>       drm/tidss: dispc: Switch VP_REG_FLD_MOD to using a mask
>       drm/tidss: dispc: Switch OVR_REG_FLD_MOD to using a mask
>       drm/tidss: dispc: Define field masks being used
> 
>  drivers/gpu/drm/tidss/tidss_dispc.c      | 297 ++++++++++++++++---------------
>  drivers/gpu/drm/tidss/tidss_dispc_regs.h |  76 ++++++++
>  2 files changed, 225 insertions(+), 148 deletions(-)
> ---
> base-commit: a6d3da9a268e3d0a20b76fb40fd3484fe219ff17
> change-id: 20250729-drm-tidss-field-api-382947a92d44
> 
> Best regards,