09 Mar, 2016
10 commits
-
This reverts commit 3fab91ea284a3b795327dda915a3c150a49e4be2.
-
GPIO chip structure already has "parent" field which is used for the
same purpose as "dev" field in gpio_bank structure - store pointer on
GPIO device.Hence, drop duplicated "dev" field from gpio_bank structure.
Signed-off-by: Grygorii Strashko
Acked-by: Santosh Shilimkar
Signed-off-by: Linus Walleij -
…ernel/git/lee/mfd into devel
-
Setting gc->direction_output to gc->direction_output looks strange.
I think this change makes the intention more clear.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
*read_reg and *write_reg can be removed because at all the places to call
them, we can just use gc->read_reg/gc->write_reg instead.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
For devtype with specific gpio_dir_out implementation, current code is
wrong because below code sets both gc->direction_output and
mpc8xxx_gc->direction_output to the same function.gc->direction_output = devtype->gpio_dir_out ?: gc->direction_output;
mpc8xxx_gc->direction_output = gc->direction_output;Set mpc8xxx_gc->direction_output = gc->direction_output first to fix it.
This way mpc8xxx_gc->direction_output actually calls the standard
bgpio_dir_out() to update register.Fixes: commit 42178e2a1e42 ("drivers/gpio: Switch gpio-mpc8xxx to use gpio-generic")
Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
This patch adds support for the mcp23s18 which is very similar to
the mcp23s17. A couple of control bits are not the same.
Notable IOCON_HAEN (s17 only) & IOCON_INTCC. Which can be ignored.Patch changes the following:
- Add mcp23s18 types.
- Always set mirror bit if the dts defines mcp23s18. regardless of type.
Mirror bit is ignored on 8 bit devices anyway.
- In mcp23s08_probe use chip.ngpio instead of logic based on type
to determine number of gpio lins to increment by. This is set
appropiately by the call to mcp23s08_probe_one.
- Add mcp23s18 to device tree documentation.
- Remove statement that irqs don't work for spi. They do.
Tested with mcp23s18.Signed-off-by: Phil Reid
Acked-by: Rob Herring
Signed-off-by: Linus Walleij -
The altr,interrupt-trigger property is not used by the driver.
Instead, altr,interrupt-type is used by the driver and the driver
does not probe if this property is not specified. Therefore, it
is expected that there are no users of the -trigger property in
the wild and that this is a typo in the documentation for the
altera-pio controller. This patch fixes the typo.Signed-off-by: Marek Vasut
Cc: Tien Hock Loh
Cc: Linus Walleij
Signed-off-by: Linus Walleij -
The 16Z127 is a 32bit GPIO controller on a MCB FPGA.
Every single line can be configured as input and output.Push pull and open drain are supported as well as setting
a debounce value for the input lines.Signed-off-by: Andreas Werner
Signed-off-by: Linus Walleij -
The implementation of lp3943_gpio_request/lp3943_gpio_free test pin_used
for tracing the pin usage. However, gpiolib already checks FLAG_REQUESTED
flag for the same purpose. So remove the redundant implementation.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij
08 Mar, 2016
3 commits
-
Replace calls to devm_request_mem_region and devm_ioremap with a
direct call to devm_ioremap_resource instead and modify error
handling.Move the call to platform_get_resource adjacent to the call to
devm_ioremap_resource to make the connection between them more
clear.A simplified version of the Coccinelle semantic patch that is used to
make this change is as follows:
//
@nm@
expression myname;
identifier i;
@@struct platform_driver i = { .driver = { .name = myname } };
@@
expression dev,res,size,e1,e;
expression nm.myname;
@@-if (!devm_request_mem_region(dev, res->start, size,
- \(res->name\|dev_name(dev)\|myname\)))
{
- ...
- return ...;
-}
... when != res->start = e1
e =
-devm_ioremap(dev,res->start,size);
+devm_ioremap_resource(dev,res);if
-(e == NULL)
+(IS_ERR(e))
{
...
-return ...;
+return PTR_ERR(e);
}
//Further, updated error handling by hand as devm_ioremap_resource
gives appropriate error messages, so remove unnecessary error
messages.Signed-off-by: Amitoj Kaur Chawla
Signed-off-by: Linus Walleij -
The TS-4800 is an i.MX515 board. Its GPIO driver should only be compiled
for this CPU or for test builds.Signed-off-by: Julien Grossholtz
Signed-off-by: Linus Walleij
07 Mar, 2016
3 commits
-
The original code of gpiodev_add_to_list is not very clear which
lead to bugs or compiling warning, reference the following patches:
Bugs:
1. Commit ef7c7553039b ("gpiolib: improve overlap check of range of
gpio").
2. Commit 96098df125c0 ("gpiolib: fix chip order in gpio list")Warning:
1. Commit e28ecca6eac4 ("gpio: fix warning about iterator").
of gpio").There is a off-list discussion about how to improve it consequently.
This commit try to follow this by rewriting the whole functions.Tested pass with my gpio mockup driver and test scripts[1].
[1] http://www.spinics.net/lists/linux-gpio/msg09598.html
Suggested-by: Arnd Bergmann
Signed-off-by: Bamvor Jian Zhang
Signed-off-by: Linus Walleij -
There is no build dependency for this driver, so enable COMPILE_TEST to get
better build coverage.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
These fields are initialized by bgpio_init() with exactly the same settings
so remove the redundant code.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij
26 Feb, 2016
1 commit
-
I named the field representing the current user of GPIO line as
"label" but this is too vague and ambiguous. Before anyone gets
confused, rename it to "consumer" and indicate clearly in the
documentation that this is a string set by the user of the line.Also clean up leftovers in the documentation.
Signed-off-by: Linus Walleij
25 Feb, 2016
19 commits
-
We started to assign the gpio_device as parent for the sysfs
but this changes the expected layout of sysfs. Restore the
previous behaviour.Signed-off-by: Bamvor Jian Zhang
Signed-off-by: Linus Walleij -
The 'dev_id' parameter of the GPIO_LOOKUP{,_IDX} macros were removed by
commit ad824783fb23 ("gpio: better lookup method for platform GPIOs").Update the documentation to reflect that.
Signed-off-by: Gabor Juhos
Signed-off-by: Linus Walleij -
There is no build dependency for this driver, so enable COMPILE_TEST to get
better build coverage.Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
This driver has .can_sleep flag set.
So the pisosr_gpio_get() can be called from contexts that can sleep.
Thus use the cansleep() variant in pisosr_gpio_refresh().Signed-off-by: Axel Lin
Signed-off-by: Linus Walleij -
gpio->load_gpio is optional, so use devm_gpiod_get_optional instead.
Signed-off-by: Axel Lin
Acked-by: Andrew F. Davis
Signed-off-by: Linus Walleij -
ret variable can contain error values and is compared with zero.
Its type must be signed.The problem has been detected using coccinelle script
scripts/coccinelle/tests/unsigned_lesser_than_zero.cocciSigned-off-by: Andrzej Hajda
Signed-off-by: Linus Walleij -
Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
This is part of an ongoing process to migrate from ARCH_SHMOBILE to
ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.Signed-off-by: Simon Horman
Signed-off-by: Linus Walleij -
Both chip->base and chip->label are correctly set by bgpio_init().
Signed-off-by: Axel Lin
Tested-by: Julien Grossholtz
Signed-off-by: Linus Walleij -
Use list_for_each_entry to simplify the code a bit.
Signed-off-by: Axel Lin
Acked-by: Gregory Fong
Signed-off-by: Linus Walleij -
Move the code checking valid gpio count to cover both DT and non-DT cases.
Signed-off-by: Axel Lin
Acked-by: Alban Bedel
Signed-off-by: Linus Walleij -
… devm-gpiochip-add-data
-
gpio->load_gpio is optional, so use devm_gpiod_get_optional instead.
Signed-off-by: Axel Lin
Acked-by: Andrew F. Davis
Signed-off-by: Linus Walleij -
ret variable can contain error values and is compared with zero.
Its type must be signed.The problem has been detected using coccinelle script
scripts/coccinelle/tests/unsigned_lesser_than_zero.cocciSigned-off-by: Andrzej Hajda
Signed-off-by: Linus Walleij -
Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
This is part of an ongoing process to migrate from ARCH_SHMOBILE to
ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.Signed-off-by: Simon Horman
Signed-off-by: Linus Walleij -
Both chip->base and chip->label are correctly set by bgpio_init().
Signed-off-by: Axel Lin
Tested-by: Julien Grossholtz
Signed-off-by: Linus Walleij -
Use list_for_each_entry to simplify the code a bit.
Signed-off-by: Axel Lin
Acked-by: Gregory Fong
Signed-off-by: Linus Walleij -
Move the code checking valid gpio count to cover both DT and non-DT cases.
Signed-off-by: Axel Lin
Acked-by: Alban Bedel
Signed-off-by: Linus Walleij -
The R-Car GPIO driver handles Runtime PM for requested GPIOs only.
When using a GPIO purely as an interrupt source, no Runtime PM handling
is done, and the GPIO module's clock may not be enabled.To fix this:
- Add .irq_request_resources() and .irq_release_resources() callbacks
to handle Runtime PM when an interrupt is requested,
- Add irq_bus_lock() and sync_unlock() callbacks to handle Runtime PM
when e.g. disabling/enabling an interrupt, or configuring the
interrupt type.Fixes: d5c3d84657db57bd "net: phy: Avoid polling PHY with PHY_IGNORE_INTERRUPTS"
Signed-off-by: Geert Uytterhoeven
Signed-off-by: Linus Walleij -
devm_get_gpiod_from_child() tries several property suffixes to find a
GPIO descriptor. If all suffixes fail and no probe deferral has been
detected, it returns the error of the last try.However, if any but the last try fails with a real error (e.g. -EBUSY),
this error is not propagated, and -ENOENT will be returned.
This confuses drivers that e.g. want to detect if a GPIO is already in
use.To fix this, change the loop logic to continue on -ENOENT, which
indicates the property was not found and the next suffix should be
tried, and propagate all other detected errors.Signed-off-by: Geert Uytterhoeven
Signed-off-by: Linus Walleij
23 Feb, 2016
4 commits
-
Use devm_gpiochip_add_data() for GPIO registration and remove the
need of driver callback .remove.Signed-off-by: Laxman Dewangan
-
Use devm_gpiochip_add_data() for GPIO registration.
Signed-off-by: Laxman Dewangan
-
Use devm_gpiochip_add_data() for GPIO registration and remove the
need of driver callback .remove.Signed-off-by: Laxman Dewangan
Cc: patches@opensource.wolfsonmicro.com
Acked-by: Charles Keepax -
Use devm_gpiochip_add_data() for GPIO registration and remove the
need of driver callback .remove.Signed-off-by: Laxman Dewangan
Cc: patches@opensource.wolfsonmicro.com
Acked-by: Charles Keepax
