Hi Philipp,

On Fri, Nov 29, 2013 at 09:22:44AM +0000, Rosenberger, Philipp wrote:
> Thanks for your replay and all the hints about submitting the correct
> way. Is it ok to paste the full git format-patch output like this?
> 
> From a2e84efe0d5335547ca3e6c0a8bf8a026b0f2353 Mon Sep 17 00:00:00 2001
> From: Philipp Rosenberger <philipp.rosenber...@xse.de>
> Date: Thu, 28 Nov 2013 12:36:52 +0100
> Subject: [PATCH] of_mtd: Fix header file include guard
> 
> It seems the include guard was copied from of_net.h.
> 
> Signed-off-by: Philipp Rosenberger <philipp.rosenber...@xse.de>
> ---
>  include/linux/of_mtd.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/of_mtd.h b/include/linux/of_mtd.h
> index 6f10e93..cb32d9c 100644
> --- a/include/linux/of_mtd.h
> +++ b/include/linux/of_mtd.h
> @@ -7,7 +7,7 @@
>   */
> 
>  #ifndef __LINUX_OF_MTD_H
> -#define __LINUX_OF_NET_H
> +#define __LINUX_OF_MTD_H
> 
>  #ifdef CONFIG_OF_MTD
> 
> --
> 1.7.9.5

That's workable for me, but that's not quite the standard way. The
git-format-patch output is meant to be mailed directly (via
git-send-email, for instance), not pasted into another email. If you
paste it, then git-am will see two sets of email headers and may not
understand it.

> On 29.11.2013 09:11, Brian Norris wrote:
> > + device tree guys
> >
> > Hi Grant/Rob,
> >
> > It looks like this header is technically owned by you guys. Once we have
> > a proper submission, should I merge this sort of change through MTD, or
> > do one of you want to take care of it?

Unless someone complains soon, I'll take this via MTD. No one else
should be creating conflicts on this file anyway.

...
> >> Confidentiality Notice: This e-mail message, including any attachments, is 
> >> for the sole use of the intended recipient(s) and may contain confidential 
> >> and privileged information. Any unauthorized review, use, disclosure or 
> >> distribution is prohibited. If you are not the intended recipient, please 
> >> contact the sender by return e-mail and destroy all copies of the original 
> >> message.
> >
> > This confidentiality notice is incongruous with sending patches to a
> > public mailing list, for inclusion in an open source project. This
> > message (combined with a lack of a Signed-off-by) means we cannot take
> > your patch. Please remove this footer and resubmit, with a CC to
> > linux-...@infradead.org and devicetree@vger.kernel.org.
> >
> > Thanks,
> > Brian
> >
> >
> 
> Thank you for this hint. I forgot that the mailserver append this
> footer. I hope now there is no company footer anymore.

Looks good now. Thanks.

Pushed to l2-mtd.git.

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

Reply via email to