diff mbox

ARM: BCM5301X: activate some additional options in pl310 cache controller

Message ID 1432493418-12933-1-git-send-email-hauke@hauke-m.de (mailing list archive)
State New, archived
Headers show

Commit Message

Hauke Mehrtens May 24, 2015, 6:50 p.m. UTC
In the default Broadcom SDK the shared override is activated for this
cache controller, do the same in the upstream code. Data and
instruction prefetching is not activated by default for this cache
controller on the bcm53xx SoC, do it manually like it is done in the
vendor SDK.

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
---
 arch/arm/boot/dts/bcm5301x.dtsi | 3 +++
 1 file changed, 3 insertions(+)

Comments

Rafał Miłecki May 24, 2015, 6:54 p.m. UTC | #1
On 24 May 2015 at 20:50, Hauke Mehrtens <hauke@hauke-m.de> wrote:
> In the default Broadcom SDK the shared override is activated for this
> cache controller, do the same in the upstream code. Data and
> instruction prefetching is not activated by default for this cache
> controller on the bcm53xx SoC, do it manually like it is done in the
> vendor SDK.
>
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
> ---
>  arch/arm/boot/dts/bcm5301x.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
> index 78aec62..7d1578a 100644
> --- a/arch/arm/boot/dts/bcm5301x.dtsi
> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
> @@ -78,6 +78,9 @@
>                         compatible = "arm,pl310-cache";
>                         reg = <0x2000 0x1000>;
>                         cache-unified;
> +                       arm,shared-override;
> +                       arm,prefetch-data = <1>;
> +                       arm,prefetch-instr = <1>;
>                         cache-level = <2>;
>                 };
>         };
> --
> 2.1.4

Fwd-ing to Felix, who was also looking at this I believe.
Hauke Mehrtens May 24, 2015, 7 p.m. UTC | #2
On 05/24/2015 08:54 PM, Rafa? Mi?ecki wrote:
> On 24 May 2015 at 20:50, Hauke Mehrtens <hauke@hauke-m.de> wrote:
>> In the default Broadcom SDK the shared override is activated for this
>> cache controller, do the same in the upstream code. Data and
>> instruction prefetching is not activated by default for this cache
>> controller on the bcm53xx SoC, do it manually like it is done in the
>> vendor SDK.
>>
>> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
>> ---
>>  arch/arm/boot/dts/bcm5301x.dtsi | 3 +++
>>  1 file changed, 3 insertions(+)
>>
This makes use of the attributes added by these two patches:

http://www.spinics.net/lists/arm-kernel/msg416406.html
http://www.spinics.net/lists/arm-kernel/msg418996.html

I hope both patches will make it into 4.2, but there wasn't a negative
comment in the last week.

Hauke
diff mbox

Patch

diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index 78aec62..7d1578a 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -78,6 +78,9 @@ 
 			compatible = "arm,pl310-cache";
 			reg = <0x2000 0x1000>;
 			cache-unified;
+			arm,shared-override;
+			arm,prefetch-data = <1>;
+			arm,prefetch-instr = <1>;
 			cache-level = <2>;
 		};
 	};