* Re: [PATCH 2/3] fsl_rio: fix non-standard HID1 register access
From: Kumar Gala @ 2010-10-14 6:14 UTC (permalink / raw)
To: Shaohui Xie; +Cc: alexandre.bounine, linuxppc-dev
In-Reply-To: <1287021866-22095-1-git-send-email-b21989@freescale.com>
On Oct 13, 2010, at 9:04 PM, Shaohui Xie wrote:
> From: Li Yang <leoli@freescale.com>
>=20
> The access to HID1 register is only legitimate for e500 v1/v2 cores.
> Also fixes magic number.
>=20
> Signed-off-by: Li Yang <leoli@freescale.com>
> Signed-off-by: Shaohui Xie <b21989@freescale.com>
> ---
> arch/powerpc/sysdev/fsl_rio.c | 9 ++++++---
> 1 files changed, 6 insertions(+), 3 deletions(-)
>=20
> diff --git a/arch/powerpc/sysdev/fsl_rio.c =
b/arch/powerpc/sysdev/fsl_rio.c
> index 4127636..dfff3b7 100644
> --- a/arch/powerpc/sysdev/fsl_rio.c
> +++ b/arch/powerpc/sysdev/fsl_rio.c
> @@ -1537,9 +1537,12 @@ int fsl_rio_setup(struct platform_device *dev)
> #ifdef CONFIG_E500
> saved_mcheck_exception =3D ppc_md.machine_check_exception;
> ppc_md.machine_check_exception =3D fsl_rio_mcheck_exception;
> -#endif
> - /* Ensure that RFXE is set */
> - mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | 0x20000));
> +
> +#ifndef CONFIG_PPC_E500MC
> + /* Ensure that RFXE is set on e500 v1/v2 */
> + mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | HID1_RFXE));
> +#endif /* !PPC_E500MC */
> +#endif /* E500 */
I've never really been happy with this code. We really should set =
HID1_RFXE in cpu_setup_fsl_booke.S instead.
- k=
^ permalink raw reply
* Re: [PATCH 2/3] fsl_rio: fix non-standard HID1 register access
From: Kumar Gala @ 2010-10-14 6:17 UTC (permalink / raw)
To: Kumar Gala; +Cc: alexandre.bounine, linuxppc-dev, Shaohui Xie
In-Reply-To: <7F0756F8-8297-43A6-913A-6DA2634E18E5@kernel.crashing.org>
On Oct 14, 2010, at 1:14 AM, Kumar Gala wrote:
>=20
> On Oct 13, 2010, at 9:04 PM, Shaohui Xie wrote:
>=20
>> From: Li Yang <leoli@freescale.com>
>>=20
>> The access to HID1 register is only legitimate for e500 v1/v2 cores.
>> Also fixes magic number.
>>=20
>> Signed-off-by: Li Yang <leoli@freescale.com>
>> Signed-off-by: Shaohui Xie <b21989@freescale.com>
>> ---
>> arch/powerpc/sysdev/fsl_rio.c | 9 ++++++---
>> 1 files changed, 6 insertions(+), 3 deletions(-)
>>=20
>> diff --git a/arch/powerpc/sysdev/fsl_rio.c =
b/arch/powerpc/sysdev/fsl_rio.c
>> index 4127636..dfff3b7 100644
>> --- a/arch/powerpc/sysdev/fsl_rio.c
>> +++ b/arch/powerpc/sysdev/fsl_rio.c
>> @@ -1537,9 +1537,12 @@ int fsl_rio_setup(struct platform_device *dev)
>> #ifdef CONFIG_E500
>> saved_mcheck_exception =3D ppc_md.machine_check_exception;
>> ppc_md.machine_check_exception =3D fsl_rio_mcheck_exception;
>> -#endif
>> - /* Ensure that RFXE is set */
>> - mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | 0x20000));
>> +
>> +#ifndef CONFIG_PPC_E500MC
>> + /* Ensure that RFXE is set on e500 v1/v2 */
>> + mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | HID1_RFXE));
>> +#endif /* !PPC_E500MC */
>> +#endif /* E500 */
>=20
> I've never really been happy with this code. We really should set =
HID1_RFXE in cpu_setup_fsl_booke.S instead.
We should also change this so we just call fsl_rio_mcheck_exception() =
from machine_check_e500 & machine_check_e500mc and get rid of this =
ppc_md.machine_check_exception manipulation.
- k=
^ permalink raw reply
* Re: [git pull] Please pull powerpc.git next branch (updated)
From: Kumar Gala @ 2010-10-14 6:19 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linuxppc-dev
In-Reply-To: <1287018131.2205.50.camel@pasglop>
[ should fix the compile issue and pulled in 2 other minor patches ]
The following changes since commit 4108d9ba9091c55cfb968d42dd7dcae9a098b876:
powerpc/Makefiles: Change to new flag variables (2010-10-13 16:19:22 +1100)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/galak/powerpc.git next
Harninder Rai (1):
powerpc/85xx: add cache-sram support
Ilya Yanok (1):
powerpc/mpc83xx: Support for MPC8308 P1M board
Kumar Gala (10):
powerpc/fsl-pci: Fix MSI support on 83xx platforms
powerpc/mpc8xxx_gpio: Add support for 'qoriq-gpio' controllers
powerpc/fsl-booke: Add PCI device ids for P2040/P3041/P5010/P5020 QoirQ chips
powerpc/fsl-booke: Add p3041 DS board support
powerpc: Fix compile error with paca code on ppc64e
powerpc/fsl-booke: Add support for FSL 64-bit e5500 core
powerpc/fsl-booke: Add support for FSL Arch v1.0 MMU in setup_page_sizes
powerpc/fsl-booke64: Use TLB CAMs to cover linear mapping on FSL 64-bit chips
powerpc/fsl-booke: Add p5020 DS board support
powerpc/fsl-booke: Add e55xx (64-bit) smp defconfig
Matthew McClintock (6):
powerpc/kexec: make masking/disabling interrupts generic
powerpc/85xx: Remove call to mpic_teardown_this_cpu in kexec
powerpc/85xx: Minor fixups for kexec on 85xx
powerpc/85xx: flush dcache before resetting cores
powerpc/fsl_soc: Search all global-utilities nodes for rstccr
powerpc/fsl_booke: Add support to boot from core other than 0
Paul Gortmaker (1):
powerpc: Fix invalid page flags in create TLB CAM path for PTE_64BIT
Scott Wood (1):
oprofile/fsl emb: Don't set MSR[PMM] until after clearing the interrupt.
Shaohui Xie (1):
fsl_rio: Add comments for sRIO registers.
Timur Tabi (5):
powerpc: export ppc_proc_freq and ppc_tb_freq as GPL symbols
powerpc/watchdog: Allow the Book-E driver to be compiled as a module
powerpc/p1022: Add probing for individual DMA channels
powerpc/85xx: add ngPIXIS FPGA device tree node to the P1022DS board
powerpc/watchdog: Make default timeout for Book-E watchdog a Kconfig option
arch/powerpc/boot/dts/mpc8308_p1m.dts | 332 ++++++++++++++++++++++++
arch/powerpc/boot/dts/p1022ds.dts | 11 +
arch/powerpc/configs/e55xx_smp_defconfig | 84 ++++++
arch/powerpc/configs/ppc64e_defconfig | 4 +-
arch/powerpc/include/asm/fsl_85xx_cache_sram.h | 48 ++++
arch/powerpc/include/asm/kexec.h | 1 +
arch/powerpc/include/asm/mmu-book3e.h | 15 +
arch/powerpc/include/asm/pte-common.h | 7 +
arch/powerpc/kernel/Makefile | 4 +-
arch/powerpc/kernel/asm-offsets.c | 4 +-
arch/powerpc/kernel/cpu_setup_fsl_booke.S | 15 +
arch/powerpc/kernel/cputable.c | 28 ++-
arch/powerpc/kernel/crash.c | 13 +-
arch/powerpc/kernel/head_fsl_booke.S | 10 +-
arch/powerpc/kernel/machine_kexec.c | 24 ++
arch/powerpc/kernel/machine_kexec_32.c | 4 +
arch/powerpc/kernel/paca.c | 2 +-
arch/powerpc/kernel/setup_32.c | 2 +-
arch/powerpc/kernel/time.c | 3 +-
arch/powerpc/kernel/traps.c | 5 +
arch/powerpc/mm/Makefile | 2 +-
arch/powerpc/mm/fsl_booke_mmu.c | 15 +-
arch/powerpc/mm/mmu_decl.h | 5 +-
arch/powerpc/mm/tlb_nohash.c | 56 ++++-
arch/powerpc/mm/tlb_nohash_low.S | 2 +-
arch/powerpc/oprofile/op_model_fsl_emb.c | 15 +-
arch/powerpc/platforms/83xx/Kconfig | 4 +-
arch/powerpc/platforms/83xx/mpc830x_rdb.c | 3 +-
arch/powerpc/platforms/85xx/Kconfig | 28 ++-
arch/powerpc/platforms/85xx/Makefile | 2 +
arch/powerpc/platforms/85xx/p1022_ds.c | 2 +
arch/powerpc/platforms/85xx/p3041_ds.c | 64 +++++
arch/powerpc/platforms/85xx/p5020_ds.c | 69 +++++
arch/powerpc/platforms/85xx/smp.c | 76 +++++-
arch/powerpc/platforms/Kconfig.cputype | 8 +-
arch/powerpc/sysdev/Makefile | 1 +
arch/powerpc/sysdev/fsl_85xx_cache_ctlr.h | 101 +++++++
arch/powerpc/sysdev/fsl_85xx_cache_sram.c | 159 +++++++++++
arch/powerpc/sysdev/fsl_85xx_l2ctlr.c | 231 ++++++++++++++++
arch/powerpc/sysdev/fsl_msi.c | 9 +-
arch/powerpc/sysdev/fsl_pci.c | 60 ++++-
arch/powerpc/sysdev/fsl_pci.h | 1 +
arch/powerpc/sysdev/fsl_rio.c | 65 +++--
arch/powerpc/sysdev/fsl_soc.c | 20 +-
arch/powerpc/sysdev/mpc8xxx_gpio.c | 3 +
drivers/watchdog/Kconfig | 22 ++-
drivers/watchdog/booke_wdt.c | 47 +++-
include/linux/pci_ids.h | 8 +
48 files changed, 1582 insertions(+), 112 deletions(-)
create mode 100644 arch/powerpc/boot/dts/mpc8308_p1m.dts
create mode 100644 arch/powerpc/configs/e55xx_smp_defconfig
create mode 100644 arch/powerpc/include/asm/fsl_85xx_cache_sram.h
create mode 100644 arch/powerpc/platforms/85xx/p3041_ds.c
create mode 100644 arch/powerpc/platforms/85xx/p5020_ds.c
create mode 100644 arch/powerpc/sysdev/fsl_85xx_cache_ctlr.h
create mode 100644 arch/powerpc/sysdev/fsl_85xx_cache_sram.c
create mode 100644 arch/powerpc/sysdev/fsl_85xx_l2ctlr.c
^ permalink raw reply
* RE: [PATCH 1/3 v4] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices
From: Zang Roy-R61911 @ 2010-10-14 6:43 UTC (permalink / raw)
To: Anton Vorontsov
Cc: Wood Scott-B07421, dedekind1, Lan Chunhe-B25806, linuxppc-dev,
linux-mtd, akpm, dwmw2, Gala Kumar-B11780
In-Reply-To: <20100920153718.GA4411@oksana.dev.rtsoft.ru>
DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogQW50b24gVm9yb250c292
IFttYWlsdG86Y2JvdWF0bWFpbHJ1QGdtYWlsLmNvbV0NCj4gU2VudDogTW9uZGF5LCBTZXB0ZW1i
ZXIgMjAsIDIwMTAgMjM6MzcgUE0NCj4gVG86IFphbmcgUm95LVI2MTkxMQ0KPiBDYzogbGludXgt
bXRkQGxpc3RzLmluZnJhZGVhZC5vcmc7IGR3bXcyQGluZnJhZGVhZC5vcmc7IGRlZGVraW5kMUBn
bWFpbC5jb207DQo+IGFrcG1AbGludXgtZm91bmRhdGlvbi5vcmc7IExhbiBDaHVuaGUtQjI1ODA2
OyBXb29kIFNjb3R0LUIwNzQyMTsgR2FsYSBLdW1hci0NCj4gQjExNzgwOyBsaW51eHBwYy1kZXZA
b3psYWJzLm9yZw0KPiBTdWJqZWN0OiBSZTogW1BBVENIIDEvMyB2NF0gUDQwODAvZUxCQzogTWFr
ZSBGcmVlc2NhbGUgZWxiYyBpbnRlcnJ1cHQgY29tbW9uDQo+IHRvIGVsYmMgZGV2aWNlcw0KPiAN
Cj4gT24gRnJpLCBTZXAgMTcsIDIwMTAgYXQgMDM6MDE6MDdQTSArMDgwMCwgUm95IFphbmcgd3Jv
dGU6DQoNCltzbmlwXQ0KDQo+ID4gIGludCBmc2xfdXBtX3J1bl9wYXR0ZXJuKHN0cnVjdCBmc2xf
dXBtICp1cG0sIHZvaWQgX19pb21lbSAqaW9fYmFzZSwgdTMyIG1hcikNCj4gPiAgew0KPiA+ICAJ
aW50IHJldCA9IDA7DQo+ID4gIAl1bnNpZ25lZCBsb25nIGZsYWdzOw0KPiBbLi4uXQ0KPiA+ICtz
dGF0aWMgaW50IF9fZGV2aW5pdCBmc2xfbGJjX2N0cmxfcHJvYmUoc3RydWN0IHBsYXRmb3JtX2Rl
dmljZSAqZGV2KQ0KPiA+ICt7DQo+ID4gKwlpbnQgcmV0Ow0KPiA+ICsNCj4gPiArCWlmICghZGV2
LT5kZXYub2Zfbm9kZSkgew0KPiA+ICsJCWRldl9lcnIoJmRldi0+ZGV2LCAiRGV2aWNlIE9GLU5v
ZGUgaXMgTlVMTCIpOw0KPiA+ICsJCXJldHVybiAtRUZBVUxUOw0KPiA+ICsJfQ0KPiA+ICsJZnNs
X2xiY19jdHJsX2RldiA9IGt6YWxsb2Moc2l6ZW9mKCpmc2xfbGJjX2N0cmxfZGV2KSwgR0ZQX0tF
Uk5FTCk7DQo+IA0KPiBCdHcsIHRoZSBjb2RlIGluIHRoZSBOQU5EIGRyaXZlciBjaGVja3MgZm9y
ICFmc2xfbGJjX2N0cmxfZGV2Lg0KPiANCj4gU28gaXQgbWlnaHQgbWFrZSBzZW5zZSB0byBhc3Np
Z24gdGhlIGdsb2JhbCB2YXJpYWJsZSBhZnRlciB0aGUNCj4gc3RydWN0IGlzIGZ1bGx5IGluaXRp
YWxpemVkLg0KV2hhdCBzdHJ1Y3Q/DQphc3NpZ24gdGhlIGdsb2JhbCB2YXJpYWJsZSBpbiBuYW5k
IGRyaXZlcj8gDQpJJ2QgcHJlZmVyIGluaXQgaXQgaW4gdGhlIGxiYyBjb2RlLg0KVGhlcmUgbWF5
IGJlIG90aGVyIGxiYyBkZXZpY2UsIHdobyB3aWxsIHVzZSB0aGlzICBnbG9iYWwgdmFyaWFibGUu
DQoNCj4gDQo+ID4gKwlpZiAoIWZzbF9sYmNfY3RybF9kZXYpDQo+ID4gKwkJcmV0dXJuIC1FTk9N
RU07DQo+ID4gKw0KPiA+ICsJZGV2X3NldF9kcnZkYXRhKCZkZXYtPmRldiwgZnNsX2xiY19jdHJs
X2Rldik7DQo+ID4gKw0KPiA+ICsJc3Bpbl9sb2NrX2luaXQoJmZzbF9sYmNfY3RybF9kZXYtPmxv
Y2spOw0KPiA+ICsJaW5pdF93YWl0cXVldWVfaGVhZCgmZnNsX2xiY19jdHJsX2Rldi0+aXJxX3dh
aXQpOw0KPiA+ICsNCj4gPiArCWZzbF9sYmNfY3RybF9kZXYtPnJlZ3MgPSBvZl9pb21hcChkZXYt
PmRldi5vZl9ub2RlLCAwKTsNCj4gPiArCWlmICghZnNsX2xiY19jdHJsX2Rldi0+cmVncykgew0K
PiA+ICsJCWRldl9lcnIoJmRldi0+ZGV2LCAiZmFpbGVkIHRvIGdldCBtZW1vcnkgcmVnaW9uXG4i
KTsNCj4gPiArCQlyZXQgPSAtRU5PREVWOw0KPiA+ICsJCWdvdG8gZXJyOw0KPiA+ICsJfQ0KPiA+
ICsNCj4gPiArCWZzbF9sYmNfY3RybF9kZXYtPmlycSA9IGlycV9vZl9wYXJzZV9hbmRfbWFwKGRl
di0+ZGV2Lm9mX25vZGUsIDApOw0KPiA+ICsJaWYgKGZzbF9sYmNfY3RybF9kZXYtPmlycSA9PSBO
T19JUlEpIHsNCj4gPiArCQlkZXZfZXJyKCZkZXYtPmRldiwgImZhaWxlZCB0byBnZXQgaXJxIHJl
c291cmNlXG4iKTsNCj4gPiArCQlyZXQgPSAtRU5PREVWOw0KPiA+ICsJCWdvdG8gZXJyOw0KPiA+
ICsJfQ0KPiA+ICsNCj4gPiArCWZzbF9sYmNfY3RybF9kZXYtPmRldiA9ICZkZXYtPmRldjsNCj4g
PiArDQo+ID4gKwlyZXQgPSBmc2xfbGJjX2N0cmxfaW5pdChmc2xfbGJjX2N0cmxfZGV2KTsNCj4g
PiArCWlmIChyZXQgPCAwKQ0KPiA+ICsJCWdvdG8gZXJyOw0KPiA+ICsNCj4gPiArCXJldCA9IHJl
cXVlc3RfaXJxKGZzbF9sYmNfY3RybF9kZXYtPmlycSwgZnNsX2xiY19jdHJsX2lycSwgMCwNCj4g
PiArCQkJCSJmc2wtbGJjIiwgZnNsX2xiY19jdHJsX2Rldik7DQo+ID4gKwlpZiAocmV0ICE9IDAp
IHsNCj4gPiArCQlkZXZfZXJyKCZkZXYtPmRldiwgImZhaWxlZCB0byBpbnN0YWxsIGlycSAoJWQp
XG4iLA0KPiA+ICsJCQlmc2xfbGJjX2N0cmxfZGV2LT5pcnEpOw0KPiA+ICsJCXJldCA9IGZzbF9s
YmNfY3RybF9kZXYtPmlycTsNCj4gPiArCQlnb3RvIGVycjsNCj4gPiArCX0NCj4gPiArDQo+ID4g
KwlyZXR1cm4gMDsNCj4gPiArDQo+ID4gK2VycjoNCj4gPiArCWlvdW5tYXAoZnNsX2xiY19jdHJs
X2Rldi0+cmVncyk7DQo+ID4gKwlrZnJlZShmc2xfbGJjX2N0cmxfZGV2KTsNCj4gPiArCXJldHVy
biByZXQ7DQo+ID4gK30NCj4gPiArDQo+ID4gK3N0YXRpYyBjb25zdCBzdHJ1Y3Qgb2ZfZGV2aWNl
X2lkIGZzbF9sYmNfbWF0Y2hbXSA9IHsNCj4gDQo+ICNpbmNsdWRlIDxsaW51eC9tb2RfZGV2aWNl
dGFibGUuaD4gaXMgbmVlZGVkIGZvciB0aGlzLg0KPiANCj4gDQo+IFBsdXMsIEkgdGhpbmsgdGhl
IHBhdGNoIGlzIG5vdCBydW50aW1lIGJpc2VjdGFibGUgKGkuZS4geW91DQo+IG5vdyBkbyByZXF1
ZXN0X2lycSgpIGhlcmUsIGJ1dCBub3QgcmVtb3ZpbmcgaXQgZnJvbSB0aGUgbmFuZA0KPiBkcml2
ZXIsIHNvIG5hbmQgd2lsbCBmYWlsIHRvIHByb2JlKS4NCk5hbmQgZHJpdmVyIGRvZXMgbm90IG5l
ZWQgdG8gcmVxdWVzdCBpcnEuIEl0IHdpbGwgdXNlIHRoZSBpcnEgcmVxdWVzdGVkIGJ5IGxiYy4N
CnJlbWVtYmVyLCBvdGhlciBsYmMgZGV2aWNlIG1heSBhbHNvIG5lZWQgdG8gdXNlIHRoaXMgcmVn
aXN0ZXJlZCBpcnEuIA0KSXQgc2hvdWxkIG5vdCBiZSByZW1vdmVkIGluIG5hbmQgZHJpdmVyLg0K
DQpUaGFua3MuDQpSb3kNCg==
^ permalink raw reply
* RE: [PATCH 2/3] fsl_rio: fix non-standard HID1 register access
From: Li Yang-R58472 @ 2010-10-14 7:10 UTC (permalink / raw)
To: Kumar Gala, Xie Shaohui-B21989; +Cc: alexandre.bounine, linuxppc-dev
In-Reply-To: <7F0756F8-8297-43A6-913A-6DA2634E18E5@kernel.crashing.org>
>Subject: Re: [PATCH 2/3] fsl_rio: fix non-standard HID1 register access
>
>
>On Oct 13, 2010, at 9:04 PM, Shaohui Xie wrote:
>
>> From: Li Yang <leoli@freescale.com>
>>
>> The access to HID1 register is only legitimate for e500 v1/v2 cores.
>> Also fixes magic number.
>>
>> Signed-off-by: Li Yang <leoli@freescale.com>
>> Signed-off-by: Shaohui Xie <b21989@freescale.com>
>> ---
>> arch/powerpc/sysdev/fsl_rio.c | 9 ++++++---
>> 1 files changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/powerpc/sysdev/fsl_rio.c
>> b/arch/powerpc/sysdev/fsl_rio.c index 4127636..dfff3b7 100644
>> --- a/arch/powerpc/sysdev/fsl_rio.c
>> +++ b/arch/powerpc/sysdev/fsl_rio.c
>> @@ -1537,9 +1537,12 @@ int fsl_rio_setup(struct platform_device *dev)
>> #ifdef CONFIG_E500
>> saved_mcheck_exception =3D ppc_md.machine_check_exception;
>> ppc_md.machine_check_exception =3D fsl_rio_mcheck_exception; -#endif
>> - /* Ensure that RFXE is set */
>> - mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | 0x20000));
>> +
>> +#ifndef CONFIG_PPC_E500MC
>> + /* Ensure that RFXE is set on e500 v1/v2 */
>> + mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | HID1_RFXE)); #endif /*
>> +!PPC_E500MC */ #endif /* E500 */
>
>I've never really been happy with this code. We really should set
>HID1_RFXE in cpu_setup_fsl_booke.S instead.
But this bit is not recommended to be set unless necessary. And it is =
only required by SRIO for now.
- Leo
^ permalink raw reply
* Re: [PATCH][v2] powerpc/fsl: 85xx: add cache-sram support
From: Li Yang @ 2010-10-14 10:08 UTC (permalink / raw)
To: harninder.rai, Kumar Gala; +Cc: linuxppc-dev, Vivek Mahajan
In-Reply-To: <1286971256-22878-1-git-send-email-harninder.rai@freescale.com>
Oct 13, 2010 at 8:00 PM, <harninder.rai@freescale.com> wrote:
> From: Harninder Rai <harninder.rai@freescale.com>
>
> It adds cache-sram support in P1/P2 QorIQ platforms as under:
>
> =C2=A0 =C2=A0* A small abstraction over powerpc's remote heap allocator
> =C2=A0 =C2=A0* Exports mpc85xx_cache_sram_alloc()/free() APIs
> =C2=A0 =C2=A0* Supports only one contiguous SRAM window
> =C2=A0 =C2=A0* Drivers can do the following in Kconfig to use these APIs
> =C2=A0 =C2=A0 =C2=A0 =C2=A0"select FSL_85XX_CACHE_SRAM if MPC85xx"
You are missing the definition of this option in Kconfig like:
+config FSL_85XX_CACHE_SRAM
+ bool
+ select PPC_LIB_RHEAP
+
The select won't work actually.
Furthermore, MPC85xx config has been changed to FSL_SOC_BOOKE
- Leo
^ permalink raw reply
* Re: Please pull 'next' branch of 4xx tree
From: Josh Boyer @ 2010-10-14 10:35 UTC (permalink / raw)
To: Benjamin Herrenschmidt; +Cc: linuxppc-dev
In-Reply-To: <1287017775.2205.49.camel@pasglop>
On Thu, Oct 14, 2010 at 11:56:15AM +1100, Benjamin Herrenschmidt wrote:
>On Wed, 2010-10-13 at 09:16 -0400, Josh Boyer wrote:
>> On Tue, Sep 28, 2010 at 09:09:41AM -0400, Josh Boyer wrote:
>> >Hi Ben,
>> >
>> >A few small updates for the next branch. A new board/SoC from AMCC, and
>> >some 476 changes from Shaggy. Please pull.
>>
>> OK, below is a fixed up tree that drops the patch Shaggy said was
>> broken, and rebases on top of your new -next branch. Please pull.
>
>please, rebase again without Shaggy's lazy flush patch. I'll handle that
>one separately, I think it needs a little bit more massaging.
The following changes since commit 4108d9ba9091c55cfb968d42dd7dcae9a098b876:
powerpc/Makefiles: Change to new flag variables (2010-10-13 16:19:22 +1100)
are available in the git repository at:
ssh://master.kernel.org/pub/scm/linux/kernel/git/jwboyer/powerpc-4xx.git next
Josh Boyer (1):
powerpc/44x: Update ppc44x_defconfig
Tirumala Marri (1):
powerpc/44x: Add support for the AMCC APM821xx SoC
arch/powerpc/boot/dts/bluestone.dts | 254 ++++++++++++++++++++++++++
arch/powerpc/configs/44x/bluestone_defconfig | 68 +++++++
arch/powerpc/configs/ppc44x_defconfig | 9 +-
arch/powerpc/kernel/cpu_setup_44x.S | 1 +
arch/powerpc/kernel/cputable.c | 15 ++
arch/powerpc/platforms/44x/Kconfig | 16 ++
arch/powerpc/platforms/44x/ppc44x_simple.c | 1 +
7 files changed, 360 insertions(+), 4 deletions(-)
create mode 100644 arch/powerpc/boot/dts/bluestone.dts
create mode 100644 arch/powerpc/configs/44x/bluestone_defconfig
^ permalink raw reply
* Re: [PATCH 2/3] fsl_rio: fix non-standard HID1 register access
From: Kumar Gala @ 2010-10-14 12:57 UTC (permalink / raw)
To: Li Yang-R58472; +Cc: alexandre.bounine, linuxppc-dev, Xie Shaohui-B21989
In-Reply-To: <F9BD3E0A8083BE4ABAA94D7EDD7E3F633346A7@zch01exm26.fsl.freescale.net>
On Oct 14, 2010, at 2:10 AM, Li Yang-R58472 wrote:
>> Subject: Re: [PATCH 2/3] fsl_rio: fix non-standard HID1 register =
access
>>=20
>>=20
>> On Oct 13, 2010, at 9:04 PM, Shaohui Xie wrote:
>>=20
>>> From: Li Yang <leoli@freescale.com>
>>>=20
>>> The access to HID1 register is only legitimate for e500 v1/v2 cores.
>>> Also fixes magic number.
>>>=20
>>> Signed-off-by: Li Yang <leoli@freescale.com>
>>> Signed-off-by: Shaohui Xie <b21989@freescale.com>
>>> ---
>>> arch/powerpc/sysdev/fsl_rio.c | 9 ++++++---
>>> 1 files changed, 6 insertions(+), 3 deletions(-)
>>>=20
>>> diff --git a/arch/powerpc/sysdev/fsl_rio.c
>>> b/arch/powerpc/sysdev/fsl_rio.c index 4127636..dfff3b7 100644
>>> --- a/arch/powerpc/sysdev/fsl_rio.c
>>> +++ b/arch/powerpc/sysdev/fsl_rio.c
>>> @@ -1537,9 +1537,12 @@ int fsl_rio_setup(struct platform_device =
*dev)
>>> #ifdef CONFIG_E500
>>> saved_mcheck_exception =3D ppc_md.machine_check_exception;
>>> ppc_md.machine_check_exception =3D fsl_rio_mcheck_exception; =
-#endif
>>> - /* Ensure that RFXE is set */
>>> - mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | 0x20000));
>>> +
>>> +#ifndef CONFIG_PPC_E500MC
>>> + /* Ensure that RFXE is set on e500 v1/v2 */
>>> + mtspr(SPRN_HID1, (mfspr(SPRN_HID1) | HID1_RFXE)); #endif /*
>>> +!PPC_E500MC */ #endif /* E500 */
>>=20
>> I've never really been happy with this code. We really should set
>> HID1_RFXE in cpu_setup_fsl_booke.S instead.
>=20
> But this bit is not recommended to be set unless necessary. And it is =
only required by SRIO for now.
Than wrap it in a CONFIG_RAPIDIO in cpu_setup_fsl_booke.S
- k
^ permalink raw reply
* Oops while running systemtap on the p6 machine against the kernel version 2.6.36-rc7-git3
From: divya @ 2010-10-14 13:25 UTC (permalink / raw)
To: systemtap, linuxppc-dev
While running systemtap tests on the p6 machine , against the kernel version 2.6.36-rc7-git3
Oops occured , here are the call trace
BUG: spinlock bad magic on CPU#6, stapio/20398
-- 0:conmux-control -- time-stamp -- Oct/13/10 2:49:18 --res
lock: c000000000fcfa18, .magic: 00000000, .owner:<none>/-1, .owner_cpu: 0
Call Trace:
[c0000001effbfab0] [c000000000011934] .show_stack+0x6c/0x16c (unreliable)
[c0000001effbfb60] [c0000000002c9274] .spin_bug+0xb0/0xd4
[c0000001effbfbf0] [c0000000002c953c] .do_raw_spin_lock+0x48/0x184
[c0000001effbfc90] [c00000000054af78] ._raw_spin_lock+0x10/0x24
[c0000001effbfd00] [d000000003015908] .__stp_time_timer_callback+0x94/0x13c [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c0000001effbfdc0] [c00000000009c2f8] .run_timer_softirq+0x1d8/0x2a8
[c0000001effbfec0] [c0000000000952d0] .__do_softirq+0xe4/0x1b4
[c0000001effbff90] [c00000000002a7a8] .call_do_softirq+0x14/0x24
[c00000010af1f560] [c00000000000dde4] .do_softirq+0x88/0xf0
[c00000010af1f600] [c000000000095030] .irq_exit+0x50/0xac
[c00000010af1f680] [c000000000027660] .timer_interrupt+0x110/0x13c
[c00000010af1f710] [c000000000003718] decrementer_common+0x118/0x180
--- Exception: 901 at .smp_call_function_many+0x284/0x2d0
LR = .smp_call_function_many+0x268/0x2d0
[c00000010af1fae0] [c0000000000c55c4] .smp_call_function+0x3c/0x54
[c00000010af1fb60] [c000000000094cdc] .on_each_cpu+0x24/0x84
[c00000010af1fc00] [d000000003016738] ._stp_ctl_write_cmd+0x3b0/0x9c8 [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c00000010af1fce0] [c0000000001585f4] .vfs_write+0xd0/0x1b8
[c00000010af1fd80] [c0000000001587e4] .SyS_write+0x58/0xa0
[c00000010af1fe30] [c0000000000085b4] syscall_exit+0x0/0x40
------------[ cut here ]------------
kernel BUG at kernel/timer.c:681!
Oops: Exception in kernel mode, sig: 5 [#1]
SMP NR_CPUS=1024 NUMA pSeries
last sysfs file: /sys/module/stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798/sections/__param
Modules linked in: stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798 ipv6 fuse loop dm_mod ibmveth sg sr_mod cdrom sd_mod crc_t10dif ibmvscsic scsi_transport_srp scsi_tgt scsi_mod [last unloaded: stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
NIP: c00000000009d090 LR: d00000000301597c CTR: c00000000009cfb0
REGS: c0000001effbf9d0 TRAP: 0700 Not tainted (2.6.36-rc7-git3-autotest)
MSR: 8000000000029032<EE,ME,CE,IR,DR> CR: 28000482 XER: 00000002
TASK = c000000103422410[20398] 'stapio' THREAD: c00000010af1c000 CPU: 6
GPR00: 0000000000000001 c0000001effbfc50 c000000000a31660 c000000000fcfa48
GPR04: 000000010000c8da 0000000000000070 0000000000000002 0000000000000000
GPR08: ffffffffffffffff c000000000ac7bf8 0000000000000006 c00000000009cfb0
GPR12: d000000003017090 c00000000f600f00 0000000010008fd8 0000000010008ff8
GPR16: 0000000000000000 c000000000a91180 0000000000000000 0000000000000001
GPR20: c00000010edb9030 c00000010edb9430 c00000010edb9830 c00000010edb9c30
GPR24: 0000000000000001 001167a5b590dc41 c000000000fcfa18 000000010000c8da
GPR28: c000000000fcfa48 d00000000301597c c0000000009a6470 000000010000c8da
NIP [c00000000009d090] .mod_timer+0xe0/0x24c
LR [d00000000301597c] .__stp_time_timer_callback+0x108/0x13c [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
Call Trace:
[c0000001effbfc50] [c0000001effbfd00] 0xc0000001effbfd00 (unreliable)
[c0000001effbfd00] [d00000000301597c] .__stp_time_timer_callback+0x108/0x13c [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c0000001effbfdc0] [c00000000009c2f8] .run_timer_softirq+0x1d8/0x2a8
[c0000001effbfec0] [c0000000000952d0] .__do_softirq+0xe4/0x1b4
[c0000001effbff90] [c00000000002a7a8] .call_do_softirq+0x14/0x24
[c00000010af1f560] [c00000000000dde4] .do_softirq+0x88/0xf0
[c00000010af1f600] [c000000000095030] .irq_exit+0x50/0xac
[c00000010af1f680] [c000000000027660] .timer_interrupt+0x110/0x13c
[c00000010af1f710] [c000000000003718] decrementer_common+0x118/0x180
--- Exception: 901 at .smp_call_function_many+0x284/0x2d0
LR = .smp_call_function_many+0x268/0x2d0
[c00000010af1fae0] [c0000000000c55c4] .smp_call_function+0x3c/0x54
[c00000010af1fb60] [c000000000094cdc] .on_each_cpu+0x24/0x84
[c00000010af1fc00] [d000000003016738] ._stp_ctl_write_cmd+0x3b0/0x9c8 [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c00000010af1fce0] [c0000000001585f4] .vfs_write+0xd0/0x1b8
[c00000010af1fd80] [c0000000001587e4] .SyS_write+0x58/0xa0
[c00000010af1fe30] [c0000000000085b4] syscall_exit+0x0/0x40
Instruction dump:
7ffb4878 f9210070 e93e8078 80090000 2f800000 419e0010 7fa4eb78 7f83e378
4bfffcb1 e81c0020 7c000074 7800d182<0b000000> 7f83e378 38810070 3b400000
Kernel panic - not syncing: Fatal exception in interrupt
Call Trace:
[c0000001effbf5b0] [c000000000011934] .show_stack+0x6c/0x16c (unreliable)
[c0000001effbf660] [c000000000552094] .panic+0x9c/0x204
[c0000001effbf700] [c000000000028974] .die+0x268/0x2ac
[c0000001effbf7a0] [c000000000028ca8] ._exception+0x88/0x174
[c0000001effbf960] [c000000000004f8c] program_check_common+0x10c/0x180
--- Exception: 700 at .mod_timer+0xe0/0x24c
LR = .__stp_time_timer_callback+0x108/0x13c [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c0000001effbfc50] [c0000001effbfd00] 0xc0000001effbfd00 (unreliable)
[c0000001effbfd00] [d00000000301597c] .__stp_time_timer_callback+0x108/0x13c [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c0000001effbfdc0] [c00000000009c2f8] .run_timer_softirq+0x1d8/0x2a8
[c0000001effbfec0] [c0000000000952d0] .__do_softirq+0xe4/0x1b4
[c0000001effbff90] [c00000000002a7a8] .call_do_softirq+0x14/0x24
[c00000010af1f560] [c00000000000dde4] .do_softirq+0x88/0xf0
[c00000010af1f600] [c000000000095030] .irq_exit+0x50/0xac
[c00000010af1f680] [c000000000027660] .timer_interrupt+0x110/0x13c
[c00000010af1f710] [c000000000003718] decrementer_common+0x118/0x180
--- Exception: 901 at .smp_call_function_many+0x284/0x2d0
LR = .smp_call_function_many+0x268/0x2d0
[c00000010af1fae0] [c0000000000c55c4] .smp_call_function+0x3c/0x54
[c00000010af1fb60] [c000000000094cdc] .on_each_cpu+0x24/0x84
[c00000010af1fc00] [d000000003016738] ._stp_ctl_write_cmd+0x3b0/0x9c8 [stap_75ce6f84d34f8665c9a6b8e27fb9ea95_818798]
[c00000010af1fce0] [c0000000001585f4] .vfs_write+0xd0/0x1b8
[c00000010af1fd80] [c0000000001587e4] .SyS_write+0x58/0xa0
[c00000010af1fe30] [c0000000000085b4] syscall_exit+0x0/0x40
Rebooting in 180 seconds..-- 0:conmux-control -- time-stamp -- Oct/13/10 2:51:18 --
Thanks
Divya
^ permalink raw reply
* [PATCH] powerpc/fsl-booke: Add e500mc smp defconfig
From: Kumar Gala @ 2010-10-14 13:25 UTC (permalink / raw)
To: linuxppc-dev
The p4080, p3041, and p5020 (32-bit) SoCs from Freescale use the e500mc
defconfig. There are enough minor differences between these SoCs and
the existing P1xxx/P2xxx and MPC85xx SoCs to warrant a new defconfig.
The biggest of those is the cachline size being different which we
handle at compile time.
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
---
arch/powerpc/configs/e500mc_smp_defconfig | 178 +++++++++++++++++++++++++++++
1 files changed, 178 insertions(+), 0 deletions(-)
create mode 100644 arch/powerpc/configs/e500mc_smp_defconfig
diff --git a/arch/powerpc/configs/e500mc_smp_defconfig b/arch/powerpc/configs/e500mc_smp_defconfig
new file mode 100644
index 0000000..39b5871
--- /dev/null
+++ b/arch/powerpc/configs/e500mc_smp_defconfig
@@ -0,0 +1,178 @@
+CONFIG_PPC_85xx=y
+CONFIG_SMP=y
+CONFIG_NR_CPUS=8
+CONFIG_EXPERIMENTAL=y
+CONFIG_SYSVIPC=y
+CONFIG_POSIX_MQUEUE=y
+CONFIG_BSD_PROCESS_ACCT=y
+CONFIG_AUDIT=y
+CONFIG_RCU_TRACE=y
+CONFIG_IKCONFIG=y
+CONFIG_IKCONFIG_PROC=y
+CONFIG_LOG_BUF_SHIFT=14
+CONFIG_BLK_DEV_INITRD=y
+# CONFIG_CC_OPTIMIZE_FOR_SIZE is not set
+CONFIG_EMBEDDED=y
+CONFIG_KALLSYMS_ALL=y
+CONFIG_KALLSYMS_EXTRA_PASS=y
+CONFIG_PERF_EVENTS=y
+CONFIG_SLAB=y
+CONFIG_MODULES=y
+CONFIG_MODULE_UNLOAD=y
+CONFIG_MODULE_FORCE_UNLOAD=y
+CONFIG_MODVERSIONS=y
+# CONFIG_BLK_DEV_BSG is not set
+CONFIG_P3041_DS=y
+CONFIG_P4080_DS=y
+CONFIG_P5020_DS=y
+CONFIG_HIGHMEM=y
+CONFIG_NO_HZ=y
+CONFIG_HIGH_RES_TIMERS=y
+CONFIG_BINFMT_MISC=m
+CONFIG_MATH_EMULATION=y
+CONFIG_SPARSE_IRQ=y
+CONFIG_PCI=y
+CONFIG_PCIEPORTBUS=y
+# CONFIG_PCIEASPM is not set
+CONFIG_NET=y
+CONFIG_PACKET=y
+CONFIG_UNIX=y
+CONFIG_XFRM_USER=y
+CONFIG_XFRM_SUB_POLICY=y
+CONFIG_XFRM_STATISTICS=y
+CONFIG_NET_KEY=y
+CONFIG_NET_KEY_MIGRATE=y
+CONFIG_INET=y
+CONFIG_IP_MULTICAST=y
+CONFIG_IP_ADVANCED_ROUTER=y
+CONFIG_IP_MULTIPLE_TABLES=y
+CONFIG_IP_ROUTE_MULTIPATH=y
+CONFIG_IP_ROUTE_VERBOSE=y
+CONFIG_IP_PNP=y
+CONFIG_IP_PNP_DHCP=y
+CONFIG_IP_PNP_BOOTP=y
+CONFIG_IP_PNP_RARP=y
+CONFIG_NET_IPIP=y
+CONFIG_NET_IPGRE=y
+CONFIG_NET_IPGRE_BROADCAST=y
+CONFIG_IP_MROUTE=y
+CONFIG_IP_PIMSM_V1=y
+CONFIG_IP_PIMSM_V2=y
+CONFIG_ARPD=y
+CONFIG_INET_AH=y
+CONFIG_INET_ESP=y
+CONFIG_INET_IPCOMP=y
+# CONFIG_INET_LRO is not set
+CONFIG_IPV6=y
+CONFIG_IP_SCTP=m
+CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug"
+CONFIG_MTD=y
+CONFIG_MTD_CONCAT=y
+CONFIG_MTD_PARTITIONS=y
+CONFIG_MTD_CMDLINE_PARTS=y
+CONFIG_MTD_OF_PARTS=y
+CONFIG_MTD_CHAR=y
+CONFIG_MTD_BLOCK=y
+CONFIG_MTD_CFI=y
+CONFIG_MTD_CFI_AMDSTD=y
+CONFIG_MTD_PHYSMAP_OF=y
+CONFIG_MTD_M25P80=y
+CONFIG_PROC_DEVICETREE=y
+CONFIG_BLK_DEV_LOOP=y
+CONFIG_BLK_DEV_NBD=y
+CONFIG_BLK_DEV_RAM=y
+CONFIG_BLK_DEV_RAM_SIZE=131072
+CONFIG_BLK_DEV_SD=y
+CONFIG_CHR_DEV_ST=y
+CONFIG_BLK_DEV_SR=y
+CONFIG_CHR_DEV_SG=y
+CONFIG_SCSI_MULTI_LUN=y
+CONFIG_SCSI_LOGGING=y
+CONFIG_SCSI_SYM53C8XX_2=y
+CONFIG_ATA=y
+CONFIG_SATA_AHCI=y
+CONFIG_SATA_SIL24=y
+CONFIG_SATA_SIL=y
+CONFIG_PATA_SIL680=y
+CONFIG_NETDEVICES=y
+CONFIG_VITESSE_PHY=y
+CONFIG_FIXED_PHY=y
+CONFIG_NET_ETHERNET=y
+CONFIG_E1000=y
+CONFIG_E1000E=y
+CONFIG_FSL_PQ_MDIO=y
+# CONFIG_INPUT_MOUSEDEV is not set
+# CONFIG_INPUT_KEYBOARD is not set
+# CONFIG_INPUT_MOUSE is not set
+CONFIG_SERIO_LIBPS2=y
+CONFIG_SERIAL_8250=y
+CONFIG_SERIAL_8250_CONSOLE=y
+CONFIG_SERIAL_8250_EXTENDED=y
+CONFIG_SERIAL_8250_MANY_PORTS=y
+CONFIG_SERIAL_8250_DETECT_IRQ=y
+CONFIG_SERIAL_8250_RSA=y
+# CONFIG_LEGACY_PTYS is not set
+CONFIG_HW_RANDOM=y
+CONFIG_NVRAM=y
+CONFIG_I2C=y
+CONFIG_I2C_MPC=y
+CONFIG_SPI=y
+CONFIG_SPI_GPIO=y
+# CONFIG_HWMON is not set
+CONFIG_VIDEO_OUTPUT_CONTROL=y
+CONFIG_USB_HID=m
+CONFIG_USB=y
+CONFIG_USB_DEVICEFS=y
+CONFIG_USB_MON=y
+CONFIG_USB_EHCI_HCD=y
+CONFIG_USB_EHCI_FSL=y
+CONFIG_USB_OHCI_HCD=y
+CONFIG_USB_OHCI_HCD_PPC_OF_BE=y
+CONFIG_USB_OHCI_HCD_PPC_OF_LE=y
+CONFIG_USB_STORAGE=y
+CONFIG_MMC=y
+CONFIG_MMC_SDHCI=y
+CONFIG_MMC_SDHCI_OF=y
+CONFIG_MMC_SDHCI_OF_ESDHC=y
+CONFIG_EDAC=y
+CONFIG_EDAC_MM_EDAC=y
+CONFIG_EDAC_MPC85XX=y
+CONFIG_RTC_CLASS=y
+CONFIG_RTC_DRV_DS3232=y
+CONFIG_RTC_DRV_CMOS=y
+CONFIG_EXT2_FS=y
+CONFIG_EXT3_FS=y
+# CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
+CONFIG_ISO9660_FS=m
+CONFIG_JOLIET=y
+CONFIG_ZISOFS=y
+CONFIG_UDF_FS=m
+CONFIG_MSDOS_FS=m
+CONFIG_VFAT_FS=y
+CONFIG_NTFS_FS=y
+CONFIG_PROC_KCORE=y
+CONFIG_TMPFS=y
+CONFIG_JFFS2_FS=y
+CONFIG_CRAMFS=y
+CONFIG_NFS_FS=y
+CONFIG_NFS_V3=y
+CONFIG_NFS_V4=y
+CONFIG_ROOT_NFS=y
+CONFIG_NFSD=m
+CONFIG_PARTITION_ADVANCED=y
+CONFIG_MAC_PARTITION=y
+CONFIG_NLS_ISO8859_1=y
+CONFIG_NLS_UTF8=m
+CONFIG_MAGIC_SYSRQ=y
+CONFIG_DEBUG_KERNEL=y
+CONFIG_DEBUG_SHIRQ=y
+CONFIG_DETECT_HUNG_TASK=y
+CONFIG_DEBUG_INFO=y
+CONFIG_SYSCTL_SYSCALL_CHECK=y
+CONFIG_CRYPTO_NULL=y
+CONFIG_CRYPTO_PCBC=m
+CONFIG_CRYPTO_MD4=y
+CONFIG_CRYPTO_SHA256=y
+CONFIG_CRYPTO_SHA512=y
+CONFIG_CRYPTO_AES=y
+# CONFIG_CRYPTO_ANSI_CPRNG is not set
--
1.7.2.3
^ permalink raw reply related
* Re: [PATCH] powerpc/fsl-booke: Add e500mc smp defconfig
From: Timur Tabi @ 2010-10-14 13:51 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev
In-Reply-To: <1287062710-8621-1-git-send-email-galak@kernel.crashing.org>
On Thu, Oct 14, 2010 at 8:25 AM, Kumar Gala <galak@kernel.crashing.org> wrote:
> +CONFIG_MATH_EMULATION=y
Don't these chips have hardware FP?
> +CONFIG_E1000=y
> +CONFIG_E1000E=y
Are you sure you want these on by default? We may use the E1000 cards
in-house, but I don't know if our customers do.
> +CONFIG_MSDOS_FS=m
> +CONFIG_VFAT_FS=y
> +CONFIG_NTFS_FS=y
These should probably be off as well.
--
Timur Tabi
Linux kernel developer at Freescale
^ permalink raw reply
* [PATCH] spi/fsl_spi: Fix compile errors when building on ppc64
From: Kumar Gala @ 2010-10-14 13:55 UTC (permalink / raw)
To: grant.likely; +Cc: linuxppc-dev, spi-devel-general
We get the following when building on ppc64 due to lack of include of
<asm/io.h>:
In file included from drivers/spi/spi_fsl_espi.c:25:0:
drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_write_reg':
drivers/spi/spi_fsl_lib.h:88:2: error: implicit declaration of function 'out_be32'
drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_read_reg':
drivers/spi/spi_fsl_lib.h:93:2: error: implicit declaration of function 'in_be32'
drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_remove':
drivers/spi/spi_fsl_espi.c:571:2: error: implicit declaration of function 'iounmap'
drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_probe':
drivers/spi/spi_fsl_espi.c:602:2: error: implicit declaration of function 'ioremap'
drivers/spi/spi_fsl_espi.c:602:24: warning: assignment makes pointer from integer without a cast
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
---
drivers/spi/spi_fsl_lib.h | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/spi/spi_fsl_lib.h b/drivers/spi/spi_fsl_lib.h
index 15aa6c2..281e060 100644
--- a/drivers/spi/spi_fsl_lib.h
+++ b/drivers/spi/spi_fsl_lib.h
@@ -18,6 +18,8 @@
#ifndef __SPI_FSL_LIB_H__
#define __SPI_FSL_LIB_H__
+#include <asm/io.h>
+
/* SPI/eSPI Controller driver's private data. */
struct mpc8xxx_spi {
struct device *dev;
--
1.7.2.3
^ permalink raw reply related
* Re: RTC rtc-cmos.c : Fix warning on PowerPC
From: srikanth krishnakar @ 2010-10-14 14:03 UTC (permalink / raw)
To: Michael Neuling; +Cc: linuxppc-dev, Linuxppc-embedded
In-Reply-To: <24020.1286940298@neuling.org>
>From 8435e5876fc3d629406c63b85bff82c25fc4bf75 Mon Sep 17 00:00:00 2001
From: Srikanth Krishnakar <skrishna@mvista.com>
Date: Fri, 8 Oct 2010 18:07:06 +0530
Subject: [PATCH] RTC: rtc-cmos.c: Fix warning on PowerPC
The following warning is seen while compilation of PowerPC kernel:
CC drivers/rtc/rtc-cmos.o
drivers/rtc/rtc-cmos.c:697:2: warning: #warning Assuming 128 bytes
of RTC+NVRAM address space, not 64 bytes.
Fix it by adding defined(__powerpc__).
Signed-off-by: Srikanth Krishnakar <skrishna@mvista.com>
---
drivers/rtc/rtc-cmos.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/rtc/rtc-cmos.c b/drivers/rtc/rtc-cmos.c
index 5856167..7e6ce62 100644
--- a/drivers/rtc/rtc-cmos.c
+++ b/drivers/rtc/rtc-cmos.c
@@ -687,7 +687,8 @@ cmos_do_probe(struct device *dev, struct resource
*ports, int rtc_irq)
#if defined(CONFIG_ATARI)
address_space = 64;
#elif defined(__i386__) || defined(__x86_64__) || defined(__arm__) \
- || defined(__sparc__) || defined(__mips__)
+ || defined(__sparc__) || defined(__mips__) \
+ || defined(__powerpc__)
address_space = 128;
#else
#warning Assuming 128 bytes of RTC+NVRAM address space, not 64 bytes.
--
1.6.3.3.333.gdf68
^ permalink raw reply related
* Re: [PATCH] spi/fsl_spi: Fix compile errors when building on ppc64
From: Grant Likely @ 2010-10-14 14:12 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev, spi-devel-general
In-Reply-To: <1287064547-8876-1-git-send-email-galak@kernel.crashing.org>
On Thu, Oct 14, 2010 at 08:55:47AM -0500, Kumar Gala wrote:
> We get the following when building on ppc64 due to lack of include of
> <asm/io.h>:
Is this an immediate problem (merge for .36), or is it a linux-next thing?
g.
>
> In file included from drivers/spi/spi_fsl_espi.c:25:0:
> drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_write_reg':
> drivers/spi/spi_fsl_lib.h:88:2: error: implicit declaration of function 'out_be32'
> drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_read_reg':
> drivers/spi/spi_fsl_lib.h:93:2: error: implicit declaration of function 'in_be32'
> drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_remove':
> drivers/spi/spi_fsl_espi.c:571:2: error: implicit declaration of function 'iounmap'
> drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_probe':
> drivers/spi/spi_fsl_espi.c:602:2: error: implicit declaration of function 'ioremap'
> drivers/spi/spi_fsl_espi.c:602:24: warning: assignment makes pointer from integer without a cast
>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
> drivers/spi/spi_fsl_lib.h | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/spi/spi_fsl_lib.h b/drivers/spi/spi_fsl_lib.h
> index 15aa6c2..281e060 100644
> --- a/drivers/spi/spi_fsl_lib.h
> +++ b/drivers/spi/spi_fsl_lib.h
> @@ -18,6 +18,8 @@
> #ifndef __SPI_FSL_LIB_H__
> #define __SPI_FSL_LIB_H__
>
> +#include <asm/io.h>
> +
> /* SPI/eSPI Controller driver's private data. */
> struct mpc8xxx_spi {
> struct device *dev;
> --
> 1.7.2.3
>
^ permalink raw reply
* Re: [PATCH] spi/fsl_spi: Fix compile errors when building on ppc64
From: Kumar Gala @ 2010-10-14 14:41 UTC (permalink / raw)
To: Grant Likely; +Cc: linuxppc-dev, spi-devel-general
In-Reply-To: <20101014141244.GA4312@angua.secretlab.ca>
On Oct 14, 2010, at 9:12 AM, Grant Likely wrote:
> On Thu, Oct 14, 2010 at 08:55:47AM -0500, Kumar Gala wrote:
>> We get the following when building on ppc64 due to lack of include of
>> <asm/io.h>:
>
> Is this an immediate problem (merge for .36), or is it a linux-next thing?
>
> g.
.37 / next is fine.
- k
^ permalink raw reply
* Re: [PATCH] spi/fsl_spi: Fix compile errors when building on ppc64
From: Grant Likely @ 2010-10-14 14:47 UTC (permalink / raw)
To: Kumar Gala; +Cc: linuxppc-dev, spi-devel-general
In-Reply-To: <1287064547-8876-1-git-send-email-galak@kernel.crashing.org>
On Thu, Oct 14, 2010 at 08:55:47AM -0500, Kumar Gala wrote:
> We get the following when building on ppc64 due to lack of include of
> <asm/io.h>:
Applied.
g.
>
> In file included from drivers/spi/spi_fsl_espi.c:25:0:
> drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_write_reg':
> drivers/spi/spi_fsl_lib.h:88:2: error: implicit declaration of function 'out_be32'
> drivers/spi/spi_fsl_lib.h: In function 'mpc8xxx_spi_read_reg':
> drivers/spi/spi_fsl_lib.h:93:2: error: implicit declaration of function 'in_be32'
> drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_remove':
> drivers/spi/spi_fsl_espi.c:571:2: error: implicit declaration of function 'iounmap'
> drivers/spi/spi_fsl_espi.c: In function 'fsl_espi_probe':
> drivers/spi/spi_fsl_espi.c:602:2: error: implicit declaration of function 'ioremap'
> drivers/spi/spi_fsl_espi.c:602:24: warning: assignment makes pointer from integer without a cast
>
> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
> ---
> drivers/spi/spi_fsl_lib.h | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/spi/spi_fsl_lib.h b/drivers/spi/spi_fsl_lib.h
> index 15aa6c2..281e060 100644
> --- a/drivers/spi/spi_fsl_lib.h
> +++ b/drivers/spi/spi_fsl_lib.h
> @@ -18,6 +18,8 @@
> #ifndef __SPI_FSL_LIB_H__
> #define __SPI_FSL_LIB_H__
>
> +#include <asm/io.h>
> +
> /* SPI/eSPI Controller driver's private data. */
> struct mpc8xxx_spi {
> struct device *dev;
> --
> 1.7.2.3
>
^ permalink raw reply
* eSDHC controller driver on MPC8308rdb
From: Maria Johansen @ 2010-10-14 14:42 UTC (permalink / raw)
To: linuxppc-dev
SGVsbG8sDQooYXBvbG9naWVzIGZvciB3cml0aW5nIHN1Y2ggYSBsb25nIGUtbWFpbCwgaG9wZSB5
b3UgY2FuIGJvdGhlciB0byByZWFkIGl0IGFsbCDimLogKQ0KSSBoYXZlIHNvbWUgZGlmZmljdWx0
aWVzIHdpdGggdGhlIGVTREhDIGNvbnRyb2xsZXIgZHJpdmVyIHVzZWQgb24gYSBNUEM4MzA4IGV2
YWx1YXRpb24gYm9hcmQgd2l0aCBrZXJuZWwgMi42LjM2LXJjNywgYW5kIGhvcGUgdGhhdCBzb21l
IG9mIHlvdSBtYXkgYmUgYWJsZSB0byBoZWxwIG1lIHdpdGggdGhlIGRlYnVnZ2luZy4gDQoNClRo
ZSBkcml2ZXIgaXMgbG9hZGVkIHByb3Blcmx5LCBhbmQgYmluZHMgdG8gdGhlIGVTREhDIGNvbnRy
b2xsZXIgd2l0aG91dCBwcm9ibGVtcy4gV2hlbiBpbnNlcnRpbmcgYSBzZC1jYXJkIGl0IGlzIGJv
dW5kIHRvIHRoZSAgbW1jYmxrLWRyaXZlciwgc28gbm8gcHJvYmxlbXMgdGhlcmUgZWl0aGVyLiAg
DQoNCkhvd2V2ZXIsIEkgYW0gbm90IGFibGUgdG8gcmVhZCBub3Igd3JpdGUgdG8gdGhlIGNhcmQs
IEkgaGF2ZSBhdHRhY2hlZCBvdXRwdXQgd2l0aCBlcnJvciBtZXNzYWdlcyBiZWxvdzoNCg0KKioq
KioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKg0K
bW1jMDogbmV3IFNESEMgY2FyZCBhdCBhZGRyZXNzIDkxNTUNCm1tY2JsazA6IG1tYzA6OTE1NSBT
RDA0RyAzLjY5IEdpQg0KbW1jMDogVG9vIGxhcmdlIHRpbWVvdXQgcmVxdWVzdGVkIQ0KbW1jYmxr
MDogcmV0cnlpbmcgdXNpbmcgc2luZ2xlIGJsb2NrIHJlYWQNCm1tYzA6IFRvbyBsYXJnZSB0aW1l
b3V0IHJlcXVlc3RlZCENCm1tY2JsazA6IGVycm9yIC04NCBzZW5kaW5nIHN0YXR1cyBjb21hbmQN
Cm1tY2JsazA6IGVycm9yIC0xMTAgc2VuZGluZyByZWFkL3dyaXRlIGNvbW1hbmQsIHJlc3BvbnNl
IDB4MCwgY2FyZCBzdGF0dXMgMHgwDQplbmRfcmVxdWVzdDogSS9PIGVycm9yLCBkZXYgbW1jYmxr
MCwgc2VjdG9yIDANCm1tYzA6IFRvbyBsYXJnZSB0aW1lb3V0IHJlcXVlc3RlZCENCm1tY2JsazA6
IGVycm9yIC04NCBzZW5kaW5nIHN0YXR1cyBjb21hbmQNCm1tY2JsazA6IGVycm9yIC0xMTAgc2Vu
ZGluZyByZWFkL3dyaXRlIGNvbW1hbmQsIHJlc3BvbnNlIDB4MCwgY2FyZCBzdGF0dXMgMHgwDQpl
bmRfcmVxdWVzdDogSS9PIGVycm9yLCBkZXYgbW1jYmxrMCwgc2VjdG9yIDENCuKApi4uPHRoaXMg
Y29udGludWVzIHVwIHRvIHNlY3RvciA3IGJlZm9yZSBzdGFydGluZyBhbmV3Pg0KDQpEZWJ1ZyBv
dXRwdXQgd2l0aCBubyBTRC1jYXJkOg0KKioqKioqKioqKioqKioqKioqKioqKioqKioqKioqKioq
KioqKioqKioqKioqKioqKioqKioqKioqKg0KbW1jMDogY2xvY2sgMEh6IGJ1c21vZGUgMSBwb3dl
cm1vZGUgMSBjcyAwIFZkZCAyMCB3aWR0aCAwIHRpbWluZyAwDQptbWMwOiBjbG9jayA0MDAwMDBI
eiBidXNtb2RlIDEgcG93ZXJtb2RlIDIgY3MgMCBWZGQgMjAgd2lkdGggMCB0aW1pbmcgMA0Kb2Y6
c2RoY2ktb2YgZTAwMmUwMDAuc2RoY2k6IGRlc2lyZWQgU0QgY2xvY2s6IDQwMDAwMCwgYWN0dWFs
OiAwDQptbWMwOiBzdGFydGluZyBDTUQ1MiBhcmcgMDAwMDBjMDAgZmxhZ3MgMDAwMDAxOTUNCnNk
aGNpIFtzZGhjaV9pcnEoKV06ICoqKiBtbWMwIGdvdCBpbnRlcnJ1cHQ6IDB4MDAwMTAwMDENCm1t
YzA6IHJlcSBkb25lIChDTUQ1Mik6IC0xMTA6IDAwMDAwMDAwIDAwMDAwMDAwIDAwMDAwMDAwIDAw
MDAwMDAwDQptbWMwOiBzdGFydGluZyBDTUQ1MiBhcmcgODAwMDBjMDggZmxhZ3MgMDAwMDAxOTUN
CnNkaGNpIFtzZGhjaV9pcnEoKV06ICoqKiBtbWMwIGdvdCBpbnRlcnJ1cHQ6IDB4MDAwMTAwMDEN
Cm1tYzA6IHJlcSBkb25lIChDTUQ1Mik6IC0xMTA6IDAwMDAwMDAwIDAwMDAwMDAwIDAwMDAwMDAw
IDAwMDAwMDAwDQptbWMwOiBjbG9jayA0MDAwMDBIeiBidXNtb2RlIDEgcG93ZXJtb2RlIDIgY3Mg
MSBWZGQgMjAgd2lkdGggMCB0aW1pbmcgMA0KbW1jMDogc3RhcnRpbmcgQ01EMCBhcmcgMDAwMDAw
MDAgZmxhZ3MgMDAwMDAwYzANCnNkaGNpIFtzZGhjaV9pcnEoKV06ICoqKiBtbWMwIGdvdCBpbnRl
cnJ1cHQ6IDB4MDAwMDAwMDENCm1tYzA6IHJlcSBkb25lIChDTUQwKTogMDogMDAwMDAwMDAgMDAw
MDAwMDAgMDAwMDAwMDAgMDAwMDAwMDANCm1tYzA6IGNsb2NrIDQwMDAwMEh6IGJ1c21vZGUgMSBw
b3dlcm1vZGUgMiBjcyAwIFZkZCAyMCB3aWR0aCAwIHRpbWluZyAwDQptbWMwOiBzdGFydGluZyBD
TUQ4IGFyZyAwMDAwMDFhYSBmbGFncyAwMDAwMDJmNQ0Kc2RoY2kgW3NkaGNpX2lycSgpXTogKioq
IG1tYzAgZ290IGludGVycnVwdDogMHgwMDAxMDAwMQ0KbW1jMDogcmVxIGRvbmUgKENNRDgpOiAt
MTEwOiAwMDAwMDAwMCAwMDAwMDAwMCAwMDAwMDAwMCAwMDAwMDAwMA0KbW1jMDogc3RhcnRpbmcg
Q01ENSBhcmcgMDAwMDAwMDAgZmxhZ3MgMDAwMDAyZTENCnNkaGNpIFtzZGhjaV9pcnEoKV06ICoq
KiBtbWMwIGdvdCBpbnRlcnJ1cHQ6IDB4MDAwMTAwMDENCm1tYzA6IHJlcSBmYWlsZWQgKENNRDUp
OiAtMTEwLCByZXRyeWluZy4uLg0Kc2RoY2kgW3NkaGNpX2lycSgpXTogKioqIG1tYzAgZ290IGlu
dGVycnVwdDogMHgwMDAxMDAwMQ0KbW1jMDogcmVxIGZhaWxlZCAoQ01ENSk6IC0xMTAsIHJldHJ5
aW5nLi4uDQpzZGhjaSBbc2RoY2lfaXJxKCldOiAqKiogbW1jMCBnb3QgaW50ZXJydXB0OiAweDAw
MDEwMDAxDQptbWMwOiByZXEgZmFpbGVkIChDTUQ1KTogLTExMCwgcmV0cnlpbmcuLi4NCuKApg0K
DQpJIGhhdmUgdGhlIGZvbGxvd2luZyBpbiBteSBkdHM6DQogICAgICAgc2RoY2lAMmUwMDAgew0K
CQljb21wYXRpYmxlID0gImZzbCxtcGM4MzA4LWVzZGhjIiwgImZzbCxlc2RoYyI7DQoJCXJlZyA9
IDwweDJlMDAwIDB4MTAwMD47DQoJCWludGVycnVwdHMgPSA8NDIgMHg4PjsNCgkJaW50ZXJydXB0
LXBhcmVudCA9IDwmaXBpYz47DQoJCWNsb2NrLWZyZXF1ZW5jeSA9IDwwPjsNCgl9Ow0KDQoNCkNv
dWxkIHRoaXMgYmUgYSBwcm9ibGVtIHJlbGF0ZWQgdGhlIGVTREhDIGNvbnRyb2xsZXIgKG9yIHRo
ZSBkcml2ZXIpLCBvciBpcyBpdCB0aGUgbWVtb3J5IGNhcmQ/IChhIDRHQiBTYW5EaXNrIEV4dHJl
bWUgU0RIQyBjYXJkLCB3aGljaCB1bmZvcnR1bmF0ZWx5IGlzIHRoZSBvbmx5IGNhcmQgSSBoYXZl
IGF2YWlsYWJsZSBhdCB0aGUgbW9tZW50LikNCkkgd2lsbCBrZWVwIGRpZ2dpbmcgaW50byBkcml2
ZXJzL21tYy9ob3N0L3NkaGNpLmMgaW4gc2VhcmNoIG9mIGEgc29sdXRpb24sIGFuZCBhbnkgdGlw
cyB0byBob3cgSSBzaG91bGQgcHJvY2VlZCB3b3VsZCBiZSBncmVhdGx5IGFwcHJlY2lhdGVkIQ0K
DQotLQ0KTWFyaWEgDQo=
^ permalink raw reply
* Re: [PATCH] powerpc/5121: pdm360ng: fix touch irq if 8xxx gpio driver is enabled
From: Anatolij Gustschin @ 2010-10-14 14:55 UTC (permalink / raw)
To: Grant Likely; +Cc: linuxppc-dev
In-Reply-To: <1284581577-2217-1-git-send-email-agust@denx.de>
Hi Grant,
On Wed, 15 Sep 2010 22:12:57 +0200
Anatolij Gustschin <agust@denx.de> wrote:
> Enabling the MPC8xxx GPIO driver with MPC512x GPIO extension
> breaks touch screen support on this board since the GPIO
> interrupt will be mapped to 8xxx GPIO irq host resulting in
> a not requestable interrupt in the touch screen driver. Fix
> it by mapping the touch interrupt on 8xxx GPIO irq host.
>
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> ---
> arch/powerpc/platforms/512x/pdm360ng.c | 26 ++++++++++++++++++++++----
> 1 files changed, 22 insertions(+), 4 deletions(-)
Can this patch go in for 2.6.37 ?
Thanks,
Anatolij
^ permalink raw reply
* Re: Questions on interrupt vector assignment on MPC8641D
From: Scott Wood @ 2010-10-14 15:51 UTC (permalink / raw)
To: tiejun.chen; +Cc: david.hagood, linuxppc-dev@lists.ozlabs.org
In-Reply-To: <4CB6788D.60705@windriver.com>
On Thu, 14 Oct 2010 11:27:09 +0800
"tiejun.chen" <tiejun.chen@windriver.com> wrote:
> tiejun.chen wrote:
> > Scott Wood wrote:
> >> On Wed, 13 Oct 2010 09:17:01 +0800
> >> "tiejun.chen" <tiejun.chen@windriver.com> wrote:
> >>
> >>> Scott Wood wrote:
> >>>> The crash is happening somewhere in mpic_set_irq_type():
> >>> Agreed. That is just where I pointed out on my email replied for OOPS. To enable
> >>> DBG to figure out 'src' and 'mpic->irq_count' from the file,
> >>> arch/powerpc/sysdev/mpic.c, .
> >>> ======
> >>> int mpic_set_irq_type(unsigned int virq, unsigned int flow_type)
> >>> {
> >>> ......
> >>> if (src >= mpic->irq_count)
> >>> return -EINVAL;
> >>> ^
> >>> I think this OOPS may be from here.
> >> No, it's after that. His board code is using the mpic's "isu" remapping
> >
> > I means OOPS is *from* here. According to David's call trace,
> > mpic_set_irq_type() is the last issued function. And that corresponding return
> > value, R3, is '0xffffffea', -22, and also '-EINVAL'.
Just because that value is in r3 doesn't mean that src >=
mpic->irq_count.
Consider something like:
cmplw r4, r5
li r3, -EINVAL
bgelr
...
-Scott
^ permalink raw reply
* Re: [PATCH 1/3 v4] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices
From: Scott Wood @ 2010-10-14 16:02 UTC (permalink / raw)
To: Zang Roy-R61911
Cc: Wood Scott-B07421, dedekind1, Lan Chunhe-B25806, linuxppc-dev,
linux-mtd, akpm, dwmw2, Gala Kumar-B11780
In-Reply-To: <3850A844E6A3854C827AC5C0BEC7B60A2B0329@zch01exm23.fsl.freescale.net>
On Wed, 13 Oct 2010 23:43:38 -0700
"Zang Roy-R61911" <r61911@freescale.com> wrote:
> > Plus, I think the patch is not runtime bisectable (i.e. you
> > now do request_irq() here, but not removing it from the nand
> > driver, so nand will fail to probe).
> Nand driver does not need to request irq. It will use the irq requested by lbc.
> remember, other lbc device may also need to use this registered irq.
> It should not be removed in nand driver.
The point is that you need to make both changes in the same patch.
-Scott
^ permalink raw reply
* Re: Questions on interrupt vector assignment on MPC8641D
From: david.hagood @ 2010-10-14 16:22 UTC (permalink / raw)
To: Scott Wood; +Cc: tiejun.chen, david.hagood, linuxppc-dev@lists.ozlabs.org
In-Reply-To: <20101014105126.08da9dd7@udp111988uds.am.freescale.net>
I may have a clue (you might not think so, but...):
I've configured the init thusly:
mpic1 = mpic_alloc(np, res.start,
MPIC_PRIMARY | MPIC_WANTS_RESET | MPIC_BIG_ENDIAN ,
0, 256,
" MPIC ");
Which, as I read the code, should disable the ISU stuff.
I've seeing this on boot:
mpic: Setting up MPIC " MPIC " version 1.2 at e0040000, max 2 CPUs
mpic: ISU size: 88, shift: 7, mask: 7f
mpic: Initializing for 88 sources
Now, since the interrupt number I want is 224, which, last time I checked,
was > 88, this may be the root cause.
As I read the code:
/* Read feature register, calculate num CPUs and, for non-ISU
* MPICs, num sources as well. On ISU MPICs, sources are counted
* as ISUs are added
*/
greg_feature = mpic_read(mpic->gregs, MPIC_INFO(GREG_FEATURE_0));
mpic->num_cpus = ((greg_feature & MPIC_GREG_FEATURE_LAST_CPU_MASK)
>> MPIC_GREG_FEATURE_LAST_CPU_SHIFT) + 1;
if (isu_size == 0)
mpic->num_sources =
((greg_feature & MPIC_GREG_FEATURE_LAST_SRC_MASK)
>> MPIC_GREG_FEATURE_LAST_SRC_SHIFT) + 1;
So it would seem to me that the "greg_feature" is saying I have 88
interrupts.
I've tried setting the ISU size to 256:
mpic1 = mpic_alloc(np, res.start,
MPIC_PRIMARY | MPIC_WANTS_RESET | MPIC_BIG_ENDIAN ,
256, 256,
" MPIC ");
And that kills the kernel as we init the mpic.
SO, I guess the question in, what sets "greg_feature", as it would seem to
be incorrect.
Or, am I on the wrong trail?
^ permalink raw reply
* Re: [PATCH 2/3 v4] P4080/mtd: Only make elbc nand driver detect nand flash partitions
From: Scott Wood @ 2010-10-14 16:01 UTC (permalink / raw)
To: Zang Roy-R61911
Cc: Wood Scott-B07421, dedekind1, Hu Mingkai-B21284,
Lan Chunhe-B25806, linuxppc-dev, linux-mtd, akpm, dwmw2,
Gala Kumar-B11780
In-Reply-To: <3850A844E6A3854C827AC5C0BEC7B60A2B02B7@zch01exm23.fsl.freescale.net>
On Wed, 13 Oct 2010 20:09:02 -0700
"Zang Roy-R61911" <r61911@freescale.com> wrote:
> > > > > + struct fsl_elbc_fcm_ctrl *elbc_fcm_ctrl = NULL;
> > > >
> > > > No need for = NULL.
> > > Any harm? Or just personal habit or style? Can you explain about
> why?
> >
> > Besides not wanting superfluous code on general principle, it could
> > hide a bug if in the future the real initialization is missing on some
> > code path. It would become a runtime NULL dereference rather than a
> > compiler warning.
>
> Not exactly.
> Per my understand, if the pointer will definitely be assigned in code
> path,
> it is not necessary to init it when define. for example,
>
> char c;
> char b;
> char *a;
> if (condition)
> a = &c;
> else
> a = &b;
> ...
>
> for other case, if the path will not ensure the pointer assignment, it
> will be inited
> when define to avoid warning. for example,
>
> char c;
> char *a = NULL;
> if (condition)
> a = &c;
> ...
Yes, but this patch looks like the former case, not the
latter. Is GCC giving a warning without the initializer?
-Scott
^ permalink raw reply
* Re: Questions on interrupt vector assignment on MPC8641D
From: Scott Wood @ 2010-10-14 16:32 UTC (permalink / raw)
To: david.hagood; +Cc: tiejun.chen, linuxppc-dev@lists.ozlabs.org
In-Reply-To: <dccb3790f1fbb04a8df74bd920e57580.squirrel@localhost>
On Thu, 14 Oct 2010 11:22:45 -0500
<david.hagood@gmail.com> wrote:
> As I read the code:
> /* Read feature register, calculate num CPUs and, for non-ISU
> * MPICs, num sources as well. On ISU MPICs, sources are counted
> * as ISUs are added
> */
> greg_feature = mpic_read(mpic->gregs, MPIC_INFO(GREG_FEATURE_0));
> mpic->num_cpus = ((greg_feature & MPIC_GREG_FEATURE_LAST_CPU_MASK)
> >> MPIC_GREG_FEATURE_LAST_CPU_SHIFT) + 1;
> if (isu_size == 0)
> mpic->num_sources =
> ((greg_feature & MPIC_GREG_FEATURE_LAST_SRC_MASK)
> >> MPIC_GREG_FEATURE_LAST_SRC_SHIFT) + 1;
> So it would seem to me that the "greg_feature" is saying I have 88
> interrupts.
>
> I've tried setting the ISU size to 256:
> mpic1 = mpic_alloc(np, res.start,
> MPIC_PRIMARY | MPIC_WANTS_RESET | MPIC_BIG_ENDIAN ,
> 256, 256,
> " MPIC ");
> And that kills the kernel as we init the mpic.
>
> SO, I guess the question in, what sets "greg_feature", as it would seem to
> be incorrect.
It comes from FRR[NIRQ]. It seems that this chip takes a
less-than-useful interpretation of what that field means -- it gives
the actual number of sources, not the size of the sparsely populated
array.
If you look at current kernels, you'll find an MPIC_BROKEN_FRR_NIRQS
flag to work around this.
It's not very clear to me what distinction the MPIC code is
trying to make between irq_count and num_sources in the first place,
though.
-Scott
^ permalink raw reply
* Re: Questions on interrupt vector assignment on MPC8641D
From: david.hagood @ 2010-10-14 17:20 UTC (permalink / raw)
To: Scott Wood; +Cc: tiejun.chen, david.hagood, linuxppc-dev@lists.ozlabs.org
In-Reply-To: <20101014113230.051b967a@udp111988uds.am.freescale.net>
Hallelujah and Huzzah! I finally got my vector!
I back-ported the MPIC_BROKEN_FRR_NIRQS flag and code to our kernel, and
the kernel is now letting me have my vector! Now I can actually see if the
dang thing works!
THANK YOU EVERYBODY for putting up with me on this!
> It comes from FRR[NIRQ]. It seems that this chip takes a
> less-than-useful interpretation of what that field means -- it gives
> the actual number of sources, not the size of the sparsely populated
> array.
Perhaps you might want to have a talk with your cow-orkers there, Scott,
since this is a Freescale part.
> It's not very clear to me what distinction the MPIC code is
> trying to make between irq_count and num_sources in the first place,
> though.
/me looks at Scott's email again.
If you, working FOR Freescale, and following the Linux development
(presumably for some time) are confused, imagine what I've been going
through!
Hot damn, and time for a quick version control commit, a hot lunch, and
really testing the code.
Thanks again!
^ permalink raw reply
* [RFC PATCH] ppc: don't override CONFIG_PPC_PSERIES_DEBUG
From: Nishanth Aravamudan @ 2010-10-14 17:48 UTC (permalink / raw)
To: nacc
Cc: Michael Neuling, Frans Pop, linux-kernel, Paul Mackerras,
Anton Blanchard, Linas Vepstas, linuxppc-dev, Thomas Gleixner
These files undef DEBUG, but I think they were added before the ability
to control this from Kconfig. It's really annoying to only get some of
the debug messages!
Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com>
---
Because the lpar and pci_dlpar code is pretty low-level & verbose,
perhaps it makes sense to add another Kconfig variable for really
low-level stuff? But it's annoying to have DEBUG *somewhat* effective,
especially in the EEH area when doing PCI stuff.
---
arch/powerpc/platforms/pseries/eeh.c | 2 --
arch/powerpc/platforms/pseries/lpar.c | 3 ---
arch/powerpc/platforms/pseries/pci_dlpar.c | 2 --
3 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/arch/powerpc/platforms/pseries/eeh.c b/arch/powerpc/platforms/pseries/eeh.c
index 34b7dc1..17a11c8 100644
--- a/arch/powerpc/platforms/pseries/eeh.c
+++ b/arch/powerpc/platforms/pseries/eeh.c
@@ -21,8 +21,6 @@
* Please address comments and feedback to Linas Vepstas <linas@austin.ibm.com>
*/
-#undef DEBUG
-
#include <linux/delay.h>
#include <linux/init.h>
#include <linux/list.h>
diff --git a/arch/powerpc/platforms/pseries/lpar.c b/arch/powerpc/platforms/pseries/lpar.c
index cf79b46..4b31a66 100644
--- a/arch/powerpc/platforms/pseries/lpar.c
+++ b/arch/powerpc/platforms/pseries/lpar.c
@@ -19,9 +19,6 @@
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*/
-/* Enables debugging of low-level hash table routines - careful! */
-#undef DEBUG
-
#include <linux/kernel.h>
#include <linux/dma-mapping.h>
#include <linux/console.h>
diff --git a/arch/powerpc/platforms/pseries/pci_dlpar.c b/arch/powerpc/platforms/pseries/pci_dlpar.c
index 4b7a062..5fcc92a 100644
--- a/arch/powerpc/platforms/pseries/pci_dlpar.c
+++ b/arch/powerpc/platforms/pseries/pci_dlpar.c
@@ -25,8 +25,6 @@
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*/
-#undef DEBUG
-
#include <linux/pci.h>
#include <asm/pci-bridge.h>
#include <asm/ppc-pci.h>
--
1.7.1
^ permalink raw reply related
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox