diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2011-07-05 08:59:05 -0400 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2011-07-05 09:41:15 -0400 |
commit | 1425b80e0dc0d4b002aee0705dd78c2efc913fee (patch) | |
tree | e2d7325317c7e906272df61348b87b610cae0df4 /drivers/pci/pci-acpi.c | |
parent | c2b65f8422a3f51435c9f60f9752a6ed82d47e13 (diff) |
usb: gadget: fusb300_udc: fix compile warnings
- remove pointer u32 abuse in fusb300_fill_idma_prdtbl().
It is assigned the dma_addr to a pointer and then back.
Poor families may have to recycle variables but we don't
- don't free req.buf in error case. We don't do it in the
ok case so it is probably wrong to do it in error case.
- return in error case. There is no reason to continue
without data and performing ops on an invalid pointer.
- The if (d) statement is bogus since an invalid DMA pointer
is ~0 on some architecutres. And since we return for the
invalid case we don't need it.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/pci/pci-acpi.c')
0 files changed, 0 insertions, 0 deletions