All of lore.kernel.org
 help / color / mirror / Atom feed
From: Beomho Seo <beomho.seo@samsung.com>
To: Kukjin Kim <kgene.kim@samsung.com>,
	linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org,
	'Tomasz Figa' <t.figa@samsung.com>
Cc: 'Myungjoo Ham' <myungjoo.ham@samsung.com>,
	'Chanwoo Choi' <cw00.choi@samsung.com>,
	'Jaehoon Chung' <jh80.chung@samsung.com>
Subject: Re: [RESEND PATCH 0/3] ARM: dts: add device data for exynos4412-trats2
Date: Fri, 02 May 2014 14:50:18 +0900	[thread overview]
Message-ID: <5363321A.10302@samsung.com> (raw)
In-Reply-To: <04ea01cf65c7$6bfd8db0$43f8a910$@samsung.com>

On 05/02/2014 02:29 PM, Kukjin Kim wrote:
> Beomho Seo wrote:
>>
>> Hi all,
>>
>> Didn't have Any other comment for this patch set?
>>
>> On 04/25/2014 11:04 AM, Beomho Seo wrote:
>>> This patchset add some device node for exynos4412-trats2.
>>> It is based on v3.15-next/dt-samsung-2 branch.
>>>
>>> exynos4412-trats2.dts
>>> - Fix incorrect compatible. Compatible of AK8975 are "ak8975" or "asahi-
>> kasei,ak8975".
>>> - Add cm36651 light/proximity sensor device node.
>>> - Change gpio-key device node. fix incorrect gpio property, and then add
>> "ok-key" node.
>>>
>>> Beomho Seo (3):
>>>   ARM: dts: fix incorrect compatible for exynos4412
>>>   ARM: dts: add cm36651 light/proximity sensor node
>>>   ARM: dts: fixed gpio key node for exynos4412
>>>
>>>  arch/arm/boot/dts/exynos4412-trats2.dts |   50
>> ++++++++++++++++++++++++++-----
>>>  1 file changed, 43 insertions(+), 7 deletions(-)
>>>
>>
> I have no objection on this series but need to get ack from dt guys.
> 
> BTW, shouldn't it include vendor name in compatible?...
> 
> And according to dt bindings(iio/magnetometer/ak8975.txt), it should be:
> 
> "compatible = "asahi-kasei,ak8975";"
> 
> even though "ak8975" also included in the driver "drivers/iio/magnetometer/ak8975.c"...
> 
> Thanks,
> Kukjin
> 

Thank you for comment.

Compatible of ak8975 will be fixed according to dt bindings(iio/magnetometer/ak8975.txt).
I will fix it and send v2 patch soon.

Thanks,
Beomho


> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

      reply	other threads:[~2014-05-02  5:50 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-26  7:08 [PATCH 0/3] ARM: dts: add device data for exynos4412-trats2 Beomho Seo
     [not found] ` <53327CE3.8030800-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-04-25  2:04   ` [RESEND PATCH " Beomho Seo
2014-05-02  4:29     ` Beomho Seo
2014-05-02  5:29       ` Kukjin Kim
2014-05-02  5:50         ` Beomho Seo [this message]

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=5363321A.10302@samsung.com \
    --to=beomho.seo@samsung.com \
    --cc=cw00.choi@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=jh80.chung@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=myungjoo.ham@samsung.com \
    --cc=t.figa@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.