> +static int fu540_macb_tx_set_rate(struct clk_hw *hw, unsigned long rate,
> + unsigned long parent_rate)
> +{
> + rate = fu540_macb_tx_round_rate(hw, rate, &parent_rate);
> + iowrite32(rate != 125000000, mgmt->reg);
That looks odd. Writing the result of a comparison to a register?
Andrew

