Re: [linuxtv-media:master 447/499] drivers/media/common/saa7146/saa7146_fops.c:536:13: sparse: incorrect type in assignment (different base types)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday 18 July 2014 13:58:37 Hans Verkuil wrote:
> On 07/18/14 13:53, Laurent Pinchart wrote:
> > On Friday 18 July 2014 07:56:52 kbuild test robot wrote:
> >> tree:   git://linuxtv.org/media_tree.git master
> >> head:   0ca1ba2aac5f6b26672099b13040c5b40db93486
> >> commit: d52e23813672c3c72f92e7b39c7408d4b9a40a96 [447/499] [media] v4l:
> >> Support extending the v4l2_pix_format structure reproduce: make C=1
> >> CF=-D__CHECK_ENDIAN__
> >> 
> >> 
> >> sparse warnings: (new ones prefixed by >>)
> >> 
> >>>> drivers/media/common/saa7146/saa7146_fops.c:536:13: sparse: incorrect
> >>>> type in assignment (different base types)
> >>>> 
> >>    drivers/media/common/saa7146/saa7146_fops.c:536:13:    expected struct
> >> 
> >> v4l2_pix_format *fmt drivers/media/common/saa7146/saa7146_fops.c:536:13:
> >> got struct <noident> *<noident>
> >> drivers/media/common/saa7146/saa7146_fops.c: In function
> >> 'saa7146_vv_init':
> >> drivers/media/common/saa7146/saa7146_fops.c:536:6: warning: assignment
> >> from
> >> incompatible pointer type [enabled by default] fmt = &vv->ov_fb.fmt;
> >> 
> >>          ^
> > 
> > I'll send a patch to fix that.
> 
> I posted a fix for that already:
> 
> [PATCH for v3.17] saa7146: fix compile warning
> 
> Part of pull request https://patchwork.linuxtv.org/patch/24885/

Thank you. For what it's worth,

Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux