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 *

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

2015-03-18 Thread Suzuki K. Poulose
From: "Suzuki K. Poulose" 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 secure transaction. This puts a limitation on the usage of the driver on systems running

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

2015-03-18 Thread Suzuki K. Poulose
From: Suzuki K. Poulose suzuki.poul...@arm.com 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 secure transaction. This puts a limitation on the usage of the driver on

[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,

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

2015-03-10 Thread Suzuki K. Poulose
From: "Suzuki K. Poulose" 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 secure transaction. This puts a limitation on the usage of the driver on systems running

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

2015-03-10 Thread Suzuki K. Poulose
From: Suzuki K. Poulose suzuki.poul...@arm.com 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 secure transaction. This puts a limitation on the usage of the driver on

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

2015-03-04 Thread Suzuki K. Poulose
On 03/03/15 15:44, Sudeep Holla wrote: On 02/03/15 11:29, Suzuki K. Poulose wrote: From: "Suzuki K. Poulose" 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

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

2015-03-04 Thread Suzuki K. Poulose
On 03/03/15 15:44, Sudeep Holla wrote: On 02/03/15 11:29, Suzuki K. Poulose wrote: From: Suzuki K. Poulose suzuki.poul...@arm.com 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,

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

2015-03-03 Thread Sudeep Holla
On 02/03/15 11:29, Suzuki K. Poulose wrote: From: "Suzuki K. Poulose" 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 secure transaction. This puts a limitation on

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

2015-03-03 Thread Sudeep Holla
On 02/03/15 11:29, Suzuki K. Poulose wrote: From: Suzuki K. Poulose suzuki.poul...@arm.com 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 secure transaction. This

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

2015-03-02 Thread Suzuki K. Poulose
From: "Suzuki K. Poulose" 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 secure transaction. This puts a limitation on the usage of the driver on systems running

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

2015-03-02 Thread Suzuki K. Poulose
From: Suzuki K. Poulose suzuki.poul...@arm.com 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 secure transaction. This puts a limitation on the usage of the driver on