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.

Sign up for GitHub

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

Jump to bottom

[spi_device] Read Status, Read JEDEC ID, Read SFDP to have fixed value (read-only) in CMD_INFO #10009

Open
eunchan opened this issue Jan 11, 2022 · 4 comments
Open

[spi_device] Read Status, Read JEDEC ID, Read SFDP to have fixed value (read-only) in CMD_INFO #10009

eunchan opened this issue Jan 11, 2022 · 4 comments
Assignees
@a-will
Labels
Component:RTL Component:Software Issue related to Software Earlgrey-PROD Candidate Temporary label to triage issues into Earlgrey-PROD Milestones IP:spi_device Priority:P2 Priority: medium Type:Enhancement Feature requests, enhancements Type:FutureRelease Not relevant to currently planned releases/milestones
Milestone
Backlog

Comments

@eunchan
Copy link
Contributor

eunchan commented Jan 11, 2022

As same as EN4B / EX4B , Read Status (1,2,3) , Read JEDEC ID, Read SFDP commands have fixe values in their command information entries. It is better to split out those entries into new CSRs like CMD_INFO_EN4B and only valid and opcode to be modifiable by SW.

Related #9974
@eunchan eunchan self-assigned this Jan 11, 2022
@eunchan eunchan mentioned this issue Jan 11, 2022
Closed
53 tasks
@eunchan eunchan added Component:RTL Component:Software Issue related to Software Priority:P2 Priority: medium Type:Enhancement Feature requests, enhancements labels Jan 13, 2022
@tjaychen
Copy link

is this one still open @eunchan ?
It kind of seems like it probably falls into future release category also

@eunchan eunchan added the Type:FutureRelease Not relevant to currently planned releases/milestones label May 11, 2022
@andreaskurth
Copy link
Contributor

Triaged for spi_device. I think this is not necessary for this release and thus correctly labeled Type:FutureRelease Not relevant to currently planned releases/milestones .

@msfschaffner msfschaffner added the Earlgrey-PROD Candidate Temporary label to triage issues into Earlgrey-PROD Milestones label Oct 6, 2023
@msfschaffner
Copy link
Contributor

CC @a-will

@msfschaffner msfschaffner added this to the Earlgrey-PROD.M2 milestone Nov 3, 2023
@msfschaffner
Copy link
Contributor

Discussed with @a-will, this is a nice to have and is not required from a functionality standpoint.

@antmarzam antmarzam mentioned this issue Feb 27, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@a-will a-will

Labels
Component:RTL Component:Software Issue related to Software Earlgrey-PROD Candidate Temporary label to triage issues into Earlgrey-PROD Milestones IP:spi_device Priority:P2 Priority: medium Type:Enhancement Feature requests, enhancements Type:FutureRelease Not relevant to currently planned releases/milestones
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
5 participants
@a-will @tjaychen @eunchan @andreaskurth @msfschaffner