Messages in this thread | | | From | Stephen Boyd <> | Subject | Re: [PATCH 2/5] clk: qcom: apcs-msm8916: get parent clock names from DT | Date | Mon, 09 Sep 2019 03:23:54 -0700 |
| |
Quoting Jorge Ramirez-Ortiz (2019-08-26 09:45:07) > @@ -61,6 +63,16 @@ static int qcom_apcs_msm8916_clk_probe(struct platform_device *pdev) > if (!a53cc) > return -ENOMEM; > > + /* legacy bindings only defined the pll parent clock (index = 0) with no
Another nitpick: This is wrong multi-line comment style. SHould be a bare /* on this line.
> + * name; when both of the parents are specified in the bindings, the > + * pll is the second one (index = 1). > + */
| |