On Mon, Dec 14, 2015 at 03:46:22PM -0600, Frank Li wrote: > show correct soc version infomation. s/infomation/information I feel you should write your commit log and comments a bit more seriously. The commit log above is not even a sentence. For arch/arm/ patches sent to me, please use "ARM" rather than "arm" as subject prefix. > > Signed-off-by: Frank Li <Frank.Li@xxxxxxxxxxxxx> > --- > arch/arm/mach-imx/anatop.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/arch/arm/mach-imx/anatop.c b/arch/arm/mach-imx/anatop.c > index 231bb25..23a4fc1 100644 > --- a/arch/arm/mach-imx/anatop.c > +++ b/arch/arm/mach-imx/anatop.c > @@ -151,7 +151,14 @@ void __init imx_init_revision_from_anatop(void) > revision = IMX_CHIP_REVISION_1_5; > break; > default: > - revision = IMX_CHIP_REVISION_UNKNOWN; > + /* > + * Fail back to return raw register value instead of 0xff. > + * It will be easy know version information in SOC if it > + * can't recongized by known version. And some chip like > + * i.MX7D soc digprog value match linux version format, > + * needn't map again and direct use register value. > + */ There are quite a few grammar issues in this piece of comment. Please fix them. > + revision = digprog & 0xff; I'm fine with it. But we should be clear about that with this change, the "unknown revision" message in imx_print_silicon_rev() will never work then for digprog. Shawn > } > > mxc_set_cpu_type(digprog >> 16 & 0xff); > -- > 2.5.2 > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html