Hi Christoph, On Tue, 25 Dec 2018 at 14:49, Christoph Müllner <christoph.muell...@theobroma-systems.com> wrote: > > 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?
Yes that's fine with me. Could you add a comment saying that this function is temporary for this release and will be removed in patches by February? Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
