linux-kernel - Re: [PATCH] ARM: dts: dra7: Fix efuse register size for ABB

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150504162609.GN24469@atomide.com>
Date:	Mon, 4 May 2015 09:26:09 -0700
From:	Tony Lindgren <tony@...mide.com>
To:	Nishanth Menon <nm@...com>
Cc:	Benoît Cousson <bcousson@...libre.com>,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	devicetree@...r.kernel.org, linux-omap@...r.kernel.org,
	Matt Gessner <Matt.Gessner@...driver.com>,
	Suman Anna <s-anna@...com>
Subject: Re: [PATCH] ARM: dts: dra7: Fix efuse register size for ABB

* Nishanth Menon <nm@...com> [150416 14:57]:
> Fix a typo in DRA7 dtsi where 12 bytes are needed for register
> description of ABB efuse registers, however only 8 bytes are provided
> to map. For some weird reason, this does not generate abort at offset
> 0x8, probably due to default maps already provided in io.c for the bus
> register ranges.
> 
> Reported-by: Matt Gessner <Matt.Gessner@...driver.com>
> Reported-by: Suman Anna <s-anna@...com>
> Signed-off-by: Nishanth Menon <nm@...com>

Applying into omap-for-v4.1/fixes thanks.

Tony

> ---
>  arch/arm/boot/dts/dra7.dtsi |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
> index 5332b57b4950..92b8bb873f43 100644
> --- a/arch/arm/boot/dts/dra7.dtsi
> +++ b/arch/arm/boot/dts/dra7.dtsi
> @@ -911,7 +911,7 @@
>  			ti,clock-cycles = <16>;
>  
>  			reg = <0x4ae07ddc 0x4>, <0x4ae07de0 0x4>,
> -			      <0x4ae06014 0x4>, <0x4a003b20 0x8>,
> +			      <0x4ae06014 0x4>, <0x4a003b20 0xc>,
>  			      <0x4ae0c158 0x4>;
>  			reg-names = "setup-address", "control-address",
>  				    "int-address", "efuse-address",
> @@ -944,7 +944,7 @@
>  			ti,clock-cycles = <16>;
>  
>  			reg = <0x4ae07e34 0x4>, <0x4ae07e24 0x4>,
> -			      <0x4ae06010 0x4>, <0x4a0025cc 0x8>,
> +			      <0x4ae06010 0x4>, <0x4a0025cc 0xc>,
>  			      <0x4a002470 0x4>;
>  			reg-names = "setup-address", "control-address",
>  				    "int-address", "efuse-address",
> @@ -977,7 +977,7 @@
>  			ti,clock-cycles = <16>;
>  
>  			reg = <0x4ae07e30 0x4>, <0x4ae07e20 0x4>,
> -			      <0x4ae06010 0x4>, <0x4a0025e0 0x8>,
> +			      <0x4ae06010 0x4>, <0x4a0025e0 0xc>,
>  			      <0x4a00246c 0x4>;
>  			reg-names = "setup-address", "control-address",
>  				    "int-address", "efuse-address",
> @@ -1010,7 +1010,7 @@
>  			ti,clock-cycles = <16>;
>  
>  			reg = <0x4ae07de4 0x4>, <0x4ae07de8 0x4>,
> -			      <0x4ae06010 0x4>, <0x4a003b08 0x8>,
> +			      <0x4ae06010 0x4>, <0x4a003b08 0xc>,
>  			      <0x4ae0c154 0x4>;
>  			reg-names = "setup-address", "control-address",
>  				    "int-address", "efuse-address",
> -- 
> 1.7.9.5
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/