| Message ID | 20220217133549.27961-1-oneukum@suse.com |
|---|---|
| State | New |
| Headers | show |
| Series | xhci: omit mem read just after allocation of trb | expand |
On 17.2.2022 15.35, Oliver Neukum wrote: > This has been allocated just a few lines earlier with a > zalloc(). The value is known and "|=" is a waste of memory > cycles. > > Signed-off-by: Oliver Neukum <oneukum@suse.com> Thanks In case Greg wants to pick this up now directly: Acked-by: Mathias Nyman <mathias.nyman@linux.intel.com> Otherwise I'll submit it later with other small changes -Mathias
On Fri, Feb 18, 2022 at 10:26:45AM +0200, Mathias Nyman wrote: > On 17.2.2022 15.35, Oliver Neukum wrote: > > This has been allocated just a few lines earlier with a > > zalloc(). The value is known and "|=" is a waste of memory > > cycles. > > > > Signed-off-by: Oliver Neukum <oneukum@suse.com> > > Thanks > > In case Greg wants to pick this up now directly: > Acked-by: Mathias Nyman <mathias.nyman@linux.intel.com> Thanks, I'll take it now, it's easy enough to :)
diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c index 0e312066c5c6..99a3ce689235 100644 --- a/drivers/usb/host/xhci-mem.c +++ b/drivers/usb/host/xhci-mem.c @@ -57,7 +57,7 @@ static struct xhci_segment *xhci_segment_alloc(struct xhci_hcd *xhci, /* If the cycle state is 0, set the cycle bit to 1 for all the TRBs */ if (cycle_state == 0) { for (i = 0; i < TRBS_PER_SEGMENT; i++) - seg->trbs[i].link.control |= cpu_to_le32(TRB_CYCLE); + seg->trbs[i].link.control = cpu_to_le32(TRB_CYCLE); } seg->dma = dma; seg->next = NULL;
This has been allocated just a few lines earlier with a zalloc(). The value is known and "|=" is a waste of memory cycles. Signed-off-by: Oliver Neukum <oneukum@suse.com> --- drivers/usb/host/xhci-mem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)