Hi Viresh,
> On 23 January 2015 at 17:44, Lukasz Majewski <l.majewski@xxxxxxxxxxx>
> wrote:
> > + cpus = of_find_node_by_path("/cpus");
> > + if (!cpus) {
> > + pr_err("failed to find cpus node\n");
> > + return 0;
> > + }
> > +
> > + np = of_get_next_child(cpus, NULL);
> > + if (!np) {
> > + pr_err("failed to find cpus child node\n");
> > + of_node_put(cpus);
> > return 0;
> > + }
>
> Why making it complex? Just get device node for cpu 0 and
> do cpu_dev->np.
Please pay a note about following problem:
Previously we got: cpu0: cpu@0 for all Exynos devices.
Now, however, cpu numbering has changed (due to GIC rework).
For example:
Exynos4412:
cpus {
cpu0: cpu@A00 {
...
#cooling-cells = <2>; /* min followed by max */
};
cpu@A01 {
};
cpu@A02 {
};
cpu@A03 {
};
}
Exynos 4210:
cpus {
cpu0: cpu@900 {
#cooling-cells = <2>; /* min followed by max */
};
cpu@901 {
};
};
Exynos 5250:
cpus {
cpu0: cpu@0 {
#cooling-cells = <2>; /* min followed by max */
};
cpu@1 {
};
};
As you can see different cpu@XXY nodes we have and simply calling cpu@0
won't work.
--
Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
