From: Tony Lindgren <tony@atomide.com>
To: Hiroki Kaminaga <kaminaga@sm.sony.co.jp>
Cc: linux-omap-open-source@linux.omap.com, Craps@kristian-m.de
Subject: Re: build error on Linux-2.6.15-omap1
Date: Thu, 12 Jan 2006 14:43:50 -0800 [thread overview]
Message-ID: <20060112224350.GJ5461@atomide.com> (raw)
In-Reply-To: <20060112.202904.59649446.kaminaga@sm.sony.co.jp>
* Hiroki Kaminaga <kaminaga@sm.sony.co.jp> [060112 03:36]:
> Hi,
>
> > I guess there must be something wrong with your patch since in
> > linux-2.6.15-omap1 pm.c and Sleep.S are already in mach-omap1
> >
> > That is since 1st december last year:
> > http://source.mvista.com/git/gitweb.cgi?p=linux-omap-2.6.git;a=commit;h=a57a89ee9ad70186e4f7be46e0930fcb8fb53bb6
> >
>
> Oh!
> It is fixed in source code management system!
>
> However, I looked Linux-2.6.15-omap1 patch again, and it was as follows:
>
> * * *
>
> diff --git a/arch/arm/plat-omap/pm.c b/arch/arm/mach-omap1/pm.c
> similarity index 83%
> rename from arch/arm/plat-omap/pm.c
> rename to arch/arm/mach-omap1/pm.c
> index 1a24e2c..d483ac4 100644
> --- a/arch/arm/plat-omap/pm.c
> +++ b/arch/arm/mach-omap1/pm.c
> @@ -40,22 +40,26 @@
> #include <linux/proc_fs.h>
> #include <linux/pm.h>
> #include <linux/interrupt.h>
> +#include <linux/sysfs.h>
> +#include <linux/module.h>
> ...
>
> * * *
>
> the `+++' part is mach-omap1, but `---' part was plat-omap, and
> after patching, pm.c was still in plat-omap...
>
> I'm using FC3 as host, diff version 2.8.1, patch version 2.5.4,
> I guess `rename from' and `rename to' line didn't do what is intended?!
Ah, thanks for catching it! This sounds like a bug in git. I've been
using this to generate the patch:
$ git diff 88026842b0a760145aa71d69e74fbc9ec118ca44..165e0bcd61404f0418fc70f07a6a23124fa0d206
> # Or I'm downloading different file? I downloaded:
> # http://www.muru.com/linux/omap/ -> Linux-2.6.15-omap1
No that's right. I'll produce 2.6.15-omap2 patch soon and will fix it in
the patch.
Regards,
Tony
prev parent reply other threads:[~2006-01-12 22:43 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-12 4:51 build error on Linux-2.6.15-omap1 Hiroki Kaminaga
2006-01-12 3:21 ` Kristian Mueller
[not found] ` <1137035283.11250.50.camel@pismo>
2006-01-12 11:29 ` Hiroki Kaminaga
2006-01-12 22:43 ` Tony Lindgren [this message]
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=20060112224350.GJ5461@atomide.com \
--to=tony@atomide.com \
--cc=Craps@kristian-m.de \
--cc=kaminaga@sm.sony.co.jp \
--cc=linux-omap-open-source@linux.omap.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.