Hello.

On 06-04-2013 1:26, Matthijs Kooijman wrote:

Before, the value of the hcchar register was shifted and masked and then
the debug output extracted the individual values out of the complete
hcchar register again. This commit makes the debug prints directly use
the original values instead.

Signed-off-by: Matthijs Kooijman <matth...@stdin.nl>
---
  drivers/staging/dwc2/core.c | 29 ++++++++---------------------
  1 file changed, 8 insertions(+), 21 deletions(-)

diff --git a/drivers/staging/dwc2/core.c b/drivers/staging/dwc2/core.c
index e64eb95..4068dd5 100644
--- a/drivers/staging/dwc2/core.c
+++ b/drivers/staging/dwc2/core.c
@@ -881,24 +881,13 @@ void dwc2_hc_init(struct dwc2_hsotg *hsotg, struct 
dwc2_host_chan *chan)
[...]
-               dev_vdbg(hsotg->dev, "   Multi Cnt: %d\n",
-                        hcchar >> HCCHAR_MULTICNT_SHIFT &
-                        HCCHAR_MULTICNT_MASK >> HCCHAR_MULTICNT_SHIFT);
+               dev_vdbg(hsotg->dev, "   Dev Addr: %d\n", chan->dev_addr);
+               dev_vdbg(hsotg->dev, "   Ep Num: %d\n", chan->ep_num);
+               dev_vdbg(hsotg->dev, "   Is In: %d\n", chan->ep_is_in);
+               dev_vdbg(hsotg->dev, "   Is Low Speed: %d\n", chan->speed == 
USB_SPEED_LOW);
+               dev_vdbg(hsotg->dev, "   Ep Type: %d\n", chan->ep_type);
+               dev_vdbg(hsotg->dev, "   Max Pkt: %d\n", chan->max_packet);
+               dev_vdbg(hsotg->dev, "   Multi Cnt: %d\n", 0);

   So it's always 0? Why use "%d" then at all? Or why even print it?

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to