From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161110AbbEEJdJ (ORCPT ); Tue, 5 May 2015 05:33:09 -0400 Received: from mga14.intel.com ([192.55.52.115]:5996 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030978AbbEEJdB (ORCPT ); Tue, 5 May 2015 05:33:01 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,372,1427785200"; d="scan'208";a="566442039" From: Shobhit Kumar To: intel-gfx , linux-kernel , linux-gpio , linux-pwm , dri-devel Cc: Linus Walleij , Alexandre Courbot , Daniel Vetter , David Airlie , Samuel Ortiz , Thierry Reding , Jani Nikula , Lee Jones , Povilas Staniulis , Chih-Wei Huang , Shobhit Kumar Subject: [PATCH 1/8] gpiolib: Add support for removing registered consumer lookup table Date: Tue, 5 May 2015 15:02:48 +0530 Message-Id: <1430818368-26790-1-git-send-email-shobhit.kumar@intel.com> X-Mailer: git-send-email 2.1.0 In-Reply-To: <1430316005-16480-2-git-send-email-shobhit.kumar@intel.com> References: <1430316005-16480-2-git-send-email-shobhit.kumar@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org In case we unload and load a driver module again that is registering a lookup table, without this it will result in multiple entries. Provide an option to remove the lookup table on driver unload v2: Ccing maintainers v3: Correct the subject line (Lee jones) Cc: Samuel Ortiz Cc: Linus Walleij Cc: Alexandre Courbot Cc: Thierry Reding Reviewed-by: Alexandre Courbot Signed-off-by: Shobhit Kumar --- drivers/gpio/gpiolib.c | 13 +++++++++++++ include/linux/gpio/machine.h | 1 + 2 files changed, 14 insertions(+) diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c index 59eaa23..2420af9 100644 --- a/drivers/gpio/gpiolib.c +++ b/drivers/gpio/gpiolib.c @@ -1658,6 +1658,19 @@ void gpiod_add_lookup_table(struct gpiod_lookup_table *table) mutex_unlock(&gpio_lookup_lock); } +/** + * gpiod_remove_lookup_table() - unregister GPIO device consumers + * @table: table of consumers to unregister + */ +void gpiod_remove_lookup_table(struct gpiod_lookup_table *table) +{ + mutex_lock(&gpio_lookup_lock); + + list_del(&table->list); + + mutex_unlock(&gpio_lookup_lock); +} + static struct gpio_desc *of_find_gpio(struct device *dev, const char *con_id, unsigned int idx, enum gpio_lookup_flags *flags) diff --git a/include/linux/gpio/machine.h b/include/linux/gpio/machine.h index e270614..c0d712d 100644 --- a/include/linux/gpio/machine.h +++ b/include/linux/gpio/machine.h @@ -57,5 +57,6 @@ struct gpiod_lookup_table { } void gpiod_add_lookup_table(struct gpiod_lookup_table *table); +void gpiod_remove_lookup_table(struct gpiod_lookup_table *table); #endif /* __LINUX_GPIO_MACHINE_H */ -- 2.1.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shobhit Kumar Subject: [PATCH 1/8] gpiolib: Add support for removing registered consumer lookup table Date: Tue, 5 May 2015 15:02:48 +0530 Message-ID: <1430818368-26790-1-git-send-email-shobhit.kumar@intel.com> References: <1430316005-16480-2-git-send-email-shobhit.kumar@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1430316005-16480-2-git-send-email-shobhit.kumar@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: intel-gfx , linux-kernel , linux-gpio , linux-pwm , dri-devel Cc: Alexandre Courbot , Samuel Ortiz , Povilas Staniulis , David Airlie , Shobhit Kumar , Linus Walleij , Jani Nikula , Chih-Wei Huang , Thierry Reding , Daniel Vetter , Lee Jones List-Id: linux-gpio@vger.kernel.org SW4gY2FzZSB3ZSB1bmxvYWQgYW5kIGxvYWQgYSBkcml2ZXIgbW9kdWxlIGFnYWluIHRoYXQgaXMg cmVnaXN0ZXJpbmcgYQpsb29rdXAgdGFibGUsIHdpdGhvdXQgdGhpcyBpdCB3aWxsIHJlc3VsdCBp biBtdWx0aXBsZSBlbnRyaWVzLiBQcm92aWRlCmFuIG9wdGlvbiB0byByZW1vdmUgdGhlIGxvb2t1 cCB0YWJsZSBvbiBkcml2ZXIgdW5sb2FkCgp2MjogQ2NpbmcgbWFpbnRhaW5lcnMKdjM6IENvcnJl Y3QgdGhlIHN1YmplY3QgbGluZSAoTGVlIGpvbmVzKQoKQ2M6IFNhbXVlbCBPcnRpeiA8c2FtZW9A bGludXguaW50ZWwuY29tPgpDYzogTGludXMgV2FsbGVpaiA8bGludXMud2FsbGVpakBsaW5hcm8u b3JnPgpDYzogQWxleGFuZHJlIENvdXJib3QgPGdudXJvdUBnbWFpbC5jb20+CkNjOiBUaGllcnJ5 IFJlZGluZyA8dGhpZXJyeS5yZWRpbmdAZ21haWwuY29tPgpSZXZpZXdlZC1ieTogQWxleGFuZHJl IENvdXJib3QgPGFjb3VyYm90QG52aWRpYS5jb20+ClNpZ25lZC1vZmYtYnk6IFNob2JoaXQgS3Vt YXIgPHNob2JoaXQua3VtYXJAaW50ZWwuY29tPgotLS0KIGRyaXZlcnMvZ3Bpby9ncGlvbGliLmMg ICAgICAgfCAxMyArKysrKysrKysrKysrCiBpbmNsdWRlL2xpbnV4L2dwaW8vbWFjaGluZS5oIHwg IDEgKwogMiBmaWxlcyBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEvZHJp dmVycy9ncGlvL2dwaW9saWIuYyBiL2RyaXZlcnMvZ3Bpby9ncGlvbGliLmMKaW5kZXggNTllYWEy My4uMjQyMGFmOSAxMDA2NDQKLS0tIGEvZHJpdmVycy9ncGlvL2dwaW9saWIuYworKysgYi9kcml2 ZXJzL2dwaW8vZ3Bpb2xpYi5jCkBAIC0xNjU4LDYgKzE2NTgsMTkgQEAgdm9pZCBncGlvZF9hZGRf bG9va3VwX3RhYmxlKHN0cnVjdCBncGlvZF9sb29rdXBfdGFibGUgKnRhYmxlKQogCW11dGV4X3Vu bG9jaygmZ3Bpb19sb29rdXBfbG9jayk7CiB9CiAKKy8qKgorICogZ3Bpb2RfcmVtb3ZlX2xvb2t1 cF90YWJsZSgpIC0gdW5yZWdpc3RlciBHUElPIGRldmljZSBjb25zdW1lcnMKKyAqIEB0YWJsZTog dGFibGUgb2YgY29uc3VtZXJzIHRvIHVucmVnaXN0ZXIKKyAqLwordm9pZCBncGlvZF9yZW1vdmVf bG9va3VwX3RhYmxlKHN0cnVjdCBncGlvZF9sb29rdXBfdGFibGUgKnRhYmxlKQoreworCW11dGV4 X2xvY2soJmdwaW9fbG9va3VwX2xvY2spOworCisJbGlzdF9kZWwoJnRhYmxlLT5saXN0KTsKKwor CW11dGV4X3VubG9jaygmZ3Bpb19sb29rdXBfbG9jayk7Cit9CisKIHN0YXRpYyBzdHJ1Y3QgZ3Bp b19kZXNjICpvZl9maW5kX2dwaW8oc3RydWN0IGRldmljZSAqZGV2LCBjb25zdCBjaGFyICpjb25f aWQsCiAJCQkJICAgICAgdW5zaWduZWQgaW50IGlkeCwKIAkJCQkgICAgICBlbnVtIGdwaW9fbG9v a3VwX2ZsYWdzICpmbGFncykKZGlmZiAtLWdpdCBhL2luY2x1ZGUvbGludXgvZ3Bpby9tYWNoaW5l LmggYi9pbmNsdWRlL2xpbnV4L2dwaW8vbWFjaGluZS5oCmluZGV4IGUyNzA2MTQuLmMwZDcxMmQg MTAwNjQ0Ci0tLSBhL2luY2x1ZGUvbGludXgvZ3Bpby9tYWNoaW5lLmgKKysrIGIvaW5jbHVkZS9s aW51eC9ncGlvL21hY2hpbmUuaApAQCAtNTcsNSArNTcsNiBAQCBzdHJ1Y3QgZ3Bpb2RfbG9va3Vw X3RhYmxlIHsKIH0KIAogdm9pZCBncGlvZF9hZGRfbG9va3VwX3RhYmxlKHN0cnVjdCBncGlvZF9s b29rdXBfdGFibGUgKnRhYmxlKTsKK3ZvaWQgZ3Bpb2RfcmVtb3ZlX2xvb2t1cF90YWJsZShzdHJ1 Y3QgZ3Bpb2RfbG9va3VwX3RhYmxlICp0YWJsZSk7CiAKICNlbmRpZiAvKiBfX0xJTlVYX0dQSU9f TUFDSElORV9IICovCi0tIAoyLjEuMAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJl ZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5m by9pbnRlbC1nZngK