Re: [PATCH 0/3] Apple M1 power management controller support

2021-12-28 Thread Jaehoon Chung
Dear Mark,

On 12/24/21 6:37 AM, Mark Kettenis wrote:
>> From: Jaehoon Chung 
>> Date: Wed, 22 Dec 2021 18:56:18 +0900
> 
> Dear Jaehoon,
> 
> Just noticed that I accidentally included some unrelated
> driver/mailbox changes in the first patch from this series.  So I sent
> a v2 without those.

I have checked your patch V2. After applied the watchdog patches, I will check 
your patches one more.

Best Regards,
Jaehoon Chung

> 
> Thanks,
> 
> Mark
> 
> 
>> Hi Mark,
>>
>> On 12/21/21 5:30 PM, Mark Kettenis wrote:
 From: Jaehoon Chung 
 Date: Tue, 21 Dec 2021 08:00:38 +0900
>>> Hello Jaehoon,
>>>
 Dear Mark,

 On 12/7/21 4:03 AM, Mark Kettenis wrote:
> This series adds support for the power management controller found on
> Apple SoCs based on the device tree bindings submitted to upstream
> Linux.  This is needed to enable power domains for devices that
> haven't been enabled by earlier boot stages.
 Is there any patch before applied this patchset?
>>> This is based on next, with the watchdog patch applied:
>>>
>>> https://protect2.fireeye.com/v1/url?k=3d0f182a-62942121-3d0e9365-002590f5b904-fe718e0a7d074ee3=1=eeb98822-5c4a-4946-9af9-cd3bcec1aef0=https%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D271919
>>>
>>> The drivers are independent, but I suppose you get conflicts in
>>> Kconfig and maybe the device tree without that series applied.
>>
>> Thanks for sharing information. I will check on next branch. 
>>
>> Best Regards,
>> Jaehoon Chung
>>
>>>
>>> Cheers,
>>>
>>> Mark
>>>
>
> Mark Kettenis (3):
>   arm: dts: apple: Update Apple M1 device trees
>   arm: dts: apple: Add u-boot,dm-pre-reloc properties
>   power: domain: Add Apple pmgr driver
>
>  arch/arm/Kconfig|4 +
>  arch/arm/dts/Makefile   |5 +-
>  arch/arm/dts/t8103-j274-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j274.dts |  122 +--
>  arch/arm/dts/t8103-j293-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j293.dts |   92 +--
>  arch/arm/dts/t8103-j313-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j313.dts |   57 ++
>  arch/arm/dts/t8103-j456-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j456.dts |   71 ++
>  arch/arm/dts/t8103-j457-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j457.dts |   59 ++
>  arch/arm/dts/t8103-jxxx.dtsi|  140 
>  arch/arm/dts/t8103-pmgr.dtsi| 1136 +++
>  arch/arm/dts/t8103-u-boot.dtsi  |   25 +
>  arch/arm/dts/t8103.dtsi |  585 +++---
>  drivers/mailbox/Kconfig |9 +
>  drivers/mailbox/Makefile|1 +
>  drivers/power/domain/Kconfig|8 +
>  drivers/power/domain/Makefile   |1 +
>  drivers/power/domain/apple-pmgr.c   |  113 +++
>  21 files changed, 2005 insertions(+), 428 deletions(-)
>  create mode 100644 arch/arm/dts/t8103-j274-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j293-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j313-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j313.dts
>  create mode 100644 arch/arm/dts/t8103-j456-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j456.dts
>  create mode 100644 arch/arm/dts/t8103-j457-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j457.dts
>  create mode 100644 arch/arm/dts/t8103-jxxx.dtsi
>  create mode 100644 arch/arm/dts/t8103-pmgr.dtsi
>  create mode 100644 arch/arm/dts/t8103-u-boot.dtsi
>  create mode 100644 drivers/power/domain/apple-pmgr.c
>

>>
>>
> 



Re: [PATCH 0/3] Apple M1 power management controller support

2021-12-23 Thread Mark Kettenis
> From: Jaehoon Chung 
> Date: Wed, 22 Dec 2021 18:56:18 +0900

Dear Jaehoon,

Just noticed that I accidentally included some unrelated
driver/mailbox changes in the first patch from this series.  So I sent
a v2 without those.

Thanks,

Mark


