On 10/22/2014 11:08 AM, Andrew Lunn wrote: >> You don't clearly state whether your patch series keep compatibility >> with the 375 Z1 or not. And in fact, it doesn't keep compatibility with >> Z1. > > Hi Thomas > > That answers my question. Thanks > >> I'm fine with that, but then it means we should officially declare >> the Z1 support in mainline as dead, and get rid of the workarounds that >> applied only to 375 Z1. > > A big ACK. > > How bad is it if you run these patches on a Z1? Is it worth adding a > revision check and calling BUG()? > So, I just tried this on a Z1. Here's the result: sleep 3 -> sleeps for 6 seconds Configuring the watchdog for 3 seconds and: echo "barf" > /dev/watchdog -> resets after 6 seconds So using the 25 MHz reference clock is broken on Z1, but the point at least boots; it will "just" keep track of time wrongly. -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html