From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lino Sanfilippo Subject: Re: [PATCH net-next v3] Add support of Cavium Liquidio ethernet adapters Date: Fri, 19 Dec 2014 22:07:03 +0100 Message-ID: <54949377.5010009@gmx.de> References: <1418959519-31681-1-git-send-email-rvatsavayi@caviumnetworks.com> <54948E1F.2080405@gmx.de> <20141219125434.594f4d46@urahara> <549491D2.1090504@gmx.de> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: Raghu Vatsavayi , davem@davemloft.net, netdev@vger.kernel.org, Derek Chickles , Satanand Burla , Felix Manlunas , Raghu Vatsavayi To: Stephen Hemminger Return-path: Received: from mout.gmx.net ([212.227.15.18]:54969 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751414AbaLSVHJ (ORCPT ); Fri, 19 Dec 2014 16:07:09 -0500 In-Reply-To: <549491D2.1090504@gmx.de> Sender: netdev-owner@vger.kernel.org List-ID: On 19.12.2014 22:00, Lino Sanfilippo wrote: > On 19.12.2014 21:54, Stephen Hemminger wrote: >> On Fri, 19 Dec 2014 21:44:15 +0100 >> Lino Sanfilippo wrote: >> >>> > + OCTEON_PCI_WIN_READ(oct, CN66XX_CIU_SOFT_RST); >>> > + OCTEON_PCI_WIN_WRITE(oct, CN66XX_CIU_SOFT_RST, 1); >> >> Missing a following PCI read to force PCI posting? >> > > Yes, I think so. I thought posting means "writes are delayed", so I wrote it should be avoided (instead of forced). However I think there should be a PCI read before the call to mdelay. Regards, Lino