Hi Akhil, You can squash the entire series. Or should I send a revised patch? Either way is fine.
Anoob > -----Original Message----- > From: Akhil Goyal <[email protected]> > Sent: 12 November 2018 16:51 > To: Joseph, Anoob <[email protected]>; Pablo de Lara > <[email protected]> > Cc: Joseph, Anoob <[email protected]>; Jacob, Jerin > <[email protected]>; Athreya, Narayana Prasad > <[email protected]>; Dwivedi, Ankur > <[email protected]>; [email protected] > Subject: Re: [dpdk-dev] [PATCH 2/3] crypto/octeontx: fix null pointer > dereferencing > > External Email > > Hi Anoob, > > On 11/11/2018 8:54 PM, Anoob Joseph wrote: > > Fixes: bfe2ae495ee2 ("crypto/octeontx: add PMD skeleton") > > > > Signed-off-by: Anoob Joseph <[email protected]> > > --- > > drivers/crypto/octeontx/otx_cryptodev.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/crypto/octeontx/otx_cryptodev.c > b/drivers/crypto/octeontx/otx_cryptodev.c > > index 269f045..b201e0a 100644 > > --- a/drivers/crypto/octeontx/otx_cryptodev.c > > +++ b/drivers/crypto/octeontx/otx_cryptodev.c > > @@ -100,8 +100,8 @@ otx_cpt_pci_remove(struct rte_pci_device *pci_dev) > > if (rte_eal_process_type() == RTE_PROC_PRIMARY) > > rte_free(cryptodev->data->dev_private); > > > > - cryptodev->device = NULL; > > cryptodev->device->driver = NULL; > > + cryptodev->device = NULL; > > cryptodev->data = NULL; > > > > /* free metapool memory */ > Can we squash the entire series into a single patch or at least 2/3 and > 3/3 should be merged. > You can have more than one fixes lines in a single patch. > > -Akhil
