On 03/11/2016 03:36 AM, Martin K. Petersen wrote: >>>>>> "Yaniv" == Yaniv Gardi <ygardi@xxxxxxxxxxxxxx> writes: > > Yaniv> V8: fixed cyclic dependency by removing ufs_quirk.c that was > Yaniv> previously added in V7 06/17, and moving its code into ufshcd.c > > Patch 6 is now different yet it retains Reviewed-by: tags from the > previous version. These should have been removed. > > But if Hannes is OK with the changed version you don't need to > resend. Hannes, if you would be so kind to take a second look? > As indicated, I'm still not happy with the timeout handler, it really should be sending real command aborts and not just extending the timeout. But it's better then nothing, and having this patchset in leaves us with more time to discuss other matters. EG ioctl interfaces ;-) So: Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking hare@xxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
