On Tue, Apr 28, 2009 at 05:53:15PM +0530, Viral Mehta wrote:
>
>> + if (last_ep < 31)
>> + last_ep_ctx = last_ep + 1;
>> + for (i = 0; i < last_ep_ctx; ++i) {
>>
> I would appreciate if you help me understand why last_ep_ctx is set to
> 31 (and not 32) or i< last_ep_ctx (and not i<= last_ep_ctx).....
There are 31 endpoint contexts in a device context, because control
endpoint 0 only takes one context instead of two.
>> + xhci_dbg(xhci, "Endpoint %02d Context:\n", i);
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - ep_info\n",
>> + (unsigned int) &ctx->ep[i].ep_info,
>> + dma, ctx->ep[i].ep_info);
>> + dma += field_size;
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - ep_info2\n",
>> + (unsigned int) &ctx->ep[i].ep_info2,
>> + dma, ctx->ep[i].ep_info2);
>> + dma += field_size;
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - deq[0]\n",
>> + (unsigned int) &ctx->ep[i].deq[0],
>> + dma, ctx->ep[i].deq[0]);
>> + dma += field_size;
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - deq[1]\n",
>> + (unsigned int) &ctx->ep[i].deq[1],
>> + dma, ctx->ep[i].deq[1]);
>> + dma += field_size;
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - tx_info\n",
>> + (unsigned int) &ctx->ep[i].tx_info,
>> + dma, ctx->ep[i].tx_info);
>> + dma += field_size;
>> + for (j = 0; j < 3; ++j) {
>> + xhci_dbg(xhci, "@%08x (virt) @%08x (dma) %#08x - rsvd[%d]\n",
>> + (unsigned int) &ctx->ep[i].reserved[j],
>> + dma, ctx->ep[i].reserved[j], j);
>> + dma += field_size;
>> + }
>> + }
>> +}
> Thanks,
> Viral Mehta, Embedded Software Engineer, eInfochips Ltd.
> www.einfochips.com
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
