All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartosz Golaszewski <brgl@bgdev.pl>
To: "Linus Walleij" <linus.walleij@linaro.org>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Marc Zyngier" <marc.zyngier@arm.com>,
	"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
	Bartosz Golaszewski <bgolaszewski@baylibre.com>
Subject: [PATCH v4 0/7] gpio: mockup: improve the user-space testing interface
Date: Thu, 14 Feb 2019 14:42:25 +0100	[thread overview]
Message-ID: <20190214134232.3821-1-brgl@bgdev.pl> (raw)

From: Bartosz Golaszewski <bgolaszewski@baylibre.com>

Hi Marc,

another try at the irq_sim stuff. This time I added a notifier chain to
irq_sim and made gpio-mockup register a notifier block. This way interested
users will get notified about a type config change.

v1 -> v2:
- instead of providing the irq_sim_get_type() helper, move the irq type
  logic into the simulator and provide a helper that allows users to specify
  the type of the fired interrupt

v2 -> v3:
- switch back to having irq_sim_type() and put the line state logic into the
  GPIO testing module

v3- > v4:
- drop irq_sim_get_type() and use a notifier chain instead so that any change
  in type configuration can be pushed out to interested users
- change the locking mechanism in gpio-mockup to a spinlock as we can't take
  a mutex when a hardirq-safe spinlock in irq_desc is being held when the
  irq_set_type() callback is called
- refuse to set any other type than falling or rising edge in irq_set_config

Bartosz Golaszewski (7):
  irq/irq_sim: add a notifier chain
  gpio: mockup: add locking
  gpio: mockup: implement get_multiple()
  gpio: mockup: don't create the debugfs link named after the label
  gpio: mockup: change the type of 'offset' to unsigned int
  gpio: mockup: change the signature of unlocked get/set helpers
  gpio: mockup: rework debugfs interface

 drivers/gpio/gpio-mockup.c | 215 ++++++++++++++++++++++++++++++++-----
 include/linux/irq_sim.h    |  22 +++-
 kernel/irq/irq_sim.c       |  40 +++++++
 3 files changed, 246 insertions(+), 31 deletions(-)

-- 
2.20.1

             reply	other threads:[~2019-02-14 13:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-14 13:42 Bartosz Golaszewski [this message]
2019-02-14 13:42 ` [PATCH v4 1/7] irq/irq_sim: add a notifier chain Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 2/7] gpio: mockup: add locking Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 3/7] gpio: mockup: implement get_multiple() Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 4/7] gpio: mockup: don't create the debugfs link named after the label Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 5/7] gpio: mockup: change the type of 'offset' to unsigned int Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 6/7] gpio: mockup: change the signature of unlocked get/set helpers Bartosz Golaszewski
2019-02-14 13:42 ` [PATCH v4 7/7] gpio: mockup: rework debugfs interface Bartosz Golaszewski
2019-02-18 16:44 ` [PATCH v4 0/7] gpio: mockup: improve the user-space testing interface Bartosz Golaszewski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190214134232.3821-1-brgl@bgdev.pl \
    --to=brgl@bgdev.pl \
    --cc=bgolaszewski@baylibre.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=tglx@linutronix.de \
    --cc=u.kleine-koenig@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.