> Hi Mark,
> 
> On 12/21/21 5:30 PM, Mark Kettenis wrote:
> >> From: Jaehoon Chung 
> >> Date: Tue, 21 Dec 2021 08:00:38 +0900
> > Hello Jaehoon,
> >
> >> Dear Mark,
> >>
> >> On 12/7/21 4:03 AM, Mark Kettenis wrote:
> >>> This series adds support for the power management controller found on
> >>> Apple SoCs based on the device tree bindings submitted to upstream
> >>> Linux.  This is needed to enable power domains for devices that
> >>> haven't been enabled by earlier boot stages.
> >> Is there any patch before applied this patchset?
> > This is based on next, with the watchdog patch applied:
> >
> > https://patchwork.ozlabs.org/project/uboot/list/?series=271919
> >
> > The drivers are independent, but I suppose you get conflicts in
> > Kconfig and maybe the device tree without that series applied.
> 
> Thanks for sharing information. I will check on next branch. 
> 
> Best Regards,
> Jaehoon Chung
> 
> >
> > Cheers,
> >
> > Mark
> >
> >>>
> >>> Mark Kettenis (3):
> >>>   arm: dts: apple: Update Apple M1 device trees
> >>>   arm: dts: apple: Add u-boot,dm-pre-reloc properties
> >>>   power: domain: Add Apple pmgr driver
> >>>
> >>>  arch/arm/Kconfig|4 +
> >>>  arch/arm/dts/Makefile   |5 +-
> >>>  arch/arm/dts/t8103-j274-u-boot.dtsi |1 +
> >>>  arch/arm/dts/t8103-j274.dts |  122 +--
> >>>  arch/arm/dts/t8103-j293-u-boot.dtsi |1 +
> >>>  arch/arm/dts/t8103-j293.dts |   92 +--
> >>>  arch/arm/dts/t8103-j313-u-boot.dtsi |1 +
> >>>  arch/arm/dts/t8103-j313.dts |   57 ++
> >>>  arch/arm/dts/t8103-j456-u-boot.dtsi |1 +
> >>>  arch/arm/dts/t8103-j456.dts |   71 ++
> >>>  arch/arm/dts/t8103-j457-u-boot.dtsi |1 +
> >>>  arch/arm/dts/t8103-j457.dts |   59 ++
> >>>  arch/arm/dts/t8103-jxxx.dtsi|  140 
> >>>  arch/arm/dts/t8103-pmgr.dtsi| 1136 +++
> >>>  arch/arm/dts/t8103-u-boot.dtsi  |   25 +
> >>>  arch/arm/dts/t8103.dtsi |  585 +++---
> >>>  drivers/mailbox/Kconfig |9 +
> >>>  drivers/mailbox/Makefile|1 +
> >>>  drivers/power/domain/Kconfig|8 +
> >>>  drivers/power/domain/Makefile   |1 +
> >>>  drivers/power/domain/apple-pmgr.c   |  113 +++
> >>>  21 files changed, 2005 insertions(+), 428 deletions(-)
> >>>  create mode 100644 arch/arm/dts/t8103-j274-u-boot.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-j293-u-boot.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-j313-u-boot.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-j313.dts
> >>>  create mode 100644 arch/arm/dts/t8103-j456-u-boot.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-j456.dts
> >>>  create mode 100644 arch/arm/dts/t8103-j457-u-boot.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-j457.dts
> >>>  create mode 100644 arch/arm/dts/t8103-jxxx.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-pmgr.dtsi
> >>>  create mode 100644 arch/arm/dts/t8103-u-boot.dtsi
> >>>  create mode 100644 drivers/power/domain/apple-pmgr.c
> >>>
> >>
> 
> 


Re: [PATCH 0/3] Apple M1 power management controller support

2021-12-22 Thread Jaehoon Chung
Hi Mark,

On 12/21/21 5:30 PM, Mark Kettenis wrote:
>> From: Jaehoon Chung 
>> Date: Tue, 21 Dec 2021 08:00:38 +0900
> Hello Jaehoon,
>
>> Dear Mark,
>>
>> On 12/7/21 4:03 AM, Mark Kettenis wrote:
>>> This series adds support for the power management controller found on
>>> Apple SoCs based on the device tree bindings submitted to upstream
>>> Linux.  This is needed to enable power domains for devices that
>>> haven't been enabled by earlier boot stages.
>> Is there any patch before applied this patchset?
> This is based on next, with the watchdog patch applied:
>
> https://patchwork.ozlabs.org/project/uboot/list/?series=271919
>
> The drivers are independent, but I suppose you get conflicts in
> Kconfig and maybe the device tree without that series applied.

