All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names
@ 2025-12-15 21:29 Rob Herring (Arm)
  2025-12-16  9:14 ` Sudeep Holla
  2025-12-16 16:41 ` Rob Herring (Arm)
  0 siblings, 2 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-12-15 21:29 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Conor Dooley, Liviu Dudau, Sudeep Holla,
	Lorenzo Pieralisi, Andre Przywara
  Cc: devicetree, linux-arm-kernel, linux-kernel

The clock and regulator node names were updated to use standard node names,
but the binding wasn't updated.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 Documentation/devicetree/bindings/arm/vexpress-config.yaml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/vexpress-config.yaml b/Documentation/devicetree/bindings/arm/vexpress-config.yaml
index b74380da3198..41c53e3acc12 100644
--- a/Documentation/devicetree/bindings/arm/vexpress-config.yaml
+++ b/Documentation/devicetree/bindings/arm/vexpress-config.yaml
@@ -103,7 +103,7 @@ required:
   - arm,vexpress,config-bridge
 
 patternProperties:
-  'clk[0-9]*$':
+  '^clock-controller.*$':
     type: object
     description:
       clocks
@@ -137,7 +137,7 @@ patternProperties:
       - arm,vexpress-sysreg,func
       - "#clock-cells"
 
-  "^volt-.+$":
+  "^regulator-.+$":
     $ref: /schemas/regulator/regulator.yaml#
     properties:
       compatible:
@@ -272,7 +272,7 @@ examples:
         compatible = "arm,vexpress,config-bus";
         arm,vexpress,config-bridge = <&v2m_sysreg>;
 
