[PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-08-04 Thread Martin Kepplinger
remove dprintk() and replace it with dev_dbg() in order to use the common kernel coding style. Signed-off-by: Martin Kepplinger mart...@posteo.de --- Thanks Dan. And since it continues to succeed if (dev == NULL), differntiate if (dev) or not. drivers/staging/media/as102/as102_drv.c | 15

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-08-04 Thread Dan Carpenter
On thinking about it some more the proper way to redo this would have been to remove the bogus check. Also a couple trivial things below. On Mon, Aug 04, 2014 at 12:17:14PM +0200, Martin Kepplinger wrote: @@ -82,10 +83,17 @@ static int as102_fe_get_tune_settings(struct dvb_frontend *fe,

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-08-04 Thread Joe Perches
On Mon, 2014-08-04 at 13:40 +0300, Dan Carpenter wrote: On Mon, Aug 04, 2014 at 12:17:14PM +0200, Martin Kepplinger wrote: [] + if (dev) { + dev_dbg(dev-bus_adap.usb_dev-dev, + tuner parameters: freq: %d bw: 0x%02x gi: 0x%02x\n, +

[PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-05-17 Thread Martin Kepplinger
don't reinvent dev_dbg(). remove dprintk() in as102_drv.c. use the common kernel coding style. Signed-off-by: Martin Kepplinger mart...@posteo.de --- this applies to next-20140516. any more suggestions? more cleanup can be done when dprintk() is completely gone.

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-05-17 Thread 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 mart...@posteo.de Reviewed-by: Antti Palosaari cr...@iki.fi --- this applies to next-20140516. any more

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-05-17 Thread Martin Kepplinger
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 mart...@posteo.de Reviewed-by: Antti Palosaari cr...@iki.fi ---

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-05-17 Thread Gianluca Gennari
Il 17/05/2014 19:52, Martin Kepplinger ha scritto: 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 mart...@posteo.de

Re: [PATCHv2] staging: media: as102: replace custom dprintk() with dev_dbg()

2014-05-17 Thread Dan Carpenter
On Sat, May 17, 2014 at 08:21:03PM +0300, Antti Palosaari wrote: 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 mart...@posteo.de Reviewed-by: Antti Palosaari