> +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

Reply via email to