* [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
@ 2013-02-04 23:41 ` Kukjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Kukjin Kim @ 2013-02-04 23:41 UTC (permalink / raw)
To: 'Arnd Bergmann', 'Olof Johansson'
Cc: linux-arm-kernel, linux-samsung-soc
Hi Arnd and Olof,
Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
pull-request branch, 'next/cleanup-header'.
Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
into mach-s3c24xx.
Note, this is 2nd pull request and first is 'next/cleanup-header' branch.
If any problems, please let me know.
Thanks.
- Kukjin
The following changes since commit 3a71c5c375e29fc9c1493b5ead1cf36572139c3e:
ARM: S3C64XX: Fix missing header error with CONFIG_CPU_IDLE enabled
(2013-01-10 10:45:41 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
next/cleanup-s3c24xx
for you to fetch changes up to acf2d41d8595829eb0ac7bf6891f4875a78d4d6e:
ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/ (2013-01-21
17:05:23 -0800)
----------------------------------------------------------------
Kukjin Kim (6):
ARM: S3C24XX: Move mach-s3c2410/ cpufreq into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2410/ pll into mach-s3c24xx/
ARM: S3C2412: Remove useless codes mach-s3c2412/gpio.c
ARM: S3C24XX: Move mach-s3c2412/ cpufreq driver into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2440/ cpufreq driver into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/
arch/arm/Kconfig
| 4 ----
arch/arm/Makefile
| 2 +-
arch/arm/mach-s3c2410/Kconfig
| 20 ----------------
arch/arm/mach-s3c2410/Makefile
| 14 -----------
arch/arm/mach-s3c2412/Kconfig
| 13 ----------
arch/arm/mach-s3c2412/Makefile
| 12 ----------
arch/arm/mach-s3c2412/gpio.c
| 62 ------------------------------------------------
arch/arm/mach-s3c2440/Kconfig
| 37 -----------------------------
arch/arm/mach-s3c2440/Makefile
| 15 ------------
arch/arm/mach-s3c24xx/Kconfig
| 56 +++++++++++++++++++++++++++++++++++++++++++
arch/arm/mach-s3c24xx/Makefile
| 6 +++++
arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
| 7 ++----
arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
| 6 ++---
arch/arm/{mach-s3c2440/s3c2440-cpufreq.c => mach-s3c24xx/cpufreq-s3c2440.c}
| 4 +---
arch/arm/mach-s3c24xx/include/mach/hardware.h
| 6 -----
arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c}
| 5 +---
arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
mach-s3c24xx/pll-s3c2440-12000000.c} | 5 +---
arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
mach-s3c24xx/pll-s3c2440-16934400.c} | 5 +---
18 files changed, 71 insertions(+), 208 deletions(-)
delete mode 100644 arch/arm/mach-s3c2410/Kconfig
delete mode 100644 arch/arm/mach-s3c2410/Makefile
delete mode 100644 arch/arm/mach-s3c2412/Kconfig
delete mode 100644 arch/arm/mach-s3c2412/Makefile
delete mode 100644 arch/arm/mach-s3c2412/gpio.c
delete mode 100644 arch/arm/mach-s3c2440/Kconfig
delete mode 100644 arch/arm/mach-s3c2440/Makefile
rename arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
(98%)
rename arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
(99%)
rename arch/arm/{mach-s3c2440/s3c2440-cpufreq.c =>
mach-s3c24xx/cpufreq-s3c2440.c} (99%)
rename arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c} (99%)
rename arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
mach-s3c24xx/pll-s3c2440-12000000.c} (98%)
rename arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
mach-s3c24xx/pll-s3c2440-16934400.c} (99%)
^ permalink raw reply [flat|nested] 8+ messages in thread
* [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
@ 2013-02-04 23:41 ` Kukjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Kukjin Kim @ 2013-02-04 23:41 UTC (permalink / raw)
To: linux-arm-kernel
Hi Arnd and Olof,
Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
pull-request branch, 'next/cleanup-header'.
Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
into mach-s3c24xx.
Note, this is 2nd pull request and first is 'next/cleanup-header' branch.
If any problems, please let me know.
Thanks.
- Kukjin
The following changes since commit 3a71c5c375e29fc9c1493b5ead1cf36572139c3e:
ARM: S3C64XX: Fix missing header error with CONFIG_CPU_IDLE enabled
(2013-01-10 10:45:41 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
next/cleanup-s3c24xx
for you to fetch changes up to acf2d41d8595829eb0ac7bf6891f4875a78d4d6e:
ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/ (2013-01-21
17:05:23 -0800)
----------------------------------------------------------------
Kukjin Kim (6):
ARM: S3C24XX: Move mach-s3c2410/ cpufreq into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2410/ pll into mach-s3c24xx/
ARM: S3C2412: Remove useless codes mach-s3c2412/gpio.c
ARM: S3C24XX: Move mach-s3c2412/ cpufreq driver into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2440/ cpufreq driver into mach-s3c24xx/
ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/
arch/arm/Kconfig
| 4 ----
arch/arm/Makefile
| 2 +-
arch/arm/mach-s3c2410/Kconfig
| 20 ----------------
arch/arm/mach-s3c2410/Makefile
| 14 -----------
arch/arm/mach-s3c2412/Kconfig
| 13 ----------
arch/arm/mach-s3c2412/Makefile
| 12 ----------
arch/arm/mach-s3c2412/gpio.c
| 62 ------------------------------------------------
arch/arm/mach-s3c2440/Kconfig
| 37 -----------------------------
arch/arm/mach-s3c2440/Makefile
| 15 ------------
arch/arm/mach-s3c24xx/Kconfig
| 56 +++++++++++++++++++++++++++++++++++++++++++
arch/arm/mach-s3c24xx/Makefile
| 6 +++++
arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
| 7 ++----
arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
| 6 ++---
arch/arm/{mach-s3c2440/s3c2440-cpufreq.c => mach-s3c24xx/cpufreq-s3c2440.c}
| 4 +---
arch/arm/mach-s3c24xx/include/mach/hardware.h
| 6 -----
arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c}
| 5 +---
arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
mach-s3c24xx/pll-s3c2440-12000000.c} | 5 +---
arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
mach-s3c24xx/pll-s3c2440-16934400.c} | 5 +---
18 files changed, 71 insertions(+), 208 deletions(-)
delete mode 100644 arch/arm/mach-s3c2410/Kconfig
delete mode 100644 arch/arm/mach-s3c2410/Makefile
delete mode 100644 arch/arm/mach-s3c2412/Kconfig
delete mode 100644 arch/arm/mach-s3c2412/Makefile
delete mode 100644 arch/arm/mach-s3c2412/gpio.c
delete mode 100644 arch/arm/mach-s3c2440/Kconfig
delete mode 100644 arch/arm/mach-s3c2440/Makefile
rename arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
(98%)
rename arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
(99%)
rename arch/arm/{mach-s3c2440/s3c2440-cpufreq.c =>
mach-s3c24xx/cpufreq-s3c2440.c} (99%)
rename arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c} (99%)
rename arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
mach-s3c24xx/pll-s3c2440-12000000.c} (98%)
rename arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
mach-s3c24xx/pll-s3c2440-16934400.c} (99%)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
2013-02-04 23:41 ` Kukjin Kim
@ 2013-02-06 0:15 ` Olof Johansson
-1 siblings, 0 replies; 8+ messages in thread
From: Olof Johansson @ 2013-02-06 0:15 UTC (permalink / raw)
To: Kukjin Kim; +Cc: 'Arnd Bergmann', linux-arm-kernel, linux-samsung-soc
On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
> Hi Arnd and Olof,
>
> Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
> pull-request branch, 'next/cleanup-header'.
> Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
> into mach-s3c24xx.
>
> Note, this is 2nd pull request and first is 'next/cleanup-header' branch.
> If any problems, please let me know.
Hi,
So, your first pull request didn't contain a "1/5", so it's hard to match them
up.
Also, see how the word wrapping looks below? I wonder if you run your "git
request-pull" in a terminal window that is wider than 80 characters, and then
copy-and-paste it over? That doesn't work well. (I suspect you do since your
git URL is always wrapped too :( )
Finally, would you mind starting to use signed tags with branch descriptions,
or at least give a short overview of the branches in the pull requests?
Thanks.
-Olof
>
> Thanks.
>
> - Kukjin
>
>
> The following changes since commit 3a71c5c375e29fc9c1493b5ead1cf36572139c3e:
>
> ARM: S3C64XX: Fix missing header error with CONFIG_CPU_IDLE enabled
> (2013-01-10 10:45:41 -0800)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> next/cleanup-s3c24xx
>
> for you to fetch changes up to acf2d41d8595829eb0ac7bf6891f4875a78d4d6e:
>
> ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/ (2013-01-21
> 17:05:23 -0800)
>
> ----------------------------------------------------------------
> Kukjin Kim (6):
> ARM: S3C24XX: Move mach-s3c2410/ cpufreq into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2410/ pll into mach-s3c24xx/
> ARM: S3C2412: Remove useless codes mach-s3c2412/gpio.c
> ARM: S3C24XX: Move mach-s3c2412/ cpufreq driver into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2440/ cpufreq driver into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/
>
> arch/arm/Kconfig
> | 4 ----
> arch/arm/Makefile
> | 2 +-
> arch/arm/mach-s3c2410/Kconfig
> | 20 ----------------
> arch/arm/mach-s3c2410/Makefile
> | 14 -----------
> arch/arm/mach-s3c2412/Kconfig
> | 13 ----------
> arch/arm/mach-s3c2412/Makefile
> | 12 ----------
> arch/arm/mach-s3c2412/gpio.c
> | 62 ------------------------------------------------
> arch/arm/mach-s3c2440/Kconfig
> | 37 -----------------------------
> arch/arm/mach-s3c2440/Makefile
> | 15 ------------
> arch/arm/mach-s3c24xx/Kconfig
> | 56 +++++++++++++++++++++++++++++++++++++++++++
> arch/arm/mach-s3c24xx/Makefile
> | 6 +++++
> arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
> | 7 ++----
> arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
> | 6 ++---
> arch/arm/{mach-s3c2440/s3c2440-cpufreq.c => mach-s3c24xx/cpufreq-s3c2440.c}
> | 4 +---
> arch/arm/mach-s3c24xx/include/mach/hardware.h
> | 6 -----
> arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c}
> | 5 +---
> arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
> mach-s3c24xx/pll-s3c2440-12000000.c} | 5 +---
> arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
> mach-s3c24xx/pll-s3c2440-16934400.c} | 5 +---
> 18 files changed, 71 insertions(+), 208 deletions(-)
> delete mode 100644 arch/arm/mach-s3c2410/Kconfig
> delete mode 100644 arch/arm/mach-s3c2410/Makefile
> delete mode 100644 arch/arm/mach-s3c2412/Kconfig
> delete mode 100644 arch/arm/mach-s3c2412/Makefile
> delete mode 100644 arch/arm/mach-s3c2412/gpio.c
> delete mode 100644 arch/arm/mach-s3c2440/Kconfig
> delete mode 100644 arch/arm/mach-s3c2440/Makefile
> rename arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
> (98%)
> rename arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
> (99%)
> rename arch/arm/{mach-s3c2440/s3c2440-cpufreq.c =>
> mach-s3c24xx/cpufreq-s3c2440.c} (99%)
> rename arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c} (99%)
> rename arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
> mach-s3c24xx/pll-s3c2440-12000000.c} (98%)
> rename arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
> mach-s3c24xx/pll-s3c2440-16934400.c} (99%)
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
@ 2013-02-06 0:15 ` Olof Johansson
0 siblings, 0 replies; 8+ messages in thread
From: Olof Johansson @ 2013-02-06 0:15 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
> Hi Arnd and Olof,
>
> Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
> pull-request branch, 'next/cleanup-header'.
> Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
> into mach-s3c24xx.
>
> Note, this is 2nd pull request and first is 'next/cleanup-header' branch.
> If any problems, please let me know.
Hi,
So, your first pull request didn't contain a "1/5", so it's hard to match them
up.
Also, see how the word wrapping looks below? I wonder if you run your "git
request-pull" in a terminal window that is wider than 80 characters, and then
copy-and-paste it over? That doesn't work well. (I suspect you do since your
git URL is always wrapped too :( )
Finally, would you mind starting to use signed tags with branch descriptions,
or at least give a short overview of the branches in the pull requests?
Thanks.
-Olof
>
> Thanks.
>
> - Kukjin
>
>
> The following changes since commit 3a71c5c375e29fc9c1493b5ead1cf36572139c3e:
>
> ARM: S3C64XX: Fix missing header error with CONFIG_CPU_IDLE enabled
> (2013-01-10 10:45:41 -0800)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> next/cleanup-s3c24xx
>
> for you to fetch changes up to acf2d41d8595829eb0ac7bf6891f4875a78d4d6e:
>
> ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/ (2013-01-21
> 17:05:23 -0800)
>
> ----------------------------------------------------------------
> Kukjin Kim (6):
> ARM: S3C24XX: Move mach-s3c2410/ cpufreq into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2410/ pll into mach-s3c24xx/
> ARM: S3C2412: Remove useless codes mach-s3c2412/gpio.c
> ARM: S3C24XX: Move mach-s3c2412/ cpufreq driver into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2440/ cpufreq driver into mach-s3c24xx/
> ARM: S3C24XX: Move mach-s3c2440/ pll into mach-s3c24xx/
>
> arch/arm/Kconfig
> | 4 ----
> arch/arm/Makefile
> | 2 +-
> arch/arm/mach-s3c2410/Kconfig
> | 20 ----------------
> arch/arm/mach-s3c2410/Makefile
> | 14 -----------
> arch/arm/mach-s3c2412/Kconfig
> | 13 ----------
> arch/arm/mach-s3c2412/Makefile
> | 12 ----------
> arch/arm/mach-s3c2412/gpio.c
> | 62 ------------------------------------------------
> arch/arm/mach-s3c2440/Kconfig
> | 37 -----------------------------
> arch/arm/mach-s3c2440/Makefile
> | 15 ------------
> arch/arm/mach-s3c24xx/Kconfig
> | 56 +++++++++++++++++++++++++++++++++++++++++++
> arch/arm/mach-s3c24xx/Makefile
> | 6 +++++
> arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
> | 7 ++----
> arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
> | 6 ++---
> arch/arm/{mach-s3c2440/s3c2440-cpufreq.c => mach-s3c24xx/cpufreq-s3c2440.c}
> | 4 +---
> arch/arm/mach-s3c24xx/include/mach/hardware.h
> | 6 -----
> arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c}
> | 5 +---
> arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
> mach-s3c24xx/pll-s3c2440-12000000.c} | 5 +---
> arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
> mach-s3c24xx/pll-s3c2440-16934400.c} | 5 +---
> 18 files changed, 71 insertions(+), 208 deletions(-)
> delete mode 100644 arch/arm/mach-s3c2410/Kconfig
> delete mode 100644 arch/arm/mach-s3c2410/Makefile
> delete mode 100644 arch/arm/mach-s3c2412/Kconfig
> delete mode 100644 arch/arm/mach-s3c2412/Makefile
> delete mode 100644 arch/arm/mach-s3c2412/gpio.c
> delete mode 100644 arch/arm/mach-s3c2440/Kconfig
> delete mode 100644 arch/arm/mach-s3c2440/Makefile
> rename arch/arm/{mach-s3c2410/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2410.c}
> (98%)
> rename arch/arm/{mach-s3c2412/cpu-freq.c => mach-s3c24xx/cpufreq-s3c2412.c}
> (99%)
> rename arch/arm/{mach-s3c2440/s3c2440-cpufreq.c =>
> mach-s3c24xx/cpufreq-s3c2440.c} (99%)
> rename arch/arm/{mach-s3c2410/pll.c => mach-s3c24xx/pll-s3c2410.c} (99%)
> rename arch/arm/{mach-s3c2440/s3c2440-pll-12000000.c =>
> mach-s3c24xx/pll-s3c2440-12000000.c} (98%)
> rename arch/arm/{mach-s3c2440/s3c2440-pll-16934400.c =>
> mach-s3c24xx/pll-s3c2440-16934400.c} (99%)
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
2013-02-06 0:15 ` Olof Johansson
@ 2013-02-06 0:54 ` Kukjin Kim
-1 siblings, 0 replies; 8+ messages in thread
From: Kukjin Kim @ 2013-02-06 0:54 UTC (permalink / raw)
To: 'Olof Johansson'
Cc: 'Arnd Bergmann', linux-arm-kernel, linux-samsung-soc
Olof Johansson wrote:
>
> On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
> > Hi Arnd and Olof,
> >
> > Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
> > pull-request branch, 'next/cleanup-header'.
> > Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
> > into mach-s3c24xx.
> >
> > Note, this is 2nd pull request and first is 'next/cleanup-header'
branch.
> > If any problems, please let me know.
>
> Hi,
>
Hi,
> So, your first pull request didn't contain a "1/5", so it's hard to match
them
> up.
>
Could be, sorry for that.
> Also, see how the word wrapping looks below? I wonder if you run your "git
> request-pull" in a terminal window that is wider than 80 characters, and
then
> copy-and-paste it over? That doesn't work well. (I suspect you do since
your
> git URL is always wrapped too :( )
>
Yeah, I know...sigh, it's hard to me how to avoid wrapped git URL...anyway
let me check it more closely from now on.
> Finally, would you mind starting to use signed tags with branch
descriptions,
Hmm, I see...but unfortunately, I can't use signed tag in my office, because
I can't use my own laptop in my office due to company security. Let me find
the way but probably need more time to use it.
> or at least give a short overview of the branches in the pull requests?
>
OK, let me add it next time.
So you want to re-send this pull request? Even though I'm not sure how I can
cover your ask except a short overview...
Thanks.
- Kukjin
^ permalink raw reply [flat|nested] 8+ messages in thread
* [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
@ 2013-02-06 0:54 ` Kukjin Kim
0 siblings, 0 replies; 8+ messages in thread
From: Kukjin Kim @ 2013-02-06 0:54 UTC (permalink / raw)
To: linux-arm-kernel
Olof Johansson wrote:
>
> On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
> > Hi Arnd and Olof,
> >
> > Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
> > pull-request branch, 'next/cleanup-header'.
> > Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
> > into mach-s3c24xx.
> >
> > Note, this is 2nd pull request and first is 'next/cleanup-header'
branch.
> > If any problems, please let me know.
>
> Hi,
>
Hi,
> So, your first pull request didn't contain a "1/5", so it's hard to match
them
> up.
>
Could be, sorry for that.
> Also, see how the word wrapping looks below? I wonder if you run your "git
> request-pull" in a terminal window that is wider than 80 characters, and
then
> copy-and-paste it over? That doesn't work well. (I suspect you do since
your
> git URL is always wrapped too :( )
>
Yeah, I know...sigh, it's hard to me how to avoid wrapped git URL...anyway
let me check it more closely from now on.
> Finally, would you mind starting to use signed tags with branch
descriptions,
Hmm, I see...but unfortunately, I can't use signed tag in my office, because
I can't use my own laptop in my office due to company security. Let me find
the way but probably need more time to use it.
> or at least give a short overview of the branches in the pull requests?
>
OK, let me add it next time.
So you want to re-send this pull request? Even though I'm not sure how I can
cover your ask except a short overview...
Thanks.
- Kukjin
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
2013-02-06 0:54 ` Kukjin Kim
@ 2013-02-06 1:08 ` Olof Johansson
-1 siblings, 0 replies; 8+ messages in thread
From: Olof Johansson @ 2013-02-06 1:08 UTC (permalink / raw)
To: Kukjin Kim; +Cc: Arnd Bergmann, linux-arm-kernel, linux-samsung-soc
On Tue, Feb 5, 2013 at 4:54 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Olof Johansson wrote:
>>
>> On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
>> > Hi Arnd and Olof,
>> >
>> > Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
>> > pull-request branch, 'next/cleanup-header'.
>> > Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
>> > into mach-s3c24xx.
>> >
>> > Note, this is 2nd pull request and first is 'next/cleanup-header'
> branch.
>> > If any problems, please let me know.
>>
>> Hi,
>>
> Hi,
>
>> So, your first pull request didn't contain a "1/5", so it's hard to match
> them
>> up.
>>
> Could be, sorry for that.
>
>> Also, see how the word wrapping looks below? I wonder if you run your "git
>> request-pull" in a terminal window that is wider than 80 characters, and
> then
>> copy-and-paste it over? That doesn't work well. (I suspect you do since
> your
>> git URL is always wrapped too :( )
>>
> Yeah, I know...sigh, it's hard to me how to avoid wrapped git URL...anyway
> let me check it more closely from now on.
>
>> Finally, would you mind starting to use signed tags with branch
> descriptions,
>
> Hmm, I see...but unfortunately, I can't use signed tag in my office, because
> I can't use my own laptop in my office due to company security. Let me find
> the way but probably need more time to use it.
>
>> or at least give a short overview of the branches in the pull requests?
>>
> OK, let me add it next time.
>
> So you want to re-send this pull request? Even though I'm not sure how I can
> cover your ask except a short overview...
No, it's ok for this time. I've pulled in all the branches now. It was
more for future branches.
-Olof
^ permalink raw reply [flat|nested] 8+ messages in thread
* [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9
@ 2013-02-06 1:08 ` Olof Johansson
0 siblings, 0 replies; 8+ messages in thread
From: Olof Johansson @ 2013-02-06 1:08 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Feb 5, 2013 at 4:54 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Olof Johansson wrote:
>>
>> On Mon, Feb 04, 2013 at 03:41:55PM -0800, Kukjin Kim wrote:
>> > Hi Arnd and Olof,
>> >
>> > Here is 2nd cleanup for Samsung S3C24XX stuff and based on previous
>> > pull-request branch, 'next/cleanup-header'.
>> > Basically, this patch merges mach-s3c2410, mach-s3c2412 and mach-s3c2440
>> > into mach-s3c24xx.
>> >
>> > Note, this is 2nd pull request and first is 'next/cleanup-header'
> branch.
>> > If any problems, please let me know.
>>
>> Hi,
>>
> Hi,
>
>> So, your first pull request didn't contain a "1/5", so it's hard to match
> them
>> up.
>>
> Could be, sorry for that.
>
>> Also, see how the word wrapping looks below? I wonder if you run your "git
>> request-pull" in a terminal window that is wider than 80 characters, and
> then
>> copy-and-paste it over? That doesn't work well. (I suspect you do since
> your
>> git URL is always wrapped too :( )
>>
> Yeah, I know...sigh, it's hard to me how to avoid wrapped git URL...anyway
> let me check it more closely from now on.
>
>> Finally, would you mind starting to use signed tags with branch
> descriptions,
>
> Hmm, I see...but unfortunately, I can't use signed tag in my office, because
> I can't use my own laptop in my office due to company security. Let me find
> the way but probably need more time to use it.
>
>> or at least give a short overview of the branches in the pull requests?
>>
> OK, let me add it next time.
>
> So you want to re-send this pull request? Even though I'm not sure how I can
> cover your ask except a short overview...
No, it's ok for this time. I've pulled in all the branches now. It was
more for future branches.
-Olof
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-02-06 1:08 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-04 23:41 [GIT PULL 2/5] Samsung cleanup-s3c24xx for v3.9 Kukjin Kim
2013-02-04 23:41 ` Kukjin Kim
2013-02-06 0:15 ` Olof Johansson
2013-02-06 0:15 ` Olof Johansson
2013-02-06 0:54 ` Kukjin Kim
2013-02-06 0:54 ` Kukjin Kim
2013-02-06 1:08 ` Olof Johansson
2013-02-06 1:08 ` Olof Johansson
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.