devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: Rahul Sharma <r.sh.open@gmail.com>
Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
	Rahul Sharma <rahul.sharma@samsung.com>,
	linux-samsung-soc@vger.kernel.org,
	devicetree-discuss@lists.ozlabs.org, kgene.kim@samsung.com,
	inki.dae@samsung.com, joshi@samsung.com
Subject: Re: [PATCH 2/4] ARM: dts: add support for mixer to exynos4
Date: Mon, 28 Jan 2013 09:38:55 +0100	[thread overview]
Message-ID: <1461207.5d9zQy0FHx@flatron> (raw)
In-Reply-To: <CAPdUM4P_jf0w+B22p1hmso8uhmTXs=fLdF8WrzDWL8cPyLeTCw@mail.gmail.com>

Hi Rahul, Sylwester,

On Monday 28 of January 2013 12:14:38 Rahul Sharma wrote:
> On Mon, Jan 28, 2013 at 2:34 AM, Sylwester Nawrocki
> 
> <sylvester.nawrocki@gmail.com> wrote:
> > On 01/25/2013 11:39 AM, Rahul Sharma wrote:
> >> diff --git a/arch/arm/boot/dts/exynos4.dtsi
> >> b/arch/arm/boot/dts/exynos4.dtsi
> >> index 6c2c79d..07cb7d4 100644
> >> --- a/arch/arm/boot/dts/exynos4.dtsi
> >> +++ b/arch/arm/boot/dts/exynos4.dtsi
> >> @@ -296,4 +296,12 @@
> >> 
> >>                 interrupts =<0 95 0>;
> >>                 hpd-gpio =<&gpx3 7 0xf 1 3>;
> >>         
> >>         };
> >> 
> >> +
> >> +       mixer: mixer {
> >> +               compatible = "samsung,exynos4-mixer";
> > 
> > I'm afraid this compatible property it too generic. Is the video mixer
> > IP really identical an all the Exynos4 series SoCs ?
> 
> It is same in Exynos4210, 4212 and 4412.

The convention is to use the name of SoC on which this IP version appeared 
first. In this case it would be samsung,exynos4210-mixer.

Best regards,
Tomasz

  reply	other threads:[~2013-01-28  8:38 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-25 10:39 [PATCH 0/4] 0/4] ARM: dts: add support for hdmi to exynos4412 Rahul Sharma
2013-01-25 10:39 ` [PATCH 1/4] ARM: dts: add support for hdmi to exynos4 Rahul Sharma
     [not found]   ` <1359110384-28402-2-git-send-email-rahul.sharma-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-01-25 17:40     ` Thomas Abraham
2013-01-28  6:55       ` Rahul Sharma
2013-01-28  8:44         ` Tomasz Figa
2013-01-28  9:23           ` Rahul Sharma
2013-01-28 18:48             ` Tomasz Figa
2013-01-27 20:54   ` Sylwester Nawrocki
2013-01-28  6:19     ` Rahul Sharma
2013-01-29 20:42       ` Sylwester Nawrocki
2013-01-31  8:09         ` Rahul Sharma
2013-01-31 22:25           ` Kukjin Kim
2013-02-01  4:04             ` Rahul Sharma
2013-01-25 10:39 ` [PATCH 2/4] ARM: dts: add support for mixer " Rahul Sharma
2013-01-27 21:04   ` Sylwester Nawrocki
2013-01-28  6:44     ` Rahul Sharma
2013-01-28  8:38       ` Tomasz Figa [this message]
2013-01-29 20:15         ` Sylwester Nawrocki
2013-01-29 20:24       ` Sylwester Nawrocki
2013-01-31  8:33         ` Rahul Sharma
2013-01-25 10:39 ` [PATCH 3/4] ARM: dts: add support for hdmi-ddc to exynos4412-origen Rahul Sharma
2013-01-27 20:37   ` Sylwester Nawrocki
2013-01-25 10:39 ` [PATCH 4/4] ARM: dts: add support for hdmi-phy to exynos4412 Rahul Sharma
2013-01-27 20:43 ` [PATCH 0/4] 0/4] ARM: dts: add support for hdmi " Sylwester Nawrocki

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=1461207.5d9zQy0FHx@flatron \
    --to=tomasz.figa@gmail.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=inki.dae@samsung.com \
    --cc=joshi@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=r.sh.open@gmail.com \
    --cc=rahul.sharma@samsung.com \
    --cc=sylvester.nawrocki@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).