From: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
To: Sachin Kamat <sachin.kamat@linaro.org>
Cc: linux-media <linux-media@vger.kernel.org>,
Inki Dae <inki.dae@samsung.com>,
Kamil Debski <k.debski@samsung.com>
Subject: Re: exynos4 / g2d
Date: Mon, 10 Feb 2014 09:28:16 +0100 [thread overview]
Message-ID: <52F88DA0.8000201@math.uni-bielefeld.de> (raw)
In-Reply-To: <CAK9yfHz7_9UWh=tTJeKyJh7V8yFtn8B4+ub=coV67ovRtebRfQ@mail.gmail.com>
Hello!
Sachin Kamat wrote:
> +cc linux-media list and some related maintainers
>
> Hi,
>
> On 10 February 2014 00:22, Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>> Hello!
>>
>> I noticed while here
>> (https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/boot/dts/exynos4x12.dtsi?id=3a0d48f6f81459c874165ffb14b310c0b5bb0c58)
>> the necessary entry for the dts was made, on the drm driver side
>> (https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/gpu/drm/exynos/exynos_drm_g2d.c)
>> this was never added.
>>
>> Shouldn't "samsung,exynos4212-g2d" go into exynos_g2d_match as well?
> The DRM version of G2D driver does not support Exynos4 based G2D IP
> yet. The support for this IP
> is available only in the V4L2 version of the driver. Please see the file:
> drivers/media/platform/s5p-g2d/g2d.c
>
That doesn't make sense to me. From the initial commit message of the
DRM code:
"The G2D is a 2D graphic accelerator that supports Bit Block Transfer.
This G2D driver is exynos drm specific and supports only G2D(version
4.1) of later Exynos series from Exynos4X12 because supporting DMA."
(https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/gpu/drm/exynos/exynos_drm_g2d.c?id=d7f1642c90ab5eb2d7c48af0581c993094f97e1a)
In fact, this doesn't even mention the Exynos5?!
Greets,
Tobias
next prev parent reply other threads:[~2014-02-10 8:37 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <52F7CE59.90009@math.uni-bielefeld.de>
2014-02-10 4:18 ` exynos4 / g2d Sachin Kamat
2014-02-10 8:28 ` Tobias Jakobi [this message]
2014-02-10 8:44 ` Sachin Kamat
2014-02-10 8:58 ` Inki Dae
2014-02-10 9:04 ` Sachin Kamat
2014-02-10 10:27 ` Tobias Jakobi
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=52F88DA0.8000201@math.uni-bielefeld.de \
--to=tjakobi@math.uni-bielefeld.de \
--cc=inki.dae@samsung.com \
--cc=k.debski@samsung.com \
--cc=linux-media@vger.kernel.org \
--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 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.