All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alim Akhtar <alim.akhtar@samsung.com>
To: Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	linux-samsung-soc@vger.kernel.org
Cc: kgene@kernel.org, javier@osg.samsung.com,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/7] Switch to generic syscon regmap based drivers
Date: Mon, 19 Oct 2015 13:10:14 +0530	[thread overview]
Message-ID: <56249E5E.80405@samsung.com> (raw)
In-Reply-To: <562493BF.9080301@samsung.com>



On 10/19/2015 12:24 PM, Krzysztof Kozlowski wrote:
> On 19.10.2015 15:03, Alim Akhtar wrote:
> ...
>
> Hi,
>
> I am happy to see more contributions from your side! Keep it up. :)
>
Thank you! will keep finding time and doing it.

> However this empty space in your cover letter is the place for
> description of the patchset. What do you want to achieve here? What kind
> of problem are you solving? What features are you adding? Any dependencies?
>
Ah, my bad, I used --compose option with git-send-email and did composed 
the message, somehow it got missed. Will take care.

The motivation came when I tried to used reboot/poweroff on exynos7, and 
since exynos7 has not added/enabled PMU and does not have access to 
arch/arm/mach-exynos, so option are to have a reboot/restart driver or 
we move pmu.c out of mach-exynos. I know there where some effort in past 
to move pmu.c to drivers, but I feel that is still WIP.
And then I came across this generic syscon regmap based driver which are 
already in place, so just used it.
Hope that helps.

> Best regards,
> Krzysztof
>
>> Alim Akhtar (7):
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos3250 SoCs
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos4
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5410 SoC
>>    ARM: exynos_defconfig: Normalize exynos defconfig
>>    ARM: exynos_defconfig: Enable generic syscon-{reboot, poweroff}
>>      drivers
>>    ARM: EXYNOS: Remove code for restart and poweroff for exynos SoCs
>>
>>   arch/arm/boot/dts/exynos3250.dtsi |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos4.dtsi    |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos5.dtsi    |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos5410.dtsi |   14 ++++++++++++
>>   arch/arm/configs/exynos_defconfig |   11 +++++-----
>>   arch/arm/mach-exynos/pmu.c        |   43 -------------------------------------
>>   6 files changed, 61 insertions(+), 49 deletions(-)
>>
>
>

WARNING: multiple messages have this Message-ID (diff)
From: alim.akhtar@samsung.com (Alim Akhtar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/7] Switch to generic syscon regmap based drivers
Date: Mon, 19 Oct 2015 13:10:14 +0530	[thread overview]
Message-ID: <56249E5E.80405@samsung.com> (raw)
In-Reply-To: <562493BF.9080301@samsung.com>



On 10/19/2015 12:24 PM, Krzysztof Kozlowski wrote:
> On 19.10.2015 15:03, Alim Akhtar wrote:
> ...
>
> Hi,
>
> I am happy to see more contributions from your side! Keep it up. :)
>
Thank you! will keep finding time and doing it.

> However this empty space in your cover letter is the place for
> description of the patchset. What do you want to achieve here? What kind
> of problem are you solving? What features are you adding? Any dependencies?
>
Ah, my bad, I used --compose option with git-send-email and did composed 
the message, somehow it got missed. Will take care.

The motivation came when I tried to used reboot/poweroff on exynos7, and 
since exynos7 has not added/enabled PMU and does not have access to 
arch/arm/mach-exynos, so option are to have a reboot/restart driver or 
we move pmu.c out of mach-exynos. I know there where some effort in past 
to move pmu.c to drivers, but I feel that is still WIP.
And then I came across this generic syscon regmap based driver which are 
already in place, so just used it.
Hope that helps.

