Tried live in action.

Any comments?

it seems more common through the tree to just use "version" here, e.g kern.version or hw.nvidia.version.

In D56110#1287717, @ziaee wrote:

it seems more common through the tree to just use "version" here, e.g kern.version or hw.nvidia.version.

That's a good point. What to do with the already existing name for mrsas? Add a new one and deprecate the old one in < 16?

I checked other Broadcom/AVAGO/LSI drivers (mpr, mpt). They all use "driver_version". So I think they should be either consistently "driver_version" or "version".

Good point. We should probably keep it consistent with those. I think aligning all of them to be more similar should probably be considered over the mailing list and in a different revision, if we decide to go down that route. Should we RELNOTES: yes or discuss this with others?

In D56110#1287730, @ziaee wrote:

Good point. We should probably keep it consistent with those. I think aligning all of them to be more similar should probably be considered over the mailing list and in a different revision, if we decide to go down that route. Should we RELNOTES: yes or discuss this with others?

I'd at least move it to a separate revision and keep it out of scope of this review. I'd raise a new review next week for all Broadcom/AVAGO/LSI drivers.

@ziaee, given the conditions: Is this one good as it is?

Guys, I'd like to complete this. Can someone review and consent?