<div dir="ltr"><div><div>Hi David,<br></div>The aim of this patch is to make gpiochip_remove() behavior consistent, especially when issuing a remove request while the chipio chip is <br>still requested. A patch has been submitted to change the return value of gpiochip_remove() from int to void. This one updates users of the return value:<br>

</div>Cheers,<br>Abdoulaye.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 30, 2014 at 12:14 AM, David Daney <span dir="ltr"><<a href="mailto:ddaney.cavm@gmail.com" target="_blank">ddaney.cavm@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 05/29/2014 02:54 PM, abdoulaye berthe wrote:<br>
<br>
Did you forget a changelog explaining why this is either needed, or even a good idea?  I joined the conversation late and don't know why you are doing this.<br>
<br>
Thanks,<br>
David Daney<div><div class="h5"><br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Signed-off-by: abdoulaye berthe <<a href="mailto:berthe.ab@gmail.com" target="_blank">berthe.ab@gmail.com</a>><br>
---<br>
  arch/arm/common/scoop.c                        | 10 ++--------<br>
  arch/mips/txx9/generic/setup.c                 |  4 ++--<br>
  arch/powerpc/platforms/83xx/<u></u>mcu_mpc8349emitx.c |  3 ++-<br>
  arch/sh/boards/mach-x3proto/<u></u>gpio.c             |  6 ++----<br>
  drivers/bcma/driver_gpio.c                     |  3 ++-<br>
  drivers/gpio/gpio-74x164.c                     |  8 +++-----<br>
  drivers/gpio/gpio-adnp.c                       |  9 +--------<br>
  drivers/gpio/gpio-adp5520.c                    |  8 +-------<br>
  drivers/gpio/gpio-adp5588.c                    |  6 +-----<br>
  drivers/gpio/gpio-amd8111.c                    |  3 +--<br>
  drivers/gpio/gpio-arizona.c                    |  3 ++-<br>
  drivers/gpio/gpio-cs5535.c                     |  8 +-------<br>
  drivers/gpio/gpio-da9052.c                     |  3 ++-<br>
  drivers/gpio/gpio-da9055.c                     |  3 ++-<br>
  drivers/gpio/gpio-dwapb.c                      |  2 +-<br>
  drivers/gpio/gpio-em.c                         |  5 +----<br>
  drivers/gpio/gpio-f7188x.c                     | 18 ++----------------<br>
  drivers/gpio/gpio-generic.c                    |  3 ++-<br>
  drivers/gpio/gpio-grgpio.c                     |  4 +---<br>
  drivers/gpio/gpio-ich.c                        |  9 +--------<br>
  drivers/gpio/gpio-it8761e.c                    |  6 +-----<br>
  drivers/gpio/gpio-janz-ttl.c                   |  8 +-------<br>
  drivers/gpio/gpio-kempld.c                     |  3 ++-<br>
  drivers/gpio/gpio-lp3943.c                     |  3 ++-<br>
  drivers/gpio/gpio-lynxpoint.c                  |  5 +----<br>
  drivers/gpio/gpio-max730x.c                    | 12 ++++--------<br>
  drivers/gpio/gpio-max732x.c                    |  7 +------<br>
  drivers/gpio/gpio-mc33880.c                    | 11 +++--------<br>
  drivers/gpio/gpio-mc9s08dz60.c                 |  3 ++-<br>
  drivers/gpio/gpio-mcp23s08.c                   | 26 +++++++-------------------<br>
  drivers/gpio/gpio-ml-ioh.c                     |  8 ++------<br>
  drivers/gpio/gpio-msm-v2.c                     |  5 +----<br>
  drivers/gpio/gpio-mxc.c                        |  2 +-<br>
  drivers/gpio/gpio-octeon.c                     |  3 ++-<br>
  drivers/gpio/gpio-palmas.c                     |  3 ++-<br>
  drivers/gpio/gpio-pca953x.c                    |  7 +------<br>
  drivers/gpio/gpio-pcf857x.c                    |  4 +---<br>
  drivers/gpio/gpio-pch.c                        | 10 ++--------<br>
  drivers/gpio/gpio-rc5t583.c                    |  3 ++-<br>
  drivers/gpio/gpio-rcar.c                       |  5 +----<br>
  drivers/gpio/gpio-rdc321x.c                    |  7 ++-----<br>
  drivers/gpio/gpio-sch.c                        | 16 +++-------------<br>
  drivers/gpio/gpio-sch311x.c                    |  6 ++----<br>
  drivers/gpio/gpio-sodaville.c                  |  4 +---<br>
  drivers/gpio/gpio-stmpe.c                      |  8 +-------<br>
  drivers/gpio/gpio-sx150x.c                     |  7 ++-----<br>
  drivers/gpio/gpio-syscon.c                     |  3 ++-<br>
  drivers/gpio/gpio-tb10x.c                      |  5 +----<br>
  drivers/gpio/gpio-tc3589x.c                    |  8 +-------<br>
  drivers/gpio/gpio-timberdale.c                 |  5 +----<br>
  drivers/gpio/gpio-tps6586x.c                   |  3 ++-<br>
  drivers/gpio/gpio-tps65910.c                   |  3 ++-<br>
  drivers/gpio/gpio-tps65912.c                   |  3 ++-<br>
  drivers/gpio/gpio-ts5500.c                     |  6 +++---<br>
  drivers/gpio/gpio-twl4030.c                    |  4 +---<br>
  drivers/gpio/gpio-twl6040.c                    |  3 ++-<br>
  drivers/gpio/gpio-ucb1400.c                    |  2 +-<br>
  drivers/gpio/gpio-viperboard.c                 | 10 +++-------<br>
  drivers/gpio/gpio-vx855.c                      |  3 +--<br>
  drivers/gpio/gpio-wm831x.c                     |  3 ++-<br>
  drivers/gpio/gpio-wm8350.c                     |  3 ++-<br>
  drivers/gpio/gpio-wm8994.c                     |  3 ++-<br>
  drivers/hid/hid-cp2112.c                       |  6 ++----<br>
  drivers/input/keyboard/<u></u>adp5588-keys.c          |  4 +---<br>
  drivers/input/keyboard/<u></u>adp5589-keys.c          |  4 +---<br>
  drivers/input/touchscreen/<u></u>ad7879.c             | 10 +++-------<br>
  drivers/leds/leds-pca9532.c                    | 10 ++--------<br>
  drivers/leds/leds-tca6507.c                    |  7 ++-----<br>
  drivers/media/dvb-frontends/<u></u>cxd2820r_core.c    | 10 +++-------<br>
  drivers/mfd/asic3.c                            |  3 ++-<br>
  drivers/mfd/htc-i2cpld.c                       |  8 +-------<br>
  drivers/mfd/sm501.c                            | 17 +++--------------<br>
  drivers/mfd/tc6393xb.c                         | 13 ++++---------<br>
  drivers/mfd/ucb1x00-core.c                     |  8 ++------<br>
  drivers/pinctrl/pinctrl-<u></u>abx500.c               | 15 +++------------<br>
  drivers/pinctrl/pinctrl-adi2.c                 |  9 ++++-----<br>
  drivers/pinctrl/pinctrl-<u></u>as3722.c               | 11 ++---------<br>
  drivers/pinctrl/pinctrl-<u></u>baytrail.c             |  5 +----<br>
  drivers/pinctrl/pinctrl-<u></u>coh901.c               | 10 ++--------<br>
  drivers/pinctrl/pinctrl-<u></u>exynos5440.c           |  6 +-----<br>
  drivers/pinctrl/pinctrl-msm.c                  | 11 ++---------<br>
  drivers/pinctrl/pinctrl-<u></u>nomadik.c              |  2 +-<br>
  drivers/pinctrl/pinctrl-<u></u>rockchip.c             | 16 ++++------------<br>
  drivers/pinctrl/pinctrl-<u></u>samsung.c              | 14 ++++----------<br>
  drivers/pinctrl/pinctrl-sunxi.<u></u>c                |  3 +--<br>
  drivers/pinctrl/sh-pfc/gpio.c                  |  9 +++------<br>
  drivers/pinctrl/spear/pinctrl-<u></u>plgpio.c         |  3 +--<br>
  drivers/pinctrl/vt8500/<u></u>pinctrl-wmt.c           |  9 ++-------<br>
  drivers/platform/x86/intel_<u></u>pmic_gpio.c         |  3 +--<br>
  drivers/ssb/driver_gpio.c                      |  3 ++-<br>
  drivers/staging/vme/devices/<u></u>vme_pio2_gpio.c    |  4 +---<br>
  drivers/tty/serial/max310x.c                   | 10 ++++------<br>
  drivers/video/fbdev/via/via-<u></u>gpio.c             | 10 +++-------<br>
  sound/soc/codecs/wm5100.c                      |  5 +----<br>
  sound/soc/codecs/wm8903.c                      |  6 +-----<br>
  sound/soc/codecs/wm8962.c                      |  5 +----<br>
  sound/soc/codecs/wm8996.c                      |  6 +-----<br>
  97 files changed, 182 insertions(+), 460 deletions(-)<br>
<br>
</blockquote></div></div>
[...]<br>
</blockquote></div><br></div>