[PATCH phy-next] phy: lynx-10g: fix lynx_10g_pccr_val_enabled()

Vinod Koul vkoul at kernel.org
Wed Jul 22 05:12:36 PDT 2026


On Fri, 12 Jun 2026 15:57:31 +0300, Vladimir Oltean wrote:
> The intention of the code is to extract the PCCR8_SGMIIa_CFG field out
> of the "pccr" value, not to create a new value with the PCCR8_SGMIIa_CFG
> field set to the "pccr" value.
> 
> Since FIELD_GET() is implemented as ((reg) & (mask)) >> __bf_shf(mask)
> and FIELD_PREP() as (val) << __bf_shf(mask)) & (mask) and since "mask"
> is GENMASK(2, 0), in practice there is no functional difference between
> FIELD_GET() and FIELD_PREP(). But FIELD_GET() is logically the correct
> helper.
> 
> [...]

Applied, thanks!

[1/1] phy: lynx-10g: fix lynx_10g_pccr_val_enabled()
      commit: 1e0c8d50663ae2afd2307d617fd94a53f6a1dc02

Best regards,
-- 
~Vinod





More information about the linux-phy mailing list