From: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
To: Sachin Kamat <sachin.kamat@linaro.org>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>,
linux-media@vger.kernel.org,
Kyungmin Park <kyungmin.park@samsung.com>
Subject: Re: [PATCH] Documentation: Update driver's directory in video4linux/fimc.txt
Date: Tue, 18 Jun 2013 23:57:25 +0200 [thread overview]
Message-ID: <51C0D7C5.6030707@gmail.com> (raw)
In-Reply-To: <CAK9yfHxpUPaKzrjCAX44rAB6pqOW2A2KD23A5SbwAWK+vJCbww@mail.gmail.com>
Hi Sachin,
On 06/18/2013 05:45 AM, Sachin Kamat wrote:
> Hi Sylwester,
>
>> -The machine code (plat-s5p and arch/arm/mach-*) must select following options
>> +The machine code (arch/arm/plat-samsung and arch/arm/mach-*) must select
>> +following options:
>
> After the recent platform code cleanup the below entries are not found
> in arch/arm/mach-* (checked in linux-next).
>
>> CONFIG_S5P_DEV_FIMC0 mandatory
>> CONFIG_S5P_DEV_FIMC1 \
AFAICS some are still there:
$ git grep S5P_DEV_FIMC0
Documentation/video4linux/fimc.txt:CONFIG_S5P_DEV_FIMC0 mandatory
arch/arm/mach-s5pc100/Kconfig: select S5P_DEV_FIMC0
arch/arm/mach-s5pv210/Kconfig: select S5P_DEV_FIMC0
arch/arm/mach-s5pv210/Kconfig: select S5P_DEV_FIMC0
arch/arm/mach-s5pv210/Kconfig: select S5P_DEV_FIMC0
arch/arm/mach-s5pv210/Kconfig: select S5P_DEV_FIMC0
arch/arm/plat-samsung/Kconfig:config S5P_DEV_FIMC0
arch/arm/plat-samsung/devs.c:#ifdef CONFIG_S5P_DEV_FIMC0
arch/arm/plat-samsung/devs.c:#endif /* CONFIG_S5P_DEV_FIMC0 */
arch/arm/plat-samsung/include/plat/fimc-core.h:#ifdef CONFIG_S5P_DEV_FIMC0
Those symbols have been removed only for Exynos. The above can be dropped
once we convert S5PV210 and S5PC100 (or drop support for this platform)
to Device Tree.
Anyway, I'm planning to update this FIMC documentation in a separate patch
to account for all the recent changes, especially addition of the FIMC-IS
support. But that will likely be only for 3.12.
Regards,
Sylwester
next prev parent reply other threads:[~2013-06-18 21:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-17 16:34 [PATCH] Documentation: Update driver's directory in video4linux/fimc.txt Sylwester Nawrocki
2013-06-18 3:45 ` Sachin Kamat
2013-06-18 21:57 ` Sylwester Nawrocki [this message]
2013-06-19 3:11 ` Sachin Kamat
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=51C0D7C5.6030707@gmail.com \
--to=sylvester.nawrocki@gmail.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-media@vger.kernel.org \
--cc=s.nawrocki@samsung.com \
--cc=sachin.kamat@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox