From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4B01DC43381 for ; Mon, 18 Mar 2019 18:09:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 112C32133D for ; Mon, 18 Mar 2019 18:09:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DEI+8cXl" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726983AbfCRSJc (ORCPT ); Mon, 18 Mar 2019 14:09:32 -0400 Received: from mail-io1-f66.google.com ([209.85.166.66]:34646 "EHLO mail-io1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726623AbfCRSJc (ORCPT ); Mon, 18 Mar 2019 14:09:32 -0400 Received: by mail-io1-f66.google.com with SMTP id n11so15487113ioh.1 for ; Mon, 18 Mar 2019 11:09:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=wVsK2q9fVhn7KnAHh4OPa7X7MXz16yP43ek/dvK+DBo=; b=DEI+8cXlrAnEnp4O5s5YwdprQ5U/Mj0JZNgqMxcTwYcjd73MKELdlb/hjN6aunne8z geMXvytz0by9uWKAomjk7BFtN5cUZS/dWa2TsEaRYa/jCs5uGZ+hBu1MyheF9Olwy3iA odShMyeQTlB/1ciIX0oFah+/XdSHUZHLmPJONZq42MMY44WGL/n484eEyOduXiEtWJCL ff9pIChoU2RDzdToZ7mubsdpjSITfcEZWgb/+XKddspkkk6Hsf+uy2zp4IAC9PJCXGb8 nRl8GfSIoKvpJzCrrgX44tfccAEoBb61O9VoCQBwNwZ4vq6rSoqduODUumtfushiX8ew kW5g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=wVsK2q9fVhn7KnAHh4OPa7X7MXz16yP43ek/dvK+DBo=; b=oLvzBwrkcpU0nCaan6oXBIXlaBet6QxvEVGjJr5wnavu6RGymjiWgvgupGyuBteHzJ N1OHWYKUontJOVhuZTyZHHJX/cpRJ7sxqw7BW+oTEo+UMmVWaRKbb2DWdwe+zG1FGn4o hd3tiuJkzFnPYhkrDqHenLySSQj+j0X5O9/uqspofLYvNcCi0axFouiscErcW9UeGkyM 5za0rrWOTLXE02N+8M0gLxDBmcTo5dv0kLW1BG6iaNxIMlqlO26m9o+jLUwJQ9Jzd6NW 55kTUBmMuSkFkR6FiUJaC4JCKGWDGFYXPs5JXE5RQ2D/D57nYzU/SbpZdycyXhFBrfaA IHgQ== X-Gm-Message-State: APjAAAU8Hd8p14spKEaOZa6o2pA2ibkMisH65XL3uT08IIKX5VzPT5XO gYwJvWcddIeFcziBuAFuM2o1g8hH X-Google-Smtp-Source: APXvYqycYYTzWejzzPk+OH2I9aBe1pB+EK+dv14qOfn/GZimJyVRshpwDgVtwJG9txuBK+Dl3ZSNTw== X-Received: by 2002:a6b:7b08:: with SMTP id l8mr13416140iop.33.1552932570755; Mon, 18 Mar 2019 11:09:30 -0700 (PDT) Received: from svens-asus.arcx.com ([184.94.50.30]) by smtp.gmail.com with ESMTPSA id 134sm25374itx.29.2019.03.18.11.09.29 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 18 Mar 2019 11:09:30 -0700 (PDT) From: Sven Van Asbroeck X-Google-Original-From: Sven Van Asbroeck To: netdev@vger.kernel.org Subject: [PATCH v9 0/6] Add Fieldbus subsystem + support HMS Profinet card Date: Mon, 18 Mar 2019 14:09:21 -0400 Message-Id: <20190318180927.24620-1-TheSven73@gmail.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org This patch: 1. adds a Fieldbus subsystem 2. adds support for the HMS Industrial Networks AB Profinet card. 1. Fieldbus subsystem --------------------- Fieldbus device (client) adapters allow data exchange with a PLC aka. "Fieldbus Controller" over a fieldbus (Profinet, FLNet, etc.) They are typically used when a Linux device wants to expose itself as an actuator, motor, console light, switch, etc. over the fieldbus. The framework is designed to provide a generic interface to Fieldbus Devices from both the Linux Kernel and the userspace. 2. Add support for HMS Profinet Card ------------------------------------ Profinet is an industry technical standard for data communication over Industrial Ethernet, designed for collecting data from, and controlling, equipment in industrial systems, with a particular strength in delivering data under tight time constraints (on the order of 1ms or less). The profinet card itself is connected to the system via an industrial bus called 'anybus'. I have followed the bus driver/client driver pattern, and created an anybus bus driver, plus a client driver for the profinet card. In case this patch set gets (eventually) accepted, drivers for other anybus client cards may follow: flnet, cc-link, ... The anybus slot on the host is located on an 'anybus controller', which is custom h/w designed by arcx. It exposes a dual anybus host implementation, plus a power readout unrelated to the anybus. v9: Moved everything to staging/ so new userspace ABI can still be tweaked Suggested cross-posting to linux-netdev to widen audience Greg KH: call ida_destroy() on module exits replace arcx anybus controller's reset spinlock with mutex (no need for spinlock as never run in atomic context) udelay() can now be replaced with usleep_range() harmonized email addresses across patchset various whitespace changes to silence checkpatch.pl --strict v8: allow fieldbus core to be built as a module (define as tristate). use IS_ENABLED() in fieldbus_dev.h to conditionally compile dummy functions. anybuss_core.ko uses symbols from fieldbus_dev.ko. Use "top level kbuild file" method to make this work. See Documentation/kbuild/modules.txt, section 6.3 Confirm licence choice: GPL-v2 only. no changes. v7: use %zu printf formatter to display size_t v6: Randy Dunlap: correctly/consistently indent Kconfig spelling/punctuation improvements Greg KH: suggestion: simplify /sys/class/fieldbus_dev/fieldbus_devX/ -> /sys/class/fieldbus_dev/X/ can't simplify: create_device() also determines the name of the associated char device. simplification would result in a char device named /dev/X ? applied above suggestion to arcx-controller driver, this works ok as it does not have associated char device class name left as-is (fieldbus_dev) to differentiate between "fieldbus controller" and "fieldbus device" in the future makefile clean-up sysfs attributes: snprintf -> sprintf except card_name, as it's externally provided and could potentially be > PAGE_SIZE enabled/online sysfs attributes converted to "0/1" values using kstrtobool() v5: Greg KH: fix licence nit consistent use of e-mail address replaced raw sysfs attribute with kobject_uevent() call keep __ATTRIBUTE_GROUPS because we need is_visible field Rob Herring: controller devicetree entry now requires multiple address regions and interrupts controller child node(s) now mapped to child devices by slot/host position (using devicetree reg = <>) architectural overhaul to align closely to existing pci arch: pci anybus -------------------------------------------------------------------------------- Common bus code (spec), pci/pci_driver.c anybus/dev_core.c calls bus_register() no of compatible no of compatible -------------------------------------------------------------------------------- Device on bus, calls XXX_client_driver_register() hwmon/k8temp.c fieldbus/hms-profinet.c -------------------------------------------------------------------------------- Controller, silicon driver pci/pcie-tango.c anybus/arcx-anybus.c platform_driver calls platform_driver calls pci_host_probe() anybus_host_common_probe() v4: general create fieldbus_dev subsystem, with standardized 'simple' userspace interface (sysfs + cdev) remove redundant __packed keywords use __be16/32 types wherever values are explicitly big-endian hms-profinet: remove configuration code, and uapi headers: not supported (yet) when registering as a fieldbus_dev. anybuss-host: use struct kref to reference-count tasks replace busy loops with usleep_range() loop after 10 tries use threaded irq so time_before_eq(jiffies, timeout) will continue to work allow client devices to be assigned a devicetree node, in the same way as pci/mmc/... v3: devicetree-bindings adding the vendor prefix is now a separate commit anybus-bridge: moved misc driver to drivers/bus/ converted of_gpio_* to gpiod_* abstractions can power readout is now a fixed-voltage regulator anybuss-host: converted refcounts from atomic_t to refcount_t fixed potential use-after-free hms-profinet: applied minor kernel build robot suggestion v2: added architecture overview comments to host driver completely reworked anybus-bridge driver, it becomes a reset controller anybuss-host driver now needs devicetree entry, link to reset controller I will hold off on kernel-doc until the overall architecture gets more validation / approval fixed Kconfig, comment-style, document ioctl magic numbers removed redundant pwm dependency renamed enable-gpios to reset-gpios stop driving reset-gpio after unloading driver use interrupt-parent / interrupts method to describe interrupts in the devicetree convert references 'i.MX WEIM parallel bus' to 'parallel bus' replace devicetree functions with more generic platform_get_resource() platform_get_irq() added device unique data to add_device_randomness() v1: first shot Sven Van Asbroeck (6): fieldbus_dev: add Fieldbus Device subsystem. anybus-s: support HMS Anybus-S bus anybus-s: support the Arcx anybus controller dt-bindings: anybus-controller: document devicetree binding dt-bindings: Add vendor prefix for arcx / Archronix fieldbus_dev: support HMS Profinet IRT industrial controller Documentation/ABI/testing/fieldbus-dev-cdev | 31 + .../ABI/testing/sysfs-class-fieldbus-dev | 62 + .../fieldbus/arcx,anybus-controller.txt | 71 + .../devicetree/bindings/vendor-prefixes.txt | 1 + Documentation/fieldbus_dev/fieldbus_dev.txt | 66 + drivers/staging/Kconfig | 2 + drivers/staging/Makefile | 1 + drivers/staging/fieldbus/Kconfig | 18 + drivers/staging/fieldbus/Makefile | 7 + drivers/staging/fieldbus/anybuss/Kconfig | 39 + drivers/staging/fieldbus/anybuss/Makefile | 10 + .../staging/fieldbus/anybuss/arcx-anybus.c | 398 +++++ .../staging/fieldbus/anybuss/hms-profinet.c | 224 +++ drivers/staging/fieldbus/anybuss/host.c | 1459 +++++++++++++++++ drivers/staging/fieldbus/dev_core.c | 349 ++++ include/linux/anybuss-client.h | 102 ++ include/linux/anybuss-controller.h | 47 + include/linux/fieldbus_dev.h | 106 ++ 18 files changed, 2993 insertions(+) create mode 100644 Documentation/ABI/testing/fieldbus-dev-cdev create mode 100644 Documentation/ABI/testing/sysfs-class-fieldbus-dev create mode 100644 Documentation/devicetree/bindings/fieldbus/arcx,anybus-controller.txt create mode 100644 Documentation/fieldbus_dev/fieldbus_dev.txt create mode 100644 drivers/staging/fieldbus/Kconfig create mode 100644 drivers/staging/fieldbus/Makefile create mode 100644 drivers/staging/fieldbus/anybuss/Kconfig create mode 100644 drivers/staging/fieldbus/anybuss/Makefile create mode 100644 drivers/staging/fieldbus/anybuss/arcx-anybus.c create mode 100644 drivers/staging/fieldbus/anybuss/hms-profinet.c create mode 100644 drivers/staging/fieldbus/anybuss/host.c create mode 100644 drivers/staging/fieldbus/dev_core.c create mode 100644 include/linux/anybuss-client.h create mode 100644 include/linux/anybuss-controller.h create mode 100644 include/linux/fieldbus_dev.h -- 2.17.1