Re: [PATCH] net/nfp: support 48-bit DMA address for firmware with NFDk

2023-02-16 Thread Niklas Soderlund
Hello again, On 2023-02-16 11:41:13 +0100, Chaoyong He wrote: > > > > -Original Message- > > From: Niklas Soderlund > > Sent: Thursday, February 16, 2023 6:37 PM > > To: Kevin Traynor > > Cc: Ferruh Yigit ; Xueming(Steven) Li > > ; Chaoyong

Re: [PATCH] doc: add supported APIs section to nfp guide

2023-02-08 Thread Niklas Soderlund
Hello Ajit, On 2023-01-24 09:13:20 +, Ferruh Yigit wrote: > On 1/24/2023 2:44 AM, Chaoyong He wrote: > >> On 12/1/2022 1:38 AM, Chaoyong He wrote: > >>> Add a new section 'Supported APIs', inculding the supported > >>> APIs/items/actions of rte_flow. > >>> > >> > >> Isn't this information alre

Re: no printf in drivers code

2023-02-06 Thread Niklas Soderlund
Hi Thomas, On 2023-02-03 14:57:06 +0100, Thomas Monjalon wrote: > Hello, > > We have too many drivers using printf() instead of rte_log(). > Please propose a plan to replace or remove the calls to printf(). > If no plan, I could propose one but you may not like it :) > > Affected drivers are: >

Re: FW: [PATCH v1] buildtools: ensure the NUMA nodes are counted correct

2022-09-20 Thread Niklas Soderlund
Hi Thomas, Have you checked if this address the same issue you where seeing? Do you think we can move forward with this fix? On 2022-08-31 10:47:24 +0200, Nole Zhang wrote: > > > > > -Original Message- > > From: Thomas Monjalon > > Sent: 2022年8月29日 21:15 > > To: Nole Zhang ; Chaoyong

Re: [PATCH v2 13/20] remove repeated word 'that'

2022-07-25 Thread Niklas Soderlund
Hi Stephen, Thanks for your work. On 2022-07-25 20:34:56 -0700, Stephen Hemminger wrote: > Found by doing duplicate word scan. > > Signed-off-by: Stephen Hemminger Reviewed-by: Niklas Söderlund > --- > drivers/net/nfp/nfp_ctrl.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > d

Re: [PATCH] net/nfp: remove unneeded header inclusion

2022-04-26 Thread Niklas Soderlund
Hi David, Thanks for your work. On 2022-04-08 11:41:16 +0200, David Marchand wrote: > Looking at this driver history, there was never a need for including > execinfo.h. > > Signed-off-by: David Marchand Reviewed-by: Niklas Söderlund > --- > drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c | 3 ---