Hi Ralf,
On Fri, Mar 25, 2011 at 01:58:20PM +0100, Ralf Baechle wrote:
> On Fri, Mar 25, 2011 at 02:26:57PM +0300, Sergei Shtylyov wrote:
>
> > >+static inline void cpu_probe_netlogic(struct cpuinfo_mips *c, int cpu)
> > >+{
> > >+ decode_configs(c);
> > >+
> > >+ c->options = (MIPS_CPU_TLB |
> >
> > Perhaps should align | with others...
> >
> > >+ MIPS_CPU_4KEX |
> > >+ MIPS_CPU_COUNTER |
> > >+ MIPS_CPU_DIVEC |
> > >+ MIPS_CPU_WATCH |
> > >+ MIPS_CPU_EJTAG |
> > >+ MIPS_CPU_LLSC);
>
> I reformatted that.
>
> > [...]
> > >+ default:
> > >+ printk(KERN_INFO "Unknown Netlogic chip id [%02x]!\n",
> >
> > Not %04x?
>
> I changed this into a panic call. An unknown CPU type means very little
> chance for the system to actually succesfully boot the system.
>
> Queued for 2.6.39. Thanks,
Thanks!
I noticed that the commit messages for the patchset are messed up, it contains
part of the mail header too.
This is what I see in 'git log'
|commit 7271e23649991af8d70da576052312571b71dba3
|Author: Jayachandran C <jayachandranc@netlogicmicro.com>
|Date: Fri Mar 25 10:29:09 2011 +0530
|
| From jayachandranc@netlogicmicro.com Fri Mar 25 05:52:44 2011
| X-Spam-Checker-Version: SpamAssassin 3.3.2-r929478 (2010-03-31) on
| eddie.linux-mips.org
| X-Spam-Level:
| X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,RDNS_NONE,
| UPPERCASE_75_100 autolearn=no version=3.3.2-r929478
Not sure what caused this, let me know if it was my fault.
Thanks,
JC.
|