Skip to content

ASoC: SOF: Intel: SRAM_REG_FW mailbox update - #976

Closed
bkokoszx wants to merge 1 commit into
thesofproject:topic/sof-devfrom
bkokoszx:sram-reg-fw-mailbox-update
Closed

bkokoszx wants to merge 1 commit into
thesofproject:topic/sof-devfrom
bkokoszx:sram-reg-fw-mailbox-update

Conversation

@bkokoszx

Copy link
Copy Markdown

Update SRAM_REG mailbox due to change on FW side (tracing
ipc received and processed counters - thesofproject/sof#1441).

Signed-off-by: Bartosz Kokoszko bartoszx.kokoszko@linux.intel.com

Update SRAM_REG mailbox due to change on FW side (tracing
ipc received and processed counters).

Signed-off-by: Bartosz Kokoszko <bartoszx.kokoszko@linux.intel.com>
@bkokoszx bkokoszx changed the title ASoC: SOF: Intel: SRAM_REG mailbox update ASoC: SOF: Intel: SRAM_REG_FW mailbox update May 23, 2019
@plbossart

Copy link
Copy Markdown
Member

@bkokoszx there are only definitions here and no code, what is the intent here? To have someone else add kernel support? @jajanusz had a PR for this earlier, not sure I get the directions

@jajanusz

Copy link
Copy Markdown

@plbossart Yea, a bit of mess, I think we should stop doing commits and figure out what we wanna do with things like this -> thesofproject/sof#1441 (comment)

@plbossart plbossart added the Unclear No agreement on problem statement and resolution label Jun 11, 2019
@bkokoszx

Copy link
Copy Markdown
Author

@lgirdwood
Should we add debug SRAM_REG_FW_IPC_RECEIVED_COUNT and SRAM_REG_FW_IPC_PROCESSED_COUNT definitions in kernel since they are only conditional (debug option) on firmware?

@bkokoszx

Copy link
Copy Markdown
Author

@lgirdwood
Could you answer my question posted in my previous comment (#976 (comment)) in a meantime? I would like to complete/close this PR. In my humble opinion adding this mailboxes definitions are not needed (since they are optional in FW), but I might be wrong. Thanks

@plbossart

Copy link
Copy Markdown
Member

@bkokoszx this PR has not moved since July 30, please close or update the status

@bkokoszx bkokoszx closed this Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Unclear No agreement on problem statement and resolution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants