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

store firmware version info into debugfs for logger usage #222

Merged
merged 3 commits into from
Nov 8, 2018
Merged

store firmware version info into debugfs for logger usage #222

merged 3 commits into from
Nov 8, 2018

Conversation

xiulipan
Copy link

@xiulipan xiulipan commented Oct 29, 2018

As discussed in thesofproject/soft#111
Stored fw version info into a debugfs to be used with logger
thesofproject/sof#512

Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix the checks on the status and error handling, thanks!

sound/soc/sof/loader.c Outdated Show resolved Hide resolved
sound/soc/sof/intel/bdw.c Show resolved Hide resolved
Store fw_version for logger to use

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
Create the debugfs to store the first boot fw verion info.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>
@xiulipan
Copy link
Author

xiulipan commented Nov 2, 2018

@plbossart
Rebased to top and fixed the

checks on the status and error handling

Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about the delay, I missed the update

@plbossart plbossart merged commit 6ff8e4d into thesofproject:topic/sof-dev Nov 8, 2018
@xiulipan xiulipan deleted the pr/fwverfs branch November 9, 2018 08:09
aiChaoSONG pushed a commit to aiChaoSONG/linux that referenced this pull request May 6, 2021
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

Successfully merging this pull request may close these issues.

2 participants