On 19-06-28 08:56:35, Leonard Crestez wrote:

On 28.06.2019 11:45, Abel Vesa wrote:

quoted

On 19-06-28 10:39:50, Leonard Crestez wrote:

quoted

quoted

This allows consumers to use min_rate max_rate.

@@ -45,10 +45,12 @@ static unsigned long imx8m_clk_composite_divider_recalc_rate(struct clk_hw *hw,
  				   divider->flags, PCG_DIV_WIDTH);
  }
  
  static int imx8m_clk_composite_compute_dividers(unsigned long rate,
  						unsigned long parent_rate,
+						unsigned long min_rate,
+						unsigned long max_rate,

You should pass on the req instead of min_rate and max_rate here.

Then I'd have to switch imx8m_clk_composite_divider_set_rate to allocate 
a dummy struct clk_rate_request on the stack. It's clearer if I just 
pass the minimum parameters required.

That is correct. Nevermind my earlier comment then.

--
Regards,
Leonard

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel