linux-kernel - Re: [PATCH V8 3/5] phy: freescale: fsl-samsung-hdmi: Support dynamic integer

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: <cddbe1ad-047a-4c28-9594-db574cfa77f9@kontron.de>
Date: Mon, 16 Sep 2024 09:15:49 +0200
From: Frieder Schrempf <frieder.schrempf@...tron.de>
To: Adam Ford <aford173@...il.com>, linux-phy@...ts.infradead.org
Cc: dominique.martinet@...ark-techno.com, linux-imx@....com,
 festevam@...il.com, aford@...conembedded.com, Sandor.yu@....com,
 neil.armstrong@...aro.org, Vinod Koul <vkoul@...nel.org>,
 Kishon Vijay Abraham I <kishon@...nel.org>,
 Marco Felsch <m.felsch@...gutronix.de>, Lucas Stach
 <l.stach@...gutronix.de>, Uwe Kleine-König
 <u.kleine-koenig@...gutronix.de>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH V8 3/5] phy: freescale: fsl-samsung-hdmi: Support dynamic
 integer

On 14.09.24 1:27 PM, Adam Ford wrote:
> There is currently a look-up table for a variety of resolutions.
> Since the phy has the ability to dynamically calculate the values
> necessary to use the intger divider which should allow more
> resolutions without having to update the look-up-table.
> 
> If the lookup table cannot find an exact match, fall back to the
> dynamic calculator of the integer divider.
> 
> Previously, the value of P was hard-coded to 1, this required an
> update to the phy_pll_cfg table to add in the extra value into the
> table, so if the value of P is calculated to be something else
> by the PMS calculator, the calculated_phy_pll_cfg structure
> can be used instead without having to keep track of which method
> was used.
> 
> Signed-off-by: Adam Ford <aford173@...il.com>
> Reviewed-by: Dominique Martinet <dominique.martinet@...ark-techno.com>
> Tested-by: Dominique Martinet <dominique.martinet@...ark-techno.com>

Reviewed-by: Frieder Schrempf <frieder.schrempf@...tron.de>