- Status Closed
- Percent Complete
- Task Type Patches
- Category Drivers
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10166 - USB arrangement - use constants and some cosmetics
This patch contains the following:
1. Use some constants to make code clearer.
2. Some cosmetics
3. Remove redundant definitions of various places which are similar to those in ch9.h and other headers.
4. Add some verbosity (logf) in usb-drv-arc.c
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Some comments:
- usb_drv_request_endpoint() gets a new type parameter which isn’t used anywhere else. Included here by accident?
- you remove EPCTRL_EP_TYPE_ISO and friends in usb-drv-arc.c without adding them elsewhere or updating the places that use them
- usb.c contains non-style things (the usb_core_enable_driver(USB_DRIVER_HID, false) lines). Are they supposed to be included here? (I don’t object to them, I just think they belong in a different commit)
In general though, are all the indentation and whitespace changes really needed? In some places, you actually change the indentation style (e.g. usb.c), something that docs/CONTRIBUTING specifically asks not to do.
Fixed patch.
- Removed changes that belong to future patches.
- EPCTRL_EP_TYPE_XXX no replaced with USB_ENDPOINT_XFER_XXX which is defined in usb_ch9.h
- Removed most cosmetics changes.