| Date | Tue, 25 Oct 2016 12:11:58 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH 13/37] staging:r8188eu: remove debug messages after memory allocation failed |
| |
On Wed, Oct 19, 2016 at 10:07:41PM +0700, Ivan Safonov wrote: > These messages are redundant. > Also break rtw_usb_if1_init function if HalData allocation failed. > > Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Since you're redoing this anyway, please split this one into two patches. We really frown on mixing bugfixes with cleanups. I wasn't frowning on it enough to ask you to redo the series but since you're already doing that...
regards, dan carpenter
|