Re: [PATCH 1/2] pinctrl: pinconf_generic: add utility functions foradd map/configs
From: Laxman Dewangan
Date: Sat Jul 27 2013 - 06:03:17 EST
On Friday 26 July 2013 11:06 PM, Stephen Warren wrote:
On 07/26/2013 04:15 AM, Laxman Dewangan wrote:
Some of pincontrol driver needs the utility function to create mapWhich other drivers already contain this code? Are you planning on
list. The utility function needed for adding mux, configs etc.
converting them over to use these new functions? The code looks
familiar, so I assume it's just cut/paste from Tegra with some renames,
so I didn't read it through in detail.
Yes, these functions are taken from the pinctrl-tegra driver. Other
drivers which have similar implementation are nomadik, tegra, abx500,
So All can use these utility function to avoid duplication and reduce
Yes, I have plan to use this in Tegra atleast but once this patch get
drivers/pinctrl/pinconf-generic.c | 105 +++++++++++++++++++++++++++++++I don't think those are the correct files for this code. Presumably
include/linux/pinctrl/pinconf-generic.h | 52 +++++++++++++++
there's no reason at all why a pinctrl driver that doesn't require
CONFIG_GENERIC_PINCONF can't use these basic utility functions. Perhaps
add a new pinctrl-utils file?
I thought generic implementation of parsing of dt is pinctrl-generic
file and as these newly added APIs are require for the parsing the DT
node, I added here.
I can move this to pinctrl-util.c/h also.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/