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 <matthijs@xxxxxxxx> --- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