Thanks for sharing information. I will check on next branch. 

Best Regards,
Jaehoon Chung

>
> Cheers,
>
> Mark
>
>>>
>>> Mark Kettenis (3):
>>>   arm: dts: apple: Update Apple M1 device trees
>>>   arm: dts: apple: Add u-boot,dm-pre-reloc properties
>>>   power: domain: Add Apple pmgr driver
>>>
>>>  arch/arm/Kconfig|4 +
>>>  arch/arm/dts/Makefile   |5 +-
>>>  arch/arm/dts/t8103-j274-u-boot.dtsi |1 +
>>>  arch/arm/dts/t8103-j274.dts |  122 +--
>>>  arch/arm/dts/t8103-j293-u-boot.dtsi |1 +
>>>  arch/arm/dts/t8103-j293.dts |   92 +--
>>>  arch/arm/dts/t8103-j313-u-boot.dtsi |1 +
>>>  arch/arm/dts/t8103-j313.dts |   57 ++
>>>  arch/arm/dts/t8103-j456-u-boot.dtsi |1 +
>>>  arch/arm/dts/t8103-j456.dts |   71 ++
>>>  arch/arm/dts/t8103-j457-u-boot.dtsi |1 +
>>>  arch/arm/dts/t8103-j457.dts |   59 ++
>>>  arch/arm/dts/t8103-jxxx.dtsi|  140 
>>>  arch/arm/dts/t8103-pmgr.dtsi| 1136 +++
>>>  arch/arm/dts/t8103-u-boot.dtsi  |   25 +
>>>  arch/arm/dts/t8103.dtsi |  585 +++---
>>>  drivers/mailbox/Kconfig |9 +
>>>  drivers/mailbox/Makefile|1 +
>>>  drivers/power/domain/Kconfig|8 +
>>>  drivers/power/domain/Makefile   |1 +
>>>  drivers/power/domain/apple-pmgr.c   |  113 +++
>>>  21 files changed, 2005 insertions(+), 428 deletions(-)
>>>  create mode 100644 arch/arm/dts/t8103-j274-u-boot.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-j293-u-boot.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-j313-u-boot.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-j313.dts
>>>  create mode 100644 arch/arm/dts/t8103-j456-u-boot.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-j456.dts
>>>  create mode 100644 arch/arm/dts/t8103-j457-u-boot.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-j457.dts
>>>  create mode 100644 arch/arm/dts/t8103-jxxx.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-pmgr.dtsi
>>>  create mode 100644 arch/arm/dts/t8103-u-boot.dtsi
>>>  create mode 100644 drivers/power/domain/apple-pmgr.c
>>>
>>



Re: [PATCH 0/3] Apple M1 power management controller support

2021-12-21 Thread Mark Kettenis
> From: Jaehoon Chung 
> Date: Tue, 21 Dec 2021 08:00:38 +0900

Hello Jaehoon,

> Dear Mark,
> 
> On 12/7/21 4:03 AM, Mark Kettenis wrote:
> > This series adds support for the power management controller found on
> > Apple SoCs based on the device tree bindings submitted to upstream
> > Linux.  This is needed to enable power domains for devices that
> > haven't been enabled by earlier boot stages.
> 
> Is there any patch before applied this patchset?

This is based on next, with the watchdog patch applied:

https://patchwork.ozlabs.org/project/uboot/list/?series=271919

The drivers are independent, but I suppose you get conflicts in
Kconfig and maybe the device tree without that series applied.

Cheers,

Mark

