All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
To: Sudeep Holla <sudeep.holla@arm.com>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	Kukjin Kim <kgene@kernel.org>
Cc: linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property with wakeup-source
Date: Thu, 17 Dec 2015 08:38:03 +0900	[thread overview]
Message-ID: <5671F5DB.305@samsung.com> (raw)
In-Reply-To: <56713DD8.1050001@arm.com>

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

WARNING: multiple messages have this Message-ID (diff)
From: k.kozlowski@samsung.com (Krzysztof Kozlowski)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup property with wakeup-source
Date: Thu, 17 Dec 2015 08:38:03 +0900	[thread overview]
Message-ID: <5671F5DB.305@samsung.com> (raw)
In-Reply-To: <56713DD8.1050001@arm.com>

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

  reply	other threads:[~2015-12-16 23:38 UTC|newest]

Thread overview: 180+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-21 10:09 [PATCH 00/19] devicetree: standardize/consolidate on "wakeup-source" property Sudeep Holla
2015-10-21 10:09 ` Sudeep Holla
2015-10-21 10:09 ` [PATCH 01/19] Documentation: " Sudeep Holla
2015-10-21 10:09   ` Sudeep Holla
2015-10-21 10:09   ` Sudeep Holla
2015-10-21 22:02   ` Rob Herring
2015-10-21 22:02     ` Rob Herring
2015-10-21 22:02     ` Rob Herring
2015-10-21 10:09 ` [PATCH 02/19] Documentation: devicetree: fix reference to legacy wakeup properties Sudeep Holla
2015-10-21 10:09   ` Sudeep Holla
2015-10-21 10:09   ` Sudeep Holla
2015-10-21 22:04   ` Rob Herring
2015-10-21 22:04     ` Rob Herring
2015-10-21 22:04     ` Rob Herring
2015-10-28 18:23     ` Sudeep Holla
2015-10-28 18:23       ` Sudeep Holla
2015-10-30 18:29       ` Rob Herring
2015-10-30 18:29         ` Rob Herring
2015-10-30 18:29         ` Rob Herring
2015-10-21 10:10 ` [rtc-linux] [PATCH 03/19] rtc: opal: enable support for the stardard "wakeup-source" property Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-27 22:53   ` [rtc-linux] " Alexandre Belloni
2015-10-27 22:53     ` Alexandre Belloni
2015-10-27 22:53     ` Alexandre Belloni
2015-10-27 22:53     ` Alexandre Belloni
2015-10-21 10:10 ` [rtc-linux] [PATCH 04/19] rtc: isl12057: enable support for the standard " Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-27 22:53   ` [rtc-linux] " Alexandre Belloni
2015-10-27 22:53     ` Alexandre Belloni
2015-10-27 22:53     ` Alexandre Belloni
2015-10-21 10:10 ` [PATCH 05/19] mmc: core/host: " Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
     [not found]   ` <1445422216-29375-6-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2015-11-19 11:32     ` Ulf Hansson
2015-11-19 11:32       ` Ulf Hansson
2015-11-19 11:32       ` Ulf Hansson
2015-10-21 10:10 ` [PATCH 06/19] input: tegra-kbc: " Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
     [not found]   ` <1445422216-29375-7-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2015-10-23  6:39     ` Dmitry Torokhov
2015-10-23  6:39       ` Dmitry Torokhov
2015-10-23  6:39       ` Dmitry Torokhov
2015-10-23 10:23       ` Sudeep Holla
2015-10-23 10:23         ` Sudeep Holla
2015-10-23 10:23         ` Sudeep Holla
2015-10-26  8:33         ` Dmitry Torokhov
2015-10-26  8:33           ` Dmitry Torokhov
2015-10-21 10:10 ` [PATCH 08/19] ARM: dts: spear: replace gpio-key, wakeup with wakeup-source property Sudeep Holla
2015-10-21 10:10   ` [PATCH 08/19] ARM: dts: spear: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 08/19] ARM: dts: spear: replace gpio-key, wakeup " Sudeep Holla
2015-10-21 10:10 ` [PATCH 09/19] ARM: dts: am335x: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 09/19] ARM: dts: am335x: replace gpio-key, wakeup " Sudeep Holla
     [not found]   ` <1445422216-29375-10-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2015-12-15 16:33     ` [PATCH 09/19] ARM: dts: am335x: replace gpio-key,wakeup " Sudeep Holla
