From: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
To: linux-gpio@vger.kernel.org
Cc: linus.walleij@linaro.org, broonie@kernel.org,
Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
Subject: [PATCH 0/2] Fix bugs in the insertion of gpiochip.
Date: Sat, 14 Nov 2015 16:38:54 +0800 [thread overview]
Message-ID: <1447490336-10209-1-git-send-email-bamvor.zhangjian@linaro.org> (raw)
These patches try to fix following bugs which is found by my gpio
mockup driver and testscript[1](will send them later):
1. Could not check the overlap if the new gpiochip is the secondly
gpiochip.
2. Could not check the overlap if the new gpiochip is overlap
with the left of gpiochip.
3. Allow overlap of base of different gpiochip.
4. Allow to insert an empty gpiochip
The first patch fix the first three by rewriting the logic in the
gpiochip_add_to_list.
The second patch fix the fourth bug in gpiochip_add. I do not
found the checker in gpiolib.c. Hope it is not a redundant logic.
[1] https://github.com/bjzhang/linux/tree/gpio-fix-and-mockup-driver
Bamvor Jian Zhang (2):
gpiolib: improve overlap check of range of gpio
gpiolib: do not allow to insert an empty gpiochip
drivers/gpio/gpiolib.c | 44 +++++++++++++++++++++++++++++++++-----------
1 file changed, 33 insertions(+), 11 deletions(-)
--
2.1.4
next reply other threads:[~2015-11-14 8:39 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-14 8:38 Bamvor Jian Zhang [this message]
2015-11-14 8:38 ` [PATCH 1/2] gpiolib: improve overlap check of range of gpio Bamvor Jian Zhang
2015-11-14 11:01 ` Linus Walleij
2015-11-14 11:20 ` Bamvor Zhang Jian
2015-11-14 11:30 ` Linus Walleij
2015-11-14 8:38 ` [PATCH 2/2] gpiolib: do not allow to insert an empty gpiochip Bamvor Jian Zhang
2015-11-14 11:03 ` Linus Walleij
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=1447490336-10209-1-git-send-email-bamvor.zhangjian@linaro.org \
--to=bamvor.zhangjian@linaro.org \
--cc=broonie@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).