Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-03-04 Thread Frederic Barrat
The point of this patch was to break a dependency with cxlflash, so that they could drop their powerVM changes independently from cxl. Since this hasn't hit upstream or next yet, change of plan: we've agreed with cxlflash that the cxl powerVM patchset will do the (small) modification to

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-22 Thread Frederic Barrat
Le 22/01/2016 01:38, Michael Neuling a écrit : On Thu, 2016-01-21 at 19:48 +0100, Frederic Barrat wrote: Le 20/01/2016 03:20, Michael Neuling a écrit : The only thing I'm a bit concerned about is are we going to end up duplicating a lot of the linux PCI API, but I guess we are only going to

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-21 Thread Frederic Barrat
Le 20/01/2016 03:20, Michael Neuling a écrit : The only thing I'm a bit concerned about is are we going to end up duplicating a lot of the linux PCI API, but I guess we are only going to do this for things the papr HCALL interface mimics. There are actually very few operations we can do on

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-21 Thread Michael Neuling
On Thu, 2016-01-21 at 19:48 +0100, Frederic Barrat wrote: > > Le 20/01/2016 03:20, Michael Neuling a écrit : > > The only thing I'm a bit concerned about is are we going to end up > > duplicating a lot of the linux PCI API, but I guess we are only going > > to do this for things the papr HCALL

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-19 Thread Michael Neuling
On Tue, 2016-01-19 at 18:14 +0100, Frederic Barrat wrote: > Introduce a new API to read the VPD of the adapter. In bare-metal, a > kernel driver can find out the adapter pci_dev behind the AFU device > and call pci_read_vpd() directly, but it won't work in a (powerVM) > guest. > Current

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-19 Thread Andrew Donnellan
On 20/01/16 04:14, Frederic Barrat wrote: Introduce a new API to read the VPD of the adapter. In bare-metal, a kernel driver can find out the adapter pci_dev behind the AFU device and call pci_read_vpd() directly, but it won't work in a (powerVM) guest. Current implementation is a stub to allow

Re: [PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-19 Thread Ian Munsie
Looks good :) Acked-by: Ian Munsie ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH] cxl: Add cxl_read_adapter_vpd() to the kernel API

2016-01-19 Thread Frederic Barrat
Introduce a new API to read the VPD of the adapter. In bare-metal, a kernel driver can find out the adapter pci_dev behind the AFU device and call pci_read_vpd() directly, but it won't work in a (powerVM) guest. Current implementation is a stub to allow existing drivers (cxlflash) to start using