mbox series

[GIT,PULL] ARM: at91: dt for 5.13

Message ID 20210407114415.13180-1-nicolas.ferre@microchip.com
State New
Headers show
Series [GIT,PULL] ARM: at91: dt for 5.13 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/at91/linux.git tags/at91-dt-5.13

Message

Nicolas Ferre April 7, 2021, 11:44 a.m. UTC
From: Nicolas Ferre <nicolas.ferre@microchip.com>

Arnd, Olof,

Here is first batch of dt changes for 5.13. Please pull.

Thanks, best regards,
  Nicolas

The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:

  Linux 5.12-rc2 (2021-03-05 17:33:41 -0800)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/at91/linux.git tags/at91-dt-5.13

for you to fetch changes up to 4d930c421e3b4f5bb3a9dd6b2eba2b15d458fcb5:

  ARM: dts: at91: sama5d2: add ETB and ETM unit name (2021-04-07 13:33:19 +0200)

----------------------------------------------------------------
AT91 dt for 5.13:

- two little fixes (typo, W=1)
- a change in gpio button keycode for recent boards

----------------------------------------------------------------
Bhaskar Chowdhury (1):
      ARM: dts: at91: Fix a typo

Ludovic Desroches (1):
      ARM: dts: at91: change the key code of the gpio key

Nicolas Ferre (1):
      ARM: dts: at91: sama5d2: add ETB and ETM unit name

 arch/arm/boot/dts/at91-sam9x60ek.dts          | 3 ++-
 arch/arm/boot/dts/at91-sama5d27_som1_ek.dts   | 3 ++-
 arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dts | 3 ++-
 arch/arm/boot/dts/at91-sama5d2_icp.dts        | 3 ++-
 arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts     | 3 ++-
 arch/arm/boot/dts/at91-sama5d2_xplained.dts   | 3 ++-
 arch/arm/boot/dts/at91-sama5d3_xplained.dts   | 3 ++-
 arch/arm/boot/dts/at91sam9260ek.dts           | 3 ++-
 arch/arm/boot/dts/at91sam9g20ek_common.dtsi   | 3 ++-
 arch/arm/boot/dts/sama5d2.dtsi                | 4 ++--
 arch/arm/boot/dts/sama5d3.dtsi                | 2 +-
 11 files changed, 21 insertions(+), 12 deletions(-)

Comments

Arnd Bergmann April 8, 2021, 3:24 p.m. UTC | #1
From: Arnd Bergmann <arnd@arndb.de>

On Wed, 7 Apr 2021 13:44:15 +0200, nicolas.ferre@microchip.com wrote:
> Arnd, Olof,
> 
> Here is first batch of dt changes for 5.13. Please pull.
> 
> Thanks, best regards,
>   Nicolas
> 
> The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:
> 
> [...]

Merged into arm/dt, thanks!

I saw two new warnings from 'make dtbs_check W=1':

arch/arm/boot/dts/at91-sama5d2_ptc_ek.dt.yaml: /: 'etm@73C000' does not match any of the regexes: '@(0|[1-9a-f][0-9a-f]*)$', '^[^@]+$', 'pinctrl-[0-9]+'
arch/arm/boot/dts/at91-kizbox3-hs.dt.yaml: /: 'etm@73C000' does not match any of the regexes: '@(0|[1-9a-f][0-9a-f]*)$', '^[^@]+$', 'pinctrl-[0-9]+'

merge commit: e2b064fec8e49112f7dac779fcec12ded40728c2

       Arnd
Nicolas Ferre April 8, 2021, 4:34 p.m. UTC | #2
Hi Arnd,

On 08/04/2021 at 17:24, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> On Wed, 7 Apr 2021 13:44:15 +0200, nicolas.ferre@microchip.com wrote:
>> Arnd, Olof,
>>
>> Here is first batch of dt changes for 5.13. Please pull.
>>
>> Thanks, best regards,
>>    Nicolas
>>
>> The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:
>>
>> [...]
> 
> Merged into arm/dt, thanks!
> 
> I saw two new warnings from 'make dtbs_check W=1':
> 
> arch/arm/boot/dts/at91-sama5d2_ptc_ek.dt.yaml: /: 'etm@73C000' does not match any of the regexes: '@(0|[1-9a-f][0-9a-f]*)$', '^[^@]+$', 'pinctrl-[0-9]+'
> arch/arm/boot/dts/at91-kizbox3-hs.dt.yaml: /: 'etm@73C000' does not match any of the regexes: '@(0|[1-9a-f][0-9a-f]*)$', '^[^@]+$', 'pinctrl-[0-9]+'

Oh, got it: it's the upper case letter withing the etm hex address. I 
used this one to mach what was done in the reg property. I'm fixing both 
of them and sending the patch to the ml right now.

Tell me if I add it to a subsequent pull-request or if you prefer to 
take it the soonest in order to not generate additional warnings upstream.

BTW, I now have a dtschema at the proper level of support for running 
"make dtbs_check W=1" and will do it before sending pull-requests in the 
future.

Thanks for the heads-up. Best regards,
   Nicolas

> 
> merge commit: e2b064fec8e49112f7dac779fcec12ded40728c2
> 
>         Arnd
>
Arnd Bergmann April 8, 2021, 8:15 p.m. UTC | #3
On Thu, Apr 8, 2021 at 6:35 PM Nicolas Ferre
<nicolas.ferre@microchip.com> wrote:
> On 08/04/2021 at 17:24, Arnd Bergmann wrote:
> Oh, got it: it's the upper case letter withing the etm hex address. I
> used this one to mach what was done in the reg property. I'm fixing both
> of them and sending the patch to the ml right now.

Ok

> Tell me if I add it to a subsequent pull-request or if you prefer to
> take it the soonest in order to not generate additional warnings upstream.

I'd like to have it before the merge window, but don't bother making it
an additional pull request if you already plan for another pull request with
5.13 material.

Thanks,

       Arnd
Nicolas Ferre April 12, 2021, 11:42 a.m. UTC | #4
Arnd,

On 08/04/2021 at 22:15, Arnd Bergmann wrote:
> On Thu, Apr 8, 2021 at 6:35 PM Nicolas Ferre
> <nicolas.ferre@microchip.com> wrote:
>> On 08/04/2021 at 17:24, Arnd Bergmann wrote:
>> Oh, got it: it's the upper case letter withing the etm hex address. I
>> used this one to mach what was done in the reg property. I'm fixing both
>> of them and sending the patch to the ml right now.
> 
> Ok
> 
>> Tell me if I add it to a subsequent pull-request or if you prefer to
>> take it the soonest in order to not generate additional warnings upstream.
> 
> I'd like to have it before the merge window, but don't bother making it
> an additional pull request if you already plan for another pull request with
> 5.13 material.

I don't plan to make another PR on DT for 5.13 so I would like you to 
take it as a patch.
https://lore.kernel.org/linux-arm-kernel/20210408164443.38941-1-nicolas.ferre@microchip.com/

Thanks for your help. Best regards,
   Nicolas