On 22/11/16 12:20, Linus Walleij wrote: > On Tue, Nov 22, 2016 at 8:58 AM, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote: >> On 21/11/16 17:34, Linus Walleij wrote: >>> On Mon, Jun 27, 2016 at 3:22 PM, Ritesh Harjani <riteshh@xxxxxxxxxxxxxx> wrote: >>> >>>> From: Asutosh Das <asutoshd@xxxxxxxxxxxxxx> >>>> >>>> eMMC cards with EXT_CSD version >= 8, optionally support command >>>> queuing feature as defined by JEDEC eMMC5.1. Add support for probing >>>> command queue feature for such type of cards. >>>> >>>> Signed-off-by: Sujit Reddy Thumma <sthumma@xxxxxxxxxxxxxx> >>>> Signed-off-by: Asutosh Das <asutoshd@xxxxxxxxxxxxxx> >>>> Signed-off-by: Venkat Gopalakrishnan <venkatg@xxxxxxxxxxxxxx> >>>> [subhashj@xxxxxxxxxxxxxx: fixed trivial merge conflicts] >>>> Signed-off-by: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx> >>>> Signed-off-by: Ritesh Harjani <riteshh@xxxxxxxxxxxxxx> >>> >>> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> >>> >>> Even if we don't merge the specific mechanism provided by the >>> rest of the patches, this patch just make us know more about >>> the capabilities of the hardware we're running on, which is good. >> >> I think SW CMDQ is a better starting point: > > When I supplied the Reviewe-by, as stated below it I am not talking > about the patch set as a whole. I am talking about *this* patch. I should have been more explicit. SW CMDQ covers some of the same ground and has basically the same patch: https://marc.info/?l=linux-mmc&m=147729859922297&w=2 > > As you can see in my reply to 00/10 I have serious concerns about > this patchset overall. I didn't mean to imply otherwise. -- 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
