Modify

Opened 7 years ago

Last modified 4 years ago

#9388 assigned defect

improve lantiq nor code (lantiq.c)

Reported by: anonymous Owned by: blogic
Priority: high Milestone: Barrier Breaker 14.07
Component: kernel Version: Trunk
Keywords: Cc:

Description

Hi All,

Lantiq xRX CPUs seems to have a bug on not aligned access to EBU bus (and flash...). For this reason the standard memcpy cannot used because, for optimization purpose, not aligned access can be uses (using lwl and lwr MIPS opcodes). The bus exception is also delayed... making the software debug hard.

The proposed patch fix the lq_copy_to function and use a 32-copy way to have better performance. The pointers are aligned in order to have the access to flash aligned.

Thanks to Lauterbach for the best jtag debugger in the history...

Attachments (3)

lantiq_nor.patch (4.7 KB) - added by luigi.mantellini@… 7 years ago.
Lantiq NOR fixes
add_ebu_adswp.patch (374 bytes) - added by luigi.mantellini@… 7 years ago.
Define EBU_ADSWP symbol
add_ebu_adswp.2.patch (374 bytes) - added by luigi.mantellini@… 7 years ago.
Define EBU_ADSWP symbol

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by luigi.mantellini@…

Lantiq NOR fixes

comment:1 Changed 7 years ago by anonymous

Error in make:

 CC      drivers/mtd/maps/lantiq.o
drivers/mtd/maps/lantiq.c: In function 'lq_mtd_probe':
drivers/mtd/maps/lantiq.c:202:32: error: 'EBU_ADSWP' undeclared (first use in this function)
drivers/mtd/maps/lantiq.c:202:32: note: each undeclared identifier is reported only once for each function it appears in
make[8]: *** [drivers/mtd/maps/lantiq.o] Error 1
make[7]: *** [drivers/mtd/maps] Error 2
make[6]: *** [drivers/mtd] Error 2
make[5]: *** [drivers] Error 2

Changed 7 years ago by luigi.mantellini@…

Define EBU_ADSWP symbol

Changed 7 years ago by luigi.mantellini@…

Define EBU_ADSWP symbol

comment:2 Changed 7 years ago by luigi.mantellini@…

Hi,

I'm sorry, I forgot to add a patch to xway.h

Please add

#define EBU_ADSWP 0x40000000

to your sources.

best regards,

luigi

comment:3 Changed 7 years ago by anonymous

hi,

next time please put patches onto the mailing list. this was not even noticed as i dont spent time browsing trac tickets

blogic

comment:4 Changed 5 years ago by florian

  • Owner changed from developers to blogic
  • Status changed from new to assigned

comment:5 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as assigned .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.