All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
To: Hans Verkuil <hverkuil@xs4all.nl>, Benoit Parrot <bparrot@ti.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [GIT PULL FOR v4.5] New ti-cal driver
Date: Thu, 3 Dec 2015 11:19:12 -0200	[thread overview]
Message-ID: <20151203111912.6fdfc076@recife.lan> (raw)
In-Reply-To: <56585B0E.8090907@xs4all.nl>

Em Fri, 27 Nov 2015 14:30:54 +0100
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> The following changes since commit 10897dacea26943dd80bd6629117f4620fc320ef:
> 
>   Merge tag 'v4.4-rc2' into patchwork (2015-11-23 14:16:58 -0200)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git cal
> 
> for you to fetch changes up to 5df9a7909b737e9725c1005b3b39383e9c2490ca:
> 
>   media: v4l: ti-vpe: Document DRA72 CAL h/w module (2015-11-27 14:25:07 +0100)
> 
> ----------------------------------------------------------------
> Benoit Parrot (2):
>       media: v4l: ti-vpe: Add CAL v4l2 camera capture driver
>       media: v4l: ti-vpe: Document DRA72 CAL h/w module

Please put the DT documentation patch before the patch itself. Documenting
first makes easier to review, and removes a checkpatch warning:

WARNING: DT compatible string "ti,dra72-cal" appears un-documented -- check ./Documentation/devicetree/bindings/
#2210: FILE: drivers/media/platform/ti-vpe/cal.c:2128:
+	{ .compatible = "ti,dra72-cal", },


Also, with regards to:

>       media: v4l: ti-vpe: Add CAL v4l2 camera capture driver

WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#78: 
new file mode 100644

Please add an entry to MAINTAINERS for platform/ti-vpe.

Thanks,
Mauro






      reply	other threads:[~2015-12-03 13:19 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-27 13:30 [GIT PULL FOR v4.5] New ti-cal driver Hans Verkuil
2015-12-03 13:19 ` Mauro Carvalho Chehab [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151203111912.6fdfc076@recife.lan \
    --to=mchehab@osg.samsung.com \
    --cc=bparrot@ti.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.