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.