Re: [PATCH] phy: freescale: fsl-samsung-hdmi: return closest rate instead LUT

[Uwe Kleine-König]

Hello Adam,

On Sun, May 04, 2025 at 03:44:26PM -0500, Adam Ford wrote:
> On Mon, Mar 10, 2025 at 11:12 AM Uwe Kleine-König
> <u.kleine-koenig@xxxxxxxxxxxx> wrote:
> >
> > Hello Dominique,
> >
> > On Mon, Mar 10, 2025 at 06:48:50PM +0900, Dominique Martinet wrote:
> > > [...] and I'm sure there are other improvements that could be made at
> > > the edges.
> >
> > One thing that irritated me is the function names. `phy_clk_round_rate`
> > sounds too generic. `fsl_samsung_hdmi_phy_clk_round_rate` is long, but
> > I'd say would be a better match.
> Uwe,
> 
> I just sent a patch to rename round_rate and set rate functions to be
> more explicit.  I also tried to refactor the driver as you requested
> by simplifying the code, but I didn't have time to integrate my
> fractional-divder code yet.
> 
> I will be traveling for most of May, so I won't be able to revisit
> this again until after May 24.  Hopefully the patches I submitted meet
> your satisfaction,

I don't feel authoritative for that driver, so please do stuff to please
the phy maintainers and consider my comments as suggestion from the side
line.

Best regards
Uwe

Attachment: signature.asc
Description: PGP signature