From: Mauro Carvalho Chehab <m.chehab@samsung.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Cc: g.liakhovetski@gmx.de, linux-media@vger.kernel.org,
magnus.damm@gmail.com, linux-sh@vger.kernel.org,
phil.edworthy@renesas.com, matsu@igel.co.jp,
vladimir.barinov@cogentembedded.com
Subject: Re: [PATCH v9] V4L2: soc_camera: Renesas R-Car VIN driver
Date: Sun, 18 Aug 2013 14:59:06 +0000 [thread overview]
Message-ID: <20130818115906.6010a695@samsung.com> (raw)
In-Reply-To: <5204063E.7050707@cogentembedded.com>
Em Fri, 09 Aug 2013 00:57:34 +0400
Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> escreveu:
> Hello.
>
> On 07/26/2013 12:23 AM, Sergei Shtylyov wrote:
>
> > From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> > Add Renesas R-Car VIN (Video In) V4L2 driver.
>
> > Based on the patch by Phil Edworthy <phil.edworthy@renesas.com>.
>
> > Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > [Sergei: removed deprecated IRQF_DISABLED flag, reordered/renamed 'enum chip_id'
> > values, reordered rcar_vin_id_table[] entries, removed senseless parens from
> > to_buf_list() macro, used ALIGN() macro in rcar_vin_setup(), added {} to the
> > *if* statement and used 'bool' values instead of 0/1 where necessary, removed
> > unused macros, done some reformatting and clarified some comments.]
> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> Guennadi, Mauro, if you don't have issues with this version, perhaps we
> still can merge it to 3.11 using "the new drivers can't cause regressions, so
> mergeable any time" rule?
AFAICT, while sometimes that happens, this is not a rule. The rule is to
merge new drivers during the merge window. What we during -rc is to accept
fix patches and even some updates for the new drivers merged at the very
latest merge window, as those won't cause regressions.
Anyway, IMO, we're too late at -rc cycle to propose this as an exception.
Regards,
Mauro
>
> WBR, Sergei
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Cheers,
Mauro
WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <m.chehab@samsung.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Cc: g.liakhovetski@gmx.de, linux-media@vger.kernel.org,
magnus.damm@gmail.com, linux-sh@vger.kernel.org,
phil.edworthy@renesas.com, matsu@igel.co.jp,
vladimir.barinov@cogentembedded.com
Subject: Re: [PATCH v9] V4L2: soc_camera: Renesas R-Car VIN driver
Date: Sun, 18 Aug 2013 11:59:06 -0300 [thread overview]
Message-ID: <20130818115906.6010a695@samsung.com> (raw)
In-Reply-To: <5204063E.7050707@cogentembedded.com>
Em Fri, 09 Aug 2013 00:57:34 +0400
Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> escreveu:
> Hello.
>
> On 07/26/2013 12:23 AM, Sergei Shtylyov wrote:
>
> > From: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
>
> > Add Renesas R-Car VIN (Video In) V4L2 driver.
>
> > Based on the patch by Phil Edworthy <phil.edworthy@renesas.com>.
>
> > Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > [Sergei: removed deprecated IRQF_DISABLED flag, reordered/renamed 'enum chip_id'
> > values, reordered rcar_vin_id_table[] entries, removed senseless parens from
> > to_buf_list() macro, used ALIGN() macro in rcar_vin_setup(), added {} to the
> > *if* statement and used 'bool' values instead of 0/1 where necessary, removed
> > unused macros, done some reformatting and clarified some comments.]
> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> Guennadi, Mauro, if you don't have issues with this version, perhaps we
> still can merge it to 3.11 using "the new drivers can't cause regressions, so
> mergeable any time" rule?
AFAICT, while sometimes that happens, this is not a rule. The rule is to
merge new drivers during the merge window. What we during -rc is to accept
fix patches and even some updates for the new drivers merged at the very
latest merge window, as those won't cause regressions.
Anyway, IMO, we're too late at -rc cycle to propose this as an exception.
Regards,
Mauro
>
> WBR, Sergei
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Cheers,
Mauro
next prev parent reply other threads:[~2013-08-18 14:59 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-25 20:23 [PATCH v9] V4L2: soc_camera: Renesas R-Car VIN driver Sergei Shtylyov
2013-07-25 20:23 ` Sergei Shtylyov
2013-08-08 20:57 ` Sergei Shtylyov
2013-08-08 20:57 ` Sergei Shtylyov
2013-08-18 14:59 ` Mauro Carvalho Chehab [this message]
2013-08-18 14:59 ` Mauro Carvalho Chehab
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=20130818115906.6010a695@samsung.com \
--to=m.chehab@samsung.com \
--cc=g.liakhovetski@gmx.de \
--cc=linux-media@vger.kernel.org \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=matsu@igel.co.jp \
--cc=phil.edworthy@renesas.com \
--cc=sergei.shtylyov@cogentembedded.com \
--cc=vladimir.barinov@cogentembedded.com \
/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.