On 28/10/2016 18:33, Mark Brown wrote:
On Fri, Oct 28, 2016 at 04:32:50PM +0200, Sebastien Guiriec wrote:
+ dev_info(sst_drv_ctx->dev, "FW Version %02x.%02x.%02x.%02x\n",
+ init->fw_version.type, init->fw_version.major,
+ init->fw_version.minor, init->fw_version.build);
+ dev_dbg(sst_drv_ctx->dev, "Build date %s Time %s\n",
+ init->build_info.date, init->build_info.time);
Seems like an odd mix of info/dbg but whatever. I'd also recommend
adding sysfs or at least debugfs files exposing this information, that
way you can still get at it even if the kernel log has been filled with
other things.
Thansk Mark. I will remove the build time and date as the most important
info is the FW version in order to have traceability of the different
FW. I will add sysfs entry also.
Sebastien
---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris,
92196 Meudon Cedex, France
Registration Number: 302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros
This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
