Hi,
On Wed, Feb 20, 2013 at 08:57:07PM +0530, Santosh Shilimkar wrote:
> case 0xb998:
> switch (rev) {
> case 0:
> - default:
> omap_revision = OMAP5432_REV_ES1_0;
> + break;
> + case 1:
> + omap_revision = OMAP5432_REV_ES2_0;
> + break;
> + default:
> + omap_revision = OMAP5432_REV_ES2_0;
no need for the duplication:
+ case 1:
+ default:
+ omap_revision = OMAP5432_REV_ES2_0;
will work just fine.
> }
> break;
>
> default:
> /* Unknown default to latest silicon rev as default*/
> - omap_revision = OMAP5430_REV_ES1_0;
> + omap_revision = OMAP5430_REV_ES2_0;
how about we default to 0xffffffff ? That's the highest possible
version, just not sure if our revision macros can cope with it.
--
balbi
Attachment:
signature.asc
Description: Digital signature
