On 12/20/18 10:16 PM, Simon Glass wrote: > Hi Christoph, > > On Mon, 17 Dec 2018 at 06:30, Christoph Muellner > <christoph.muell...@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.muell...@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 > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
