On 12/20/18 10:16 PM, Simon Glass wrote:
Hi Christoph,
On Mon, 17 Dec 2018 at 06:30, Christoph Muellner christoph.muellner@theobroma-systems.com wrote:
pinctrl_decode_pin_config_dm() is basically a feature-equivalent implementation of pinctrl_decode_pin_config(), which operates on struct udevice devices and uses the dev_read_*() API.
So is it possible to remove the old function and avoid the _dm suffix here?
Seems like the right way to go for me. But I don't have the option to test on devices, which need this function (RK3188 and RK3288) and I don't want to risk touching drivers, which I cannot test outside of the regular merge window.
My plan was to do this early during the next merge window (together with dropping the old RK3399 pinctrl driver and the refactoring of the RK3399 pinctrl driver into a generic RK pinctrl driver and a RK3399 mini-driver).
Acceptable?
Thank's, Christoph
Signed-off-by: Christoph Muellner christoph.muellner@theobroma-systems.com
Changes in v3: None Changes in v2: None
drivers/pinctrl/pinctrl-uclass.c | 15 +++++++++++++++ include/dm/pinctrl.h | 12 ++++++++++++ 2 files changed, 27 insertions(+)
Regards, Simon