RE: [PATCH v2 01/49] IB/core: Add OPA Port header definitions

Hefty, Sean
2015-06-17T15:32:55Z

> >>> include/rdma/opa_port_info.h | 433
> >>
> >> This matches the current code structure, but is this the best location
> for this file?
> >
> > Do you have a suggestion?
>
> Couldn't it be a header in the hfi1 driver directory ? Isn't it OPA
> specific ?

I don't have a specific suggestion. If the contents are specific to this driver, I would move them into the driver directory. For values that are OPA, but not driver, specific, include/rdma is probably the best option for now. If the content is mixed (I didn't look closely), I would split the file.