Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property with wakeup-source

From	Krzysztof Kozlowski <k.kozlowski@samsung.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property with wakeup-source
Date	2015-12-17 00:40 +0100
Message-ID	<qGtPQ-8mr-5@gated-at.bofh.it> (permalink)
References	<qlYEV-8lO-1@gated-at.bofh.it> <qlYEX-8lO-55@gated-at.bofh.it> <qG0NR-6Hh-41@gated-at.bofh.it> <qG7mh-2sQ-11@gated-at.bofh.it> <qGhF0-zW-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 16.12.2015 19:32, Sudeep Holla wrote:
> 
> 
> On 15/12/15 23:32, Krzysztof Kozlowski wrote:
>> On 16.12.2015 01:35, Sudeep Holla wrote:
>>>
>>>
>>> On 21/10/15 11:10, Sudeep Holla wrote:
>>>> Though the keyboard and other driver will continue to support the
>>>> legacy
>>>> "gpio-key,wakeup", "linux-keypad,wakeup" boolean property to enable the
>>>> wakeup source, "wakeup-source" is the new standard binding.
>>>>
>>>> This patch replaces all the legacy wakeup properties with the unified
>>>> "wakeup-source" property in order to avoid any futher copy-paste
>>>> duplication.
>>>>
>>>> Cc: Kukjin Kim <kgene@kernel.org>
>>>> Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>>
>>> Ping, do you prefer taking via your tree or should I send to armsoc
>>> directly ?
>>
>> Usually it depends on the dependencies... but:
>> 1. I just received this patch - 12/19.
> 
> Sorry for that. The initial idea was to show all the changes but merge
> the binding and dependencies first, so didn't cc all.
> 
>> 2. No cover letter, no other patches, no explanation about any
>> dependencies or other related changes. Nothing. Just one patch.
> 
> Yes this was initially posted as part of the series, but only
> dependencies were merged for v4.4
> 
>> 3. The patch mentions new standard binding. Was the new binding already
>> merged? Or is it part of these series? What about backward compatibility?
> 
> Not a brand new binding, but made existing one a standard so that we
> don't continue to be creative and generate more bindings for the same
> thing. Yes binding and dependent patches are already in mainline. And
> yes backward compatibility is maintained as mentioned in the commit
> message.

That explains everything, thanks. We'll take it through samsung-soc,
just give me some time for review.

As for other patch - the s5pv210 - I did not get it directly but I think
I can find it on list subscription archives.

Best regards,
Krzysztof


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property  with wakeup-source Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-12-16 00:40 +0100
  Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property  with wakeup-source Sudeep Holla <sudeep.holla@arm.com> - 2015-12-16 11:40 +0100
    Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property  with wakeup-source Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-12-17 00:40 +0100

csiph-web