Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1417822

Re: [PATCH] arm64: dts: apm: Use lowercase consistently for hex constants

Show all headers | View raw

On Mon, Jun 06, 2016 at 03:17:56PM -0500, Bjorn Helgaas wrote:
> The convention in these files is to use lowercase for "0x" prefixes and for
> the hex constants themselves, but a few changes didn't follow that
> convention, which makes the file annoying to read.
> 
> Use lowercase consistently for the hex constants.  No functional change
> intended.
> 
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> ---
>  .../devicetree/bindings/net/apm-xgene-enet.txt     |    4 +-
>  arch/arm64/boot/dts/apm/apm-shadowcat.dtsi         |   40 ++++++++++----------
>  arch/arm64/boot/dts/apm/apm-storm.dtsi             |   36 +++++++++---------
>  3 files changed, 40 insertions(+), 40 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/apm-xgene-enet.txt b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> index 05f705e3..e41b2d5 100644
> --- a/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> +++ b/Documentation/devicetree/bindings/net/apm-xgene-enet.txt
> @@ -59,8 +59,8 @@ Example:
>  		compatible = "apm,xgene-enet";
>  		status = "disabled";
>  		reg = <0x0 0x17020000 0x0 0xd100>,
> -		      <0x0 0X17030000 0x0 0X400>,
> -		      <0x0 0X10000000 0x0 0X200>;
> +		      <0x0 0x17030000 0x0 0x400>,
> +		      <0x0 0x10000000 0x0 0x200>;
>  		reg-names = "enet_csr", "ring_csr", "ring_cmd";
>  		interrupts = <0x0 0x3c 0x4>;
>  		port-id = <0>;
> diff --git a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi b/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
> index c569f76..9773687 100644
> --- a/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
> +++ b/arch/arm64/boot/dts/apm/apm-shadowcat.dtsi
> @@ -106,9 +106,9 @@
>  		interrupts = <1 9 0xf04>;	/* GIC Maintenence IRQ */
>  		ranges = <0 0 0 0x79000000 0x0 0x800000>; /* MSI Range */
>  		reg = <0x0 0x78090000 0x0 0x10000>,	/* GIC Dist */
> -		      <0x0 0x780A0000 0x0 0x20000>,	/* GIC CPU */
> -		      <0x0 0x780C0000 0x0 0x10000>,	/* GIC VCPU Control */
> -		      <0x0 0x780E0000 0x0 0x20000>;	/* GIC VCPU */
> +		      <0x0 0x780a0000 0x0 0x20000>,	/* GIC CPU */
> +		      <0x0 0x780c0000 0x0 0x10000>,	/* GIC VCPU Control */
> +		      <0x0 0x780e0000 0x0 0x20000>;	/* GIC VCPU */
>  		v2m0: v2m@0x00000 {

Unit addresses should not have '0x' or leading 0s.

>  			compatible = "arm,gic-v2m-frame";
>  			msi-controller;
> @@ -159,35 +159,35 @@
>  			msi-controller;
>  			reg = <0x0 0x90000 0x0 0x1000>;
>  		};
> -		v2m10: v2m@0xA0000 {
> +		v2m10: v2m@0xa0000 {

and here...

Rob

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] arm64: dts: apm: Use lowercase consistently for hex  constants Bjorn Helgaas <bhelgaas@google.com> - 2016-06-06 22:20 +0200
  Re: [PATCH] arm64: dts: apm: Use lowercase consistently for hex  constants Rob Herring <robh@kernel.org> - 2016-06-08 21:50 +0200

csiph-web