BMC/Intel-BMC/linux.git/drivers/pinctrl/pinconf.c, branch v4.4

pinctrl: pinconf: remove needless loop

2015-10-31T21:13:07+00:00

This removes a needless loop which was caught in pinconf.c. Suggested-by: Andy Shevchenko Signed-off-by: Laurent Meunier Reviewed-by: Andy Shevchenko Signed-off-by: Linus Walleij

pinctrl: join dev_dbg strings into a single line

2015-08-03T08:13:00+00:00

These are user-visible strings, so can exceed 80 columns. Signed-off-by: Masahiro Yamada Signed-off-by: Linus Walleij

pinctrl: pinconf: pinconf_show_config() can be static

2015-07-17T19:56:23+00:00

Signed-off-by: Fengguang Wu Signed-off-by: Linus Walleij

pinctrl: pinconf: Fix display of configs

2015-07-17T12:20:49+00:00

The function pinconf_dbg_config_print() only prints the configuration of the 1st pin config in an array of pin configurations. Fix this so that all pin configurations in the array are displayed. There are a few places in the code where the pin configs are displayed and so add a helper function to display the pin configs to simplify the code. Signed-off-by: Jon Hunter Signed-off-by: Linus Walleij

pinctrl: pinconf: Allow groups to be configured via debugfs

2015-07-17T12:19:34+00:00

The function pinconf_dbg_config_write() currently only supports configuring a pin configuration mapping via the debugfs. Allow group mappings to also be configured via the debugfs. Signed-off-by: Jon Hunter Signed-off-by: Linus Walleij

pinctrl: pinconf-generic: Allow driver to specify DT params

2015-01-11T21:32:19+00:00

Additionally to the generic DT parameters, allow drivers to provide driver-specific DT parameters to be used with the generic parser infrastructure. To achieve this 'struct pinctrl_desc' is extended to pass custom pinconf option to the core. In order to pass this kind of information, the related data structures - 'struct pinconf_generic_dt_params', 'pin_config_item' - are moved from pinconf internals to the pinconf-generic header. Additionally pinconfg-generic is refactored to not only iterate over the generic pinconf parameters but also take the parameters into account that are provided through the driver's 'struct pinctrl_desc'. In particular 'pinconf_generic_parse_dt_config()' and 'pinconf_generic_dump' helpers are split into two parts each. In order to have a more generic helper that can be used to process the generic parameters as well as the driver-specific ones. v2: - fix typo - add missing documentation for @conf_items member in struct - rebase to pinctrl/devel: conflict in abx500 - rename _pinconf_generic_dump() to pinconf_generic_dump_one() - removed '_' from _parse_dt_cfg() - removed BUG_ONs, error condition is handled in if statements - removed pinconf_generic_dump_group() & pinconf_generic_dump_pin helpers - fixed up corresponding call sites - renamed pinconf_generic_dump() to pinconf_generic_dump_pins() - added kernel-doc to pinconf_generic_dump_pins() - add kernel-doc - more verbose commit message Signed-off-by: Soren Brinkmann Tested-by: Andreas FÃ¤rber Signed-off-by: Linus Walleij

pinctrl: pinconf: remove warning: unused variable 'ops'

2013-12-13T12:55:15+00:00

Signed-off-by: Alexandre Belloni Signed-off-by: Linus Walleij

pinctrl: pinconf: remove checks on ops->pin_config_get

2013-12-12T18:11:07+00:00

ops->pin_config_get() is only used in one specific path that will only be taken for generic pinconf drivers (ops->is_generic == true) when dumping the pinconf by using debugfs. By removing the check in pinconf_check_ops(), let's stop pressuring people to write a pin_config_get() function that will never be used and so will probably never be tested. Removing the check in pinconf_pins_show() allows driver to not implement pin_config_get() but still get a dump of the pinconf in debugfs by implementing pin_config_dbg_show(). Finally, not implementing pin_config_get() now results in returning -ENOTSUPP instead of -EINVAL. While this doesn't have any real impact for now, this feels more right. Signed-off-by: Alexandre Belloni Signed-off-by: Linus Walleij

pinctrl: remove an unnecessary cast

2013-09-23T08:47:14+00:00

sizeof() is already size_t so there is no need to cast here. Generally, casting inside the min() macro instead of using min_t() is considered bad style. Signed-off-by: Dan Carpenter Signed-off-by: Linus Walleij

pinctrl: fix pinconf_dbg_config_write return type

2013-09-23T08:47:03+00:00

Have pinconf_dbg_config_write() return a ssize_t. This fixes the following compilation warning: drivers/pinctrl/pinconf.c:617:2: warning: initialization from incompatible pointer type [enabled by default] drivers/pinctrl/pinconf.c:617:2: warning: (near initialization for ‘pinconf_dbg_pinconfig_fops.write’) [enabled by default] Signed-off-by: Vincent Stehlé Cc: Linus Walleij Signed-off-by: Linus Walleij