Messages in this thread Patch in this message | | | Date | Tue, 22 Jul 2014 16:56:06 -0400 | Subject | Re: [PATCH] edac: Remove fixmes in e7xxx_edac.c | From | Nick Krause <> |
| |
On Tue, Jul 22, 2014 at 3:33 PM, Steven Rostedt <rostedt@goodmis.org> wrote: > On Tue, Jul 22, 2014 at 02:25:24PM -0400, Nick Krause wrote: >> >> I understand what your saying and I should have searched for Page shift. >> In addition I am already got commits in the kernel for fix mes so I feel >> that your comment on fix mes is incorrect. > > After seeing your other solutions to "fix mes" around the kernel, I now have to > investigate the patches that were accepted, and see if they did not cause > any new bugs. If any of them do, I will ask to have all your patches reverted. > > You are not helping. You are actually doing quite the opposite. Who do you work > for? Microsoft or Apple? > > -- Steve >
I don't work for any companies. And here is one of them that has been accepted. I am removing two fix mes in this file as after dicussing then it seems there is no reason to check against Null for usb_device as it can never be NULL and this is check is therefore not needed.
Signed-off-by: Nicholas Krause <xerofoify@gmail.com> --- drivers/usb/core/hcd.c | 4 ---- 1 file changed, 4 deletions(-)
diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c index bec31e2..487abcf 100644 --- a/drivers/usb/core/hcd.c +++ b/drivers/usb/core/hcd.c @@ -855,8 +855,6 @@ static ssize_t authorized_default_show(struct device *dev, struct usb_bus *usb_bus = rh_usb_dev->bus; struct usb_hcd *usb_hcd;
- if (usb_bus == NULL) /* FIXME: not sure if this case is possible */ - return -ENODEV; usb_hcd = bus_to_hcd(usb_bus); return snprintf(buf, PAGE_SIZE, "%u\n", usb_hcd->authorized_default); } @@ -871,8 +869,6 @@ static ssize_t authorized_default_store( struct device *dev, struct usb_bus *usb_bus = rh_usb_dev->bus; struct usb_hcd *usb_hcd;
- if (usb_bus == NULL) /* FIXME: not sure if this case is possible */ - return -ENODEV; usb_hcd = bus_to_hcd(usb_bus); result = sscanf(buf, "%u\n", &val); if (result == 1) { Nick
| |