On 11/17/16 07:32, Sudeep Holla wrote:
Currently platforms/drivers needing to get the machine model name are
replicating the same snippet of code. In some case, the OF reference
counting is either missing or incorrect.
This patch adds support to read the machine model name either using
the "model" or the "compatible" property in the device tree root node
to the core OF/DT code.
This can be used to remove all the duplicate code snippets doing exactly
same thing later.
I find five instances of reading only property "model":
arch/arm/mach-imx/cpu.c
arch/arm/mach-mxs/mach-mxs.c
arch/c6x/kernel/setup.c
arch/mips/cavium-octeon/setup.c
arch/sh/boards/of-generic.c
I find one instance of reading property "model", then if
that does not exist, property "compatible":
arch/mips/generic/proc.c
The proposed patch matches the code used in one place, and thus
current usage does not match the patch description.
Is my search bad? Are you planning to add additional instances
of reading "model" then "compatible"?