Re: [PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-29 Thread Logan Gunthorpe
On 6/29/2017 12:11 PM, Allen Hubbe wrote: This could get in the way of letting the driver support more than two ports later on. Is there already a plan to change this to support more than two ports? Well, as I mentioned this patchset is only to support 2 ports. Future work will expand this

Re: [PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-29 Thread Logan Gunthorpe
On 6/29/2017 12:11 PM, Allen Hubbe wrote: This could get in the way of letting the driver support more than two ports later on. Is there already a plan to change this to support more than two ports? Well, as I mentioned this patchset is only to support 2 ports. Future work will expand this

RE: [PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-29 Thread Allen Hubbe
From: Logan Gunthorpe > Seeing there is no dedicated hardware for this, we simply add > these as entries in the shared memory window. Thus, we could support > any number of them but 128 seems like enough, for now. Scratchpads are not natively supported by this hardware, - but the hardware does

RE: [PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-29 Thread Allen Hubbe
From: Logan Gunthorpe > Seeing there is no dedicated hardware for this, we simply add > these as entries in the shared memory window. Thus, we could support > any number of them but 128 seems like enough, for now. Scratchpads are not natively supported by this hardware, - but the hardware does

[PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-28 Thread Logan Gunthorpe
Seeing there is no dedicated hardware for this, we simply add these as entries in the shared memory window. Thus, we could support any number of them but 128 seems like enough, for now. Signed-off-by: Logan Gunthorpe Reviewed-by: Stephen Bates

[PATCH 14/16] switchtec_ntb: implement scratchpad registers

2017-06-28 Thread Logan Gunthorpe
Seeing there is no dedicated hardware for this, we simply add these as entries in the shared memory window. Thus, we could support any number of them but 128 seems like enough, for now. Signed-off-by: Logan Gunthorpe Reviewed-by: Stephen Bates Reviewed-by: Kurt Schwemmer ---