From: Adam Ford <aford173@gmail.com>
To: linux-phy@lists.infradead.org
Cc: dominique.martinet@atmark-techno.com, linux-imx@nxp.com,
festevam@gmail.com, frieder.schrempf@kontron.de,
aford@beaconembedded.com, Sandor.yu@nxp.com,
"Adam Ford" <aford173@gmail.com>, "Vinod Koul" <vkoul@kernel.org>,
"Kishon Vijay Abraham I" <kishon@kernel.org>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Lucas Stach" <l.stach@pengutronix.de>,
"Marco Felsch" <m.felsch@pengutronix.de>,
linux-kernel@vger.kernel.org
Subject: [PATCH V4 0/5] phy: freescale: fsl-samsung-hdmi: Expand phy clock options
Date: Mon, 2 Sep 2024 20:30:42 -0500 [thread overview]
Message-ID: <20240903013113.139698-1-aford173@gmail.com> (raw)
Currently, there is a look-up-table to describe all the clock options the HDMI PHY
can use. Some of these entries in the LUT are using a fractional divider which does
not have a well documented algorithm for determinging values, but the the integer
divider can use an algorithm to calculate the integer divder values dynamically
beyond those listed in the LUT and also duplicates some of the entries.
The first two patches do not do anything functionally other than simplify
some of the register accesses and de-duplicates some of the register look-ups.
The third patch adds support for the integer divider and uses it whenever the
clock request is an exact match. Otherwise, it will use the LUT as before.
The rouding is still based on the LUT if the integer clock isn't an exact match.
The forth patch updates thes set_rate and round_rate functions to use either
the fractional clock LUT or the the integer divder mechanism to determine
which ever clock rate might be closest match.
The last patch removes the integer divider entries from the LUT since by then
it'll be comparing both the integer divider calculator and the closest value
in the LUT.
In my testing with a AOC 4K monitor, I was able to add 4 entries in my modetest
table. I do not have an HDMI analyzer, so I just used my monitor to determine
if this series worked.
Adam Ford (5):
phy: freescale: fsl-samsung-hdmi: Replace register defines with macro
phy: freescale: fsl-samsung-hdmi: Simplify REG21_PMS_S_MASK lookup
phy: freescale: fsl-samsung-hdmi: Support dynamic integer
phy: freescale: fsl-samsung-hdmi: Use closest divider
phy: freescale: fsl-samsung-hdmi: Remove unnecessary LUT entries
drivers/phy/freescale/phy-fsl-samsung-hdmi.c | 575 ++++++++++---------
1 file changed, 291 insertions(+), 284 deletions(-)
--
2.43.0
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
next reply other threads:[~2024-09-03 1:31 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-03 1:30 Adam Ford [this message]
2024-09-03 1:30 ` [PATCH V4 1/5] phy: freescale: fsl-samsung-hdmi: Replace register defines with macro Adam Ford
2024-09-03 7:47 ` Marco Felsch
2024-09-03 1:30 ` [PATCH V4 2/5] phy: freescale: fsl-samsung-hdmi: Simplify REG21_PMS_S_MASK lookup Adam Ford
2024-09-03 7:57 ` Marco Felsch
2024-09-03 1:30 ` [PATCH V4 3/5] phy: freescale: fsl-samsung-hdmi: Support dynamic integer Adam Ford
2024-09-04 8:28 ` Dominique Martinet
2024-09-04 10:37 ` Adam Ford
2024-09-03 1:30 ` [PATCH V4 4/5] phy: freescale: fsl-samsung-hdmi: Use closest divider Adam Ford
2024-09-03 5:12 ` Dominique Martinet
2024-09-03 1:30 ` [PATCH V4 5/5] phy: freescale: fsl-samsung-hdmi: Remove unnecessary LUT entries Adam Ford
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240903013113.139698-1-aford173@gmail.com \
--to=aford173@gmail.com \
--cc=Sandor.yu@nxp.com \
--cc=aford@beaconembedded.com \
--cc=dominique.martinet@atmark-techno.com \
--cc=festevam@gmail.com \
--cc=frieder.schrempf@kontron.de \
--cc=kishon@kernel.org \
--cc=l.stach@pengutronix.de \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=m.felsch@pengutronix.de \
--cc=u.kleine-koenig@pengutronix.de \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).