>>-----Original Message----- >>From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of >>Nishanth Menon >>Sent: Friday, August 06, 2010 4:38 PM >>To: Gopinath, Thara >>Cc: Menon, Nishanth; linux-omap; Kevin Hilman >>Subject: Re: [PM-SR][PATCH 01/12] omap3: voltage: cleanup pr_xxxx >> >>On 08/06/2010 02:42 AM, Gopinath, Thara wrote: >>> >>> >>>>> -----Original Message----- >>>>> From: Menon, Nishanth >>>>> Sent: Friday, August 06, 2010 3:54 AM >>>>> To: linux-omap >>>>> Cc: Menon, Nishanth; Kevin Hilman; Gopinath, Thara >>>>> Subject: [PM-SR][PATCH 01/12] omap3: voltage: cleanup pr_xxxx >>>>> >>>>> Few more pr_xxxx need cleanup for printing the function name and >>>>> not using multiline prints when c allows us to do "". >>>>> >>>>> Cc: Kevin Hilman<khilman@xxxxxxxxxxxxxxxxxxx> >>>>> Cc: Thara Gopinath<thara@xxxxxx> >>>>> >>>>> Signed-off-by: Nishanth Menon<nm@xxxxxx> >>>>> --- >>>>> arch/arm/mach-omap2/voltage.c | 7 ++++--- >>>>> 1 files changed, 4 insertions(+), 3 deletions(-) >>>>> >>>>> diff --git a/arch/arm/mach-omap2/voltage.c b/arch/arm/mach-omap2/voltage.c >>>>> index 148e4d4..3431fa3 100644 >>>>> --- a/arch/arm/mach-omap2/voltage.c >>>>> +++ b/arch/arm/mach-omap2/voltage.c >>>>> @@ -253,8 +253,9 @@ static int vp_debug_set(void *data, u64 val) >>>>> u32 *option = data; >>>>> *option = val; >>>>> } else { >>>>> - pr_notice("DEBUG option not enabled!\n \ >>>>> - echo 1> pm_debug/enable_sr_vp_debug - to enable\n"); >>>>> + pr_notice("%s: DEBUG option not enabled! " >>>>> + "echo 1> pm_debug/enable_sr_vp_debug - to enable\n", >>>>> + __func__); >>>>> } >>> >>> I do not think this is needed as these fns get called only from user space. >>> >> >>have you tried working on someone else's unfamiliar code and grepping >>for a warning message at the middle of the night because some developer >>forgot to give a "%s:...", __func__ with half a dozen people watching >>behind your back to know it's meaning because the product is hitting the >>shelves the next day? Instead of being able to use cscope and pop the >>function in and go straight to it and understand what is happening?? I >>believe there are few in this list who had the fortune to be in that sitn.. >> >> >>Well that is my motivation here. if driver reports a warning to kernel >>syslog, well, I expect the log to contain the function name for me to >>maintain faster. Bingo!! Kudos on managing to prove that you are a cut above the rest. Poor moi wont understand at all as moi do not work at all!! But unless there are others in the community who feel that this is absolutely necessary I am not pulling this in. For me you do a cat /debug/pm_debg<some entry> and get an error it is easy to figure out in the kernel with the limited experience I have working on linux kernel. Regards Thara >> >>> Regards >>> Thara >>> >>>>> return 0; >>>>> } >>>>> @@ -265,7 +266,7 @@ static int vp_volt_debug_get(void *data, u64 *val) >>>>> u8 vsel; >>>>> >>>>> vsel = voltage_read_reg(info->vp_offs.voltage_reg); >>>>> - pr_notice("curr_vsel = %x\n", vsel); >>>>> + pr_notice("%s: curr_vsel = %x\n", __func__, vsel); >>>>> *val = vsel * 12500 + 600000; >>>>> >>>>> return 0; >>>>> -- >>>>> 1.6.3.3 >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in >>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >>-- >>To unsubscribe from this list: send the line "unsubscribe linux-omap" in >>the body of a message to majordomo@xxxxxxxxxxxxxxx >>More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
