Messages in this thread | | | Date | Sat, 17 May 2014 19:52:12 +0200 | From | Martin Kepplinger <> | Subject | Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg() |
| |
Am 2014-05-17 19:21, schrieb Antti Palosaari: > On 05/17/2014 07:05 PM, Martin Kepplinger wrote: >> don't reinvent dev_dbg(). remove dprintk() in as102_drv.c. >> use the common kernel coding style. >> >> Signed-off-by: Martin Kepplinger <martink@posteo.de> > > Reviewed-by: Antti Palosaari <crope@iki.fi> > >> --- >> this applies to next-20140516. any more suggestions? >> more cleanup can be done when dprintk() is completely gone. > > Do you have the device? I am a bit reluctant patching that driver > without any testing as it has happened too many times something has gone > totally broken. I don't have the device and will, at most, change such style issues.
> > IIRC Devin said it is in staging because of style issues and nothing > more. Is that correct? I haven't heard anything. A TODO file would help.
> > regards > Antti >
| |