From: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
To: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Cc: Hans Verkuil <hverkuil@xs4all.nl>,
linux-samsung-soc@vger.kernel.org, linux-media@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
Jacek Anaszewski <j.anaszewski@samsung.com>,
Kukjin Kim <kgene@kernel.org>,
Marek Szyprowski <m.szyprowski@samsung.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Krzysztof Kozlowski <k.kozlowski@samsung.com>
Subject: Re: [PATCH 4/4] ARM64: dts: exynos5433: add jpeg node
Date: Mon, 21 Sep 2015 14:55:28 +0200 [thread overview]
Message-ID: <55FFFE40.8070801@samsung.com> (raw)
In-Reply-To: <20150921084133.175b1bda@recife.lan>
Hi Mauro,
W dniu 21.09.2015 o 13:41, Mauro Carvalho Chehab pisze:
<snip>
>>>
>>> I think the media subsystem can take patches 1-3 and whoever does DT patches can
>>> take this patch, right?
>>>
>>
>> The cover letter explains that the series is rebased onto Mauro's
>> master with Kukjin's branch merged. The latter does contain
>> the exynos5433.dtsi. That said, yes, taking patches 1-3 in
>> media subsystem and leaving DT patch to someone else is the
>> way to go.
>
> I'm ok with such strategy, provided that the new driver builds fine with
> COMPILE_TEST without the need of the dtsi patch.
>
I've checked. It does compile with COMPILE_TEST.
> Please also notice that, if the driver uses MC, it has to wait for
> the MC next gen support to be merged,
No, it does not. It is a rather simple mem2mem device.
Sylwester has answered about camera support which is a different
topic.
Andrzej
WARNING: multiple messages have this Message-ID (diff)
From: andrzej.p@samsung.com (Andrzej Pietrasiewicz)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/4] ARM64: dts: exynos5433: add jpeg node
Date: Mon, 21 Sep 2015 14:55:28 +0200 [thread overview]
Message-ID: <55FFFE40.8070801@samsung.com> (raw)
In-Reply-To: <20150921084133.175b1bda@recife.lan>
Hi Mauro,
W dniu 21.09.2015 o 13:41, Mauro Carvalho Chehab pisze:
<snip>
>>>
>>> I think the media subsystem can take patches 1-3 and whoever does DT patches can
>>> take this patch, right?
>>>
>>
>> The cover letter explains that the series is rebased onto Mauro's
>> master with Kukjin's branch merged. The latter does contain
>> the exynos5433.dtsi. That said, yes, taking patches 1-3 in
>> media subsystem and leaving DT patch to someone else is the
>> way to go.
>
> I'm ok with such strategy, provided that the new driver builds fine with
> COMPILE_TEST without the need of the dtsi patch.
>
I've checked. It does compile with COMPILE_TEST.
> Please also notice that, if the driver uses MC, it has to wait for
> the MC next gen support to be merged,
No, it does not. It is a rather simple mem2mem device.
Sylwester has answered about camera support which is a different
topic.
Andrzej
next prev parent reply other threads:[~2015-09-21 12:55 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-18 14:20 [PATCH 0/4] Exynos 5433 jpeg h/w codec support Andrzej Pietrasiewicz
2015-09-18 14:20 ` Andrzej Pietrasiewicz
2015-09-18 14:20 ` [PATCH 1/4] s5p-jpeg: generalize clocks handling Andrzej Pietrasiewicz
2015-09-18 14:20 ` Andrzej Pietrasiewicz
2015-09-18 14:29 ` Jacek Anaszewski
2015-09-18 14:29 ` Jacek Anaszewski
2015-09-18 14:20 ` [PATCH 2/4] s5p-jpeg: add support for 5433 Andrzej Pietrasiewicz
2015-09-18 14:20 ` Andrzej Pietrasiewicz
2015-09-18 14:30 ` Jacek Anaszewski
2015-09-18 14:30 ` Jacek Anaszewski
2015-09-18 14:20 ` [PATCH 3/4] MAINTAINERS: add exynos jpeg codec maintainers Andrzej Pietrasiewicz
2015-09-18 14:20 ` Andrzej Pietrasiewicz
2015-09-18 14:30 ` Jacek Anaszewski
2015-09-18 14:30 ` Jacek Anaszewski
2015-09-18 14:21 ` [PATCH 4/4] ARM64: dts: exynos5433: add jpeg node Andrzej Pietrasiewicz
2015-09-18 14:21 ` Andrzej Pietrasiewicz
2015-09-21 9:50 ` Hans Verkuil
2015-09-21 9:50 ` Hans Verkuil
2015-09-21 9:59 ` Andrzej Pietrasiewicz
2015-09-21 9:59 ` Andrzej Pietrasiewicz
2015-09-21 11:41 ` Mauro Carvalho Chehab
2015-09-21 11:41 ` Mauro Carvalho Chehab
2015-09-21 12:28 ` Sylwester Nawrocki
2015-09-21 12:28 ` Sylwester Nawrocki
2015-09-21 12:55 ` Andrzej Pietrasiewicz [this message]
2015-09-21 12:55 ` Andrzej Pietrasiewicz
2015-09-28 8:52 ` Krzysztof Kozlowski
2015-09-28 8:52 ` Krzysztof Kozlowski
2015-09-28 11:54 ` Bartlomiej Zolnierkiewicz
2015-09-28 11:54 ` Bartlomiej Zolnierkiewicz
2015-09-28 12:02 ` Krzysztof Kozlowski
2015-09-28 12:02 ` Krzysztof Kozlowski
2015-09-30 6:35 ` Kukjin Kim
2015-09-30 6:35 ` Kukjin Kim
2015-09-30 6:46 ` Chanwoo Choi
2015-09-30 6:46 ` Chanwoo Choi
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=55FFFE40.8070801@samsung.com \
--to=andrzej.p@samsung.com \
--cc=b.zolnierkie@samsung.com \
--cc=hverkuil@xs4all.nl \
--cc=j.anaszewski@samsung.com \
--cc=k.kozlowski@samsung.com \
--cc=kgene@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=mchehab@osg.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.