All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@deeprootsystems.com>
To: "Munegowda, Keshava" <keshava_mgowda@ti.com>
Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"tony@atomide.com" <tony@atomide.com>
Subject: Re: [PATCH V2]omap: mux.c warning removal
Date: Mon, 10 May 2010 09:11:55 -0700	[thread overview]
Message-ID: <874oiflos4.fsf@deeprootsystems.com> (raw)
In-Reply-To: <0680EC522D0CC943BC586913CF3768C003B31655C3@dbde02.ent.ti.com> (Keshava Munegowda's message of "Fri\, 7 May 2010 12\:17\:48 +0530")

"Munegowda, Keshava" <keshava_mgowda@ti.com> writes:

> From: Keshava Munegowda <a0393220@ti.com>
>
> This patch removes the below warning 
> arch/arm/mach-omap2/mux.c:52: warning: 'mux_phys' defined but not used
> The definition of variable mux_phys should be enclosed in the macro CONFIG_ARCH_OMAP3 
>
> Signed-off-by: Keshava Munegowda <a0393220@ti.com>

Thanks, this version looks better.

Acked-by: Kevin Hilman <khilman@deeprootsystems.com>


One silly question though... is your a0....@ti.com address the one you
want as your From: and Signed-off-by?  Since the mail actually comes
from a more human readable address, I'm guessing that would be
preferred?  If so, create a ~/.gitconfig file and add this:

[user]
	name = Keshava Munegowda
	email = keshava_mgowda@ti.com

This way, all your local git commands will use this name/email in your
git commits, sign-offs etc.

Kevin







> Index: linux-2.6/arch/arm/mach-omap2/mux.c
> ===================================================================
> --- linux-2.6.orig/arch/arm/mach-omap2/mux.c	2010-05-05 23:59:19.000000000 +0530
> +++ linux-2.6/arch/arm/mach-omap2/mux.c	2010-05-06 00:00:07.000000000 +0530
> @@ -49,7 +49,7 @@
>  	struct list_head	node;
>  };
>  
> -static unsigned long mux_phys;
> +
>  static void __iomem *mux_base;
>  
>  u16 omap_mux_read(u16 reg)
> @@ -373,6 +373,7 @@
>  #ifdef CONFIG_ARCH_OMAP3
>  static LIST_HEAD(muxmodes);
>  static DEFINE_MUTEX(muxmode_mutex);
> +static unsigned long mux_phys;
>  
>  #ifdef CONFIG_OMAP_MUX
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2010-05-10 16:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-07  6:47 [PATCH V2]omap: mux.c warning removal Munegowda, Keshava
2010-05-10 16:11 ` Kevin Hilman [this message]
2010-05-10 22:30   ` Tony Lindgren
2010-05-11 10:30   ` Munegowda, Keshava

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874oiflos4.fsf@deeprootsystems.com \
    --to=khilman@deeprootsystems.com \
    --cc=keshava_mgowda@ti.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.