review ZC706 SDRAM size #22

New Issue

sb10q · 2020-04-26T16:51:56+08:00

sb10q commented

2020-04-26 16:51:56 +08:00

The code says 511MB:

    pub fn size(&self) -> usize {
        #[cfg(feature = "target_zc706")]
        let megabytes = 511;

But the manual says 1GB:

Shouldn't the code say 1023MB? (1MB missing because Zynq designers thought HIMEM.SYS was awesome?).

The code says 511MB: ```rust pub fn size(&self) -> usize { #[cfg(feature = "target_zc706")] let megabytes = 511; ``` But the [manual](https://www.xilinx.com/support/documentation/boards_and_kits/zc706/ug954-zc706-eval-board-xc7z045-ap-soc.pdf) says 1GB: ![image](/attachments/95b65128-cf25-4e61-9a05-469051fa5238) Shouldn't the code say 1023MB? (1MB missing because Zynq designers thought HIMEM.SYS was awesome?).

image.png

61 KiB

astro commented

2020-05-04 06:29:27 +08:00

Memtesting currently produces errors beyond 511 MB. We're still missing some bit of initialization...

astro was assigned by sb10q

2020-05-07 14:54:16 +08:00

astro commented

2020-05-09 08:37:06 +08:00

Finally found the culprit in 3e02980c20.

I wonder if this affects M-Labs/artiq-zynq#5 in any way.

Finally found the culprit in 3e02980c20429087145764c6a419ec4f68f96977. I wonder if this affects https://git.m-labs.hk/M-Labs/artiq-zynq/issues/5 in any way.

astro closed this issue

2020-05-09 08:37:08 +08:00

sb10q commented

2020-05-09 15:11:52 +08:00

Poster

I wonder if this affects M-Labs/artiq-zynq#5 in any way.

No, still the same.

> I wonder if this affects https://git.m-labs.hk/M-Labs/artiq-zynq/issues/5 in any way. No, still the same.

Sign in to join this conversation.

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/zynq-rs#22