Hi Marek On 06/13/2017 10:45 AM, Marek Vasut wrote: > On 06/12/2017 09:24 AM, Patrice CHOTARD wrote: >> Hi Tom >> >> I think this series is mature enough to be merged. > > I think there's still some discussion with Simon going on ? > Also, please don't top-post.
Yes, i saw Simon's request after sending this email. I will submit a v7 of this series Patrice > >> Thanks >> >> Patrice >> >> >> On 06/05/2017 11:42 AM, patrice.chot...@st.com wrote: >>> From: Patrice Chotard <patrice.chot...@st.com> >>> >>> This is needed in error path to assert previously deasserted >>> reset by using a saved reset_ctl reference. >>> >>> Signed-off-by: Patrice Chotard <patrice.chot...@st.com> >>> Reviewed-by: Simon Glass <s...@chromium.org> >>> --- >>> v6: _ none >>> v5: _ none >>> v4: _ none >>> v3: _ none >>> v2: _ none >>> >>> drivers/reset/reset-uclass.c | 9 +++++++++ >>> include/reset.h | 9 +++++++++ >>> 2 files changed, 18 insertions(+) >>> >>> diff --git a/drivers/reset/reset-uclass.c b/drivers/reset/reset-uclass.c >>> index de3695f..4fd82b9 100644 >>> --- a/drivers/reset/reset-uclass.c >>> +++ b/drivers/reset/reset-uclass.c >>> @@ -97,6 +97,15 @@ int reset_get_by_name(struct udevice *dev, const char >>> *name, >>> return reset_get_by_index(dev, index, reset_ctl); >>> } >>> >>> +int reset_request(struct reset_ctl *reset_ctl) >>> +{ >>> + struct reset_ops *ops = reset_dev_ops(reset_ctl->dev); >>> + >>> + debug("%s(reset_ctl=%p)\n", __func__, reset_ctl); >>> + >>> + return ops->request(reset_ctl); >>> +} >>> + >>> int reset_free(struct reset_ctl *reset_ctl) >>> { >>> struct reset_ops *ops = reset_dev_ops(reset_ctl->dev); >>> diff --git a/include/reset.h b/include/reset.h >>> index f45fcf8..4f2e35f 100644 >>> --- a/include/reset.h >>> +++ b/include/reset.h >>> @@ -100,6 +100,15 @@ int reset_get_by_name(struct udevice *dev, const char >>> *name, >>> struct reset_ctl *reset_ctl); >>> >>> /** >>> + * reset_request - Request a reset signal. >>> + * >>> + * @reset_ctl: A reset control struct. >>> + * >>> + * @return 0 if OK, or a negative error code. >>> + */ >>> +int reset_request(struct reset_ctl *reset_ctl); >>> + >>> +/** >>> * reset_free - Free a previously requested reset signal. >>> * >>> * @reset_ctl: A reset control struct that was previously successfully > > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot