On 11/22/2016 01:13 AM, Stephen Boyd wrote:
On 11/21, Georgi Djakov wrote:The clk_hw_onecell_data struct is missing references to the actual clocks. Fix this. Reported-by: Michael Scott <michael.scott@xxxxxxxxxx> Signed-off-by: Georgi Djakov <georgi.djakov@xxxxxxxxxx> --- drivers/clk/qcom/clk-smd-rpm.c | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/drivers/clk/qcom/clk-smd-rpm.c b/drivers/clk/qcom/clk-smd-rpm.c index a27013dbc0aa..58821f7213b0 100644 --- a/drivers/clk/qcom/clk-smd-rpm.c +++ b/drivers/clk/qcom/clk-smd-rpm.c @@ -148,8 +148,7 @@ struct clk_smd_rpm_req { struct rpm_cc { struct qcom_rpm *rpm; - struct clk_hw_onecell_data data; - struct clk_hw *hws[]; + struct clk_hw_onecell_data *data;How about rolling our own xlate function to return hw pointers? We already have a list of hws here, so it doesn't seem like much more to do.
We could do this, but is there any benefit of adding and using our own xlate function instead of the of_clk_hw_onecell_get, which is already there? Maybe I'm missing something..? Thanks, Georgi -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