2015-12-15 16:33       ` Sudeep Holla
2015-12-15 16:33       ` Sudeep Holla
     [not found]       ` <567040E7.7060308-5wv7dgnIgG8@public.gmane.org>
2015-12-17 17:58         ` Tony Lindgren
2015-12-17 17:58           ` Tony Lindgren
2015-12-17 17:58           ` Tony Lindgren
2015-10-21 10:10 ` [PATCH 10/19] ARM: dts: at91: replace gpio-key, wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 10/19] ARM: dts: at91: replace gpio-key, wakeup " Sudeep Holla
2015-10-21 10:21   ` [PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup " Nicolas Ferre
2015-10-21 10:21     ` Nicolas Ferre
2015-10-21 10:21     ` Nicolas Ferre
2015-10-21 10:25     ` Sudeep Holla
2015-10-21 10:25       ` Sudeep Holla
2015-10-21 10:25       ` Sudeep Holla
2015-10-21 11:01       ` Ulf Hansson
2015-10-21 11:01         ` Ulf Hansson
2015-10-21 11:13         ` Sudeep Holla
2015-10-21 11:13           ` Sudeep Holla
2015-10-21 11:13           ` Sudeep Holla
2015-10-21 12:41           ` Sudeep Holla
2015-10-21 12:41             ` Sudeep Holla
2015-10-21 12:41             ` Sudeep Holla
2015-10-21 15:27             ` Nicolas Ferre
2015-10-21 15:27               ` Nicolas Ferre
2015-10-21 15:27               ` Nicolas Ferre
2015-10-21 10:10 ` [PATCH 11/19] ARM: dts: imx: replace legacy *, wakeup property with wakeup-source Sudeep Holla
2015-10-21 10:10   ` [PATCH 11/19] ARM: dts: imx: replace legacy *,wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 11/19] ARM: dts: imx: replace legacy *, wakeup " Sudeep Holla
2015-11-23  5:37   ` [PATCH 11/19] ARM: dts: imx: replace legacy *,wakeup " Shawn Guo
2015-11-23  5:37     ` Shawn Guo
2015-11-23  5:37     ` Shawn Guo
2015-11-26 11:33     ` Sudeep Holla
2015-11-26 11:33       ` Sudeep Holla
2015-11-26 11:33       ` Sudeep Holla
2015-12-02  1:54       ` Shawn Guo
2015-12-02  1:54         ` Shawn Guo
2015-12-02 10:05         ` Sudeep Holla
2015-12-02 10:05           ` Sudeep Holla
     [not found] ` <1445422216-29375-1-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2015-10-21 10:10   ` [PATCH 07/19] ARM: dts: rockchip: replace gpio-key, wakeup with wakeup-source property Sudeep Holla
2015-10-21 10:10     ` [PATCH 07/19] ARM: dts: rockchip: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:10     ` [PATCH 07/19] ARM: dts: rockchip: replace gpio-key, wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property with wakeup-source Sudeep Holla
2015-10-21 10:10     ` Sudeep Holla
2015-10-21 10:10     ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Sudeep Holla
2015-12-15 16:35     ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Sudeep Holla
2015-12-15 16:35       ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Sudeep Holla
     [not found]       ` <56704141.8070308-5wv7dgnIgG8@public.gmane.org>
2015-12-15 23:32         ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Krzysztof Kozlowski
2015-12-15 23:32           ` Krzysztof Kozlowski
2015-12-15 23:32           ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Krzysztof Kozlowski
     [not found]           ` <5670A2F2.5060009-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-12-16 10:32             ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Sudeep Holla
