Hi Mauro,
On Mon, Oct 12, 2015 at 08:43:14PM -0300, Mauro Carvalho Chehab wrote:
Em Mon, 12 Oct 2015 22:35:21 +0300 Sakari Ailus sakari.ailus@iki.fi escreveu:
Hi Mauro,
On Mon, Oct 12, 2015 at 01:42:55PM -0300, Mauro Carvalho Chehab wrote:
diff --git a/drivers/media/media-device.c b/drivers/media/media-device.c index 81d6a130efef..3bdda16584fe 100644 --- a/drivers/media/media-device.c +++ b/drivers/media/media-device.c @@ -427,6 +427,8 @@ EXPORT_SYMBOL_GPL(media_device_unregister); int __must_check media_device_register_entity(struct media_device *mdev, struct media_entity *entity) {
- int i;
i should be unsigned, not signed.
I'm pretty sure you pointed that already and I fixed on a latter patch. It is at patch 80/83.
As I justified on that thread, rebasing a 83 patch series to touch at the 6th patch is risky, as it could cause breakages and badly solved conflicts. So, I really prefer not doing that, specially when it is a minor issue.
Oh. Indeed. I don't remember what I commented anymore back then and what not, but at least the comment was the same. ;) I'm fine with fixing this later on. So please ignore this comment.