[PATCH 4/5] phy: exynos5-usbdrd: fix definition of EXYNOS5_FSEL_26MHZ

Peter Griffin peter.griffin at linaro.org
Wed Jun 12 02:36:25 PDT 2024


Hi André,

On Tue, 7 May 2024 at 15:14, André Draszik <andre.draszik at linaro.org> wrote:
>
> Using 0x82 seems odd, where everything else is just a sequence.
>
> On E850, this macro isn't used (as a register value), only to assign
> its value to the 'extrefclk' variable, which is otherwise unused on
> that platform. Older platforms don't appear to support 26MHz in the
> first place (since this macro was added for E850).
>
> Furthermore, the downstream driver uses 0x82 to denote
> USBPHY_REFCLK_DIFF_26MHZ (whatever that means exactly), but for all the
> other values we match downstream's non-DIFF macros.
>
> Update to avoid confusion. No functional change intended.
>
> Signed-off-by: André Draszik <andre.draszik at linaro.org>
> ---

Reviewed-by:  Peter Griffin <peter.griffin at linaro.org>

regards,

Peter

[..]



More information about the linux-phy mailing list