From: Ameet Patil <ammubhai@gmail.com>
To: Ming Liu <eemingliu@hotmail.com>
Cc: linuxppc-embedded@ozlabs.org
Subject: Re: some problems on the SystemACE driver.
Date: Wed, 12 Jul 2006 10:54:13 +0100 [thread overview]
Message-ID: <44B4C6C5.1040007@gmail.com> (raw)
In-Reply-To: <BAY110-F784991A71896469EE9541B2680@phx.gbl>
Hi Ming,
> I heard that you have tested this driver. Have you got this problem?
> Why there are so many strange problems for me while you have tested
> without problem?
Yes, that is right! When I say... I have tested - "it really means I
have tested". So what's the problem? It works for me but not you? The
obvious difference: mine is a ML300 configuration and yours ML403.
There were some files which unknowing were made dependant on ML300
target. I have now made them compile for both targets. It should work
fine for you now (Hopefully!). Download the updated patch from the same
location.
http://www.cs.york.ac.uk/rtslab/demos/amos/xupv2pro/patches/linuxppc-2.6.17_sysace.patch
Since I don't have ML403 board, theres no way I can test this patch on
it. I rely on you in doing this... and thanks for letting me know the
issues.
WARNING: There might be more issues. :-)
Please DONOT hesitate to raise any issues with the driver. I am more
than happy to fix them.
-Ameet
Ming Liu wrote:
> Dear Ameet,
> Sorry to bother you again but I am totally confused on the systemACE
> driver. First let me show you the problem.
>
> 1. I downloaded the linux kernel of 2.6.17.1, also the patch for
> SystemACE driver. Applied the patch to the kernel. Replaced the
> xparameters_ml403.h with the generated file xparameters_ml300.h from
> Xilinx EDK. Make menuconfig, make dep and make zImage. Then the error
> shows like this:
>
> drivers/block/xilinx_sysace/xsysace.c:120:6: warning:
> "XPAR_XSYSACE_MEM_WIDTH" is not defined
> drivers/block/xilinx_sysace/xsysace.c: In function `XSysAce_LookupConfig':
> drivers/block/xilinx_sysace/xsysace.c:366: error:
> `XPAR_XSYSACE_NUM_INSTANCES' undeclared (first use in this function)
> drivers/block/xilinx_sysace/xsysace.c:366: error: (Each undeclared
> identifier is reported only once
> drivers/block/xilinx_sysace/xsysace.c:366: error: for each function it
> appears in.)
> make[3]: *** [drivers/block/xilinx_sysace/xsysace.o] Error 1
> make[2]: *** [drivers/block/xilinx_sysace] Error 2
> make[1]: *** [drivers/block] Error 2
> make: *** [drivers] Error 2
>
> I think this is because of the no inclusion of the xparameters header
> file. So I change #include "xparameters.h" into #include "
> /home/mingliu/linux-2.6.17.1/arch/ppc/platforms/4xx/xparameters/xparameters.h"
> in the files of xsysace.c and xsysace_g.c, using the full address to
> specify the header file. In fact, this is not a serious problem and it
> often happens. But, after the modification, another problem happened:
> GEN .version
> CHK include/linux/compile.h
> UPD include/linux/compile.h
> CC init/version.o
> LD init/built-in.o
> LD .tmp_vmlinux1
> drivers/built-in.o(.text+0x2234a): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x2235e): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x22364): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssert'
> drivers/built-in.o(.text+0x22372): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x2237a): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x22394): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssert'
> drivers/built-in.o(.text+0x223a2): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x223aa): In function `XSysAce_GetCfgAddr':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x22cd6): In function `XSysAce_Initialize':
> : undefined reference to `XAssertStatus'
> drivers/built-in.o(.text+0x22cdc): In function `XSysAce_Initialize':
> : undefined reference to `XAssert'
> drivers/built-in.o(.text+0x22cea): In function `XSysAce_Initialize':
> : undefined reference to `XAssertStatus'
>
> ......( a long information to say that undefined reference to the
> XAssert things.)
>
> Also, I tried this in the kernel 2.6.16-rc5. (In fact I prefer this
> version because the temac driver is for this version. ) The same problem
> happened. I checked the source code. The problem happened in the file
> driver/block/xilinx_sysace/adapter.c, etc. Also, the XAssert things are
> defined in the file arch/ppc/platforms/4xx/xilinx_ocp/xbasic_types.c.
> (In 2.6.16 kernel, it is also defined in
> driver/xilinx_edk/xbasic_types.c. There are two copies of this file. ) I
> think the problem is, the systemACE files cannot link together with the
> xbasic_types.c file.
> I heard that you have tested this driver. Have you got this problem? Why
> there are so many strange problems for me while you have tested without
> problem? Without the CF card, I cannot try the Temac driver and my work
> is totally blocked. So I have to ask for your suggestion. Really anxious
> for your useful guidance. Thanks a lot!!!!!!
>
> Regards
> Ming
>
> _________________________________________________________________
> 与联机的朋友进行交流,请使用 MSN Messenger: http://messenger.msn.com/cn
>
next prev parent reply other threads:[~2006-07-12 9:59 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-11 19:47 some problems on the SystemACE driver Ming Liu
2006-07-12 9:54 ` Ameet Patil [this message]
2006-07-12 15:55 ` Ming Liu
2006-07-12 18:22 ` Ameet Patil
2006-07-13 11:57 ` Ming Liu
[not found] <BAY110-F40A48564C0A5B30AB0DCCB2690@phx.gbl>
2006-07-13 16:16 ` Ameet Patil
2006-07-13 21:41 ` Ming Liu
2006-07-14 11:12 ` Ameet Patil
-- strict thread matches above, loose matches on Subject: below --
2006-07-14 13:09 Ming Liu
2006-07-14 15:10 ` Ameet Patil
2006-07-14 16:38 ` Ming Liu
2006-07-14 16:57 ` Ameet Patil
2006-07-14 21:06 Ming Liu
2006-07-16 9:40 ` Ameet Patil
2006-07-16 13:17 ` Ming Liu
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=44B4C6C5.1040007@gmail.com \
--to=ammubhai@gmail.com \
--cc=eemingliu@hotmail.com \
--cc=linuxppc-embedded@ozlabs.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).