linux-kernel - Re: [PATCH v5 3/6] drm: lcdif: Determine bus format and flags in ->atomic

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]

Message-ID: <211f0b07-568e-509d-464a-2f8821d300d7@denx.de>
Date:   Tue, 9 May 2023 02:11:10 +0200
From:   Marek Vasut <marex@...x.de>
To:     Liu Ying <victor.liu@....com>, dri-devel@...ts.freedesktop.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org
Cc:     stefan@...er.ch, airlied@...il.com, daniel@...ll.ch,
        robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
        conor+dt@...nel.org, shawnguo@...nel.org, s.hauer@...gutronix.de,
        kernel@...gutronix.de, festevam@...il.com, linux-imx@....com,
        krzysztof.kozlowski@...aro.org, LW@...o-electronics.de,
        alexander.stein@...tq-group.com
Subject: Re: [PATCH v5 3/6] drm: lcdif: Determine bus format and flags in
 ->atomic_check()

On 5/8/23 07:57, Liu Ying wrote:
> Instead of determining LCDIF output bus format and bus flags in
> ->atomic_enable(), do that in ->atomic_check().  This is a
> preparation for the upcoming patch to check consistent bus format
> and bus flags across all first downstream bridges in ->atomic_check().
> New lcdif_crtc_state structure is introduced to cache bus format
> and bus flags states in ->atomic_check() so that they can be read
> in ->atomic_enable().
> 
> Tested-by: Alexander Stein <alexander.stein@...tq-group.com>
> Reviewed-by: Alexander Stein <alexander.stein@...tq-group.com>
> Signed-off-by: Liu Ying <victor.liu@....com>

Reviewed-by: Marek Vasut <marex@...x.de>