From: "Steven A. Falco" <sfalco@harris.com>
To: Stefan Roese <sr@denx.de>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: PPC440EPx on Sequoia: /proc/iomem acts weird
Date: Mon, 12 Nov 2007 14:25:29 -0500 [thread overview]
Message-ID: <4738A8A9.2080503@harris.com> (raw)
In-Reply-To: <200711101403.48792.sr@denx.de>
[-- Attachment #1: Type: text/plain, Size: 1670 bytes --]
First I will say that I don't understand resources well enough to
suggest a fix. But I have done a little poking around. In file
arch/powerpc/platforms/44x/ppc4xx-nand.c I see one "struct resource",
which is referenced by two "struct platform_device" items (ndfc_dev and
nand_dev).
In routine ppc4xx_setup_nand_node() we have two calls to
platform_device_register():
platform_device_register(&ndfc_dev);
platform_device_register(&nand_dev);
If I comment out the second one, then there is no loop in the resource
tree, and I can cat /proc/iomem just fine. If both calls are present,
then cat /proc/iomem loops forever.
So, just a wild guess - should there be two "struct resource"s, one for
each platform_device, or is there some other way to break the loop in
the tree?
Steve
Stefan Roese wrote:
> Hi Steve,
>
> On Friday 09 November 2007, Steven A. Falco wrote:
>
>> I am using the Denx 2.6.32 kernel, which does have powerpc/sequoia.
>> Xenomai is a real-time kernel built on Adeos/Ipipe. I'll dig into it
>> further.
>>
>
> Is this arch/ppc or arch/powerpc? I remember fixing this a while ago in
> arch/ppc:
>
> commit 67a35ce785b1d11d09bf528c166ea26d489a4bd6
> Author: Stefan Roese <sr@denx.de>
> Date: Thu Aug 2 14:15:22 2007 +0200
>
> ppc: Fix problem with recursive NDFC platform_device resource management
>
> This change fixes a problem with a resursive platform_device resource
> management of the AMCC 4xx NDFC. Without this fix a "cat /proc/iomem"
> leads to an infinite loop of printing the "ndfc-nand.0" resource.
>
> Signed-off-by: Stefan Roese <sr@denx.de>
>
> Best regards,
> Stefan
>
>
[-- Attachment #2: Type: text/html, Size: 2231 bytes --]
prev parent reply other threads:[~2007-11-12 19:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-09 18:30 PPC440EPx on Sequoia: /proc/iomem acts weird Steven A. Falco
2007-11-09 18:35 ` Josh Boyer
2007-11-09 18:46 ` Steven A. Falco
2007-11-10 13:03 ` Stefan Roese
2007-11-12 15:36 ` Steven A. Falco
2007-11-12 19:25 ` Steven A. Falco [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4738A8A9.2080503@harris.com \
--to=sfalco@harris.com \
--cc=linuxppc-dev@ozlabs.org \
--cc=sr@denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.