This series fixes the EMAC clock source selection in the N5X clock driver and then cleans it up to match the Agilex clock driver.
Patch 1 is the functional fix that was sent as v1: the mask was applied after the shift instead of before, so EMAC always read the clock source as emaca_free_clk. Patch 2 is a follow-up readability change that replaces the open-coded mask-and-shift with FIELD_GET(), as suggested by Tien Fong on the v1 review, aligning N5X with the existing Agilex usage. Tested on Intel SoCFPGA N5X SoCDK hardware. v1->v2: -------- - No functional change to the fix (patch 1). - New patch 2: use FIELD_GET() for the EMAC clock source select field, for consistency with the Agilex clock driver (Tien Fong's review nit). History: -------- [v1] https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/ Chen Huei Lok (2): drivers: clk: n5x: Fix incorrect EMAC clock source selection drivers: clk: n5x: Use FIELD_GET for EMAC clock source selection drivers/clk/altera/clk-n5x.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) -- 2.43.7
