Premi, Sanjeev had written, on 11/13/2009 04:10 AM, the following:
-----Original Message-----
From: Gadiyar, Anand Sent: Friday, November 13, 2009 10:41 AM
To: Tony Lindgren; Premi, Sanjeev
Cc: [email protected]
Subject: RE: [PATCH] omap3: Change the default silicon

Tony Lindgren wrote:
* Sanjeev Premi <[email protected]> [091029 07:35]:
Currently the default silicon - in absence of
identification - is set to OMAP3630 ES1.0.

Though, condition may/should not arise; but
the default should be latest in the most
common silicon variant - currently OMAP3430
ES3.1.
Is this still needed? To me it seems more likely there will
more 3630 based silicon than 3430 based silicon?

3430 ES3.1s are the most common I believe. All boards in the wild
are 3430 based.

3630 is just coming up and will take a while to be as common.

IMO, 3430 ES3.1 should be default.

- Anand

[sp] That's exactly the reason for this patch.
     There is a small typo in the comment though :(

                /* Unknown. Default to latest among all variants */
Should be:
                /* Unknown. Default to common among all variants */

     Sending a v2 for the same.


I believe this patch should be dropped -> here is why -> if you have new silicons that are in 3430 category, you should be sending patches for them ;)..

The latest and greatest in the bucket of silicons is 3630 and I would rather go with felipe's patch [1] cleaning up the id.c than this -> in my opinion, hitting this condition is only because you have the next generation of devices..


--
Regards,
Nishanth Menon

Ref:
[1]: http://patchwork.kernel.org/patch/59540/
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to