Re: [PATCH V3 04/19] OMAP3+: voltage: remove spurious pr_notice for debugfs

2011-07-26 Thread Felipe Balbi 
On Sat, Mar 05, 2011 at 08:59:07PM +0530, Nishanth Menon wrote:
 cat of debugfs entry for vp_volt provides voltage. The additional pr_notice
 is just spam on console and provides no additional information.
 
 Signed-off-by: Nishanth Menon n...@ti.com

FWIW:

Reviewed-by: Felipe Balbi ba...@ti.com

 ---
  arch/arm/mach-omap2/voltage.c |1 -
  1 files changed, 0 insertions(+), 1 deletions(-)
 
 diff --git a/arch/arm/mach-omap2/voltage.c b/arch/arm/mach-omap2/voltage.c
 index 1b40aa0..53c399f 100644
 --- a/arch/arm/mach-omap2/voltage.c
 +++ b/arch/arm/mach-omap2/voltage.c
 @@ -150,7 +150,6 @@ static int vp_volt_debug_get(void *data, u64 *val)
   }
  
   vsel = vdd-read_reg(prm_mod_offs, vdd-vp_data-voltage);
 - pr_notice(curr_vsel = %x\n, vsel);
  
   if (!vdd-pmic_info-vsel_to_uv) {
   pr_warning(PMIC function to convert vsel to voltage
 -- 
 1.7.1
 
 --
 To unsubscribe from this list: send the line unsubscribe linux-omap in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
balbi


signature.asc
Description: Digital signature

Re: [PATCH V3 04/19] OMAP3+: voltage: remove spurious pr_notice for debugfs

2011-03-17 Thread Kevin Hilman 
Nishanth Menon n...@ti.com writes:

 cat of debugfs entry for vp_volt provides voltage. The additional pr_notice
 is just spam on console and provides no additional information.

 Signed-off-by: Nishanth Menon n...@ti.com

Thanks, queueing for 2.6.39-rc fixes cycle.

Kevin

 ---
  arch/arm/mach-omap2/voltage.c |1 -
  1 files changed, 0 insertions(+), 1 deletions(-)

 diff --git a/arch/arm/mach-omap2/voltage.c b/arch/arm/mach-omap2/voltage.c
 index 1b40aa0..53c399f 100644
 --- a/arch/arm/mach-omap2/voltage.c
 +++ b/arch/arm/mach-omap2/voltage.c
 @@ -150,7 +150,6 @@ static int vp_volt_debug_get(void *data, u64 *val)
   }
  
   vsel = vdd-read_reg(prm_mod_offs, vdd-vp_data-voltage);
 - pr_notice(curr_vsel = %x\n, vsel);
  
   if (!vdd-pmic_info-vsel_to_uv) {
   pr_warning(PMIC function to convert vsel to voltage
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH V3 04/19] OMAP3+: voltage: remove spurious pr_notice for debugfs

2011-03-05 Thread Nishanth Menon 
cat of debugfs entry for vp_volt provides voltage. The additional pr_notice
is just spam on console and provides no additional information.

Signed-off-by: Nishanth Menon n...@ti.com
---
 arch/arm/mach-omap2/voltage.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/voltage.c b/arch/arm/mach-omap2/voltage.c
index 1b40aa0..53c399f 100644
--- a/arch/arm/mach-omap2/voltage.c
+++ b/arch/arm/mach-omap2/voltage.c
@@ -150,7 +150,6 @@ static int vp_volt_debug_get(void *data, u64 *val)
}
 
vsel = vdd-read_reg(prm_mod_offs, vdd-vp_data-voltage);
-   pr_notice(curr_vsel = %x\n, vsel);
 
if (!vdd-pmic_info-vsel_to_uv) {
pr_warning(PMIC function to convert vsel to voltage
-- 
1.7.1

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html