Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
On Thu, Mar 19, 2015 at 05:52:54PM +, Suzuki K. Poulose wrote: > On 19/03/15 17:38, Sudeep Holla wrote: > > > > > > On 19/03/15 17:32, Mark Rutland wrote: > >> One more thing: > >> > >>> @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model > >>> *get_cci_model(struct platform_device

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Suzuki K. Poulose
On 19/03/15 17:38, Sudeep Holla wrote: On 19/03/15 17:32, Mark Rutland wrote: One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device * pdev->dev.of_node); if

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Sudeep Holla
On 19/03/15 17:32, Mark Rutland wrote: One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device * pdev->dev.of_node); if (!match) return NULL; +

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
One more thing: > @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model > *get_cci_model(struct platform_device * > pdev->dev.of_node); > if (!match) > return NULL; > + if (match->data) > + return

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
On Tue, Mar 17, 2015 at 09:51:41AM +, Suzuki K. Poulose wrote: > From: "Suzuki K. Poulose" > > A minor change, fixed missplled 'DEPRECATED' in the dev_warn(). > > Thanks > Suzuki > > 8> > Avoid secure transactions while probing the CCI PMU. The > existing code makes use of the

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
On Tue, Mar 17, 2015 at 09:51:41AM +, Suzuki K. Poulose wrote: From: Suzuki K. Poulose suzuki.poul...@arm.com A minor change, fixed missplled 'DEPRECATED' in the dev_warn(). Thanks Suzuki 8 Avoid secure transactions while probing the CCI PMU. The existing code makes use of

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device * pdev-dev.of_node); if (!match) return NULL; + if (match-data) + return

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Suzuki K. Poulose
On 19/03/15 17:38, Sudeep Holla wrote: On 19/03/15 17:32, Mark Rutland wrote: One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device * pdev-dev.of_node); if (!match)

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Sudeep Holla
On 19/03/15 17:32, Mark Rutland wrote: One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device * pdev-dev.of_node); if (!match) return NULL; + if

Re: [UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-19 Thread Mark Rutland
On Thu, Mar 19, 2015 at 05:52:54PM +, Suzuki K. Poulose wrote: On 19/03/15 17:38, Sudeep Holla wrote: On 19/03/15 17:32, Mark Rutland wrote: One more thing: @@ -883,7 +894,11 @@ static inline const struct cci_pmu_model *get_cci_model(struct platform_device *

[UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-17 Thread Suzuki K. Poulose
From: "Suzuki K. Poulose" A minor change, fixed missplled 'DEPRECATED' in the dev_warn(). Thanks Suzuki 8> Avoid secure transactions while probing the CCI PMU. The existing code makes use of the Peripheral ID2 (PID2) register to determine the revision of the CCI400, which requires a

[UPDATED] [PATCH 3/5] arm-cci: Get rid of secure transactions for PMU driver

2015-03-17 Thread Suzuki K. Poulose
From: Suzuki K. Poulose suzuki.poul...@arm.com A minor change, fixed missplled 'DEPRECATED' in the dev_warn(). Thanks Suzuki 8 Avoid secure transactions while probing the CCI PMU. The existing code makes use of the Peripheral ID2 (PID2) register to determine the revision of the CCI400,