Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751667AbdG0TKZ (ORCPT ); Thu, 27 Jul 2017 15:10:25 -0400 Received: from mail.kernel.org ([198.145.29.99]:41778 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751461AbdG0TKX (ORCPT ); Thu, 27 Jul 2017 15:10:23 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9863D22C97 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=robh+dt@kernel.org MIME-Version: 1.0 In-Reply-To: References: <1498441938-14046-1-git-send-email-hao.wu@intel.com> <1498441938-14046-3-git-send-email-hao.wu@intel.com> From: Rob Herring Date: Thu, 27 Jul 2017 14:10:01 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 02/22] fpga: add FPGA device framework To: Alan Tull Cc: Wu Hao , Moritz Fischer , linux-fpga@vger.kernel.org, linux-kernel , "linux-api@vger.kernel.org" , "Kang, Luwei" , "Zhang, Yi Z" , Tim Whisonant , Enno Luebbers , Shiva Rao , Christopher Rauer , "devicetree@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3247 Lines: 82 On Thu, Jul 27, 2017 at 11:35 AM, Alan Tull wrote: > On Sun, Jun 25, 2017 at 8:51 PM, Wu Hao wrote: > > Hi Rob, > > I was hoping to pick your brain a bit on a DT question. > >> During FPGA device (e.g PCI-based) discovery, platform devices are >> registered for different FPGA function units. But the device node path >> isn't quite friendly to applications. >> >> Consider this case, applications want to access child device's sysfs file >> for some information. >> >> 1) Access using bus-based path (e.g PCI) >> >> /sys/bus/pci/devices/xxxxx/fpga_func_a.0/sysfs_file >> >> From the path, it's clear which PCI device is the parent, but not perfect >> solution for applications. PCI device BDF is not fixed, application may >> need to search all PCI device to find the actual FPGA Device. >> >> 2) Or access using platform device path >> >> /sys/bus/platform/devices/fpga_func_a.0/sysfs_file >> >> Applications find the actual function by name easily, but no information >> about which fpga device it belongs to. It's quite confusing if multiple >> FPGA devices are in one system. > > There's a proposal for adding sysfs nodes that correspond to each FPGA > device., with the devices located on each FPGA under them. It makes > it easier to see which device is on which FPGA. Makes sense. >> 'FPGA Device' class is introduced to resolve this problem. Each node under >> this class represents a fpga device, which may have one or more child >> devices. Applications only need to search under this FPGA Device class >> folder to find the child device node it needs. >> >> For example, for the platform has 2 fpga devices, each fpga device has >> 3 child devices, the hierarchy looks like this. >> >> Two nodes are under /sys/class/fpga/: >> /sys/class/fpga/fpga.0 >> /sys/class/fpga/fpga.1 >> >> Each node has 1 function A device and 2 function B devices: >> /sys/class/fpga/fpga.0/func_a.0 >> /sys/class/fpga/fpga.0/func_b.0 >> /sys/class/fpga/fpga.0/func_b.1 >> >> /sys/class/fpga/fpga.1/func_a.1 >> /sys/class/fpga/fpga.1/func_b.2 >> /sys/class/fpga/fpga.1/func_b.3 A class is generally what is the function of the device, not how it is attached. Seems like what you want here is a new bus type if the existing PCI and platform bus types don't work. > > I can see the value of having sysfs nodes that correspond to fpga > devices and being able to find devices under them. I'm thinking what > that would mean for Device Tree when fpga-dev is used on DT enabled > systems. In Device Tree, what is a fpga-dev? Just properly setting the parent struct device on the functions should be enough to figure out which function is in which fpga. I don't see why a new class is needed. > Currently the DT would have a FPGA bridge corresponding to each FPGA's > hardware bridge and a heirarchy of bridges, regions and devices under > it. On systems that don't support partial reconfiguration under the > OS (so not main bridge that was controlled by the OS), there would be > a FPGA region, then its child regions, bridges, and devices. The FPGA bridges could instantiate fpga bus type devices instead of platform devices. That's really up to Linux and outside the scope of the bindings. Rob