2015-12-16 10:32               ` Sudeep Holla
2015-12-16 10:32               ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Sudeep Holla
2015-12-16 23:38               ` Krzysztof Kozlowski [this message]
2015-12-16 23:38                 ` Krzysztof Kozlowski
2015-12-16 23:49     ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Krzysztof Kozlowski
2015-12-16 23:49       ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Krzysztof Kozlowski
2016-01-28  8:11       ` [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Krzysztof Kozlowski
2016-01-28  8:11         ` [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Krzysztof Kozlowski
     [not found]         ` <56A9CD1A.8010800-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2016-01-28 15:59           ` [UPDATE] [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Sudeep Holla
2016-01-28 15:59             ` Sudeep Holla
2016-01-28 15:59             ` [UPDATE] [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Sudeep Holla
2016-01-29  0:11             ` [UPDATE] [PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup " Krzysztof Kozlowski
2016-01-29  0:11               ` [UPDATE] [PATCH 12/19] ARM: dts: exynos: replace legacy *, wakeup " Krzysztof Kozlowski
2015-10-21 10:10 ` [PATCH 13/19] ARM: dts: tegra: replace legacy *,wakeup " Sudeep Holla
2015-10-21 10:10   ` [PATCH 13/19] ARM: dts: tegra: replace legacy *, wakeup " Sudeep Holla
2015-12-15 16:35   ` [PATCH 13/19] ARM: dts: tegra: replace legacy *,wakeup " Sudeep Holla
2015-12-15 16:35     ` Sudeep Holla
     [not found]   ` <1445422216-29375-14-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org>
2015-12-23 11:24     ` Sudeep Holla
2015-12-23 11:24       ` Sudeep Holla
2015-12-23 11:24       ` Sudeep Holla
2015-10-21 10:10 ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-21 10:10   ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key, wakeup " Sudeep Holla
2015-10-21 10:18   ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup " Geert Uytterhoeven
2015-10-21 10:18     ` Geert Uytterhoeven
2015-10-21 10:18     ` Geert Uytterhoeven
2015-10-21 10:18     ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key, wakeup " Geert Uytterhoeven
2015-10-21 10:23     ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:23       ` Sudeep Holla
2015-10-21 10:23       ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key, wakeup " Sudeep Holla
2015-10-22  0:50       ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup " Simon Horman
2015-10-22  0:50         ` Simon Horman
2015-10-22  0:50         ` Simon Horman
2015-10-22  9:37         ` Sudeep Holla
2015-10-22  9:37           ` Sudeep Holla
2015-10-22  9:37           ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key, wakeup " Sudeep Holla
2015-10-23  0:18           ` [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup " Simon Horman
2015-10-23  0:18             ` Simon Horman
2015-10-23  0:18             ` Simon Horman
2015-10-23  0:18             ` Simon Horman
2015-10-21 10:10 ` [PATCH 15/19] ARM: dts: omap: replace legacy *,wakeup property with wakeup-source Sudeep Holla
2015-10-21 10:10   ` [PATCH 15/19] ARM: dts: omap: replace legacy *, wakeup " Sudeep Holla
2015-12-15 16:37   ` [PATCH 15/19] ARM: dts: omap: replace legacy *,wakeup " Sudeep Holla
2015-12-15 16:37     ` Sudeep Holla
2015-12-17 17:58     ` Tony Lindgren
2015-12-17 17:58       ` Tony Lindgren
2015-10-21 10:10 ` [PATCH 16/19] ARM: dts: s5pv210: " Sudeep Holla
2015-10-21 10:10   ` [PATCH 16/19] ARM: dts: s5pv210: replace legacy *, wakeup " Sudeep Holla
2015-12-15 16:37   ` [PATCH 16/19] ARM: dts: s5pv210: replace legacy *,wakeup " Sudeep Holla
2015-12-15 16:37     ` [PATCH 16/19] ARM: dts: s5pv210: replace legacy *, wakeup " Sudeep Holla
2015-12-16 23:52   ` [PATCH 16/19] ARM: dts: s5pv210: replace legacy *,wakeup " Krzysztof Kozlowski
2015-12-16 23:52     ` [PATCH 16/19] ARM: dts: s5pv210: replace legacy *, wakeup " Krzysztof Kozlowski
2015-10-21 10:10 ` [PATCH 17/19] ARM: dts: armada: replace isil, irq2-can-wakeup-machine with wakeup-source property Sudeep Holla
2015-10-21 10:10   ` [PATCH 17/19] ARM: dts: armada: replace isil,irq2-can-wakeup-machine " Sudeep Holla
2015-12-15 16:39   ` Sudeep Holla
2015-12-15 16:39     ` Sudeep Holla
2015-12-15 16:56     ` [PATCH 17/19] ARM: dts: armada: replace isil, irq2-can-wakeup-machine " Gregory CLEMENT
2015-12-15 16:56       ` [PATCH 17/19] ARM: dts: armada: replace isil,irq2-can-wakeup-machine " Gregory CLEMENT
2015-10-21 10:10 ` [PATCH 18/19] ARM: dts: ste: replace legacy *, wakeup property with wakeup-source Sudeep Holla
2015-10-21 10:10   ` [PATCH 18/19] ARM: dts: ste: replace legacy *,wakeup " Sudeep Holla
2015-10-27 12:34   ` Linus Walleij
2015-10-27 12:34     ` Linus Walleij
2015-10-27 12:34     ` Linus Walleij
2015-10-27 12:46     ` Sudeep Holla
2015-10-27 12:46       ` Sudeep Holla
2015-10-27 12:46       ` Sudeep Holla
2015-11-05  9:26       ` Linus Walleij
2015-11-05  9:26         ` Linus Walleij
2015-11-05  9:26         ` Linus Walleij
2015-10-21 10:10 ` [PATCH 19/19] ARM: dts: zynq: replace gpio-key, wakeup with wakeup-source property Sudeep Holla
2015-10-21 10:10   ` [PATCH 19/19] ARM: dts: zynq: replace gpio-key,wakeup " Sudeep Holla
2015-10-21 10:10   ` Sudeep Holla
2015-10-29  8:56   ` Michal Simek
2015-10-29  8:56     ` Michal Simek
2015-10-29  8:56     ` Michal Simek

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=5671F5DB.305@samsung.com \
    --to=k.kozlowski@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=sudeep.holla@arm.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.