diff for duplicates of <522557B7.8080909@gmail.com> diff --git a/a/1.txt b/N1/1.txt index 196f6ef..bcd8a71 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -136,7 +136,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > addr = &ir_set->irq_pending; > - temp = xhci_readl(xhci, addr); > + temp = xhci_readl(addr); -> if (temp = XHCI_INIT_VALUE) +> if (temp == XHCI_INIT_VALUE) > return; > > @@ -187,17 +187,17 @@ void xhci_print_ir_set(struct xhci_hcd *xhci, int set_num) @@ -263,7 +263,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > wIndex--; > - temp = xhci_readl(xhci, port_array[wIndex]); > + temp = xhci_readl(port_array[wIndex]); -> if (temp = 0xffffffff) { +> if (temp == 0xffffffff) { > retval = -ENODEV; > break; > @@ -749,7 +749,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, @@ -272,7 +272,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > wIndex--; > - temp = xhci_readl(xhci, port_array[wIndex]); > + temp = xhci_readl(port_array[wIndex]); -> if (temp = 0xffffffff) { +> if (temp == 0xffffffff) { > retval = -ENODEV; > break; > @@ -758,7 +758,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, @@ -290,7 +290,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > */ > - temp = xhci_readl(xhci, port_array[wIndex]); > + temp = xhci_readl(port_array[wIndex]); -> if ((temp & PORT_PE) = 0 || (temp & PORT_RESET) +> if ((temp & PORT_PE) == 0 || (temp & PORT_RESET) > || (temp & PORT_PLS_MASK) >= XDEV_U3) { > xhci_warn(xhci, "USB core suspending device " > @@ -796,11 +796,11 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, @@ -306,7 +306,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > + temp = xhci_readl(port_array[wIndex]); > > /* Disable port */ -> if (link_state = USB_SS_PORT_LS_SS_DISABLED) { +> if (link_state == USB_SS_PORT_LS_SS_DISABLED) { > @@ -815,7 +815,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, > PORT_CEC; > xhci_writel(xhci, temp | PORT_PE, @@ -331,7 +331,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > > - temp = xhci_readl(xhci, port_array[wIndex]); > + temp = xhci_readl(port_array[wIndex]); -> if (link_state = USB_SS_PORT_LS_U3) +> if (link_state == USB_SS_PORT_LS_U3) > bus_state->suspended_ports |= 1 << wIndex; > break; > @@ -872,7 +872,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, @@ -396,7 +396,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > wIndex--; > - temp = xhci_readl(xhci, port_array[wIndex]); > + temp = xhci_readl(port_array[wIndex]); -> if (temp = 0xffffffff) { +> if (temp == 0xffffffff) { > retval = -ENODEV; > break; > @@ -939,7 +939,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, @@ -414,7 +414,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > for (i = 0; i < max_ports; i++) { > - temp = xhci_readl(xhci, port_array[i]); > + temp = xhci_readl(port_array[i]); -> if (temp = 0xffffffff) { +> if (temp == 0xffffffff) { > retval = -ENODEV; > break; > @@ -1098,7 +1098,7 @@ int xhci_bus_suspend(struct usb_hcd *hcd) @@ -497,7 +497,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > addr = &xhci->cap_regs->hcc_params; > - offset = XHCI_HCC_EXT_CAPS(xhci_readl(xhci, addr)); > + offset = XHCI_HCC_EXT_CAPS(xhci_readl(addr)); -> if (offset = 0) { +> if (offset == 0) { > xhci_err(xhci, "No Extended Capability registers, " > "unable to set up roothub.\n"); > @@ -2115,7 +2115,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags) @@ -506,7 +506,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > u32 cap_id; > - cap_id = xhci_readl(xhci, tmp_addr); > + cap_id = xhci_readl(tmp_addr); -> if (XHCI_EXT_CAPS_ID(cap_id) = XHCI_EXT_CAPS_PROTOCOL) +> if (XHCI_EXT_CAPS_ID(cap_id) == XHCI_EXT_CAPS_PROTOCOL) > cap_count++; > tmp_offset = XHCI_EXT_CAPS_NEXT(cap_id); > @@ -2129,7 +2129,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags) @@ -515,7 +515,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > > - cap_id = xhci_readl(xhci, addr); > + cap_id = xhci_readl(addr); -> if (XHCI_EXT_CAPS_ID(cap_id) = XHCI_EXT_CAPS_PROTOCOL) +> if (XHCI_EXT_CAPS_ID(cap_id) == XHCI_EXT_CAPS_PROTOCOL) > xhci_add_in_port(xhci, num_ports, addr, > (u8) XHCI_EXT_PORT_MAJOR(cap_id), > @@ -2234,7 +2234,7 @@ int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags) @@ -586,11 +586,11 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > > - temp = xhci_readl(xhci, port_array[faked_port_index]); > + temp = xhci_readl(port_array[faked_port_index]); -> if (hcd->state = HC_STATE_SUSPENDED) { +> if (hcd->state == HC_STATE_SUSPENDED) { > xhci_dbg(xhci, "resume root hub\n"); > usb_hcd_resume_root_hub(hcd); > @@ -1684,7 +1684,7 @@ static void handle_port_status(struct xhci_hcd *xhci, -> if ((temp & PORT_PLC) && (temp & PORT_PLS_MASK) = XDEV_RESUME) { +> if ((temp & PORT_PLC) && (temp & PORT_PLS_MASK) == XDEV_RESUME) { > xhci_dbg(xhci, "port resume event for port %d\n", port_id); > > - temp1 = xhci_readl(xhci, &xhci->op_regs->command); @@ -604,7 +604,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > /* Check if the xHC generated the interrupt, or the irq is shared */ > - status = xhci_readl(xhci, &xhci->op_regs->status); > + status = xhci_readl(&xhci->op_regs->status); -> if (status = 0xffffffff) +> if (status == 0xffffffff) > goto hw_died; > > @@ -2783,7 +2783,7 @@ hw_died: @@ -635,7 +635,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > do { > - result = xhci_readl(xhci, ptr); > + result = xhci_readl(ptr); -> if (result = ~(u32)0) /* card removed */ +> if (result == ~(u32)0) /* card removed */ > return -ENODEV; > result &= mask; > @@ -82,11 +82,11 @@ void xhci_quiesce(struct xhci_hcd *xhci) @@ -667,7 +667,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > > - state = xhci_readl(xhci, &xhci->op_regs->status); > + state = xhci_readl(&xhci->op_regs->status); -> if ((state & STS_HALT) = 0) { +> if ((state & STS_HALT) == 0) { > xhci_warn(xhci, "Host controller not halted, aborting reset.\n"); > return 0; > } @@ -684,7 +684,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > for (i = 0; i < xhci->num_usb3_ports; i++) { > - temp = xhci_readl(xhci, xhci->usb3_ports[i]); > + temp = xhci_readl(xhci->usb3_ports[i]); -> if ((temp & PORT_PLS_MASK) = USB_SS_PORT_LS_COMP_MOD) { +> if ((temp & PORT_PLS_MASK) == USB_SS_PORT_LS_COMP_MOD) { > /* > * Compliance Mode Detected. Letting USB Core > @@ -611,19 +611,19 @@ int xhci_run(struct usb_hcd *hcd) @@ -832,7 +832,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > goto done; > - temp = xhci_readl(xhci, &xhci->op_regs->status); > + temp = xhci_readl(&xhci->op_regs->status); -> if (temp = 0xffffffff || (xhci->xhc_state & XHCI_STATE_HALTED)) { +> if (temp == 0xffffffff || (xhci->xhc_state & XHCI_STATE_HALTED)) { > xhci_dbg_trace(xhci, trace_xhci_dbg_cancel_urb, > "HW died, freeing TD."); > @@ -3593,7 +3593,7 @@ void xhci_free_dev(struct usb_hcd *hcd, struct usb_device *udev) @@ -841,7 +841,7 @@ On Saturday 31 August 2013 11:02 PM, Kumar Gaurav wrote: > /* Don't disable the slot if the host controller is dead. */ > - state = xhci_readl(xhci, &xhci->op_regs->status); > + state = xhci_readl(&xhci->op_regs->status); -> if (state = 0xffffffff || (xhci->xhc_state & XHCI_STATE_DYING) || +> if (state == 0xffffffff || (xhci->xhc_state & XHCI_STATE_DYING) || > (xhci->xhc_state & XHCI_STATE_HALTED)) { > xhci_free_virt_device(xhci, udev->slot_id); > @@ -4102,12 +4102,12 @@ static int xhci_usb2_software_lpm_test(struct usb_hcd *hcd, diff --git a/a/content_digest b/N1/content_digest index 20e4d2c..098762a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\01377970365-5420-1-git-send-email-kumargauravgupta3@gmail.com\0" "From\0Kumar Gaurav <kumargauravgupta3@gmail.com>\0" "Subject\0Re: [PATCH v4:No Change] xHCI:Fixing xhci_readl definition and function call\0" - "Date\0Tue, 03 Sep 2013 03:41:59 +0000\0" + "Date\0Tue, 03 Sep 2013 08:59:59 +0530\0" "To\0sarah.a.sharp@linux.intel.com" " gregkh@linuxfoundation.org\0" "Cc\0linux-usb@vger.kernel.org" @@ -147,7 +147,7 @@ "> \taddr = &ir_set->irq_pending;\n" "> -\ttemp = xhci_readl(xhci, addr);\n" "> +\ttemp = xhci_readl(addr);\n" - "> \tif (temp = XHCI_INIT_VALUE)\n" + "> \tif (temp == XHCI_INIT_VALUE)\n" "> \t\treturn;\n" "> \n" "> @@ -187,17 +187,17 @@ void xhci_print_ir_set(struct xhci_hcd *xhci, int set_num)\n" @@ -274,7 +274,7 @@ "> \t\twIndex--;\n" "> -\t\ttemp = xhci_readl(xhci, port_array[wIndex]);\n" "> +\t\ttemp = xhci_readl(port_array[wIndex]);\n" - "> \t\tif (temp = 0xffffffff) {\n" + "> \t\tif (temp == 0xffffffff) {\n" "> \t\t\tretval = -ENODEV;\n" "> \t\t\tbreak;\n" "> @@ -749,7 +749,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" @@ -283,7 +283,7 @@ "> \t\twIndex--;\n" "> -\t\ttemp = xhci_readl(xhci, port_array[wIndex]);\n" "> +\t\ttemp = xhci_readl(port_array[wIndex]);\n" - "> \t\tif (temp = 0xffffffff) {\n" + "> \t\tif (temp == 0xffffffff) {\n" "> \t\t\tretval = -ENODEV;\n" "> \t\t\tbreak;\n" "> @@ -758,7 +758,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" @@ -301,7 +301,7 @@ "> \t\t\t */\n" "> -\t\t\ttemp = xhci_readl(xhci, port_array[wIndex]);\n" "> +\t\t\ttemp = xhci_readl(port_array[wIndex]);\n" - "> \t\t\tif ((temp & PORT_PE) = 0 || (temp & PORT_RESET)\n" + "> \t\t\tif ((temp & PORT_PE) == 0 || (temp & PORT_RESET)\n" "> \t\t\t\t|| (temp & PORT_PLS_MASK) >= XDEV_U3) {\n" "> \t\t\t\txhci_warn(xhci, \"USB core suspending device \"\n" "> @@ -796,11 +796,11 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" @@ -317,7 +317,7 @@ "> +\t\t\ttemp = xhci_readl(port_array[wIndex]);\n" "> \n" "> \t\t\t/* Disable port */\n" - "> \t\t\tif (link_state = USB_SS_PORT_LS_SS_DISABLED) {\n" + "> \t\t\tif (link_state == USB_SS_PORT_LS_SS_DISABLED) {\n" "> @@ -815,7 +815,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" "> \t\t\t\t\tPORT_CEC;\n" "> \t\t\t\txhci_writel(xhci, temp | PORT_PE,\n" @@ -342,7 +342,7 @@ "> \n" "> -\t\t\ttemp = xhci_readl(xhci, port_array[wIndex]);\n" "> +\t\t\ttemp = xhci_readl(port_array[wIndex]);\n" - "> \t\t\tif (link_state = USB_SS_PORT_LS_U3)\n" + "> \t\t\tif (link_state == USB_SS_PORT_LS_U3)\n" "> \t\t\t\tbus_state->suspended_ports |= 1 << wIndex;\n" "> \t\t\tbreak;\n" "> @@ -872,7 +872,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" @@ -407,7 +407,7 @@ "> \t\twIndex--;\n" "> -\t\ttemp = xhci_readl(xhci, port_array[wIndex]);\n" "> +\t\ttemp = xhci_readl(port_array[wIndex]);\n" - "> \t\tif (temp = 0xffffffff) {\n" + "> \t\tif (temp == 0xffffffff) {\n" "> \t\t\tretval = -ENODEV;\n" "> \t\t\tbreak;\n" "> @@ -939,7 +939,7 @@ int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,\n" @@ -425,7 +425,7 @@ "> \tfor (i = 0; i < max_ports; i++) {\n" "> -\t\ttemp = xhci_readl(xhci, port_array[i]);\n" "> +\t\ttemp = xhci_readl(port_array[i]);\n" - "> \t\tif (temp = 0xffffffff) {\n" + "> \t\tif (temp == 0xffffffff) {\n" "> \t\t\tretval = -ENODEV;\n" "> \t\t\tbreak;\n" "> @@ -1098,7 +1098,7 @@ int xhci_bus_suspend(struct usb_hcd *hcd)\n" @@ -508,7 +508,7 @@ "> \taddr = &xhci->cap_regs->hcc_params;\n" "> -\toffset = XHCI_HCC_EXT_CAPS(xhci_readl(xhci, addr));\n" "> +\toffset = XHCI_HCC_EXT_CAPS(xhci_readl(addr));\n" - "> \tif (offset = 0) {\n" + "> \tif (offset == 0) {\n" "> \t\txhci_err(xhci, \"No Extended Capability registers, \"\n" "> \t\t\t\t\"unable to set up roothub.\\n\");\n" "> @@ -2115,7 +2115,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags)\n" @@ -517,7 +517,7 @@ "> \t\tu32 cap_id;\n" "> -\t\tcap_id = xhci_readl(xhci, tmp_addr);\n" "> +\t\tcap_id = xhci_readl(tmp_addr);\n" - "> \t\tif (XHCI_EXT_CAPS_ID(cap_id) = XHCI_EXT_CAPS_PROTOCOL)\n" + "> \t\tif (XHCI_EXT_CAPS_ID(cap_id) == XHCI_EXT_CAPS_PROTOCOL)\n" "> \t\t\tcap_count++;\n" "> \t\ttmp_offset = XHCI_EXT_CAPS_NEXT(cap_id);\n" "> @@ -2129,7 +2129,7 @@ static int xhci_setup_port_arrays(struct xhci_hcd *xhci, gfp_t flags)\n" @@ -526,7 +526,7 @@ "> \n" "> -\t\tcap_id = xhci_readl(xhci, addr);\n" "> +\t\tcap_id = xhci_readl(addr);\n" - "> \t\tif (XHCI_EXT_CAPS_ID(cap_id) = XHCI_EXT_CAPS_PROTOCOL)\n" + "> \t\tif (XHCI_EXT_CAPS_ID(cap_id) == XHCI_EXT_CAPS_PROTOCOL)\n" "> \t\t\txhci_add_in_port(xhci, num_ports, addr,\n" "> \t\t\t\t\t(u8) XHCI_EXT_PORT_MAJOR(cap_id),\n" "> @@ -2234,7 +2234,7 @@ int xhci_mem_init(struct xhci_hcd *xhci, gfp_t flags)\n" @@ -597,11 +597,11 @@ "> \n" "> -\ttemp = xhci_readl(xhci, port_array[faked_port_index]);\n" "> +\ttemp = xhci_readl(port_array[faked_port_index]);\n" - "> \tif (hcd->state = HC_STATE_SUSPENDED) {\n" + "> \tif (hcd->state == HC_STATE_SUSPENDED) {\n" "> \t\txhci_dbg(xhci, \"resume root hub\\n\");\n" "> \t\tusb_hcd_resume_root_hub(hcd);\n" "> @@ -1684,7 +1684,7 @@ static void handle_port_status(struct xhci_hcd *xhci,\n" - "> \tif ((temp & PORT_PLC) && (temp & PORT_PLS_MASK) = XDEV_RESUME) {\n" + "> \tif ((temp & PORT_PLC) && (temp & PORT_PLS_MASK) == XDEV_RESUME) {\n" "> \t\txhci_dbg(xhci, \"port resume event for port %d\\n\", port_id);\n" "> \n" "> -\t\ttemp1 = xhci_readl(xhci, &xhci->op_regs->command);\n" @@ -615,7 +615,7 @@ "> \t/* Check if the xHC generated the interrupt, or the irq is shared */\n" "> -\tstatus = xhci_readl(xhci, &xhci->op_regs->status);\n" "> +\tstatus = xhci_readl(&xhci->op_regs->status);\n" - "> \tif (status = 0xffffffff)\n" + "> \tif (status == 0xffffffff)\n" "> \t\tgoto hw_died;\n" "> \n" "> @@ -2783,7 +2783,7 @@ hw_died:\n" @@ -646,7 +646,7 @@ "> \tdo {\n" "> -\t\tresult = xhci_readl(xhci, ptr);\n" "> +\t\tresult = xhci_readl(ptr);\n" - "> \t\tif (result = ~(u32)0)\t\t/* card removed */\n" + "> \t\tif (result == ~(u32)0)\t\t/* card removed */\n" "> \t\t\treturn -ENODEV;\n" "> \t\tresult &= mask;\n" "> @@ -82,11 +82,11 @@ void xhci_quiesce(struct xhci_hcd *xhci)\n" @@ -678,7 +678,7 @@ "> \n" "> -\tstate = xhci_readl(xhci, &xhci->op_regs->status);\n" "> +\tstate = xhci_readl(&xhci->op_regs->status);\n" - "> \tif ((state & STS_HALT) = 0) {\n" + "> \tif ((state & STS_HALT) == 0) {\n" "> \t\txhci_warn(xhci, \"Host controller not halted, aborting reset.\\n\");\n" "> \t\treturn 0;\n" "> \t}\n" @@ -695,7 +695,7 @@ "> \tfor (i = 0; i < xhci->num_usb3_ports; i++) {\n" "> -\t\ttemp = xhci_readl(xhci, xhci->usb3_ports[i]);\n" "> +\t\ttemp = xhci_readl(xhci->usb3_ports[i]);\n" - "> \t\tif ((temp & PORT_PLS_MASK) = USB_SS_PORT_LS_COMP_MOD) {\n" + "> \t\tif ((temp & PORT_PLS_MASK) == USB_SS_PORT_LS_COMP_MOD) {\n" "> \t\t\t/*\n" "> \t\t\t * Compliance Mode Detected. Letting USB Core\n" "> @@ -611,19 +611,19 @@ int xhci_run(struct usb_hcd *hcd)\n" @@ -843,7 +843,7 @@ "> \t\tgoto done;\n" "> -\ttemp = xhci_readl(xhci, &xhci->op_regs->status);\n" "> +\ttemp = xhci_readl(&xhci->op_regs->status);\n" - "> \tif (temp = 0xffffffff || (xhci->xhc_state & XHCI_STATE_HALTED)) {\n" + "> \tif (temp == 0xffffffff || (xhci->xhc_state & XHCI_STATE_HALTED)) {\n" "> \t\txhci_dbg_trace(xhci, trace_xhci_dbg_cancel_urb,\n" "> \t\t\t\t\"HW died, freeing TD.\");\n" "> @@ -3593,7 +3593,7 @@ void xhci_free_dev(struct usb_hcd *hcd, struct usb_device *udev)\n" @@ -852,7 +852,7 @@ "> \t/* Don't disable the slot if the host controller is dead. */\n" "> -\tstate = xhci_readl(xhci, &xhci->op_regs->status);\n" "> +\tstate = xhci_readl(&xhci->op_regs->status);\n" - "> \tif (state = 0xffffffff || (xhci->xhc_state & XHCI_STATE_DYING) ||\n" + "> \tif (state == 0xffffffff || (xhci->xhc_state & XHCI_STATE_DYING) ||\n" "> \t\t\t(xhci->xhc_state & XHCI_STATE_HALTED)) {\n" "> \t\txhci_free_virt_device(xhci, udev->slot_id);\n" "> @@ -4102,12 +4102,12 @@ static int xhci_usb2_software_lpm_test(struct usb_hcd *hcd,\n" @@ -965,4 +965,4 @@ "> \treturn readl(regs);\n" > } -69519e6f778da7c37423b5d376dc4708c2dda749db36c6d42fcca85216336025 +02ee82ca4e0c49d58cffb8f68869029197102f7e998740d37833409783a7c99e
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.