-        clk0 {
+        clock-controller {
             compatible = "arm,vexpress-osc";
             arm,vexpress-sysreg,func = <1 0>;
             #clock-cells = <0>;
-- 
2.51.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names
  2025-12-15 21:29 [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names Rob Herring (Arm)
@ 2025-12-16  9:14 ` Sudeep Holla
  2025-12-16 16:41 ` Rob Herring (Arm)
  1 sibling, 0 replies; 4+ messages in thread
From: Sudeep Holla @ 2025-12-16  9:14 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Liviu Dudau, Lorenzo Pieralisi,
	Andre Przywara, devicetree, linux-arm-kernel, linux-kernel

On Mon, Dec 15, 2025 at 03:29:14PM -0600, Rob Herring (Arm) wrote:
> The clock and regulator node names were updated to use standard node names,
> but the binding wasn't updated.
> 

I assume you will be taking this via your tree itself.

Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>

-- 
Regards,
Sudeep


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names
  2025-12-15 21:29 [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names Rob Herring (Arm)
  2025-12-16  9:14 ` Sudeep Holla
@ 2025-12-16 16:41 ` Rob Herring (Arm)
  1 sibling, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2025-12-16 16:41 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Andre Przywara, Lorenzo Pieralisi,
	linux-arm-kernel, Liviu Dudau, Conor Dooley, linux-kernel,
	devicetree, Sudeep Holla


On Mon, 15 Dec 2025 15:29:14 -0600, Rob Herring (Arm) wrote:
> The clock and regulator node names were updated to use standard node names,
> but the binding wasn't updated.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/vexpress-config.yaml | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 

Applied, thanks!



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names
@ 2025-12-22 22:39 kernel test robot
  0 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2025-12-22 22:39 UTC (permalink / raw)
  To: oe-kbuild; +Cc: lkp

:::::: 
:::::: Manual check reason: "dtcheck: binding changes may go via different trees"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20251215212914.3323741-1-robh@kernel.org>
References: <20251215212914.3323741-1-robh@kernel.org>
TO: "Rob Herring (Arm)" <robh@kernel.org>

Hi Rob,

kernel test robot noticed the following build warnings:

[auto build test WARNING on robh/for-next]
[also build test WARNING on krzk/for-next krzk-mem-ctrl/for-next krzk-dt/for-next linus/master v6.16-rc1]
[cannot apply to next-20251219]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Rob-Herring-Arm/dt-bindings-arm-vexpress-config-Update-clock-and-regulator-node-names/20251216-053816
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link:    https://lore.kernel.org/r/20251215212914.3323741-1-robh%40kernel.org
patch subject: [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names
:::::: branch date: 7 days ago
:::::: commit date: 7 days ago
config: arm-randconfig-2052-20251217 (https://download.01.org/0day-ci/archive/20251222/202512222312.Twb9vQqw-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 15.1.0
dtschema version: 2025.12
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251222/202512222312.Twb9vQqw-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202512222312.Twb9vQqw-lkp@intel.com/

dtcheck warnings: (new ones prefixed by >>)
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:419.9-490.7: Warning (simple_bus_reg): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/mcc: missing or empty reg/ranges property
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:366.13-376.9: Warning (graph_child_address): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/i2c@160000/dvi-transmitter@39/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: / (arm,vexpress,v2p-ca5s): bus@8000000: Unevaluated properties are not allowed ('#interrupt-cells', 'interrupt-map', 'interrupt-map-mask' were unexpected)
   	from schema $id: http://devicetree.org/schemas/arm/arm,vexpress-juno.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: iofpga-bus@300000000 (simple-bus): mcc: 'ranges' is a required property
   	from schema $id: http://devicetree.org/schemas/simple-bus.yaml
>> arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: mcc (arm,vexpress,config-bus): 'oscclk0', 'oscclk1', 'oscclk2', 'volt-vio' do not match any of the regexes: '^amp-.+$', '^clock-controller.*$', '^energy(-.+)?$', '^pinctrl-[0-9]+$', '^power-.+$', '^regulator-.+$', '^reset[0-9]*$', '^temp-.+$'
   	from schema $id: http://devicetree.org/schemas/arm/vexpress-config.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: hdlcd@2a110000 (arm,hdlcd): 'port' is a required property
   	from schema $id: http://devicetree.org/schemas/display/arm,hdlcd.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: /memory-controller@2a150000: failed to match any schema with compatible: ['arm,pl341', 'arm,primecell']
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: timer@2c000200 (arm,cortex-a5-global-timer): compatible: ['arm,cortex-a5-global-timer', 'arm,cortex-a9-global-timer'] is too long
   	from schema $id: http://devicetree.org/schemas/timer/arm,global_timer.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: interrupt-controller@2c001000 (arm,cortex-a5-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   	['arm,cortex-a5-gic', 'arm,cortex-a9-gic'] is too long
   	arch/arm/boot/dts/arm/vexpress-v2p-ca5s.dtb: interrupt-controller@2c001000 (arm,cortex-a5-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   		['arm,cortex-a5-gic', 'arm,cortex-a9-gic'] is too long
--
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:419.9-490.7: Warning (simple_bus_reg): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/mcc: missing or empty reg/ranges property
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:366.13-376.9: Warning (graph_child_address): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/i2c@160000/dvi-transmitter@39/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: / (arm,vexpress,v2p-ca15,tc1): bus@8000000: Unevaluated properties are not allowed ('#interrupt-cells', 'interrupt-map', 'interrupt-map-mask' were unexpected)
   	from schema $id: http://devicetree.org/schemas/arm/arm,vexpress-juno.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: iofpga-bus@300000000 (simple-bus): mcc: 'ranges' is a required property
   	from schema $id: http://devicetree.org/schemas/simple-bus.yaml
>> arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: mcc (arm,vexpress,config-bus): 'oscclk0', 'oscclk1', 'oscclk2', 'volt-vio' do not match any of the regexes: '^amp-.+$', '^clock-controller.*$', '^energy(-.+)?$', '^pinctrl-[0-9]+$', '^power-.+$', '^regulator-.+$', '^reset[0-9]*$', '^temp-.+$'
   	from schema $id: http://devicetree.org/schemas/arm/vexpress-config.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: hdlcd@2b000000 (arm,hdlcd): 'port' is a required property
   	from schema $id: http://devicetree.org/schemas/display/arm,hdlcd.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: /memory-controller@2b0a0000: failed to match any schema with compatible: ['arm,pl341', 'arm,primecell']
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: wdt@2b060000 (arm,sp805): $nodename:0: 'wdt@2b060000' does not match '^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
   	from schema $id: http://devicetree.org/schemas/watchdog/arm,sp805.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: interrupt-controller@2c001000 (arm,cortex-a15-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   	['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
   	arch/arm/boot/dts/arm/vexpress-v2p-ca15-tc1.dtb: interrupt-controller@2c001000 (arm,cortex-a15-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   		['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
--
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:419.9-490.7: Warning (simple_bus_reg): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/mcc: missing or empty reg/ranges property
   arch/arm/boot/dts/arm/vexpress-v2m-rs1.dtsi:366.13-376.9: Warning (graph_child_address): /bus@8000000/motherboard-bus@8000000/iofpga-bus@300000000/i2c@160000/dvi-transmitter@39/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: / (arm,vexpress,v2p-ca15_a7): bus@8000000: Unevaluated properties are not allowed ('#interrupt-cells', 'interrupt-map', 'interrupt-map-mask' were unexpected)
   	from schema $id: http://devicetree.org/schemas/arm/arm,vexpress-juno.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: iofpga-bus@300000000 (simple-bus): mcc: 'ranges' is a required property
   	from schema $id: http://devicetree.org/schemas/simple-bus.yaml
>> arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: mcc (arm,vexpress,config-bus): 'oscclk0', 'oscclk1', 'oscclk2', 'volt-vio' do not match any of the regexes: '^amp-.+$', '^clock-controller.*$', '^energy(-.+)?$', '^pinctrl-[0-9]+$', '^power-.+$', '^regulator-.+$', '^reset[0-9]*$', '^temp-.+$'
   	from schema $id: http://devicetree.org/schemas/arm/vexpress-config.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: wdt@2a490000 (arm,sp805): $nodename:0: 'wdt@2a490000' does not match '^(timer|watchdog)(@.*|-([0-9]|[1-9][0-9]+))?$'
   	from schema $id: http://devicetree.org/schemas/watchdog/arm,sp805.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: hdlcd@2b000000 (arm,hdlcd): 'port' is a required property
   	from schema $id: http://devicetree.org/schemas/display/arm,hdlcd.yaml
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: /memory-controller@2b0a0000: failed to match any schema with compatible: ['arm,pl341', 'arm,primecell']
   arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: interrupt-controller@2c001000 (arm,cortex-a15-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   	['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long
   	arch/arm/boot/dts/arm/vexpress-v2p-ca15_a7.dtb: interrupt-controller@2c001000 (arm,cortex-a15-gic): compatible: 'oneOf' conditional failed, one must be fixed:
   		['arm,cortex-a15-gic', 'arm,cortex-a9-gic'] is too long

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-12-22 22:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-15 21:29 [PATCH] dt-bindings: arm: vexpress-config: Update clock and regulator node names Rob Herring (Arm)
2025-12-16  9:14 ` Sudeep Holla
2025-12-16 16:41 ` Rob Herring (Arm)
  -- strict thread matches above, loose matches on Subject: below --
2025-12-22 22:39 kernel test robot

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.