mbox

[GIT,PULL] Xilinx Zynq DT changes for v4.3 v2

Message ID 55D4646F.7010404@monstr.eu
State New
Headers show

Pull-request

https://github.com/Xilinx/linux-xlnx.git tags/zynq-dt-for-4.3-v2

Message

Michal Simek Aug. 19, 2015, 11:11 a.m. UTC
Hi guys,

Philipp just sent his PR to you with adding reset controller for Zynq
(http://www.spinics.net/lists/arm-kernel/msg440136.html).
Can you please consider to add this one more patch to your tree?
It is one more patch on the top of existing zynq/dt branch which you
already have in arm-soc git repo.

Thanks,
Michal

The following changes since commit b76da4d8758995437a831748dfca6f64a0e31c95:

  ARM: zynq: DT: Add zc702 pushbuttons to DT as gpio-keys (2015-07-31
10:50:06 +0200)

are available in the git repository at:

  https://github.com/Xilinx/linux-xlnx.git tags/zynq-dt-for-4.3-v2

for you to fetch changes up to d8ae96c948ccb6c4274f007eb2ffd27ed656783a:

  dts: zynq: Add devicetree entry for Xilinx Zynq reset controller.
(2015-08-19 13:03:23 +0200)

----------------------------------------------------------------
arm: Xilinx Zynq dt patches for v4.3 v2

- Enable reset driver for Zynq

----------------------------------------------------------------
Moritz Fischer (1):
      dts: zynq: Add devicetree entry for Xilinx Zynq reset controller.

 arch/arm/boot/dts/zynq-7000.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Olof Johansson Aug. 21, 2015, 2:03 a.m. UTC | #1
On Wed, Aug 19, 2015 at 01:11:43PM +0200, Michal Simek wrote:
> Hi guys,
> 
> Philipp just sent his PR to you with adding reset controller for Zynq
> (http://www.spinics.net/lists/arm-kernel/msg440136.html).
> Can you please consider to add this one more patch to your tree?
> It is one more patch on the top of existing zynq/dt branch which you
> already have in arm-soc git repo.

The change is small enough, so yes we can merge it with the below fixed:

> Moritz Fischer (1):
>       dts: zynq: Add devicetree entry for Xilinx Zynq reset controller.

ARM: dts: <....> is the format we want here -- that's what most other platforms
use as well. I.e. please add ARM as a prefix.

I can do it here and just apply as a patch too instead of merging, just let me
know.


Thanks,


-Olof
Michal Simek Aug. 21, 2015, 6:36 a.m. UTC | #2
On 08/21/2015 04:03 AM, Olof Johansson wrote:
> On Wed, Aug 19, 2015 at 01:11:43PM +0200, Michal Simek wrote:
>> Hi guys,
>>
>> Philipp just sent his PR to you with adding reset controller for Zynq
>> (http://www.spinics.net/lists/arm-kernel/msg440136.html).
>> Can you please consider to add this one more patch to your tree?
>> It is one more patch on the top of existing zynq/dt branch which you
>> already have in arm-soc git repo.
> 
> The change is small enough, so yes we can merge it with the below fixed:
> 
>> Moritz Fischer (1):
>>       dts: zynq: Add devicetree entry for Xilinx Zynq reset controller.
> 
> ARM: dts: <....> is the format we want here -- that's what most other platforms
> use as well. I.e. please add ARM as a prefix.

ok. Will keep my eye on it.

> 
> I can do it here and just apply as a patch too instead of merging, just let me
> know.

Yes that will be great if you can do that change on your side.

Thanks,
Michal
Olof Johansson Aug. 21, 2015, 4:10 p.m. UTC | #3
On Fri, Aug 21, 2015 at 08:36:09AM +0200, Michal Simek wrote:
> On 08/21/2015 04:03 AM, Olof Johansson wrote:
> > On Wed, Aug 19, 2015 at 01:11:43PM +0200, Michal Simek wrote:
> >> Hi guys,
> >>
> >> Philipp just sent his PR to you with adding reset controller for Zynq
> >> (http://www.spinics.net/lists/arm-kernel/msg440136.html).
> >> Can you please consider to add this one more patch to your tree?
> >> It is one more patch on the top of existing zynq/dt branch which you
> >> already have in arm-soc git repo.
> > 
> > The change is small enough, so yes we can merge it with the below fixed:
> > 
> >> Moritz Fischer (1):
> >>       dts: zynq: Add devicetree entry for Xilinx Zynq reset controller.
> > 
> > ARM: dts: <....> is the format we want here -- that's what most other platforms
> > use as well. I.e. please add ARM as a prefix.
> 
> ok. Will keep my eye on it.
> 
> > 
> > I can do it here and just apply as a patch too instead of merging, just let me
> > know.
> 
> Yes that will be great if you can do that change on your side.

Great, patch applied to next/dt.


-Olof