On 04/28/2015 05:43 PM, Mauro Carvalho Chehab wrote: > As reported by smatch: > drivers/media/pci/zoran/zoran_device.c:1594 zoran_init_hardware() warn: > inconsistent indenting > > Fix indent. While here, fix CodingStyle and remove dead code, as it > can always be recovered from git logs. > > Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
Acked-by: Hans Verkuil <hans.verk...@cisco.com> > > diff --git a/drivers/media/pci/zoran/zoran_device.c > b/drivers/media/pci/zoran/zoran_device.c > index b6801e035ea4..40119b3c52c1 100644 > --- a/drivers/media/pci/zoran/zoran_device.c > +++ b/drivers/media/pci/zoran/zoran_device.c > @@ -1584,14 +1584,11 @@ zoran_init_hardware (struct zoran *zr) > jpeg_codec_sleep(zr, 1); > jpeg_codec_sleep(zr, 0); > > - /* set individual interrupt enables (without GIRQ1) > - * but don't global enable until zoran_open() */ > - > - //btwrite(IRQ_MASK & ~ZR36057_ISR_GIRQ1, ZR36057_ICR); // SW > - // It looks like using only JPEGRepIRQEn is not always reliable, > - // may be when JPEG codec crashes it won't generate IRQ? So, > - /*CP*/ // btwrite(IRQ_MASK, ZR36057_ICR); // > Enable Vsync interrupts too. SM WHY ? LP > - zr36057_init_vfe(zr); > + /* > + * set individual interrupt enables (without GIRQ1) > + * but don't global enable until zoran_open() > + */ > + zr36057_init_vfe(zr); > > zr36057_enable_jpg(zr, BUZ_MODE_IDLE); > > ------------------------------------------------------------------------------ One dashboard for servers and applications across Physical-Virtual-Cloud Widest out-of-the-box monitoring support with 50+ applications Performance metrics, stats and reports that give you Actionable Insights Deep dive visibility with transaction tracing using APM Insight. http://ad.doubleclick.net/ddm/clk/290420510;117567292;y _______________________________________________ Mjpeg-users mailing list Mjpeg-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mjpeg-users
