On Mon, Nov 22, 2010 at 4:08 AM, Felipe Balbi <balbi@xxxxxx> wrote: > On Fri, Nov 19, 2010 at 03:50:02PM +0100, Cousson, Benoit wrote: >> >> Most of the time, we do not want to use the IP revision because it is >> un-accurate and does not reflect the change we'd like to track. >> For example some time a minor change in the RTL that will not impact the >> SW at all might trigger a change in the IP revision, whereas on the other >> hand a major bug fix that will impact the SW is not capture in the IP >> revision... yeah, that's bad, but this can happen. >> >> That's why we are relying on a rev field in the hwmod. > > But then, what's inside this rev field ? Is it some internal revision of > hwmod or do you read from the hw ? > >From the dmtimer stuff, it looks like the driver defines the version types, which hwmod uses to define the rev field. /* timer ip constants */ #define OMAP_TIMER_IP_LEGACY 0x1 #define OMAP_TIMER_IP_VERSION_2 0x2 Thank you, Best regards, Hari Kanigeri -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html