Thread (62 messages) 62 messages, 4 authors, 2020-02-04

Re: [PATCH 01/15] devlink: prepare to support region operations

From: Jacob Keller <jacob.e.keller@intel.com>
Date: 2020-02-03 16:48:16


On 2/3/2020 3:35 AM, Jiri Pirko wrote:
Thu, Jan 30, 2020 at 11:58:56PM CET, jacob.e.keller@intel.com wrote:
quoted
-static const char *region_cr_space_str = "cr-space";
-static const char *region_fw_health_str = "fw-health";
Just leave these as are and use in ops and messages. It is odd to use
ops.name in the message.
If I recall, I tried this and it produced some issues with const. Will
confirm that and fix or explain in the commit message next round.

Thanks,
Jake
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help