* Re: [i2c] [PATCH 0/5] Version 17, series to add device tree naming to i2c
From: Jochen Friedrich @ 2008-01-11 16:05 UTC (permalink / raw)
To: Jon Smirl; +Cc: Jean Delvare, linuxppc-dev, i2c, linux-kernel
In-Reply-To: <9e4733910801110752k57f1fd7crd5f143900fc64f0b@mail.gmail.com>
Hi Jon,
>>> The following series implements standard linux module aliasing for i2c modules on arch=powerpc. It then converts the mpc i2c driver from being a platform driver to an open firmware one. I2C device names are picked up from the device tree. Module aliasing is used to translate from device tree names into to linux kernel names. Several i2c drivers are updated to use the new aliasing.
>> Now that I have read all the previous versions of this patch series
>> and, more importantly, all objections that were raised on the way, I
>> can start reviewing the latest iteration of your patches. I'll also do
>> some testing, although I have no powerpc stuff here, but at least I
>> want to make sure that there are no regressions introduced by your
>> patches on x86.
> Various people were worried about x86. Around version 15 I altered the
> patches so that they only impacted PowerPC. If they impact x86 in
> current form that is a bug.
I can only second this. The latest version of i2c-cpm
(http://patchwork.ozlabs.org/linuxppc/patch?person=1023&id=15902) makes
use of this patch, as well. On the dbox2, loading and unloading of
modules in any order just works fine.
Thanks,
Jochen
^ permalink raw reply
* Re: [PATCH for 2.6.24][NET] fs_enet: check for phydev existence in the ethtool handlers
From: Sergej Stepanov @ 2008-01-11 16:01 UTC (permalink / raw)
To: hs; +Cc: Scott Wood, linuxppc-dev, Jeff Garzik
In-Reply-To: <478666C6.2030104@denx.de>
Am Donnerstag, den 10.01.2008, 19:41 +0100 schrieb Heiko Schocher:
> Hello Scott,
>
> Scott Wood wrote:
> > Heiko Schocher wrote:
> >> diff --git a/drivers/net/fs_enet/fs_enet-main.c
> >> b/drivers/net/fs_enet/fs_enet-main.c
> >> index f2a4d39..f432a18 100644
> >> --- a/drivers/net/fs_enet/fs_enet-main.c
> >> +++ b/drivers/net/fs_enet/fs_enet-main.c
> >> @@ -810,6 +810,10 @@ static int fs_enet_open(struct net_device *dev)
> >> if (fep->fpi->use_napi)
> >> napi_enable(&fep->napi);
> >>
> >> + /* to initialize the fep->cur_rx,... */
> >> + /* not doing this, will cause a crash in fs_enet_rx_napi */
> >> + fs_init_bds(fep->ndev);
> >
> > We should do this just before napi_enable() rather than just after, to
> > eliminate any chance of a race window.
>
> Yes, you are right.
> Here is the new patch:
>
It works fine.
Sergej.
^ permalink raw reply
* Re: [i2c] [PATCH 0/5] Version 17, series to add device tree naming to i2c
From: Jon Smirl @ 2008-01-11 15:52 UTC (permalink / raw)
To: Jean Delvare; +Cc: linuxppc-dev, i2c, linux-kernel
In-Reply-To: <20080111095638.775670ed@hyperion.delvare>
On 1/11/08, Jean Delvare <khali@linux-fr.org> wrote:
> Hi Jon,
>
> On Wed, 19 Dec 2007 23:41:36 -0500, Jon Smirl wrote:
> > Since copying i2c-mpc.c to maintain support for the ppc architecture seems to be an issue; instead rework i2c-mpc.c to use CONFIG_PPC_MERGE #ifdefs to support both the ppc and powerpc architecture. When ppc is deleted in six months these #ifdefs will need to be removed.
> >
> > Another rework of the i2c for powerpc device tree patch. This version implements standard alias naming only on the powerpc platform and only for the device tree names. The old naming mechanism of i2c_client.name,driver_name is left in place and not changed for non-powerpc platforms. This patch is fully capable of dynamically loading the i2c modules. You can modprobe in the i2c-mpc driver and the i2c modules described in the device tree will be automatically loaded. Modules also work if compiled in.
> >
> > The follow on patch to module-init-tools is also needed since the i2c subsystem has never implemented dynamic loading.
> >
> > The following series implements standard linux module aliasing for i2c modules on arch=powerpc. It then converts the mpc i2c driver from being a platform driver to an open firmware one. I2C device names are picked up from the device tree. Module aliasing is used to translate from device tree names into to linux kernel names. Several i2c drivers are updated to use the new aliasing.
>
> Now that I have read all the previous versions of this patch series
> and, more importantly, all objections that were raised on the way, I
> can start reviewing the latest iteration of your patches. I'll also do
> some testing, although I have no powerpc stuff here, but at least I
> want to make sure that there are no regressions introduced by your
> patches on x86.
Various people were worried about x86. Around version 15 I altered the
patches so that they only impacted PowerPC. If they impact x86 in
current form that is a bug.
When x86 is ready for it I do think dynamic module loading should be
implemented there also.
>
> --
> Jean Delvare
>
--
Jon Smirl
jonsmirl@gmail.com
^ permalink raw reply
* Re: PCI Failed to allocate mem for PCI ROM
From: Kumar Gala @ 2008-01-11 15:41 UTC (permalink / raw)
To: Jiri Slaby; +Cc: Greg KH, linux-pci, LKML, linuxppc-dev list
In-Reply-To: <47873333.5020202@gmail.com>
On Jan 11, 2008, at 3:13 AM, Jiri Slaby wrote:
> On 01/11/2008 10:07 AM, Kumar Gala wrote:
>> On Jan 11, 2008, at 2:41 AM, Jiri Slaby wrote:
>>> On 01/11/2008 09:29 AM, Kumar Gala wrote:
>>>> Greg,
>>>> I'm getting the following message from the kernel on an embedded
>>>> ppc32 system:
>>>> PCI: Failed to allocate mem resource #9:100000@e0000000 for
>>>> 0000:00:00.0
>>>> The HW setup is a PCIe host controller and an e1000 NIC card. It
>>>> appears that pci_bus_assign_resources() is trying to call
>>>> pci_assign_resource() for the ROM and the resource for the ROM is
>>>> [100000:1fffff] where the PHB is [c0000000:dfffffff].
>>>> It seems like the resno that pci_assign_resource is getting
>>>> called with is wrong and thus pci_update_resource() doesn't get
>>>> called.
>>>> any ideas?
>>>
>>> Kernel version, please.
>> Sorry, its 2.6.24-rc7 + some ppc patches queued for 2.6.25
>
> Could you try this patch?
> http://git.kernel.org/?p=linux/kernel/git/gregkh/patches.git;a=blob_plain;f=pci/pci-remove-default-pci-expansion-rom-memory-allocation.patch
>
> Greg: is this 2.6.25 material, please? We need this for SP2.
I saw that patch, but if you notice that its just x86 specific and I'm
having the issue on a powerpc 32-bit system.
- k
^ permalink raw reply
* Re: [alsa-devel] [PATCH v2] [ALSA] Add ASoC drivers for the Freescale MPC8610 SoC
From: Timur Tabi @ 2008-01-11 15:08 UTC (permalink / raw)
To: Olof Johansson; +Cc: linuxppc-dev, alsa-devel, david
In-Reply-To: <20080111002450.GA13291@lixom.net>
Olof Johansson wrote:
> Having been in a similar situation myself (needing to share resources
> between DMA, ethernet and function offload), I recommend creating a
> separate small library that all those drivers use, instead of making
> some sort of dependency between drivers in completely different parts
> of the kernel.
Well, the DMA driver should be in soon. Actually, it should be in now, because
I saw a blurb in this month's Linux Journal about it. As soon as I find it :-),
I'll post a new patch that adds arbitration.
--
Timur Tabi
Linux kernel developer at Freescale
^ permalink raw reply
* Re: PPC vs POWERPC
From: Grant Likely @ 2008-01-11 14:52 UTC (permalink / raw)
To: samppa@sundmangroup.com; +Cc: linuxppc-embedded
In-Reply-To: <23145.147.11.3.128.1200054808.squirrel@www.sundmangroup.com>
On 1/11/08, samppa@sundmangroup.com <samppa@sundmangroup.com> wrote:
> Hi,
> I want to port a board ( WRS SBC8265 ) from the PPC branch to the POWERPC
> branch in the Linux Kernel -- do you have any good starting points that
> describes what I need to pay attention to?
>
> I've already made a port of U-boot-1.3.1 and enabled CONFIG_OF_LIBFDT.
>
> So I need to create a DTS file, compile it with the DTC compiler and port
> the current PPC branch to the POWERPC branch to my understanding.
Yes, you're exactly right. Use one of the dts files in
arch/powerpc/boot/dts as a starting point and read
Documentation/powerpc/booting-without-of.txt to find out what is
expected in the device tree.
Cheers,
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
^ permalink raw reply
* Re: libfdt: Add fdt_set_name() function
From: Jon Loeliger @ 2008-01-11 14:38 UTC (permalink / raw)
To: Josh Boyer; +Cc: linuxppc-dev, David Gibson
In-Reply-To: <20080111083003.38b556d6@zod.rchland.ibm.com>
So, like, the other day Josh Boyer mumbled:
> On Fri, 11 Jan 2008 07:44:33 -0600
> Jon Loeliger <jdl@jdl.com> wrote:
>
> > So, like, the other day David Gibson mumbled:
> > > This patch adds an fdt_set_name() function to libfdt, mirroring
> > > fdt_get_name(). This is a r/w function which alters the name of a
> > > given device tree node.
> > >
> > > Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> >
> > Applied.
>
> Awesome. Now if we could just get that into the kernel and usable in
> the wrappers... :)
I've pushed it out now.
I guess we'll try to get the real v1.1.0 into the kernel then?
Maybe at 2.6.25?
jdl
^ permalink raw reply
* Re: libfdt: Add fdt_set_name() function
From: Josh Boyer @ 2008-01-11 14:30 UTC (permalink / raw)
To: Jon Loeliger; +Cc: linuxppc-dev, David Gibson
In-Reply-To: <E1JDKBV-0007Hq-Dm@jdl.com>
On Fri, 11 Jan 2008 07:44:33 -0600
Jon Loeliger <jdl@jdl.com> wrote:
> So, like, the other day David Gibson mumbled:
> > This patch adds an fdt_set_name() function to libfdt, mirroring
> > fdt_get_name(). This is a r/w function which alters the name of a
> > given device tree node.
> >
> > Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
>
> Applied.
Awesome. Now if we could just get that into the kernel and usable in
the wrappers... :)
josh
^ permalink raw reply
* RE: Help with device tree binding for SMC serial
From: Rune Torgersen @ 2008-01-11 14:13 UTC (permalink / raw)
To: Scott Wood; +Cc: linuxppc-dev
In-Reply-To: <DCEAAC0833DD314AB0B58112AD99B93B03D9A042@ismail.innsys.innovsys.com>
> From: Rune Torgersen
> Finally got it (sort-of) working.
> Turned out that for some reason the console init is setting=20
> the baudrate
> to 9600
> the options string passed in to the console init fuunction is NULL.
>=20
> Any idea oon how this should be passed in from u-boot?
Ok, needed a valid console=3D line on the command line. WHen we tried
that, we had a typo, so it was not recognized.
Our old 2.6.18 arch/ppc kernel didn't need a console parameter, it got
the baudrate from u-boot somehow.
Anyway of doing that here too?
^ permalink raw reply
* Re: libfdt: Add fdt_set_name() function
From: Jon Loeliger @ 2008-01-11 13:44 UTC (permalink / raw)
To: David Gibson; +Cc: linuxppc-dev
In-Reply-To: <20080111035505.GB25055@localhost.localdomain>
So, like, the other day David Gibson mumbled:
> This patch adds an fdt_set_name() function to libfdt, mirroring
> fdt_get_name(). This is a r/w function which alters the name of a
> given device tree node.
>
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Applied.
jdl
^ permalink raw reply
* [PATCH 2/2] ps3fb: fix deadlock on kexec()
From: Geert Uytterhoeven @ 2008-01-11 13:28 UTC (permalink / raw)
To: Linus Torvalds, Andrew Morton
Cc: Linux Frame Buffer Device Development, Linux Kernel Development,
Linux/PPC Development, Jeremy Kerr,
Cell Broadband Engine OSS Development
In-Reply-To: <Pine.LNX.4.64.0801111417450.12038@vixen.sonytel.be>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 3570 bytes --]
From: Jeremy Kerr <jk@ozlabs.org>
Since the introduction of the acquire_console_sem calls in
0333d83509c7d8496c8965b5ba9bc0c98e83c259, kexecing can cause the
kernel to deadlock:
ps3fb_shutdown()
-> unregister_framebuffer()
-> fb_notifier_call_chain(FB_EVENT_FB_UNBIND)
-> fbcon_fb_unbind()
-> unbind_con_driver()
-> bind_con_driver()
[ acquires console_sem ]
-> fbcon_deinit()
-> fbops->fb_release(newinfo, 0)
-> ps3fb_release()
-> ps3fb_sync()
[ acquires console_sem ]
This change avoids the deadlock by moving the acquire_console_sem()
out of ps3fb_sync(), and puts it into the two other callsites, leaving
ps3fb_release() to call ps3fb_sync() without the console semaphore.
[Geert]
- Corrected call sequence above
- ps3fb_release() may be called with and without console_sem held. This is an
inconsistency that should be fixed at the fb level, but for now, try to
acquire console_sem in ps3fb_release().
I think it's safer to let ps3fb_release() try to acquire console_sem and
not refresh the screen if it fails, than to call ps3fb_sync() without
holding console_sem, as ps3fb_par may be modified at the same time, causing
crashes or lockups.
Besides, ps3fb_release() only calls ps3fb_sync() to refresh the screen
when display flipping is disabled, which is an uncommon case (except during
shutdown/kexec).
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---
drivers/video/ps3fb.c | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
--- a/drivers/video/ps3fb.c
+++ b/drivers/video/ps3fb.c
@@ -443,8 +443,6 @@ static int ps3fb_sync(struct fb_info *in
u32 ddr_line_length, xdr_line_length;
u64 ddr_base, xdr_base;
- acquire_console_sem();
-
if (frame > par->num_frames - 1) {
dev_dbg(info->device, "%s: invalid frame number (%u)\n",
__func__, frame);
@@ -464,7 +462,6 @@ static int ps3fb_sync(struct fb_info *in
xdr_line_length);
out:
- release_console_sem();
return error;
}
@@ -479,7 +476,10 @@ static int ps3fb_release(struct fb_info
if (atomic_dec_and_test(&ps3fb.f_count)) {
if (atomic_read(&ps3fb.ext_flip)) {
atomic_set(&ps3fb.ext_flip, 0);
- ps3fb_sync(info, 0); /* single buffer */
+ if (!try_acquire_console_sem()) {
+ ps3fb_sync(info, 0); /* single buffer */
+ release_console_sem();
+ }
}
}
return 0;
@@ -865,7 +865,9 @@ static int ps3fb_ioctl(struct fb_info *i
break;
dev_dbg(info->device, "PS3FB_IOCTL_FSEL:%d\n", val);
+ acquire_console_sem();
retval = ps3fb_sync(info, val);
+ release_console_sem();
break;
default:
@@ -885,7 +887,9 @@ static int ps3fbd(void *arg)
set_current_state(TASK_INTERRUPTIBLE);
if (ps3fb.is_kicked) {
ps3fb.is_kicked = 0;
+ acquire_console_sem();
ps3fb_sync(info, 0); /* single buffer */
+ release_console_sem();
}
schedule();
}
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply
* [PATCH 1/2] ps3fb: prevent use after free of fb_info
From: Geert Uytterhoeven @ 2008-01-11 13:27 UTC (permalink / raw)
To: Linus Torvalds, Andrew Morton
Cc: Linux/PPC Development, Linux Frame Buffer Device Development,
Jeremy Kerr, Cell Broadband Engine OSS Development
In-Reply-To: <Pine.LNX.4.64.0801111417450.12038@vixen.sonytel.be>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1869 bytes --]
From: Jeremy Kerr <jk@ozlabs.org>
In ps3fb_shutdown, freeing the framebuffer will cause fb_info (in
dev->core.driver_data) to be free()ed, which we potentially access
from the ps3fbd kthread.
This change frees the framebuffer after stopping the ps3fbd kthread.
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
---
drivers/video/ps3fb.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
--- a/drivers/video/ps3fb.c
+++ b/drivers/video/ps3fb.c
@@ -1234,12 +1234,6 @@ static int ps3fb_shutdown(struct ps3_sys
ps3fb_flip_ctl(0, &ps3fb); /* flip off */
ps3fb.dinfo->irq.mask = 0;
- if (info) {
- unregister_framebuffer(info);
- fb_dealloc_cmap(&info->cmap);
- framebuffer_release(info);
- }
-
ps3av_register_flip_ctl(NULL, NULL);
if (ps3fb.task) {
struct task_struct *task = ps3fb.task;
@@ -1250,6 +1244,12 @@ static int ps3fb_shutdown(struct ps3_sys
free_irq(ps3fb.irq_no, &dev->core);
ps3_irq_plug_destroy(ps3fb.irq_no);
}
+ if (info) {
+ unregister_framebuffer(info);
+ fb_dealloc_cmap(&info->cmap);
+ framebuffer_release(info);
+ info = dev->core.driver_data = NULL;
+ }
iounmap((u8 __iomem *)ps3fb.dinfo);
status = lv1_gpu_context_free(ps3fb.context_handle);
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply
* [PATCH 0/2] ps3fb fixes for 2.6.24
From: Geert Uytterhoeven @ 2008-01-11 13:26 UTC (permalink / raw)
To: Linus Torvalds, Andrew Morton
Cc: Linux/PPC Development, Linux Frame Buffer Device Development,
Jeremy Kerr, Cell Broadband Engine OSS Development
[-- Attachment #1: Type: TEXT/PLAIN, Size: 989 bytes --]
Hi Linus, Andrew,
Here are 2 more fixes for ps3fb:
[1] ps3fb: prevent use after free of fb_info
[2] ps3fb: fix deadlock on kexec()
The first one fixes a potential use after free.
The second one fixes a lockup when using kexec or shutdown while /dev/fb0 is
open (e.g. when using the petitboot graphical bootloader to load the second
stage kernel).
Can we please get these in 2.6.24? Thanks!
With kind regards,
Geert Uytterhoeven
Software Architect
Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium
Phone: +32 (0)2 700 8453
Fax: +32 (0)2 700 8622
E-mail: Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/
Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
^ permalink raw reply
* PPC vs POWERPC
From: samppa @ 2008-01-11 12:33 UTC (permalink / raw)
To: linuxppc-embedded
Hi,
I want to port a board ( WRS SBC8265 ) from the PPC branch to the POWERPC
branch in the Linux Kernel -- do you have any good starting points that
describes what I need to pay attention to?
I've already made a port of U-boot-1.3.1 and enabled CONFIG_OF_LIBFDT.
So I need to create a DTS file, compile it with the DTC compiler and port
the current PPC branch to the POWERPC branch to my understanding.
Cheers // Matias
^ permalink raw reply
* Re: Please pull linux-2.6-virtex.git
From: Josh Boyer @ 2008-01-11 13:09 UTC (permalink / raw)
To: Grant Likely; +Cc: linuxppc-dev
In-Reply-To: <fa686aa40801090703m5f411795j890bfa714a128b1e@mail.gmail.com>
On Wed, 9 Jan 2008 08:03:31 -0700
"Grant Likely" <grant.likely@secretlab.ca> wrote:
> Gah; teach me to pick up patches right before bed. I shouldn't have
> picked up the ulite console changes patch just yet. I've dropped it
> from the series until I have a chance to rework it. Sorry.
>
> Here's the new pull request
>
> The following changes since commit 4f43143f9fbbb679c38d2ff99e44d3aaa00d0fe1:
> Paul Mackerras (1):
> Merge branch 'for-2.6.25' of git://git.kernel.org/.../olof/pasemi
>
> are available in the git repository at:
>
> git://git.secretlab.ca/git/linux-2.6-virtex.git virtex-for-2.6.25
>
> Stephen Neuendorffer (5):
> [POWERPC] Xilinx: update compatible list for interrupt controller
> [POWERPC] Xilinx: Add correct compatible list for device tree
> bus bindings.
> [POWERPC] Xilinx: Update booting-without-of.
> [POWERPC] Xilinx: updated device tree compatibility to match
> uboot bsp generator.
> [POWERPC] Xilinx uartlite: Section type fixups
Pulled and pushed out. I'll ask Paul to pull after I finish chasing
the Warp patches Sean is working on, and the RTC patch from David.
josh
^ permalink raw reply
* Re: Linux for ml310
From: Enno Lübbers @ 2008-01-11 11:40 UTC (permalink / raw)
To: linuxppc-embedded
Hi Joachim,
I haven't tried the XUP or the ML310 yet, but I just managed to get
Linux 2.4.26 from Bitkeeper running on a ML403 board (a Virtex-4FX12).
However, I'm seriously considering moving to the 2.6 kernel, for
various reasons.
Your problems seem to be related to the BSP configuration. Have you
set the corerct options in the Software Platform Settings of the EDK?
In particular, you need to set the clock frequency and the attached
peripherals in the "OS and Libraries" section.
Regards
- Enno
--
Dipl.-Ing. Enno Luebbers
Computer Science Department
University of Paderborn
Warburger Str. 100
33098 Paderborn
http://wwwcs.upb.de/cs/ag-platzner
phone: 05251 / 60-5397
fax: 05251 / 60-5377
^ permalink raw reply
* Re: [PATCH 1/5] Warp Base Platform
From: Stephen Rothwell @ 2008-01-11 10:02 UTC (permalink / raw)
To: Sean MacLennan; +Cc: linuxppc-dev
In-Reply-To: <47871675.6030602@pikatech.com>
[-- Attachment #1: Type: text/plain, Size: 1703 bytes --]
Hi Sean,
On Fri, 11 Jan 2008 02:10:45 -0500 Sean MacLennan <smaclennan@pikatech.com> wrote:
>
> Stephen Rothwell wrote:
> > On Wed, 09 Jan 2008 15:19:13 -0500 Sean MacLennan <smaclennan@pikatech.com> wrote:
> >
> >> No comments? I really thought I would get raked over the coals for this one.
> >
> > Ah ha! A challenge! :-)
> >
> I hoped somebody would respond to that ;)
Glad to oblige.
> >> +static int pika_dtm_thread(void *fpga)
> >> +{
> >> + extern int ad7414_get_temp(int index);
> >
> > no externs in C code - put it in a header file.
>
> I didn't know where to put this extern. It is fairly specific to this
> driver, although a generic function... if that makes sense. It returns
> the exact contents of the register rather than doing any conversion.
>
> Any recommendations for a location gladly accepted.
I don't where that function is actually defined - I assume it is in one
of the other recent patch sets.
/me searches ...
/me reads "ad7414 driver" email ...
/me notes spaces missing there as well :-)
Tricky. You have something that can only be built in (pika_dtm_thread in
warp.c) calling something that might be a module ... I think you need to
think of a new way of organizing these pieces.
While I am looking, the return type of ioremap is "void __iomem *", so
you need to annotate your "fpga" variable appropriately and the parameter
to pika_dtm_thread.
> And below has all the above mentioned fixes, except the extern.
You seem to have forgotten some of the spaces after "if"s, "switch"s and
"while"s.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply
* [PATCH, ppc64] improve dedotify()
From: Jan Beulich @ 2008-01-11 9:13 UTC (permalink / raw)
To: paulus; +Cc: linuxppc-dev, linux-kernel
This completely untested patch is intended to be a suggestion only:
Code inspection for an entirely different purpose made me stumble
across this, and I think that modifying the string table of an ELF
object is a bad idea, since there's nothing disallowing a linker to
merge strings inside the table, which would result in this code
possibly, but unintentionally screwing up other symbol names.
Besides that, the presented alternative is both smaller and faster.
Signed-off-by: Jan Beulich <jbeulich@novell.com>
---
arch/powerpc/kernel/module_64.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
--- linux-2.6.24-rc7/arch/powerpc/kernel/module_64.c 2007-02-04 =
19:44:54.000000000 +0100
+++ 2.6.24-rc7-ppc64-dedotify/arch/powerpc/kernel/module_64.c 2008-01-08 =
13:32:33.000000000 +0100
@@ -154,16 +154,14 @@ static void dedotify_versions(struct mod
}
=20
/* Undefined symbols which refer to .funcname, hack to funcname */
-static void dedotify(Elf64_Sym *syms, unsigned int numsyms, char *strtab)
+static void dedotify(Elf64_Sym *syms, unsigned int numsyms, const char =
*strtab)
{
unsigned int i;
=20
for (i =3D 1; i < numsyms; i++) {
- if (syms[i].st_shndx =3D=3D SHN_UNDEF) {
- char *name =3D strtab + syms[i].st_name;
- if (name[0] =3D=3D '.')
- memmove(name, name+1, strlen(name));
- }
+ if (syms[i].st_shndx =3D=3D SHN_UNDEF
+ && strtab[syms[i].st_name] =3D=3D '.')
+ syms[i].st_name++;
}
}
=20
^ permalink raw reply
* Re: [PATCH] ps3: vuart: semaphore to mutex
From: Daniel Walker @ 2008-01-11 9:17 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: linuxppc-dev, mingo
In-Reply-To: <Pine.LNX.4.64.0801110919500.10759@vixen.sonytel.be>
On Fri, 2008-01-11 at 09:20 +0100, Geert Uytterhoeven wrote:
> On Thu, 10 Jan 2008, Daniel Walker wrote:
> > This probe_mutex conforms to the new struct mutex type.
> > This patch converts it from the old semaphore to the new struct mutex.
>
> The PS3 tree already has this change.
>
> With kind regards,
Ok, thanks..
Daniel
^ permalink raw reply
* Re: PCI Failed to allocate mem for PCI ROM
From: Jiri Slaby @ 2008-01-11 9:13 UTC (permalink / raw)
To: Kumar Gala; +Cc: Greg KH, linux-pci, LKML, linuxppc-dev list
In-Reply-To: <0119CD93-D9F1-42E0-9F6A-A50B2C63B19D@kernel.crashing.org>
On 01/11/2008 10:07 AM, Kumar Gala wrote:
>
> On Jan 11, 2008, at 2:41 AM, Jiri Slaby wrote:
>
>> On 01/11/2008 09:29 AM, Kumar Gala wrote:
>>> Greg,
>>> I'm getting the following message from the kernel on an embedded
>>> ppc32 system:
>>> PCI: Failed to allocate mem resource #9:100000@e0000000 for 0000:00:00.0
>>> The HW setup is a PCIe host controller and an e1000 NIC card. It
>>> appears that pci_bus_assign_resources() is trying to call
>>> pci_assign_resource() for the ROM and the resource for the ROM is
>>> [100000:1fffff] where the PHB is [c0000000:dfffffff].
>>> It seems like the resno that pci_assign_resource is getting called
>>> with is wrong and thus pci_update_resource() doesn't get called.
>>> any ideas?
>>
>> Kernel version, please.
>
> Sorry, its 2.6.24-rc7 + some ppc patches queued for 2.6.25
Could you try this patch?
http://git.kernel.org/?p=linux/kernel/git/gregkh/patches.git;a=blob_plain;f=pci/pci-remove-default-pci-expansion-rom-memory-allocation.patch
Greg: is this 2.6.25 material, please? We need this for SP2.
thanks,
--
Jiri Slaby
Faculty of Informatics, Masaryk University
Suse Labs
^ permalink raw reply
* Re: PCI Failed to allocate mem for PCI ROM
From: Kumar Gala @ 2008-01-11 9:07 UTC (permalink / raw)
To: Jiri Slaby; +Cc: Greg KH, linux-pci, LKML, linuxppc-dev list
In-Reply-To: <47872BC8.2040204@gmail.com>
On Jan 11, 2008, at 2:41 AM, Jiri Slaby wrote:
> On 01/11/2008 09:29 AM, Kumar Gala wrote:
>> Greg,
>> I'm getting the following message from the kernel on an embedded
>> ppc32 system:
>> PCI: Failed to allocate mem resource #9:100000@e0000000 for
>> 0000:00:00.0
>> The HW setup is a PCIe host controller and an e1000 NIC card. It
>> appears that pci_bus_assign_resources() is trying to call
>> pci_assign_resource() for the ROM and the resource for the ROM is
>> [100000:1fffff] where the PHB is [c0000000:dfffffff].
>> It seems like the resno that pci_assign_resource is getting called
>> with is wrong and thus pci_update_resource() doesn't get called.
>> any ideas?
>
> Kernel version, please.
Sorry, its 2.6.24-rc7 + some ppc patches queued for 2.6.25
- k
^ permalink raw reply
* Re: [i2c] [PATCH 0/5] Version 17, series to add device tree naming to i2c
From: Jean Delvare @ 2008-01-11 8:56 UTC (permalink / raw)
To: Jon Smirl; +Cc: linuxppc-dev, i2c, linux-kernel
In-Reply-To: <20071220044136.20091.70984.stgit@terra.home>
Hi Jon,
On Wed, 19 Dec 2007 23:41:36 -0500, Jon Smirl wrote:
> Since copying i2c-mpc.c to maintain support for the ppc architecture seems to be an issue; instead rework i2c-mpc.c to use CONFIG_PPC_MERGE #ifdefs to support both the ppc and powerpc architecture. When ppc is deleted in six months these #ifdefs will need to be removed.
>
> Another rework of the i2c for powerpc device tree patch. This version implements standard alias naming only on the powerpc platform and only for the device tree names. The old naming mechanism of i2c_client.name,driver_name is left in place and not changed for non-powerpc platforms. This patch is fully capable of dynamically loading the i2c modules. You can modprobe in the i2c-mpc driver and the i2c modules described in the device tree will be automatically loaded. Modules also work if compiled in.
>
> The follow on patch to module-init-tools is also needed since the i2c subsystem has never implemented dynamic loading.
>
> The following series implements standard linux module aliasing for i2c modules on arch=powerpc. It then converts the mpc i2c driver from being a platform driver to an open firmware one. I2C device names are picked up from the device tree. Module aliasing is used to translate from device tree names into to linux kernel names. Several i2c drivers are updated to use the new aliasing.
Now that I have read all the previous versions of this patch series
and, more importantly, all objections that were raised on the way, I
can start reviewing the latest iteration of your patches. I'll also do
some testing, although I have no powerpc stuff here, but at least I
want to make sure that there are no regressions introduced by your
patches on x86.
--
Jean Delvare
^ permalink raw reply
* Re: 2.6.22-ppc8xx fec.c bugs
From: raul.moreno @ 2008-01-11 8:51 UTC (permalink / raw)
To: Scott Wood; +Cc: linuxppc-embedded
In-Reply-To: <47865A80.7050205@freescale.com>
>>raul.moreno@telvent.abengoa.com wrote:
>> Hello everyone,
>>
>> I don't know who the maintainer of the FEC (Fast Ethernet Controller=
) in
>> the ppc8xx achitecture is, so I am writing this email here.
>arch/ppc is deprecated, and arch/ppc/8xx_io especially so.
>Please use drivers/net/fs_enet (and also note that arch/ppc will be
>going away in a few months).
I didn't know ppc was deprecated. Why is it so? Could you tell me where=
can
I read anything about this issue?
I have been using this architecture since I started with Linux (not too=
long ago) because I manage a mpc866 processor.
Do you mean that I should port to arch/powerpc?
>> I've seen that some bugs were fixed in the latest kernel but others
haven't
>> been addressed (possibly undetected yet). The following is a diff wi=
th
>> corrections I made to the 2.6.22 'fec.c' file (currently seems to wo=
rk
>> fine).
>Please post patches against the latest head-of-tree, and in unified di=
ff
>format (diff -u).
I'll do it in this way for next patches.
Ra=FAl
=
^ permalink raw reply
* Re: PCI Failed to allocate mem for PCI ROM
From: Jiri Slaby @ 2008-01-11 8:41 UTC (permalink / raw)
To: Kumar Gala; +Cc: Greg KH, linux-pci, LKML, linuxppc-dev list
In-Reply-To: <1B75C5A8-E512-40CB-A6F3-351640701D0D@kernel.crashing.org>
On 01/11/2008 09:29 AM, Kumar Gala wrote:
> Greg,
>
> I'm getting the following message from the kernel on an embedded ppc32
> system:
>
> PCI: Failed to allocate mem resource #9:100000@e0000000 for 0000:00:00.0
>
> The HW setup is a PCIe host controller and an e1000 NIC card. It
> appears that pci_bus_assign_resources() is trying to call
> pci_assign_resource() for the ROM and the resource for the ROM is
> [100000:1fffff] where the PHB is [c0000000:dfffffff].
>
> It seems like the resno that pci_assign_resource is getting called with
> is wrong and thus pci_update_resource() doesn't get called.
>
> any ideas?
Kernel version, please.
^ permalink raw reply
* PCI Failed to allocate mem for PCI ROM
From: Kumar Gala @ 2008-01-11 8:29 UTC (permalink / raw)
To: Greg KH; +Cc: linuxppc-dev list, linux-pci, LKML
Greg,
I'm getting the following message from the kernel on an embedded ppc32
system:
PCI: Failed to allocate mem resource #9:100000@e0000000 for 0000:00:00.0
The HW setup is a PCIe host controller and an e1000 NIC card. It
appears that pci_bus_assign_resources() is trying to call
pci_assign_resource() for the ROM and the resource for the ROM is
[100000:1fffff] where the PHB is [c0000000:dfffffff].
It seems like the resno that pci_assign_resource is getting called
with is wrong and thus pci_update_resource() doesn't get called.
any ideas?
- k
^ permalink raw reply
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