Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change the pciex8 to x4 which cased a firmware problem #7

Open
preta21 opened this issue Mar 27, 2023 · 1 comment
Open

change the pciex8 to x4 which cased a firmware problem #7

preta21 opened this issue Mar 27, 2023 · 1 comment

Comments

@preta21
Copy link

preta21 commented Mar 27, 2023

Hello, I'm using NVMeCHA to adjust to my FPGA.
Adjusted xdc, In PCIe x8, it works fine,
However, the controller doesn't work after I changed PCIe Interface -> lane width to x4 in XDMA IP.
image

I got these logs from uart

[12:16:59:577] ======NVMe Test Start========␊
[12:16:59:612] ␍NVMe Reg CC_EN is set␊
[12:17:13:792] ␍PS Set CSTS RDY ␊
[12:17:13:811] ␍Get cmd:␍␊
[12:17:13:884] E0006 0 0 0 0 0 0 0 0 0 7C2B0000 4 0 0 0 0 ␊
[12:17:13:931] Admin Command id is 14 ␍␊
[12:17:13:956] Admin Command(Identify), cid: 14, nsid: 0, cns: 0x0␊
[12:17:14:009] ␍Get cmd:␍␊
[12:17:14:021] 0 0 1 0 0 0 0 0 0 0 0 0 0 0 0 0 ␊
[12:17:14:056] Admin Command id is 0 ␍␊
[12:17:14:081] Admin Command(Delete IO SQ), cid: 0, nsid: 0, qid: ======NVMe Test Start========␊
[12:17:14:166] ␍

It seems that the controller only received one Identify(Namespace data structure, cns=0x00) admin command, and shutdown.
(the host deletes SQ; CC_EN and CSTS_RDY are not set.)

I modified the IP configure, and also changed the PL_LINK_CAP_MAX_LINK_WIDTH to 4 in nvme_top.v.
So, is there anyting else I need to do when make a PCIe lane width modification?

Sincerely.

@yhqiu16
Copy link
Owner

yhqiu16 commented Mar 27, 2023

In your case, the data width will change from 256 to 128 bits, so you should adjust the interfaces in the data path accordingly. The PL_LINK_CAP_MAX_LINK_WIDTH is not a global parameter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants