Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp56833imm; Wed, 29 Aug 2018 13:51:26 -0700 (PDT) X-Google-Smtp-Source: ANB0VdbSdT1TuSAGdYqYxckpG9yACw63cfCK0NwHqbe1D0v8cDGFI/xYxnYMbTvzx/ApE+4f/sgB X-Received: by 2002:a63:b44c:: with SMTP id n12-v6mr2177961pgu.337.1535575886266; Wed, 29 Aug 2018 13:51:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1535575886; cv=none; d=google.com; s=arc-20160816; b=wTOlwQ8kRfdJ0IyxfWs8o8HCgIzOJf/jMYEyZBz2747Hs2qP77GaidR8Qnn+Bk/ZzD IGFJ33ywcELKbmVMcoz+o8YrWmhywk6A9QuioOk9PFOP1giji3uHRY3eFgexSmtBzbTT 9hGTjM6XjFOq2ax3nZaq3aleQJ+mdVcSLPY+IYM24AfJPm/7MO7xujlp1D3XmOZEJtJm bDkxvgpi1++a3ERmwejN/S4VEGXfmSNLg9EHgjSk2WHFsFSGRAh1mvJ4Srqytoq5KAJA ZzaiVjgm6pShIzMsxnJMJKpeiEeJ0UN5zULoE9aA86Sq4rb1TGOupAa6Q61lJ8PiAw6s el8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature:arc-authentication-results; bh=Ss6DAMy39ntUSy+xns54ZkDEt4V3I8DvbWAXkdjX6Cw=; b=CpW6Rade5Y6YrYAhwl4vNjyikw1WY9ZWA1n3POyNb+ENKgPZdP2kft2R3JW7I7l/ms O6sllOUUB3fPmIISFb13LIwQRhMSQ19sQK5AUAhnrSKidBH+hAeMe3JHFsIdNg2OjL6U j1T8c+ddlg1Di6aLFF9H+JHXbgQ7ZJn4q5kQOVTFBrTtoCS2LFSOhkmZLcNwQ1aDElI6 BqjbCktb1ea37jY9HNGxQsGznbXQzJ5jJJ/T41EoBjbPikrIktFPumXF6iipxjDNkk/B O8uuyUQXeQ/c+mq4A7b1qopxh7bIeVcfWAFIosXlM6obbL8KTBAPqmJm9Whcpr6Oiy5F uWDw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=pBw95ZWm; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j11-v6si4366045plk.513.2018.08.29.13.51.11; Wed, 29 Aug 2018 13:51:26 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=pBw95ZWm; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729046AbeH3Ari (ORCPT + 99 others); Wed, 29 Aug 2018 20:47:38 -0400 Received: from mail-lj1-f195.google.com ([209.85.208.195]:34781 "EHLO mail-lj1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727664AbeH3ArG (ORCPT ); Wed, 29 Aug 2018 20:47:06 -0400 Received: by mail-lj1-f195.google.com with SMTP id f8-v6so5497485ljk.1; Wed, 29 Aug 2018 13:48:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=Ss6DAMy39ntUSy+xns54ZkDEt4V3I8DvbWAXkdjX6Cw=; b=pBw95ZWmCh/zCe3F646sOePMgphT8gyVQJxYaXzxsIZVgZEtDpHAbdNsdczT6ITIZl ZxBlnupZ4G/dgw+e/BQon3Izo+poYV2czPhxb4TW9KqPXN/PffuTy/XYKxccBm/GuJwP fIKrjfowEO1UolbY6VUlrKmRpWDmODZSucpbTsAPxXjorMY3z0eflJiCbJcXcQyKrMNR 4+ZAEeAgUdOgXn0OPpFAOdQmIdB7Zs4vGy+gHGhNHFdAIwP/XHQxX3Vq8Es7tdzERr3/ m9fzo+gKws03zv5Txn2+ZI0yVPI1otcJkHT5D6MdgME+uNsV0oGRIpqNDGX2/tnNd/oF 9rcA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=Ss6DAMy39ntUSy+xns54ZkDEt4V3I8DvbWAXkdjX6Cw=; b=K6jJSPtC39Zb81GKWxa/eDQf17R5VYtjq6uJMI1G7WCdbS4HOW01OmLLiBhPUggFH6 Mh6wiHQwMf0IbqQvSM0Rox06jKhzOv8QuRaL8YCn9rLjIf/JdcMvRybaVGPaE2v+iInA Dw6vcRzXcgCnto+gcs/3ZOZmhPc0Ps1tT7adAVUXXbUyH0+w/7N/IDLi0DCVD6dd01Is 08zkv6dRINq5qmTpBT9Lqwe+6OYhQFjz63sceFziqeUzK9gwlOXvIzsWryqhDSA53jAV Lm4e8P/8BmWOm6S2P1XCioztRl+T9sTbGZ9RiCn8poThHUUotKFg139nQTMJbthH2Ajc UKPQ== X-Gm-Message-State: APzg51AB1/4RkzdGpm65LawiURBRg984S3ZFgshCPNSsZxEGZmhwk/kx Wyew11RC2RRLWHWSA4N8BFM= X-Received: by 2002:a2e:4557:: with SMTP id s84-v6mr5200071lja.47.1535575706812; Wed, 29 Aug 2018 13:48:26 -0700 (PDT) Received: from z50.lan (93-181-165-181.internetia.net.pl. [93.181.165.181]) by smtp.gmail.com with ESMTPSA id f66-v6sm947619lji.23.2018.08.29.13.48.24 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 29 Aug 2018 13:48:26 -0700 (PDT) From: Janusz Krzysztofik To: Linus Walleij Cc: Jonathan Corbet , Miguel Ojeda Sandonis , Peter Korsgaard , Peter Rosin , Ulf Hansson , Andrew Lunn , Florian Fainelli , "David S. Miller" , Dominik Brodowski , Greg Kroah-Hartman , Kishon Vijay Abraham I , Lars-Peter Clausen , Michael Hennerich , Jonathan Cameron , Hartmut Knaack , Peter Meerwald-Stadler , Jiri Slaby , Willy Tarreau , Geert Uytterhoeven , linux-doc@vger.kernel.org, linux-i2c@vger.kernel.org, linux-mmc@vger.kernel.org, netdev@vger.kernel.org, linux-iio@vger.kernel.org, devel@driverdev.osuosl.org, linux-serial@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Janusz Krzysztofik Subject: [PATCH v5 2/4] gpiolib: Identify arrays matching GPIO hardware Date: Wed, 29 Aug 2018 22:48:58 +0200 Message-Id: <20180829204900.19390-3-jmkrzyszt@gmail.com> X-Mailer: git-send-email 2.16.4 In-Reply-To: <20180829204900.19390-1-jmkrzyszt@gmail.com> References: <20180820234341.5271-1-jmkrzyszt@gmail.com> <20180829204900.19390-1-jmkrzyszt@gmail.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Certain GPIO array lookup results may map directly to GPIO pins of a single GPIO chip in hardware order. If that condition is recognized and handled efficiently, significant performance gain of get/set array functions may be possible. While processing a request for an array of GPIO descriptors, identify those which represent corresponding pins of a single GPIO chip. Skip over pins which require open source or open drain special processing. Moreover, identify pins which require inversion. Pass a pointer to that information with the array to the caller so it can benefit from enhanced performance as soon as get/set array functions can accept and make efficient use of it. Cc: Jonathan Corbet Signed-off-by: Janusz Krzysztofik --- Documentation/driver-api/gpio/consumer.rst | 4 +- drivers/gpio/gpiolib.c | 72 +++++++++++++++++++++++++++++- drivers/gpio/gpiolib.h | 9 ++++ include/linux/gpio/consumer.h | 9 ++++ 4 files changed, 92 insertions(+), 2 deletions(-) diff --git a/Documentation/driver-api/gpio/consumer.rst b/Documentation/driver-api/gpio/consumer.rst index ed68042ddccf..7e0298b9a7b9 100644 --- a/Documentation/driver-api/gpio/consumer.rst +++ b/Documentation/driver-api/gpio/consumer.rst @@ -109,9 +109,11 @@ For a function using multiple GPIOs all of those can be obtained with one call:: enum gpiod_flags flags) This function returns a struct gpio_descs which contains an array of -descriptors:: +descriptors. It also contains a pointer to a gpiolib private structure which, +if passed back to get/set array functions, may speed up I/O proocessing:: struct gpio_descs { + struct gpio_array *info; unsigned int ndescs; struct gpio_desc *desc[]; } diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c index f0e9ffa8cab6..c1ed1c759345 100644 --- a/drivers/gpio/gpiolib.c +++ b/drivers/gpio/gpiolib.c @@ -4174,7 +4174,9 @@ struct gpio_descs *__must_check gpiod_get_array(struct device *dev, { struct gpio_desc *desc; struct gpio_descs *descs; - int count; + struct gpio_array *array_info = NULL; + struct gpio_chip *chip; + int count, bitmap_size; count = gpiod_count(dev, con_id); if (count < 0) @@ -4190,9 +4192,77 @@ struct gpio_descs *__must_check gpiod_get_array(struct device *dev, gpiod_put_array(descs); return ERR_CAST(desc); } + descs->desc[descs->ndescs] = desc; + + chip = gpiod_to_chip(desc); + /* + * Select a chip of first array member + * whose index matches its pin hardware number + * as a candidate for fast bitmap processing. + */ + if (!array_info && gpio_chip_hwgpio(desc) == descs->ndescs) { + struct gpio_descs *array; + + bitmap_size = BITS_TO_LONGS(chip->ngpio > count ? + chip->ngpio : count); + + array = kzalloc(struct_size(descs, desc, count) + + struct_size(array_info, invert_mask, + 3 * bitmap_size), GFP_KERNEL); + if (!array) { + gpiod_put_array(descs); + return ERR_PTR(-ENOMEM); + } + + memcpy(array, descs, + struct_size(descs, desc, descs->ndescs + 1)); + kfree(descs); + + descs = array; + array_info = (void *)(descs->desc + count); + array_info->get_mask = array_info->invert_mask + + bitmap_size; + array_info->set_mask = array_info->get_mask + + bitmap_size; + + array_info->desc = descs->desc; + array_info->size = count; + array_info->chip = chip; + bitmap_set(array_info->get_mask, descs->ndescs, + count - descs->ndescs); + bitmap_set(array_info->set_mask, descs->ndescs, + count - descs->ndescs); + descs->info = array_info; + } + /* + * Unmark members which don't qualify for fast bitmap + * processing (different chip, not in hardware order) + */ + if (array_info && (chip != array_info->chip || + gpio_chip_hwgpio(desc) != descs->ndescs)) { + __clear_bit(descs->ndescs, array_info->get_mask); + __clear_bit(descs->ndescs, array_info->set_mask); + } else if (array_info) { + /* Exclude open drain or open source from fast output */ + if (gpiochip_line_is_open_drain(chip, descs->ndescs) || + gpiochip_line_is_open_source(chip, descs->ndescs)) + __clear_bit(descs->ndescs, + array_info->set_mask); + /* Identify 'fast' pins which require invertion */ + if (gpiod_is_active_low(desc)) + __set_bit(descs->ndescs, + array_info->invert_mask); + } + descs->ndescs++; } + if (array_info) + dev_dbg(dev, + "GPIO array info: chip=%s, size=%d, get_mask=%lx, set_mask=%lx, invert_mask=%lx\n", + array_info->chip->label, array_info->size, + *array_info->get_mask, *array_info->set_mask, + *array_info->invert_mask); return descs; } EXPORT_SYMBOL_GPL(gpiod_get_array); diff --git a/drivers/gpio/gpiolib.h b/drivers/gpio/gpiolib.h index 11e83d2eef89..b60905d558b1 100644 --- a/drivers/gpio/gpiolib.h +++ b/drivers/gpio/gpiolib.h @@ -183,6 +183,15 @@ static inline bool acpi_can_fallback_to_crs(struct acpi_device *adev, } #endif +struct gpio_array { + struct gpio_desc **desc; + unsigned int size; + struct gpio_chip *chip; + unsigned long *get_mask; + unsigned long *set_mask; + unsigned long invert_mask[]; +}; + struct gpio_desc *gpiochip_get_desc(struct gpio_chip *chip, u16 hwnum); int gpiod_get_array_value_complex(bool raw, bool can_sleep, unsigned int array_size, diff --git a/include/linux/gpio/consumer.h b/include/linux/gpio/consumer.h index 1b21dc7b0fad..8dede3e886af 100644 --- a/include/linux/gpio/consumer.h +++ b/include/linux/gpio/consumer.h @@ -17,11 +17,20 @@ struct device; */ struct gpio_desc; +/** + * Opaque descriptor for a structure of GPIO array attributes. This structure + * is attached to struct gpiod_descs obtained from gpiod_get_array() and can be + * passed back to get/set array functions in order to activate fast processing + * path if applicable. + */ +struct gpio_array; + /** * Struct containing an array of descriptors that can be obtained using * gpiod_get_array(). */ struct gpio_descs { + struct gpio_array *info; unsigned int ndescs; struct gpio_desc *desc[]; }; -- 2.16.4