> Best regards,
> Krzysztof
>
>> Alim Akhtar (7):
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos3250 SoCs
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos4
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5
>>    arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5410 SoC
>>    ARM: exynos_defconfig: Normalize exynos defconfig
>>    ARM: exynos_defconfig: Enable generic syscon-{reboot, poweroff}
>>      drivers
>>    ARM: EXYNOS: Remove code for restart and poweroff for exynos SoCs
>>
>>   arch/arm/boot/dts/exynos3250.dtsi |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos4.dtsi    |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos5.dtsi    |   14 ++++++++++++
>>   arch/arm/boot/dts/exynos5410.dtsi |   14 ++++++++++++
>>   arch/arm/configs/exynos_defconfig |   11 +++++-----
>>   arch/arm/mach-exynos/pmu.c        |   43 -------------------------------------
>>   6 files changed, 61 insertions(+), 49 deletions(-)
>>
>
>

  reply	other threads:[~2015-10-19  7:50 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-19  6:03 [PATCH 0/7] Switch to generic syscon regmap based drivers Alim Akhtar
2015-10-19  6:03 ` Alim Akhtar
2015-10-19  6:03 ` [PATCH 1/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos3250 SoCs Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  8:43   ` [1/7] " Pankaj Dubey
2015-10-19  8:43     ` Pankaj Dubey
2015-10-19 13:03   ` [PATCH 1/7] " Javier Martinez Canillas
2015-10-19 13:03     ` Javier Martinez Canillas
2015-10-20  9:00     ` Alim Akhtar
2015-10-20  9:00       ` Alim Akhtar
2015-10-19  6:03 ` [PATCH 2/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos4 Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  8:45   ` [2/7] " Pankaj Dubey
2015-10-19  8:45     ` Pankaj Dubey
2015-10-19 13:08   ` [PATCH 2/7] " Javier Martinez Canillas
2015-10-19 13:08     ` Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 3/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5 Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  8:47   ` [3/7] " Pankaj Dubey
2015-10-19  8:47     ` Pankaj Dubey
2015-10-19 13:09   ` [PATCH 3/7] " Javier Martinez Canillas
2015-10-19 13:09     ` Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 4/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5410 SoC Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  8:56   ` [4/7] " Pankaj Dubey
2015-10-19  8:56     ` Pankaj Dubey
2015-10-19 13:10   ` [PATCH 4/7] " Javier Martinez Canillas
2015-10-19 13:10     ` Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 5/7] ARM: exynos_defconfig: Normalize exynos defconfig Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19 13:25   ` Javier Martinez Canillas
2015-10-19 13:25     ` Javier Martinez Canillas
2015-10-19 15:58     ` Alim Akhtar
2015-10-19 15:58       ` Alim Akhtar
2015-10-19 16:38       ` Javier Martinez Canillas
2015-10-19 16:38         ` Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 6/7] ARM: exynos_defconfig: Enable generic syscon-{reboot, poweroff} drivers Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  6:03 ` [PATCH 7/7] ARM: EXYNOS: Remove code for restart and poweroff for exynos SoCs Alim Akhtar
2015-10-19  6:03   ` Alim Akhtar
2015-10-19  7:00   ` Krzysztof Kozlowski
2015-10-19  7:00     ` Krzysztof Kozlowski
2015-10-19  8:27     ` Alim Akhtar
2015-10-19  8:27       ` Alim Akhtar
2015-10-19  9:56     ` Javier Martinez Canillas
2015-10-19  9:56       ` Javier Martinez Canillas
2015-10-19 10:23       ` Alim Akhtar
2015-10-19 10:23         ` Alim Akhtar
2015-10-19 13:28       ` Krzysztof Kozlowski
2015-10-19 13:28         ` Krzysztof Kozlowski
2015-10-19 14:07         ` Javier Martinez Canillas
2015-10-19 14:07           ` Javier Martinez Canillas
2015-10-19 16:13           ` Alim Akhtar
2015-10-19 16:13             ` Alim Akhtar
2015-10-19 16:42             ` Javier Martinez Canillas
2015-10-19 16:42               ` Javier Martinez Canillas
2015-10-19 16:06         ` Alim Akhtar
2015-10-19 16:06           ` Alim Akhtar
2015-10-20  0:20           ` Krzysztof Kozlowski
2015-10-20  0:20             ` Krzysztof Kozlowski
2015-10-19  6:54 ` [PATCH 0/7] Switch to generic syscon regmap based drivers Krzysztof Kozlowski
2015-10-19  6:54   ` Krzysztof Kozlowski
2015-10-19  7:40   ` Alim Akhtar [this message]
2015-10-19  7:40     ` Alim Akhtar
2015-10-19 16:08 ` Tobias Jakobi
2015-10-19 16:08   ` Tobias Jakobi
2015-10-19 16:36   ` Alim Akhtar
2015-10-19 16:36     ` Alim Akhtar
2015-10-19 21:18 ` Moritz Fischer
2015-10-19 21:18   ` Moritz Fischer
2015-10-20  8:56   ` Alim Akhtar
2015-10-20  8:56     ` Alim Akhtar
2015-10-20 13:43 ` Markus Reichl
2015-10-20 13:43   ` Markus Reichl
2015-10-20 13:56   ` Krzysztof Kozlowski
2015-10-20 13:56     ` Krzysztof Kozlowski

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=56249E5E.80405@samsung.com \
    --to=alim.akhtar@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=javier@osg.samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.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.