Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751337AbdGaUZb (ORCPT ); Mon, 31 Jul 2017 16:25:31 -0400 Received: from mail.kernel.org ([198.145.29.99]:36316 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751052AbdGaUZa (ORCPT ); Mon, 31 Jul 2017 16:25:30 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4D4B422B5D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=helgaas@kernel.org Date: Mon, 31 Jul 2017 15:25:27 -0500 From: Bjorn Helgaas To: Logan Gunthorpe Cc: linux-ntb@googlegroups.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Jon Mason , Dave Jiang , Allen Hubbe , Bjorn Helgaas , Greg Kroah-Hartman , Kurt Schwemmer , Stephen Bates , Serge Semin Subject: Re: [PATCH v3 09/16] switchtec_ntb: initialize hardware for memory windows Message-ID: <20170731202527.GC9303@bhelgaas-glaptop.roam.corp.google.com> References: <20170725205753.4735-1-logang@deltatee.com> <20170725205753.4735-10-logang@deltatee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170725205753.4735-10-logang@deltatee.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 759 Lines: 20 On Tue, Jul 25, 2017 at 02:57:46PM -0600, Logan Gunthorpe wrote: > Add the code to initialize the memory windows in the hardware. > This includes setting up the requester ID table, and figuring out > which bar corresponds to which memory window. (Seeing the switch > can be configured with any number of bars.) s/bar/BAR/ (twice) > Also, seeing the device doesn't have hardware for scratchpads or > determining the link status, we create a shared memory window that has > these features. A magic number with a version copmonent will be used s/copmonent/component/ > to determine if the otherside's driver is actually up. s/otherside/other side/ > The shared memory window also informs the other side of the > size and count of the local memory windows.