On 02/26/2018 07:27 PM, Javier Gonzalez wrote:

quoted

On 26 Feb 2018, at 19.24, Matias Bjørling [off-list ref] wrote:

On 02/26/2018 07:21 PM, Javier Gonzalez wrote:

quoted

quoted

On 26 Feb 2018, at 19.19, Matias Bjørling [off-list ref] wrote:

On 02/26/2018 02:16 PM, Javier González wrote:

quoted

# Changes since V2:
Apply Matias' feedback:
  - Remove generic nvm_id identify structure.
  - Do not remap capabilities (cap) to media and controlled capabilities
    (mccap). Instead, add a comment to prevent confusion when
    crosschecking with 2.0 spec.
  - Change maxoc and maxocpu defaults from 1 block to the max number of
    blocks.
  - Re-implement the generic geometry to use nvm_geo on both device and
    targets. Maintain nvm_common_geo to make it easier to copy the common
    part of the geometry (without having to overwrite target-specific
    fields, which is ugly and error prone). Matias, if you still want to
    get rid of this, we can do it.

I do, the variables should go directly in nvm_geo. Thanks.

Ok. Is the rest ok with you?

I'll go through it when the rebase is posted. Most of the patches is
dependent on the first patch.

As it is now, it will be basically %s/geo->c.X/geo->X/g. If you can look
at the first patch now, I can put all comments in a single version more,
instead of going through one extra cycle...

Also the variable dev_geo -> geo. I'm good with just getting the updated 
version before posting the full patch set again.