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 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.