* [GIT PULL] Renesas SoC updates for v3.4
@ 2012-03-09 20:55 Rafael J. Wysocki
2012-03-09 21:34 ` Arnd Bergmann
2012-03-12 22:53 ` [GIT PULL] Renesas SoC updates for v3.4, take 2 Rafael J. Wysocki
0 siblings, 2 replies; 21+ messages in thread
From: Rafael J. Wysocki @ 2012-03-09 20:55 UTC (permalink / raw)
To: Arnd Bergmann
Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt
Hi,
Please pull Renesas SoC updates for v3.4 since commit
ce8fea7aa4ad9e3b40999a08622ef27c77159659:
mmap: EINVAL not ENOMEM when rejecting VM_GROWS
with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef
ARM: mach-shmobile: default to no earlytimer
from the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc
They include:
* The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid
possible future name space collision with common struct clk code.
This also affects drivers that are shared with the sh architecture,
so the branch containing this part of the material, clk_ops-rename,
will be merged into the Paul Mundt's sh tree if necessary.
* Introduction of L2 Cache support for r8a7779.
* Conversion of the mach-shmobile subarch to properly use a per-SoC
map_io and separate init_early callback for early serial console
support on platforms where that is possible.
Magnus Damm is the author of all the changes.
arch/arm/mach-shmobile/board-ag5evm.c | 39 +---------------
arch/arm/mach-shmobile/board-ap4evb.c | 44 +++----------------
arch/arm/mach-shmobile/board-bonito.c | 44 +++++-------------
arch/arm/mach-shmobile/board-g3evm.c | 38 +--------------
arch/arm/mach-shmobile/board-g4evm.c | 38 +--------------
arch/arm/mach-shmobile/board-kota2.c | 38 +--------------
arch/arm/mach-shmobile/board-mackerel.c | 44 +++----------------
arch/arm/mach-shmobile/board-marzen.c | 62 +------------------------
arch/arm/mach-shmobile/clock-r8a7740.c | 8 ++--
arch/arm/mach-shmobile/clock-r8a7779.c | 4 +-
arch/arm/mach-shmobile/clock-sh7367.c | 8 ++--
arch/arm/mach-shmobile/clock-sh7372.c | 10 ++--
arch/arm/mach-shmobile/clock-sh7377.c | 8 ++--
arch/arm/mach-shmobile/clock-sh73a0.c | 14 +++---
arch/arm/mach-shmobile/clock.c | 2 +-
arch/arm/mach-shmobile/include/mach/common.h | 9 +++-
arch/arm/mach-shmobile/setup-r8a7740.c | 45 +++++++++++++++++++
arch/arm/mach-shmobile/setup-r8a7779.c | 54 ++++++++++++++++++++++
arch/arm/mach-shmobile/setup-sh7367.c | 32 +++++++++++++
arch/arm/mach-shmobile/setup-sh7372.c | 32 +++++++++++++
arch/arm/mach-shmobile/setup-sh7377.c | 32 +++++++++++++
arch/arm/mach-shmobile/setup-sh73a0.c | 32 +++++++++++++
arch/arm/mach-shmobile/timer.c | 6 ++-
arch/sh/boards/mach-highlander/setup.c | 2 +-
arch/sh/boards/mach-sdk7786/setup.c | 2 +-
arch/sh/include/asm/clock.h | 2 +-
arch/sh/kernel/cpu/sh2/clock-sh7619.c | 12 +++---
arch/sh/kernel/cpu/sh2a/clock-sh7201.c | 12 +++---
arch/sh/kernel/cpu/sh2a/clock-sh7203.c | 12 +++---
arch/sh/kernel/cpu/sh2a/clock-sh7206.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh3.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh7705.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh7706.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh7709.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh7710.c | 12 +++---
arch/sh/kernel/cpu/sh3/clock-sh7712.c | 10 ++--
arch/sh/kernel/cpu/sh4/clock-sh4-202.c | 6 +-
arch/sh/kernel/cpu/sh4/clock-sh4.c | 12 +++---
arch/sh/kernel/cpu/sh4a/clock-sh7343.c | 4 +-
arch/sh/kernel/cpu/sh4a/clock-sh7366.c | 4 +-
arch/sh/kernel/cpu/sh4a/clock-sh7722.c | 4 +-
arch/sh/kernel/cpu/sh4a/clock-sh7723.c | 4 +-
arch/sh/kernel/cpu/sh4a/clock-sh7724.c | 6 +-
arch/sh/kernel/cpu/sh4a/clock-sh7757.c | 2 +-
arch/sh/kernel/cpu/sh4a/clock-sh7763.c | 14 +++---
arch/sh/kernel/cpu/sh4a/clock-sh7770.c | 12 +++---
arch/sh/kernel/cpu/sh4a/clock-sh7780.c | 14 +++---
arch/sh/kernel/cpu/sh4a/clock-sh7785.c | 2 +-
arch/sh/kernel/cpu/sh4a/clock-sh7786.c | 2 +-
arch/sh/kernel/cpu/sh4a/clock-shx3.c | 2 +-
arch/sh/kernel/cpu/sh5/clock-sh5.c | 12 +++---
drivers/sh/clk/cpg.c | 16 +++---
include/linux/sh_clk.h | 5 +-
53 files changed, 429 insertions(+), 459 deletions(-)
---------------
Magnus Damm (35):
sh: introduce sh_clk_ops in parallel with clk_ops
sh: convert cpg code to sh_clk_ops
ARM: mach-shmobile: sh7367 sh_clk_ops rename
ARM: mach-shmobile: sh7377 sh_clk_ops rename
ARM: mach-shmobile: sh7372 sh_clk_ops rename
ARM: mach-shmobile: sh73a0 sh_clk_ops rename
ARM: mach-shmobile: r8a7740 sh_clk_ops rename
ARM: mach-shmobile: r8a7779 sh_clk_ops rename
sh: sh header sh_clk_ops rename
sh: sh2 sh_clk_ops rename
sh: sh2a sh_clk_ops rename
sh: sh3 sh_clk_ops rename
sh: sh4 sh_clk_ops rename
sh: sh4a sh_clk_ops rename
sh: sh5 sh_clk_ops rename
sh: board sh_clk_ops rename
sh: remove clk_ops
ARM: mach-shmobile: sh7367 map_io and init_early update
ARM: mach-shmobile: sh7377 map_io and init_early update
ARM: mach-shmobile: sh7372 map_io and init_early update
ARM: mach-shmobile: sh73a0 map_io and init_early update
ARM: mach-shmobile: r8a7740 map_io and init_early update
ARM: mach-shmobile: r8a7779 map_io and init_early update
ARM: mach-shmobile: r8a7779 L2 cache support
ARM: mach-shmobile: rename clk_init() to shmobile_clk_init()
ARM: mach-shmobile: Move sh7372 Mackerel external clk setup
ARM: mach-shmobile: Move sh7372 AP4EVB external clk setup
ARM: mach-shmobile: add shmobile_earlytimer_init()
ARM: mach-shmobile: sh7367 and G3EVM timer rework
ARM: mach-shmobile: sh7377 and G4EVM timer rework
ARM: mach-shmobile: sh7372, AP4EVB and Mackerel timer rework
ARM: mach-shmobile: sh73a0, AG5EVM and Kota2 timer rework
ARM: mach-shmobile: r8a7740 and Bonito timer rework
ARM: mach-shmobile: r8a7779 and Marzen timer rework
ARM: mach-shmobile: default to no earlytimer
^ permalink raw reply [flat|nested] 21+ messages in thread* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 20:55 [GIT PULL] Renesas SoC updates for v3.4 Rafael J. Wysocki @ 2012-03-09 21:34 ` Arnd Bergmann 2012-03-09 21:48 ` Rob Herring 2012-03-09 21:52 ` Rafael J. Wysocki 2012-03-12 22:53 ` [GIT PULL] Renesas SoC updates for v3.4, take 2 Rafael J. Wysocki 1 sibling, 2 replies; 21+ messages in thread From: Arnd Bergmann @ 2012-03-09 21:34 UTC (permalink / raw) To: Rafael J. Wysocki Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt, Rob Herring On Friday 09 March 2012, Rafael J. Wysocki wrote: > Hi, > > Please pull Renesas SoC updates for v3.4 since commit > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef Hi Rafael, Please rebase this on an -rc release, otherwise we get a rather messy history in the arm-soc tree. > ARM: mach-shmobile: default to no earlytimer > > from the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc > > They include: > > * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid > possible future name space collision with common struct clk code. > > This also affects drivers that are shared with the sh architecture, > so the branch containing this part of the material, clk_ops-rename, > will be merged into the Paul Mundt's sh tree if necessary. > > * Introduction of L2 Cache support for r8a7779. > > * Conversion of the mach-shmobile subarch to properly use a per-SoC > map_io and separate init_early callback for early serial console > support on platforms where that is possible. > > Magnus Damm is the author of all the changes. For next time, I think it would be good to make those things separate pull requests, since they are all rather big. In particular, the clk rework could be a series by itself. One thing I noticed while browsing through the patches was this hunk that Rob should know about: @@ -242,6 +243,10 @@ static struct platform_device *r8a7779_late_devices[] __initdata = { void __init r8a7779_add_standard_devices(void) { +#ifdef CONFIG_CACHE_L2X0 + /* Early BRESP enable, Shared attribute override enable, 64K*16way */ + l2x0_init(__io(0xf0100000), 0x40470000, 0x82000fff); +#endif r8a7779_pm_init(); r8a7779_init_pm_domain(&r8a7779_sh4a); Here you are abusing the __io() macro rather badly as a typecast. This breaks with Rob's series to turn __io(foo) into a NULL pointer for the case where PCI is disabled. I found a few more of these, mostly in mach-shmobile but also some others: arnd@klappe2:~/linux-arm$ git grep -w __io arch/arm/mach-* | grep -v __typesafe arch/arm/mach-cns3xxx/core.c: gic_init(0, 29, __io(CNS3XXX_TC11MP_GIC_DIST_BASE_VIRT), arch/arm/mach-cns3xxx/core.c: __io(CNS3XXX_TC11MP_GIC_CPU_BASE_VIRT)); arch/arm/mach-cns3xxx/core.c: u32 __iomem *pm_base = __io(CNS3XXX_PM_BASE_VIRT); arch/arm/mach-cns3xxx/core.c: cns3xxx_tmr1 = __io(CNS3XXX_TIMER1_2_3_BASE_VIRT); arch/arm/mach-cns3xxx/devices.c: u32 __iomem *gpioa = __io(CNS3XXX_MISC_BASE_VIRT + 0x0014); arch/arm/mach-netx/generic.c: vic_init(__io(io_p2v(NETX_PA_VIC)), 0, ~0, 0); arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_SYSTEM_REG(ofs) __io(NETX_VA_SYSTEM + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_GPIO_REG(ofs) __io(NETX_VA_GPIO + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PIO_REG(ofs) __io(NETX_VA_PIO + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MIIMU __io(NETX_VA_MIIMU) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PFIFO_REG(ofs) __io(NETX_VA_PFIFO + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MEMCR_REG(ofs) __io(NETX_VA_MEMCR + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_DPMAS_REG(ofs) __io(NETX_VA_DPMAS + (ofs)) arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_I2C_REG(ofs) __io(NETX_VA_I2C, (ofs)) arch/arm/mach-realview/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) arch/arm/mach-shmobile/board-ag5evm.c: l2x0_init(__io(0xf0100000), 0x00460000, 0xc2000fff); arch/arm/mach-shmobile/board-bonito.c: l2x0_init(__io(0xf0002000), 0x40440000, 0x82000fff); arch/arm/mach-shmobile/board-kota2.c: l2x0_init(__io(0xf0100000), 0x40460000, 0x82000fff); arch/arm/mach-shmobile/include/mach/io.h:#define __io(a) ((void __iomem *)(a)) arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_dist_base = __io(0xf0001000); arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_cpu_base = __io(0xf0000100); arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_dist_base = __io(0xf0001000); arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_cpu_base = __io(0xf0000100); arch/arm/mach-shmobile/smp-r8a7779.c: __raw_writel(__pa(shmobile_secondary_vector), __io(AVECR)); arch/arm/mach-shmobile/smp-sh73a0.c: if (((__raw_readl(__io(PSTR)) >> (4 * cpu)) & 3) == 3) arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(WUPCR)); /* wake up */ arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(SRESCR)); /* reset */ arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(0, __io(APARMBAREA)); /* 4k */ arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(__pa(shmobile_secondary_vector), __io(SBAR)); arch/arm/mach-ux500/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) These are all broken and need to be changed to something else before we add the global definition for __io. Arnd ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 21:34 ` Arnd Bergmann @ 2012-03-09 21:48 ` Rob Herring 2012-03-09 21:52 ` Rafael J. Wysocki 1 sibling, 0 replies; 21+ messages in thread From: Rob Herring @ 2012-03-09 21:48 UTC (permalink / raw) To: Arnd Bergmann Cc: Rafael J. Wysocki, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On 03/09/2012 03:34 PM, Arnd Bergmann wrote: > On Friday 09 March 2012, Rafael J. Wysocki wrote: >> Hi, >> >> Please pull Renesas SoC updates for v3.4 since commit >> ce8fea7aa4ad9e3b40999a08622ef27c77159659: >> >> mmap: EINVAL not ENOMEM when rejecting VM_GROWS >> >> with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > Hi Rafael, > > Please rebase this on an -rc release, otherwise we get a rather > messy history in the arm-soc tree. > >> ARM: mach-shmobile: default to no earlytimer >> >> from the git repository at: >> >> git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc >> >> They include: >> >> * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid >> possible future name space collision with common struct clk code. >> >> This also affects drivers that are shared with the sh architecture, >> so the branch containing this part of the material, clk_ops-rename, >> will be merged into the Paul Mundt's sh tree if necessary. >> >> * Introduction of L2 Cache support for r8a7779. >> >> * Conversion of the mach-shmobile subarch to properly use a per-SoC >> map_io and separate init_early callback for early serial console >> support on platforms where that is possible. >> >> Magnus Damm is the author of all the changes. > > For next time, I think it would be good to make those things separate pull > requests, since they are all rather big. In particular, the clk rework > could be a series by itself. > > One thing I noticed while browsing through the patches was this hunk > that Rob should know about: > > @@ -242,6 +243,10 @@ static struct platform_device *r8a7779_late_devices[] __initdata = { > > void __init r8a7779_add_standard_devices(void) > { > +#ifdef CONFIG_CACHE_L2X0 > + /* Early BRESP enable, Shared attribute override enable, 64K*16way */ > + l2x0_init(__io(0xf0100000), 0x40470000, 0x82000fff); > +#endif > r8a7779_pm_init(); > > r8a7779_init_pm_domain(&r8a7779_sh4a); > > Here you are abusing the __io() macro rather badly as a typecast. This breaks > with Rob's series to turn __io(foo) into a NULL pointer for the case where > PCI is disabled. I found a few more of these, mostly in mach-shmobile but > also some others: > > arnd@klappe2:~/linux-arm$ git grep -w __io arch/arm/mach-* | grep -v __typesafe > arch/arm/mach-cns3xxx/core.c: gic_init(0, 29, __io(CNS3XXX_TC11MP_GIC_DIST_BASE_VIRT), > arch/arm/mach-cns3xxx/core.c: __io(CNS3XXX_TC11MP_GIC_CPU_BASE_VIRT)); > arch/arm/mach-cns3xxx/core.c: u32 __iomem *pm_base = __io(CNS3XXX_PM_BASE_VIRT); > arch/arm/mach-cns3xxx/core.c: cns3xxx_tmr1 = __io(CNS3XXX_TIMER1_2_3_BASE_VIRT); > arch/arm/mach-cns3xxx/devices.c: u32 __iomem *gpioa = __io(CNS3XXX_MISC_BASE_VIRT + 0x0014); > arch/arm/mach-netx/generic.c: vic_init(__io(io_p2v(NETX_PA_VIC)), 0, ~0, 0); > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_SYSTEM_REG(ofs) __io(NETX_VA_SYSTEM + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_GPIO_REG(ofs) __io(NETX_VA_GPIO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PIO_REG(ofs) __io(NETX_VA_PIO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MIIMU __io(NETX_VA_MIIMU) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PFIFO_REG(ofs) __io(NETX_VA_PFIFO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MEMCR_REG(ofs) __io(NETX_VA_MEMCR + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_DPMAS_REG(ofs) __io(NETX_VA_DPMAS + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_I2C_REG(ofs) __io(NETX_VA_I2C, (ofs)) > arch/arm/mach-realview/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > arch/arm/mach-shmobile/board-ag5evm.c: l2x0_init(__io(0xf0100000), 0x00460000, 0xc2000fff); > arch/arm/mach-shmobile/board-bonito.c: l2x0_init(__io(0xf0002000), 0x40440000, 0x82000fff); > arch/arm/mach-shmobile/board-kota2.c: l2x0_init(__io(0xf0100000), 0x40460000, 0x82000fff); > arch/arm/mach-shmobile/include/mach/io.h:#define __io(a) ((void __iomem *)(a)) > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_dist_base = __io(0xf0001000); > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_cpu_base = __io(0xf0000100); > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_dist_base = __io(0xf0001000); > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_cpu_base = __io(0xf0000100); > arch/arm/mach-shmobile/smp-r8a7779.c: __raw_writel(__pa(shmobile_secondary_vector), __io(AVECR)); > arch/arm/mach-shmobile/smp-sh73a0.c: if (((__raw_readl(__io(PSTR)) >> (4 * cpu)) & 3) == 3) > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(WUPCR)); /* wake up */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(SRESCR)); /* reset */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(0, __io(APARMBAREA)); /* 4k */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(__pa(shmobile_secondary_vector), __io(SBAR)); > arch/arm/mach-ux500/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > These are all broken and need to be changed to something else before we add the > global definition for __io. Several platforms define IO_MEM for the purpose of casting the defines to void __iomem *, so I'll make that common unless you have better suggestion. I'd guess that changing the defines themselves will cause lots of warnings in other places. Rob ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 21:34 ` Arnd Bergmann 2012-03-09 21:48 ` Rob Herring @ 2012-03-09 21:52 ` Rafael J. Wysocki 2012-03-09 22:03 ` Arnd Bergmann 2012-03-12 1:13 ` Paul Mundt 1 sibling, 2 replies; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-09 21:52 UTC (permalink / raw) To: Arnd Bergmann, Paul Mundt Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring Hi, On Friday, March 09, 2012, Arnd Bergmann wrote: > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > Hi, > > > > Please pull Renesas SoC updates for v3.4 since commit > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > Hi Rafael, > > Please rebase this on an -rc release, otherwise we get a rather > messy history in the arm-soc tree. Well, OK, I can rebase it on -rc7 if there is one, unless Paul has already pulled from clk_ops-rename. Paul? > > ARM: mach-shmobile: default to no earlytimer > > > > from the git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc > > > > They include: > > > > * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid > > possible future name space collision with common struct clk code. > > > > This also affects drivers that are shared with the sh architecture, > > so the branch containing this part of the material, clk_ops-rename, > > will be merged into the Paul Mundt's sh tree if necessary. > > > > * Introduction of L2 Cache support for r8a7779. > > > > * Conversion of the mach-shmobile subarch to properly use a per-SoC > > map_io and separate init_early callback for early serial console > > support on platforms where that is possible. > > > > Magnus Damm is the author of all the changes. > > For next time, I think it would be good to make those things separate pull > requests, since they are all rather big. In particular, the clk rework > could be a series by itself. It is a series by itself. You can readily pull it from clk_ops-rename even. :-) I'm not sure, though, what exactly the point of this would be. > One thing I noticed while browsing through the patches was this hunk > that Rob should know about: > > @@ -242,6 +243,10 @@ static struct platform_device *r8a7779_late_devices[] __initdata = { > > void __init r8a7779_add_standard_devices(void) > { > +#ifdef CONFIG_CACHE_L2X0 > + /* Early BRESP enable, Shared attribute override enable, 64K*16way */ > + l2x0_init(__io(0xf0100000), 0x40470000, 0x82000fff); > +#endif > r8a7779_pm_init(); > > r8a7779_init_pm_domain(&r8a7779_sh4a); > > Here you are abusing the __io() macro rather badly as a typecast. This breaks > with Rob's series to turn __io(foo) into a NULL pointer for the case where > PCI is disabled. I found a few more of these, mostly in mach-shmobile but > also some others: > > arnd@klappe2:~/linux-arm$ git grep -w __io arch/arm/mach-* | grep -v __typesafe > arch/arm/mach-cns3xxx/core.c: gic_init(0, 29, __io(CNS3XXX_TC11MP_GIC_DIST_BASE_VIRT), > arch/arm/mach-cns3xxx/core.c: __io(CNS3XXX_TC11MP_GIC_CPU_BASE_VIRT)); > arch/arm/mach-cns3xxx/core.c: u32 __iomem *pm_base = __io(CNS3XXX_PM_BASE_VIRT); > arch/arm/mach-cns3xxx/core.c: cns3xxx_tmr1 = __io(CNS3XXX_TIMER1_2_3_BASE_VIRT); > arch/arm/mach-cns3xxx/devices.c: u32 __iomem *gpioa = __io(CNS3XXX_MISC_BASE_VIRT + 0x0014); > arch/arm/mach-netx/generic.c: vic_init(__io(io_p2v(NETX_PA_VIC)), 0, ~0, 0); > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_SYSTEM_REG(ofs) __io(NETX_VA_SYSTEM + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_GPIO_REG(ofs) __io(NETX_VA_GPIO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PIO_REG(ofs) __io(NETX_VA_PIO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MIIMU __io(NETX_VA_MIIMU) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PFIFO_REG(ofs) __io(NETX_VA_PFIFO + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MEMCR_REG(ofs) __io(NETX_VA_MEMCR + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_DPMAS_REG(ofs) __io(NETX_VA_DPMAS + (ofs)) > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_I2C_REG(ofs) __io(NETX_VA_I2C, (ofs)) > arch/arm/mach-realview/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > arch/arm/mach-shmobile/board-ag5evm.c: l2x0_init(__io(0xf0100000), 0x00460000, 0xc2000fff); > arch/arm/mach-shmobile/board-bonito.c: l2x0_init(__io(0xf0002000), 0x40440000, 0x82000fff); > arch/arm/mach-shmobile/board-kota2.c: l2x0_init(__io(0xf0100000), 0x40460000, 0x82000fff); > arch/arm/mach-shmobile/include/mach/io.h:#define __io(a) ((void __iomem *)(a)) > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_dist_base = __io(0xf0001000); > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_cpu_base = __io(0xf0000100); > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_dist_base = __io(0xf0001000); > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_cpu_base = __io(0xf0000100); > arch/arm/mach-shmobile/smp-r8a7779.c: __raw_writel(__pa(shmobile_secondary_vector), __io(AVECR)); > arch/arm/mach-shmobile/smp-sh73a0.c: if (((__raw_readl(__io(PSTR)) >> (4 * cpu)) & 3) == 3) > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(WUPCR)); /* wake up */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(SRESCR)); /* reset */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(0, __io(APARMBAREA)); /* 4k */ > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(__pa(shmobile_secondary_vector), __io(SBAR)); > arch/arm/mach-ux500/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > These are all broken and need to be changed to something else before we add the > global definition for __io. While I generally agree with that, I think it's not super-urgent, is it? Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 21:52 ` Rafael J. Wysocki @ 2012-03-09 22:03 ` Arnd Bergmann 2012-03-09 22:27 ` Rafael J. Wysocki 2012-03-12 1:13 ` Paul Mundt 1 sibling, 1 reply; 21+ messages in thread From: Arnd Bergmann @ 2012-03-09 22:03 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Paul Mundt, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Friday 09 March 2012, Rafael J. Wysocki wrote: > Hi, > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > Hi, > > > > > > Please pull Renesas SoC updates for v3.4 since commit > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > Hi Rafael, > > > > Please rebase this on an -rc release, otherwise we get a rather > > messy history in the arm-soc tree. > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > has already pulled from clk_ops-rename. Paul? Why not rebase back to -rc6? > > > > For next time, I think it would be good to make those things separate pull > > requests, since they are all rather big. In particular, the clk rework > > could be a series by itself. > > It is a series by itself. You can readily pull it from clk_ops-rename even. :-) > > I'm not sure, though, what exactly the point of this would be. Mostly it helps make obvious which things are logically connected. Another reason is that when one of the branches has a problem, we can still pull all the other ones. Finally, we sometimes make new topic branches in the arm-soc tree when a lot of maintainers send similar things, e.g. we had a 'next/clk' branch in the past that half of your patches would have applied to (this time we don't have one of those). By having more branches for logically separate things, it allows us to group them in more flexible ways across platforms. It's not very important this time, so I didn't ask you to rebase them just for that. > > arnd@klappe2:~/linux-arm$ git grep -w __io arch/arm/mach-* | grep -v __typesafe > > arch/arm/mach-cns3xxx/core.c: gic_init(0, 29, __io(CNS3XXX_TC11MP_GIC_DIST_BASE_VIRT), > > arch/arm/mach-cns3xxx/core.c: __io(CNS3XXX_TC11MP_GIC_CPU_BASE_VIRT)); > > arch/arm/mach-cns3xxx/core.c: u32 __iomem *pm_base = __io(CNS3XXX_PM_BASE_VIRT); > > arch/arm/mach-cns3xxx/core.c: cns3xxx_tmr1 = __io(CNS3XXX_TIMER1_2_3_BASE_VIRT); > > arch/arm/mach-cns3xxx/devices.c: u32 __iomem *gpioa = __io(CNS3XXX_MISC_BASE_VIRT + 0x0014); > > arch/arm/mach-netx/generic.c: vic_init(__io(io_p2v(NETX_PA_VIC)), 0, ~0, 0); > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_SYSTEM_REG(ofs) __io(NETX_VA_SYSTEM + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_GPIO_REG(ofs) __io(NETX_VA_GPIO + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PIO_REG(ofs) __io(NETX_VA_PIO + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MIIMU __io(NETX_VA_MIIMU) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PFIFO_REG(ofs) __io(NETX_VA_PFIFO + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MEMCR_REG(ofs) __io(NETX_VA_MEMCR + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_DPMAS_REG(ofs) __io(NETX_VA_DPMAS + (ofs)) > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_I2C_REG(ofs) __io(NETX_VA_I2C, (ofs)) > > arch/arm/mach-realview/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > arch/arm/mach-shmobile/board-ag5evm.c: l2x0_init(__io(0xf0100000), 0x00460000, 0xc2000fff); > > arch/arm/mach-shmobile/board-bonito.c: l2x0_init(__io(0xf0002000), 0x40440000, 0x82000fff); > > arch/arm/mach-shmobile/board-kota2.c: l2x0_init(__io(0xf0100000), 0x40460000, 0x82000fff); > > arch/arm/mach-shmobile/include/mach/io.h:#define __io(a) ((void __iomem *)(a)) > > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_dist_base = __io(0xf0001000); > > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_cpu_base = __io(0xf0000100); > > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_dist_base = __io(0xf0001000); > > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_cpu_base = __io(0xf0000100); > > arch/arm/mach-shmobile/smp-r8a7779.c: __raw_writel(__pa(shmobile_secondary_vector), __io(AVECR)); > > arch/arm/mach-shmobile/smp-sh73a0.c: if (((__raw_readl(__io(PSTR)) >> (4 * cpu)) & 3) == 3) > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(WUPCR)); /* wake up */ > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(SRESCR)); /* reset */ > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(0, __io(APARMBAREA)); /* 4k */ > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(__pa(shmobile_secondary_vector), __io(SBAR)); > > arch/arm/mach-ux500/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > > > These are all broken and need to be changed to something else before we add the > > global definition for __io. > > While I generally agree with that, I think it's not super-urgent, is it? No, it's not. I just wanted to let you all know now so we don't forget it. Arnd ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 22:03 ` Arnd Bergmann @ 2012-03-09 22:27 ` Rafael J. Wysocki 2012-03-10 10:53 ` Arnd Bergmann 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-09 22:27 UTC (permalink / raw) To: Arnd Bergmann Cc: Paul Mundt, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring Hi, On Friday, March 09, 2012, Arnd Bergmann wrote: > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > Hi, > > > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > > Hi, > > > > > > > > Please pull Renesas SoC updates for v3.4 since commit > > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > > > Hi Rafael, > > > > > > Please rebase this on an -rc release, otherwise we get a rather > > > messy history in the arm-soc tree. > > > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > > has already pulled from clk_ops-rename. Paul? > > Why not rebase back to -rc6? Well, I can do that too, if I have to, but still I'd like to be sure that Paul hasn't pulled from the clk_ops-rename branch yet. > > > For next time, I think it would be good to make those things separate pull > > > requests, since they are all rather big. In particular, the clk rework > > > could be a series by itself. > > > > It is a series by itself. You can readily pull it from clk_ops-rename even. :-) > > > > I'm not sure, though, what exactly the point of this would be. > > Mostly it helps make obvious which things are logically connected. > Another reason is that when one of the branches has a problem, we > can still pull all the other ones. > Finally, we sometimes make new topic branches in the arm-soc tree > when a lot of maintainers send similar things, e.g. we had a > 'next/clk' branch in the past that half of your patches would > have applied to (this time we don't have one of those). By having > more branches for logically separate things, it allows us to group > them in more flexible ways across platforms. OK > It's not very important this time, so I didn't ask you to rebase > them just for that. OK, thanks! > > > arnd@klappe2:~/linux-arm$ git grep -w __io arch/arm/mach-* | grep -v __typesafe > > > arch/arm/mach-cns3xxx/core.c: gic_init(0, 29, __io(CNS3XXX_TC11MP_GIC_DIST_BASE_VIRT), > > > arch/arm/mach-cns3xxx/core.c: __io(CNS3XXX_TC11MP_GIC_CPU_BASE_VIRT)); > > > arch/arm/mach-cns3xxx/core.c: u32 __iomem *pm_base = __io(CNS3XXX_PM_BASE_VIRT); > > > arch/arm/mach-cns3xxx/core.c: cns3xxx_tmr1 = __io(CNS3XXX_TIMER1_2_3_BASE_VIRT); > > > arch/arm/mach-cns3xxx/devices.c: u32 __iomem *gpioa = __io(CNS3XXX_MISC_BASE_VIRT + 0x0014); > > > arch/arm/mach-netx/generic.c: vic_init(__io(io_p2v(NETX_PA_VIC)), 0, ~0, 0); > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_SYSTEM_REG(ofs) __io(NETX_VA_SYSTEM + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_GPIO_REG(ofs) __io(NETX_VA_GPIO + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PIO_REG(ofs) __io(NETX_VA_PIO + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MIIMU __io(NETX_VA_MIIMU) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_PFIFO_REG(ofs) __io(NETX_VA_PFIFO + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_MEMCR_REG(ofs) __io(NETX_VA_MEMCR + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_DPMAS_REG(ofs) __io(NETX_VA_DPMAS + (ofs)) > > > arch/arm/mach-netx/include/mach/netx-regs.h:#define NETX_I2C_REG(ofs) __io(NETX_VA_I2C, (ofs)) > > > arch/arm/mach-realview/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > > arch/arm/mach-shmobile/board-ag5evm.c: l2x0_init(__io(0xf0100000), 0x00460000, 0xc2000fff); > > > arch/arm/mach-shmobile/board-bonito.c: l2x0_init(__io(0xf0002000), 0x40440000, 0x82000fff); > > > arch/arm/mach-shmobile/board-kota2.c: l2x0_init(__io(0xf0100000), 0x40460000, 0x82000fff); > > > arch/arm/mach-shmobile/include/mach/io.h:#define __io(a) ((void __iomem *)(a)) > > > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_dist_base = __io(0xf0001000); > > > arch/arm/mach-shmobile/intc-r8a7779.c: void __iomem *gic_cpu_base = __io(0xf0000100); > > > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_dist_base = __io(0xf0001000); > > > arch/arm/mach-shmobile/intc-sh73a0.c: void __iomem *gic_cpu_base = __io(0xf0000100); > > > arch/arm/mach-shmobile/smp-r8a7779.c: __raw_writel(__pa(shmobile_secondary_vector), __io(AVECR)); > > > arch/arm/mach-shmobile/smp-sh73a0.c: if (((__raw_readl(__io(PSTR)) >> (4 * cpu)) & 3) == 3) > > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(WUPCR)); /* wake up */ > > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(1 << cpu, __io(SRESCR)); /* reset */ > > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(0, __io(APARMBAREA)); /* 4k */ > > > arch/arm/mach-shmobile/smp-sh73a0.c: __raw_writel(__pa(shmobile_secondary_vector), __io(SBAR)); > > > arch/arm/mach-ux500/include/mach/hardware.h:#define __io_address(n) __io(IO_ADDRESS(n)) > > > > > > These are all broken and need to be changed to something else before we add the > > > global definition for __io. > > > > While I generally agree with that, I think it's not super-urgent, is it? > > No, it's not. I just wanted to let you all know now so we don't forget it. OK Thanks, Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 22:27 ` Rafael J. Wysocki @ 2012-03-10 10:53 ` Arnd Bergmann 2012-03-10 21:26 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Arnd Bergmann @ 2012-03-10 10:53 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Paul Mundt, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Friday 09 March 2012, Rafael J. Wysocki wrote: > On Friday, March 09, 2012, Arnd Bergmann wrote: > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > Hi, > > > > > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > > > Hi, > > > > > > > > > > Please pull Renesas SoC updates for v3.4 since commit > > > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > > > > > Hi Rafael, > > > > > > > > Please rebase this on an -rc release, otherwise we get a rather > > > > messy history in the arm-soc tree. > > > > > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > > > has already pulled from clk_ops-rename. Paul? > > > > Why not rebase back to -rc6? > > Well, I can do that too, if I have to, but still I'd like to be sure that > Paul hasn't pulled from the clk_ops-rename branch yet. The point is that there might not be an -rc7 any more. I'd suggest that you rebase on -rc6 if Paul has not yet pulled them, otherwise we will merge them in the current state and put an explanation into the merge changeset. Arnd ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-10 10:53 ` Arnd Bergmann @ 2012-03-10 21:26 ` Rafael J. Wysocki 2012-03-10 22:06 ` Geert Uytterhoeven 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-10 21:26 UTC (permalink / raw) To: Arnd Bergmann Cc: Paul Mundt, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Saturday, March 10, 2012, Arnd Bergmann wrote: > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > > Hi, > > > > > > > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > > > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > > > > Hi, > > > > > > > > > > > > Please pull Renesas SoC updates for v3.4 since commit > > > > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > > > > > > > Hi Rafael, > > > > > > > > > > Please rebase this on an -rc release, otherwise we get a rather > > > > > messy history in the arm-soc tree. > > > > > > > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > > > > has already pulled from clk_ops-rename. Paul? > > > > > > Why not rebase back to -rc6? > > > > Well, I can do that too, if I have to, but still I'd like to be sure that > > Paul hasn't pulled from the clk_ops-rename branch yet. > > The point is that there might not be an -rc7 any more. Yes, I'm aware of that. :-) > I'd suggest that you rebase on -rc6 if Paul has not yet pulled them, I'm still waiting for a word from Paul. > otherwise we will merge them in the current state and put an explanation > into the merge changeset. Well, if there's no -rc7 (which means that the final v3.3 is released tomorrow) or there's no information from Paul within the next few days, I'm afraid rebasing won't be very safe. Thanks a lot and sorry for the trouble (I think I didn't understand you correctly when you said you wanted things to be based on -rc, I thought you meant the Linus' tree in general and not specific commits). Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-10 21:26 ` Rafael J. Wysocki @ 2012-03-10 22:06 ` Geert Uytterhoeven 0 siblings, 0 replies; 21+ messages in thread From: Geert Uytterhoeven @ 2012-03-10 22:06 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Arnd Bergmann, Paul Mundt, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Sat, Mar 10, 2012 at 22:26, Rafael J. Wysocki <rjw@sisk.pl> wrote: >> The point is that there might not be an -rc7 any more. > > Yes, I'm aware of that. :-) It seems there is. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-09 21:52 ` Rafael J. Wysocki 2012-03-09 22:03 ` Arnd Bergmann @ 2012-03-12 1:13 ` Paul Mundt 2012-03-12 21:30 ` Rafael J. Wysocki 1 sibling, 1 reply; 21+ messages in thread From: Paul Mundt @ 2012-03-12 1:13 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Fri, Mar 09, 2012 at 10:52:47PM +0100, Rafael J. Wysocki wrote: > On Friday, March 09, 2012, Arnd Bergmann wrote: > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > Please pull Renesas SoC updates for v3.4 since commit > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > Hi Rafael, > > > > Please rebase this on an -rc release, otherwise we get a rather > > messy history in the arm-soc tree. > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > has already pulled from clk_ops-rename. Paul? > Nope, I haven't had a chance yet. Feel free to rebase as necessary and I'll grab them afterwards. ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4 2012-03-12 1:13 ` Paul Mundt @ 2012-03-12 21:30 ` Rafael J. Wysocki 0 siblings, 0 replies; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-12 21:30 UTC (permalink / raw) To: Paul Mundt Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Rob Herring On Monday, March 12, 2012, Paul Mundt wrote: > On Fri, Mar 09, 2012 at 10:52:47PM +0100, Rafael J. Wysocki wrote: > > On Friday, March 09, 2012, Arnd Bergmann wrote: > > > On Friday 09 March 2012, Rafael J. Wysocki wrote: > > > > Please pull Renesas SoC updates for v3.4 since commit > > > > ce8fea7aa4ad9e3b40999a08622ef27c77159659: > > > > > > > > mmap: EINVAL not ENOMEM when rejecting VM_GROWS > > > > > > > > with top-most commit f57fd2100e8273af3a9d2ff67714903d2dfd1eef > > > > > > Hi Rafael, > > > > > > Please rebase this on an -rc release, otherwise we get a rather > > > messy history in the arm-soc tree. > > > > Well, OK, I can rebase it on -rc7 if there is one, unless Paul > > has already pulled from clk_ops-rename. Paul? > > > Nope, I haven't had a chance yet. Feel free to rebase as necessary and > I'll grab them afterwards. Thanks, I've just rebased clk_ops-rename on top of -rc7. Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-09 20:55 [GIT PULL] Renesas SoC updates for v3.4 Rafael J. Wysocki 2012-03-09 21:34 ` Arnd Bergmann @ 2012-03-12 22:53 ` Rafael J. Wysocki 2012-03-13 10:26 ` Arnd Bergmann 1 sibling, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-12 22:53 UTC (permalink / raw) To: Arnd Bergmann Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt Hi, Please pull Renesas SoC updates for v3.4 since commit fde7d9049e55ab85a390be7f415d74c9f62dd0f9 Linux 3.3-rc7 with top-most commit 2854903ad1329d09d7ec35639fff0949e45d496d ARM: mach-shmobile: default to no earlytimer from the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc They include: * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid possible future name space collision with common struct clk code. This also affects drivers that are shared with the sh architecture, so the branch containing this part of the material, clk_ops-rename, will be merged into the Paul Mundt's sh tree if necessary. * Introduction of L2 Cache support for r8a7779. * Conversion of the mach-shmobile subarch to properly use a per-SoC map_io and separate init_early callback for early serial console support on platforms where that is possible. Magnus Damm is the author of all the changes. arch/arm/mach-shmobile/board-ag5evm.c | 39 +--------------- arch/arm/mach-shmobile/board-ap4evb.c | 44 +++---------------- arch/arm/mach-shmobile/board-bonito.c | 44 +++++------------- arch/arm/mach-shmobile/board-g3evm.c | 38 +-------------- arch/arm/mach-shmobile/board-g4evm.c | 38 +-------------- arch/arm/mach-shmobile/board-kota2.c | 38 +-------------- arch/arm/mach-shmobile/board-mackerel.c | 44 +++---------------- arch/arm/mach-shmobile/board-marzen.c | 62 +------------------------ arch/arm/mach-shmobile/clock-r8a7740.c | 8 ++-- arch/arm/mach-shmobile/clock-r8a7779.c | 4 +- arch/arm/mach-shmobile/clock-sh7367.c | 8 ++-- arch/arm/mach-shmobile/clock-sh7372.c | 10 ++-- arch/arm/mach-shmobile/clock-sh7377.c | 8 ++-- arch/arm/mach-shmobile/clock-sh73a0.c | 14 +++--- arch/arm/mach-shmobile/clock.c | 2 +- arch/arm/mach-shmobile/include/mach/common.h | 9 +++- arch/arm/mach-shmobile/setup-r8a7740.c | 45 +++++++++++++++++++ arch/arm/mach-shmobile/setup-r8a7779.c | 54 ++++++++++++++++++++++ arch/arm/mach-shmobile/setup-sh7367.c | 32 +++++++++++++ arch/arm/mach-shmobile/setup-sh7372.c | 32 +++++++++++++ arch/arm/mach-shmobile/setup-sh7377.c | 32 +++++++++++++ arch/arm/mach-shmobile/setup-sh73a0.c | 32 +++++++++++++ arch/arm/mach-shmobile/timer.c | 6 ++- arch/sh/boards/mach-highlander/setup.c | 2 +- arch/sh/boards/mach-sdk7786/setup.c | 2 +- arch/sh/include/asm/clock.h | 2 +- arch/sh/kernel/cpu/sh2/clock-sh7619.c | 12 +++--- arch/sh/kernel/cpu/sh2a/clock-sh7201.c | 12 +++--- arch/sh/kernel/cpu/sh2a/clock-sh7203.c | 12 +++--- arch/sh/kernel/cpu/sh2a/clock-sh7206.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh3.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh7705.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh7706.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh7709.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh7710.c | 12 +++--- arch/sh/kernel/cpu/sh3/clock-sh7712.c | 10 ++-- arch/sh/kernel/cpu/sh4/clock-sh4-202.c | 6 +- arch/sh/kernel/cpu/sh4/clock-sh4.c | 12 +++--- arch/sh/kernel/cpu/sh4a/clock-sh7343.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7366.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7722.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7723.c | 4 +- arch/sh/kernel/cpu/sh4a/clock-sh7724.c | 6 +- arch/sh/kernel/cpu/sh4a/clock-sh7757.c | 2 +- arch/sh/kernel/cpu/sh4a/clock-sh7763.c | 14 +++--- arch/sh/kernel/cpu/sh4a/clock-sh7770.c | 12 +++--- arch/sh/kernel/cpu/sh4a/clock-sh7780.c | 14 +++--- arch/sh/kernel/cpu/sh4a/clock-sh7785.c | 2 +- arch/sh/kernel/cpu/sh4a/clock-sh7786.c | 2 +- arch/sh/kernel/cpu/sh4a/clock-shx3.c | 2 +- arch/sh/kernel/cpu/sh5/clock-sh5.c | 12 +++--- drivers/sh/clk/cpg.c | 16 +++--- include/linux/sh_clk.h | 5 +- 53 files changed, 429 insertions(+), 459 deletions(-) --------------- Magnus Damm (35): sh: introduce sh_clk_ops in parallel with clk_ops sh: convert cpg code to sh_clk_ops ARM: mach-shmobile: sh7367 sh_clk_ops rename ARM: mach-shmobile: sh7377 sh_clk_ops rename ARM: mach-shmobile: sh7372 sh_clk_ops rename ARM: mach-shmobile: sh73a0 sh_clk_ops rename ARM: mach-shmobile: r8a7740 sh_clk_ops rename ARM: mach-shmobile: r8a7779 sh_clk_ops rename sh: sh header sh_clk_ops rename sh: sh2 sh_clk_ops rename sh: sh2a sh_clk_ops rename sh: sh3 sh_clk_ops rename sh: sh4 sh_clk_ops rename sh: sh4a sh_clk_ops rename sh: sh5 sh_clk_ops rename sh: board sh_clk_ops rename sh: remove clk_ops ARM: mach-shmobile: sh7367 map_io and init_early update ARM: mach-shmobile: sh7377 map_io and init_early update ARM: mach-shmobile: sh7372 map_io and init_early update ARM: mach-shmobile: sh73a0 map_io and init_early update ARM: mach-shmobile: r8a7740 map_io and init_early update ARM: mach-shmobile: r8a7779 map_io and init_early update ARM: mach-shmobile: r8a7779 L2 cache support ARM: mach-shmobile: rename clk_init() to shmobile_clk_init() ARM: mach-shmobile: Move sh7372 Mackerel external clk setup ARM: mach-shmobile: Move sh7372 AP4EVB external clk setup ARM: mach-shmobile: add shmobile_earlytimer_init() ARM: mach-shmobile: sh7367 and G3EVM timer rework ARM: mach-shmobile: sh7377 and G4EVM timer rework ARM: mach-shmobile: sh7372, AP4EVB and Mackerel timer rework ARM: mach-shmobile: sh73a0, AG5EVM and Kota2 timer rework ARM: mach-shmobile: r8a7740 and Bonito timer rework ARM: mach-shmobile: r8a7779 and Marzen timer rework ARM: mach-shmobile: default to no earlytimer ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-12 22:53 ` [GIT PULL] Renesas SoC updates for v3.4, take 2 Rafael J. Wysocki @ 2012-03-13 10:26 ` Arnd Bergmann 2012-03-13 19:36 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Arnd Bergmann @ 2012-03-13 10:26 UTC (permalink / raw) To: Rafael J. Wysocki Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Monday 12 March 2012, Rafael J. Wysocki wrote: > Please pull Renesas SoC updates for v3.4 since commit > fde7d9049e55ab85a390be7f415d74c9f62dd0f9 > > Linux 3.3-rc7 > > with top-most commit 2854903ad1329d09d7ec35639fff0949e45d496d > > ARM: mach-shmobile: default to no earlytimer > > from the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc > > They include: > > * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid > possible future name space collision with common struct clk code. > > This also affects drivers that are shared with the sh architecture, > so the branch containing this part of the material, clk_ops-rename, > will be merged into the Paul Mundt's sh tree if necessary. > > * Introduction of L2 Cache support for r8a7779. > > * Conversion of the mach-shmobile subarch to properly use a per-SoC > map_io and separate init_early callback for early serial console > support on platforms where that is possible. > > Magnus Damm is the author of all the changes. > Thanks for rebasing this, Olof will merge this soon. Note that the __io() issue has turned out to be more urgent than I first thought when we discussed it, so it would be good to apply the patch below on top of your series. Arnd 8<----- ARM: shmobile: remove additional __io() macro use setup-r8a7779.c has grown a new user of the __io() macro. Rob Herring's PIO cleanup series already gets rid of all other uses in shmobile, so we should ensure that this one gets removed as well. Signed-off-by: Arnd Bergmann <arnd@arndb.de> diff --git a/arch/arm/mach-shmobile/setup-r8a7779.c b/arch/arm/mach-shmobile/setup-r8a7779.c index ce57d90..9545d82 100644 --- a/arch/arm/mach-shmobile/setup-r8a7779.c +++ b/arch/arm/mach-shmobile/setup-r8a7779.c @@ -246,7 +246,7 @@ void __init r8a7779_add_standard_devices(void) { #ifdef CONFIG_CACHE_L2X0 /* Early BRESP enable, Shared attribute override enable, 64K*16way */ - l2x0_init(__io(0xf0100000), 0x40470000, 0x82000fff); + l2x0_init((void __iomem __force *)(0xf0100000), 0x40470000, 0x82000fff); #endif r8a7779_pm_init(); ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-13 10:26 ` Arnd Bergmann @ 2012-03-13 19:36 ` Rafael J. Wysocki 2012-03-13 19:39 ` Arnd Bergmann 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-13 19:36 UTC (permalink / raw) To: Arnd Bergmann Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tuesday, March 13, 2012, Arnd Bergmann wrote: > On Monday 12 March 2012, Rafael J. Wysocki wrote: > > Please pull Renesas SoC updates for v3.4 since commit > > fde7d9049e55ab85a390be7f415d74c9f62dd0f9 > > > > Linux 3.3-rc7 > > > > with top-most commit 2854903ad1329d09d7ec35639fff0949e45d496d > > > > ARM: mach-shmobile: default to no earlytimer > > > > from the git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/rafael/renesas.git soc > > > > They include: > > > > * The rename of shmobile struct clk_ops to struct sh_clk_ops to avoid > > possible future name space collision with common struct clk code. > > > > This also affects drivers that are shared with the sh architecture, > > so the branch containing this part of the material, clk_ops-rename, > > will be merged into the Paul Mundt's sh tree if necessary. > > > > * Introduction of L2 Cache support for r8a7779. > > > > * Conversion of the mach-shmobile subarch to properly use a per-SoC > > map_io and separate init_early callback for early serial console > > support on platforms where that is possible. > > > > Magnus Damm is the author of all the changes. > > > > Thanks for rebasing this, Olof will merge this soon. Thanks! > Note that the __io() issue has turned out to be more urgent than I first > thought when we discussed it, so it would be good to apply the patch below > on top of your series. Do you want me to apply it now or will you do that in your tree? Rafael > 8<----- > ARM: shmobile: remove additional __io() macro use > > setup-r8a7779.c has grown a new user of the __io() macro. Rob Herring's > PIO cleanup series already gets rid of all other uses in shmobile, so > we should ensure that this one gets removed as well. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > > diff --git a/arch/arm/mach-shmobile/setup-r8a7779.c b/arch/arm/mach-shmobile/setup-r8a7779.c > index ce57d90..9545d82 100644 > --- a/arch/arm/mach-shmobile/setup-r8a7779.c > +++ b/arch/arm/mach-shmobile/setup-r8a7779.c > @@ -246,7 +246,7 @@ void __init r8a7779_add_standard_devices(void) > { > #ifdef CONFIG_CACHE_L2X0 > /* Early BRESP enable, Shared attribute override enable, 64K*16way */ > - l2x0_init(__io(0xf0100000), 0x40470000, 0x82000fff); > + l2x0_init((void __iomem __force *)(0xf0100000), 0x40470000, 0x82000fff); > #endif > r8a7779_pm_init(); > > > ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-13 19:36 ` Rafael J. Wysocki @ 2012-03-13 19:39 ` Arnd Bergmann 2012-03-13 20:17 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Arnd Bergmann @ 2012-03-13 19:39 UTC (permalink / raw) To: Rafael J. Wysocki Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tuesday 13 March 2012, Rafael J. Wysocki wrote: > > Note that the __io() issue has turned out to be more urgent than I first > > thought when we discussed it, so it would be good to apply the patch below > > on top of your series. > > Do you want me to apply it now or will you do that in your tree? Doesn't matter to me, as long as one of you applies it ;-) Arnd ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-13 19:39 ` Arnd Bergmann @ 2012-03-13 20:17 ` Rafael J. Wysocki 2012-03-13 21:42 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-13 20:17 UTC (permalink / raw) To: Arnd Bergmann Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tuesday, March 13, 2012, Arnd Bergmann wrote: > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: > > > Note that the __io() issue has turned out to be more urgent than I first > > > thought when we discussed it, so it would be good to apply the patch below > > > on top of your series. > > > > Do you want me to apply it now or will you do that in your tree? > > Doesn't matter to me, as long as one of you applies it ;-) OK, so I will add it to the soc branch in the Renesas tree. Thanks, Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-13 20:17 ` Rafael J. Wysocki @ 2012-03-13 21:42 ` Rafael J. Wysocki 2012-03-14 0:02 ` Olof Johansson 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-13 21:42 UTC (permalink / raw) To: Arnd Bergmann Cc: LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tuesday, March 13, 2012, Rafael J. Wysocki wrote: > On Tuesday, March 13, 2012, Arnd Bergmann wrote: > > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: > > > > Note that the __io() issue has turned out to be more urgent than I first > > > > thought when we discussed it, so it would be good to apply the patch below > > > > on top of your series. > > > > > > Do you want me to apply it now or will you do that in your tree? > > > > Doesn't matter to me, as long as one of you applies it ;-) > > OK, so I will add it to the soc branch in the Renesas tree. Applied and pushed back the updated tree. Thanks, Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-13 21:42 ` Rafael J. Wysocki @ 2012-03-14 0:02 ` Olof Johansson 2012-03-14 0:16 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Olof Johansson @ 2012-03-14 0:02 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tue, Mar 13, 2012 at 2:42 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Tuesday, March 13, 2012, Rafael J. Wysocki wrote: >> On Tuesday, March 13, 2012, Arnd Bergmann wrote: >> > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: >> > > > Note that the __io() issue has turned out to be more urgent than I first >> > > > thought when we discussed it, so it would be good to apply the patch below >> > > > on top of your series. >> > > >> > > Do you want me to apply it now or will you do that in your tree? >> > >> > Doesn't matter to me, as long as one of you applies it ;-) >> >> OK, so I will add it to the soc branch in the Renesas tree. > > Applied and pushed back the updated tree. Seems like you applied the patch but without the subject line, so now git thinks the whole first paragraph is the patch subject. That's a bit messy and I'd prefer if you could fix it up. Do you have downstream users of your tree that you'll mess up if you rebase the last commit though? Alternatively I can take everything on the branch but the last patch and apply it myself, but that will still cause trouble for your downstream developers once things go upstream and come down. -Olof ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-14 0:02 ` Olof Johansson @ 2012-03-14 0:16 ` Rafael J. Wysocki 2012-03-14 0:40 ` Olof Johansson 0 siblings, 1 reply; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-14 0:16 UTC (permalink / raw) To: Olof Johansson Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Wednesday, March 14, 2012, Olof Johansson wrote: > On Tue, Mar 13, 2012 at 2:42 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > > On Tuesday, March 13, 2012, Rafael J. Wysocki wrote: > >> On Tuesday, March 13, 2012, Arnd Bergmann wrote: > >> > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: > >> > > > Note that the __io() issue has turned out to be more urgent than I first > >> > > > thought when we discussed it, so it would be good to apply the patch below > >> > > > on top of your series. > >> > > > >> > > Do you want me to apply it now or will you do that in your tree? > >> > > >> > Doesn't matter to me, as long as one of you applies it ;-) > >> > >> OK, so I will add it to the soc branch in the Renesas tree. > > > > Applied and pushed back the updated tree. > > Seems like you applied the patch but without the subject line, I didn't notice that, sorry. > so now git thinks the whole first paragraph is the patch subject. That's a > bit messy and I'd prefer if you could fix it up. I've just replaced the broken commit with a new fixed one. > Do you have downstream users of your tree that you'll mess up if you > rebase the last commit though? Alternatively I can take everything on > the branch but the last patch and apply it myself, but that will still > cause trouble for your downstream developers once things go upstream > and come down. Hopefully, no one has pulled from the tree yet. :-) Thanks, Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-14 0:16 ` Rafael J. Wysocki @ 2012-03-14 0:40 ` Olof Johansson 2012-03-14 9:03 ` Rafael J. Wysocki 0 siblings, 1 reply; 21+ messages in thread From: Olof Johansson @ 2012-03-14 0:40 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Tue, Mar 13, 2012 at 5:16 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Wednesday, March 14, 2012, Olof Johansson wrote: >> On Tue, Mar 13, 2012 at 2:42 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: >> > On Tuesday, March 13, 2012, Rafael J. Wysocki wrote: >> >> On Tuesday, March 13, 2012, Arnd Bergmann wrote: >> >> > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: >> >> > > > Note that the __io() issue has turned out to be more urgent than I first >> >> > > > thought when we discussed it, so it would be good to apply the patch below >> >> > > > on top of your series. >> >> > > >> >> > > Do you want me to apply it now or will you do that in your tree? >> >> > >> >> > Doesn't matter to me, as long as one of you applies it ;-) >> >> >> >> OK, so I will add it to the soc branch in the Renesas tree. >> > >> > Applied and pushed back the updated tree. >> >> Seems like you applied the patch but without the subject line, > > I didn't notice that, sorry. > >> so now git thinks the whole first paragraph is the patch subject. That's a >> bit messy and I'd prefer if you could fix it up. > > I've just replaced the broken commit with a new fixed one. > >> Do you have downstream users of your tree that you'll mess up if you >> rebase the last commit though? Alternatively I can take everything on >> the branch but the last patch and apply it myself, but that will still >> cause trouble for your downstream developers once things go upstream >> and come down. > > Hopefully, no one has pulled from the tree yet. :-) Cool. Pulled into next/soc. Thanks!! -Olof ^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [GIT PULL] Renesas SoC updates for v3.4, take 2 2012-03-14 0:40 ` Olof Johansson @ 2012-03-14 9:03 ` Rafael J. Wysocki 0 siblings, 0 replies; 21+ messages in thread From: Rafael J. Wysocki @ 2012-03-14 9:03 UTC (permalink / raw) To: Olof Johansson Cc: Arnd Bergmann, LKML, Magnus Damm, Linux-sh list, arm@kernel.org, Paul Mundt On Wednesday, March 14, 2012, Olof Johansson wrote: > On Tue, Mar 13, 2012 at 5:16 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > > On Wednesday, March 14, 2012, Olof Johansson wrote: > >> On Tue, Mar 13, 2012 at 2:42 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > >> > On Tuesday, March 13, 2012, Rafael J. Wysocki wrote: > >> >> On Tuesday, March 13, 2012, Arnd Bergmann wrote: > >> >> > On Tuesday 13 March 2012, Rafael J. Wysocki wrote: > >> >> > > > Note that the __io() issue has turned out to be more urgent than I first > >> >> > > > thought when we discussed it, so it would be good to apply the patch below > >> >> > > > on top of your series. > >> >> > > > >> >> > > Do you want me to apply it now or will you do that in your tree? > >> >> > > >> >> > Doesn't matter to me, as long as one of you applies it ;-) > >> >> > >> >> OK, so I will add it to the soc branch in the Renesas tree. > >> > > >> > Applied and pushed back the updated tree. > >> > >> Seems like you applied the patch but without the subject line, > > > > I didn't notice that, sorry. > > > >> so now git thinks the whole first paragraph is the patch subject. That's a > >> bit messy and I'd prefer if you could fix it up. > > > > I've just replaced the broken commit with a new fixed one. > > > >> Do you have downstream users of your tree that you'll mess up if you > >> rebase the last commit though? Alternatively I can take everything on > >> the branch but the last patch and apply it myself, but that will still > >> cause trouble for your downstream developers once things go upstream > >> and come down. > > > > Hopefully, no one has pulled from the tree yet. :-) > > Cool. Pulled into next/soc. Thanks!! Great, thanks a lot! Rafael ^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2012-03-14 8:59 UTC | newest] Thread overview: 21+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-03-09 20:55 [GIT PULL] Renesas SoC updates for v3.4 Rafael J. Wysocki 2012-03-09 21:34 ` Arnd Bergmann 2012-03-09 21:48 ` Rob Herring 2012-03-09 21:52 ` Rafael J. Wysocki 2012-03-09 22:03 ` Arnd Bergmann 2012-03-09 22:27 ` Rafael J. Wysocki 2012-03-10 10:53 ` Arnd Bergmann 2012-03-10 21:26 ` Rafael J. Wysocki 2012-03-10 22:06 ` Geert Uytterhoeven 2012-03-12 1:13 ` Paul Mundt 2012-03-12 21:30 ` Rafael J. Wysocki 2012-03-12 22:53 ` [GIT PULL] Renesas SoC updates for v3.4, take 2 Rafael J. Wysocki 2012-03-13 10:26 ` Arnd Bergmann 2012-03-13 19:36 ` Rafael J. Wysocki 2012-03-13 19:39 ` Arnd Bergmann 2012-03-13 20:17 ` Rafael J. Wysocki 2012-03-13 21:42 ` Rafael J. Wysocki 2012-03-14 0:02 ` Olof Johansson 2012-03-14 0:16 ` Rafael J. Wysocki 2012-03-14 0:40 ` Olof Johansson 2012-03-14 9:03 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox