On Fri, 27 Jan 2017 15:27:19 +0200, Andy Shevchenko wrote: > On Fri, 2017-01-27 at 10:21 +0100, Jean Delvare wrote: > > That break isn't needed ;-) > > Would be better to have in my opinion. I already did myself some > mistakes regarding to missed break. OK, fine with me. > > The code looks sane to me, although I can't verify its correctness > > without a datasheet. > > Datasheet is public. > > Intel® Atom™ Processor Z3600 and Z3700 Series Datasheet (Volume 2 of 2) > Number: 329518-002 Apparently it was removed in December 2014 and replaced by a more concise document... https://communities.intel.com/thread/57313 But no big deal, I trust your fix is correct. -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html