From patchwork Tue Sep 30 04:51:53 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rajat Jain X-Patchwork-Id: 394759 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id C12411400BB for ; Tue, 30 Sep 2014 14:51:42 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750908AbaI3Evk (ORCPT ); Tue, 30 Sep 2014 00:51:40 -0400 Received: from mail-pd0-f178.google.com ([209.85.192.178]:43696 "EHLO mail-pd0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750780AbaI3Evj (ORCPT ); Tue, 30 Sep 2014 00:51:39 -0400 Received: by mail-pd0-f178.google.com with SMTP id y10so3953825pdj.9 for ; Mon, 29 Sep 2014 21:51:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :content-type:content-transfer-encoding; bh=vmblg1s04tVH2QvMPNZCFKlm4dECBsbm+HLPZ0XcEqc=; b=rla8eVWw8I9KdZdGT5A1+kotXmYqOBmuSpzgJt5qC0fq0kzSDkzQWum/8iECxpxNsn 6m1yVRZb8QACU5qQGKhGCfrNdYltMVwbjySPqyieHte5z9RhJcTomojc+T6wx/gCzJKm 8VAa4ke92KNhoO4/D4weY74hbxZ7d8XbiliSB9lhZ9f8zKfnKc2s+rM/xHskfFkqiQ6B 1Yg+asmYKgV0eVVMqYAXWQLkRPndPdFp4SjUU8/cosekQQ98oWSh70ixRAUIKec3/bOa l2L4gzDjylfbEiXkBaJMkMmzsDjWRB00PPQErfm4oimYtctmhiMmyV4OETJYNgDSNZwC xC7g== X-Received: by 10.68.215.106 with SMTP id oh10mr40133641pbc.98.1412052698654; Mon, 29 Sep 2014 21:51:38 -0700 (PDT) Received: from [192.168.95.133] ([66.129.239.11]) by mx.google.com with ESMTPSA id ve10sm13825941pbc.65.2014.09.29.21.51.37 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 29 Sep 2014 21:51:38 -0700 (PDT) Message-ID: <542A36E9.9020205@gmail.com> Date: Mon, 29 Sep 2014 21:51:53 -0700 From: Rajat Jain User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 MIME-Version: 1.0 To: Bjorn Helgaas , Jiri Kosina , Andrew Morton , "David S. Miller" , Greg Kroah-Hartman , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org, linux-doc@vger.kernel.org CC: rajatjain@juniper.net, groeck@juniper.net Subject: [PATCH 4/4] Documentation: Add documentation for the PCI switch PEX8xxx I2C driver Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org Signed-off-by: Rajat Jain Signed-off-by: Rajat Jain Signed-off-by: Guenter Roeck --- Documentation/PCI/pex8xxx_i2c.txt | 134 +++++++++++++++++++++++++++++++++++++ 1 file changed, 134 insertions(+) create mode 100644 Documentation/PCI/pex8xxx_i2c.txt diff --git a/Documentation/PCI/pex8xxx_i2c.txt b/Documentation/PCI/pex8xxx_i2c.txt new file mode 100644 index 0000000..9195242 --- /dev/null +++ b/Documentation/PCI/pex8xxx_i2c.txt @@ -0,0 +1,134 @@ +================================================================================ + The PEX8xxx I2C Interface driver + + Rajat Jain - Sep 2014 +================================================================================ + +0. Why have an I2C interface to a PCIe switch? +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +Other than the regular PCI express interface, most modern PCIe switches (e.g. +from IDT and PLX) have an I2C based secondary interface. This interface allows +access to all the registers of the switch. Some of these registers may not even +be accessible over the regular PCI interface. Also, there are certain registers +that can be written to, using only the I2C interface and may only be read-only +using the PCI interface. + +This I2C interface is often used in designs involving these switches, and can +be used for a variety of use cases where the switch needs to be configured +independent of the PCI subsystem (and likely before PCI enumeration). Some +examples: + +* Dividing a PCIe switch into multiple "virtual" switches. Using this feature, + a switch could be connected to 2 root ports for instance, each managing its + own PCI hierarchy, and the traffic from one virtual switch does not leak into + another. + +* Managing Transparent / Non-transparent bridging, and changing them on-the-fly. + There are ports that can be converted into "Non-transparent" bridge ports. + Essentially this is used to create different domains (not visible to + software). In a dynamic distributed system, it may be desirable to change a + transparent bridge to non-transparent or vice versa, for example, to handle a + failover situation. + +* Buggy hardware / Bad EEPROM configuration. There may be cases where an errata + involving register writes need to be applied before enumerating over PCI. + Also these switches are typically attached to an EEPROM that is supposed to + initialize the switch. If that EEPROM is not present, or contains bad + initialization data, this I2C interface can be used to fix that. + +* Changing switch configuration on the fly. In a multi-homed or complex + distributed systemsystem, there may be a need to change the switch + configuration (eg. change the upstream port, or the port or lane + configuration etc) to address run time scenarios (CPU plug out etc). + +1. What devices does this driver support? +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +PEX8xxx represents a family of PCI Express switches from the vendor PLX. +(http://www.plxtech.com/products/expresslane/switches). Currently this driver +supports the following PLX switch devices: +PEX8614 +PEX8618 +PEX8713 + +2. What does this "PEX8xxx I2C Interface driver" do? +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +This driver is an I2C client driver that allows talking to the said PEX8XXX +PCIe switches over the I2C interface. In a nutshell, it currently provides: + +* API calls to read / write the PEX8xxx switch device. +* A sysfs interface, to read / write the PEX8xxx switch device. + +The API calls are self explanatory (all reads / writes are 32 bit wide, but +the argument "byte_mask" can be used to selectively mask out the bytes): + +int pex8xxx_read(struct i2c_client *client, u8 stn, u8 mode, u8 byte_mask, + u8 port, u32 reg, u32 *val); +int pex8xxx_write(struct i2c_client *client, u8 stn, u8 mode, u8 byte_mask, + u8 port, u32 reg, u32 val) + +The arguments correspond to the arguments as described in the Chapter 7 +"I2C/SMBus Slave Interface Operation" of the all the switch datasheets. +http://www.plxtech.com/products/expresslane/pex8614#technicaldocumentation +http://www.plxtech.com/products/expresslane/pex8618#technicaldocumentation +http://www.plxtech.com/products/expresslane/pex8713#technicaldocumentation + +The sysfs interface is described in the next section. + +3. The PEX8xxx I2C driver sysfs Interface +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +The sysfs interface allows to read / write / dump the registers of any given +port of the pex8xxx switch. Note that all reads / writes are 32 bit wide. For +all pex8xxx devices, the following sysfs attributes are provided by this driver +in the directory /sys/bus/i2c/drivers/pex8xxx// + +* "port_num" (RW) - The port number whose registers are to be read / written. +* "reg_addr" (RW) - The register offset (within the port register space) that + is to be read / written. +* "reg_value"(RW) - When read, it gives the value of the register at offset + "reg_addr" in the port "port_num" of the switch. When + written, it writes the value to the same register. +* "port_config_regs" (RO) - A binary dump of the 4KB register address space of + the port "port_num". + +In addition, some devices (currently PEX8713) support and require additional +parameters, and hence these will appear for PEX8713 only currently: + +* "port_mode" (RW) - Denotes the port mode to use to talk to the switch. Valid + values are: "transparent" / "nt-link" / "nt-virtual" / + "dma". +* "port_stn" (RW) - Port Station number. + +Note all the attributes preserve their value unless explicitly changed. Thus if +port_num is set to 5, then all subsequent reads / writes will be directed to +that port unless the port_num attribute is explicitly changed. + +4. Examples using sysfs interface +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + +* Instantiate the device (refer "Documentation/i2c/instantiating-devices"). + Valid strings of device names for this driver are "pex8614", "pex8618" and + "pex8713". + Example (pex8614 at I2C address 0x38 on I2C bus 55): + echo pex8614 0x38 > /sys/bus/i2c/devices/i2c-55/new_device + +* Verify if device was instantiated: + cd /sys/bus/i2c/drivers/pex8xxx/55-0038 + cat name (should give pex8614) + +* Dump all registers of port number 4: + echo 4 > port_num + od -x port_config_regs + +* Get register values at offsets 0x230, 0x234, 0x238 of port number 1: + echo 1 > port_num + echo 0x230 > reg_addr + cat reg_value + echo 0x234 > reg_addr + cat reg_value + echo 0x238 > reg_addr + cat reg_value + +* Write value 0x101 at offset 0x530 of port number 5: + echo 5 > port_num + echo 0x530 > reg_addr + echo 0x101 > reg_value