> > 
> > 
> > Mark Kettenis (3):
> >   arm: dts: apple: Update Apple M1 device trees
> >   arm: dts: apple: Add u-boot,dm-pre-reloc properties
> >   power: domain: Add Apple pmgr driver
> > 
> >  arch/arm/Kconfig|4 +
> >  arch/arm/dts/Makefile   |5 +-
> >  arch/arm/dts/t8103-j274-u-boot.dtsi |1 +
> >  arch/arm/dts/t8103-j274.dts |  122 +--
> >  arch/arm/dts/t8103-j293-u-boot.dtsi |1 +
> >  arch/arm/dts/t8103-j293.dts |   92 +--
> >  arch/arm/dts/t8103-j313-u-boot.dtsi |1 +
> >  arch/arm/dts/t8103-j313.dts |   57 ++
> >  arch/arm/dts/t8103-j456-u-boot.dtsi |1 +
> >  arch/arm/dts/t8103-j456.dts |   71 ++
> >  arch/arm/dts/t8103-j457-u-boot.dtsi |1 +
> >  arch/arm/dts/t8103-j457.dts |   59 ++
> >  arch/arm/dts/t8103-jxxx.dtsi|  140 
> >  arch/arm/dts/t8103-pmgr.dtsi| 1136 +++
> >  arch/arm/dts/t8103-u-boot.dtsi  |   25 +
> >  arch/arm/dts/t8103.dtsi |  585 +++---
> >  drivers/mailbox/Kconfig |9 +
> >  drivers/mailbox/Makefile|1 +
> >  drivers/power/domain/Kconfig|8 +
> >  drivers/power/domain/Makefile   |1 +
> >  drivers/power/domain/apple-pmgr.c   |  113 +++
> >  21 files changed, 2005 insertions(+), 428 deletions(-)
> >  create mode 100644 arch/arm/dts/t8103-j274-u-boot.dtsi
> >  create mode 100644 arch/arm/dts/t8103-j293-u-boot.dtsi
> >  create mode 100644 arch/arm/dts/t8103-j313-u-boot.dtsi
> >  create mode 100644 arch/arm/dts/t8103-j313.dts
> >  create mode 100644 arch/arm/dts/t8103-j456-u-boot.dtsi
> >  create mode 100644 arch/arm/dts/t8103-j456.dts
> >  create mode 100644 arch/arm/dts/t8103-j457-u-boot.dtsi
> >  create mode 100644 arch/arm/dts/t8103-j457.dts
> >  create mode 100644 arch/arm/dts/t8103-jxxx.dtsi
> >  create mode 100644 arch/arm/dts/t8103-pmgr.dtsi
> >  create mode 100644 arch/arm/dts/t8103-u-boot.dtsi
> >  create mode 100644 drivers/power/domain/apple-pmgr.c
> > 
> 
> 


Re: [PATCH 0/3] Apple M1 power management controller support

2021-12-20 Thread Jaehoon Chung
Dear Mark,

On 12/7/21 4:03 AM, Mark Kettenis wrote:
> This series adds support for the power management controller found on
> Apple SoCs based on the device tree bindings submitted to upstream
> Linux.  This is needed to enable power domains for devices that
> haven't been enabled by earlier boot stages.

Is there any patch before applied this patchset?

Best Regards,
Jaehoon Chung

> 
> 
> Mark Kettenis (3):
>   arm: dts: apple: Update Apple M1 device trees
>   arm: dts: apple: Add u-boot,dm-pre-reloc properties
>   power: domain: Add Apple pmgr driver
> 
>  arch/arm/Kconfig|4 +
>  arch/arm/dts/Makefile   |5 +-
>  arch/arm/dts/t8103-j274-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j274.dts |  122 +--
>  arch/arm/dts/t8103-j293-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j293.dts |   92 +--
>  arch/arm/dts/t8103-j313-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j313.dts |   57 ++
>  arch/arm/dts/t8103-j456-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j456.dts |   71 ++
>  arch/arm/dts/t8103-j457-u-boot.dtsi |1 +
>  arch/arm/dts/t8103-j457.dts |   59 ++
>  arch/arm/dts/t8103-jxxx.dtsi|  140 
>  arch/arm/dts/t8103-pmgr.dtsi| 1136 +++
>  arch/arm/dts/t8103-u-boot.dtsi  |   25 +
>  arch/arm/dts/t8103.dtsi |  585 +++---
>  drivers/mailbox/Kconfig |9 +
>  drivers/mailbox/Makefile|1 +
>  drivers/power/domain/Kconfig|8 +
>  drivers/power/domain/Makefile   |1 +
>  drivers/power/domain/apple-pmgr.c   |  113 +++
>  21 files changed, 2005 insertions(+), 428 deletions(-)
>  create mode 100644 arch/arm/dts/t8103-j274-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j293-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j313-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j313.dts
>  create mode 100644 arch/arm/dts/t8103-j456-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j456.dts
>  create mode 100644 arch/arm/dts/t8103-j457-u-boot.dtsi
>  create mode 100644 arch/arm/dts/t8103-j457.dts
>  create mode 100644 arch/arm/dts/t8103-jxxx.dtsi
>  create mode 100644 arch/arm/dts/t8103-pmgr.dtsi
>  create mode 100644 arch/arm/dts/t8103-u-boot.dtsi
>  create mode 100644 drivers/power/domain/apple-pmgr.c
>