On Tue, Nov 22, 2016 at 11:31 AM, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote: > 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 OK it seems to define more of the constants, I'll go and review it. Yours, Linus Walleij -- 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