All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>
Cc: LMML <linux-media@vger.kernel.org>
Subject: Re: [GIT PULL FOR 3.7] Samsung media drivers fixes
Date: Sat, 27 Oct 2012 08:19:59 -0200	[thread overview]
Message-ID: <20121027081959.4db3aecc@redhat.com> (raw)
In-Reply-To: <508A62CD.5090909@samsung.com>

Em Fri, 26 Oct 2012 12:15:41 +0200
Sylwester Nawrocki <s.nawrocki@samsung.com> escreveu:

> On 10/25/2012 08:59 PM, Sylwester Nawrocki wrote:
> > On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote:
> >> Em Thu, 25 Oct 2012 18:01:17 +0200
> >> Sylwester Nawrocki<s.nawrocki@samsung.com>  escreveu:
> >>
> >>> Hi Mauro,
> >>>
> >>> please pull following fixes for v3.7-rc.
> >>>
> >>> The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c:
> >>>
> >>>    [media] m5mols: Add missing #include<linux/sizes.h>  (2012-10-10 08:17:16 -0300)
> >>>
> >>> are available in the git repository at:
> >>>
> >>>    git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7
> >>>
> >>> for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b:
> >>>
> >>>    s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200)
> >>>
> >>> ----------------------------------------------------------------
> >>> Jesper Juhl (1):
> >>>        s5p-tv: don't include linux/version.h in mixer_video.c
> >>>
> >>> Sachin Kamat (5):
> >>>        s5p-mfc: Fix compilation warning
> >>>        exynos-gsc: Fix compilation warning
> >>>        s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c
> >>>        s5p-fimc: Make 'fimc_pipeline_s_stream' function static
> >>>        s5p-fimc: Fix potential NULL pointer dereference
> >>>
> >>> Shaik Ameer Basha (3):
> >>>        exynos-gsc: change driver compatible string
> >>>        exynos-gsc: fix variable type in gsc_m2m_device_run()
> >>>        s5p-fimc: fix variable type in fimc_device_run()
> >>>
> >>> Sylwester Nawrocki (4):
> >>>        s5p-fimc: Don't ignore return value of vb2_queue_init()
> >>>        s5p-csis: Select S5P_SETUP_MIPIPHY
> >>>        s5p-fimc: Add missing new line character
> >>>        s5p-fimc: Fix platform entities registration
> >>
> >>
> >> Only a few of the above seems to be material for -rc:
> >> 	s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago)
> >> 	s5p-fimc: Fix platform entities registration (60 seconds ago)
> >> 	s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago)
> >> 	s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago)
> >>
> >> The other ones are warnings/sparse warnings and cleanups. I'll
> >> be applying only the 4 above patches for 3.7, adding the other
> >> ones for 3.8.
> 
> I've noticed "s5p-fimc: Fix platform entities registration" will cause
> a race condition. Can you please drop entirely these two patches from
> the 3.7 queue:
> 
>  	s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago)
>  	s5p-fimc: Fix platform entities registration (60 seconds ago)
> ?
> 
> Alternatively I can send a subsequent follow up patch.

As I already applied pinchartl patches on my 3.7 sandbox, I prefer if you
could send it as a follow up patch.

Regards,
Mauro

  reply	other threads:[~2012-10-27 10:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-25 16:01 [GIT PULL FOR 3.7] Samsung media drivers fixes Sylwester Nawrocki
2012-10-25 16:46 ` Mauro Carvalho Chehab
2012-10-25 18:59   ` Sylwester Nawrocki
2012-10-26 10:15     ` Sylwester Nawrocki
2012-10-27 10:19       ` Mauro Carvalho Chehab [this message]
2012-10-27 10:19     ` 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=20121027081959.4db3aecc@redhat.com \
    --to=mchehab@redhat.com \
    --cc=linux-media@vger.kernel.org \
    --cc=s.nawrocki@samsung.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.