mbox series

[U-Boot,v6,0/5] drivers: Add reset ctrl to drivers

Message ID 1528973123-21858-1-git-send-email-ley.foon.tan@intel.com
Headers show
Series drivers: Add reset ctrl to drivers | expand

Message

Ley Foon Tan June 14, 2018, 10:45 a.m. UTC
Add reset ctrl to dwmmc socfpga, designware Ethernet and ns16550 serial drivers.

A reset property is an optional feature, so only print out a warning and
do not fail if a reset property is not present.
    
If a reset property is discovered, then use it to deassert, thus bringing the
IP out of reset.

v6:
- Include change history to patches and cover letter
- Added Joe's Acked-by in designware emac patch.

v5: https://patchwork.ozlabs.org/cover/924857/
- Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
- Change to use CONFIG_IS_ENABLED(DM_RESET) in reset.h
- Added Simon's Reviewed-by in dwmmc, 16550 serial and designware emac patches.

v4: https://patchwork.ozlabs.org/cover/923883/
- Add patch to check CONFIG_SPL_RESET_SUPPORT in reset.h

v3: https://patchwork.ozlabs.org/cover/910018/
- remove #ifdef CONFIG_DM_RESET switch
- add maintainer emails

v2: https://patchwork.ozlabs.org/cover/908667/
- remove 'return' in designware emac driver
- keep reset control in socfpga_dw_mmc.c because it didn't call to common dwmmc probe
  function when in SPL.
- add reviewed-by in ns16550 patch

v1: https://patchwork.ozlabs.org/cover/905519/

Ley Foon Tan (5):
  reset: Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
  include: reset: Change to use CONFIG_IS_ENABLED(DM_RESET)
  mmc: dwmmc: socfpga: Add reset ctrl to driver
  serial: ns16550: Add reset ctrl to driver
  net: designware: Add reset ctrl to driver

 arch/arm/mach-stm32mp/Kconfig |  2 +-
 common/spl/Kconfig            |  2 +-
 drivers/Makefile              |  2 +-
 drivers/mmc/socfpga_dw_mmc.c  | 17 +++++++++++++++++
 drivers/net/designware.c      |  8 ++++++++
 drivers/serial/ns16550.c      |  8 ++++++++
 include/reset.h               |  2 +-
 7 files changed, 37 insertions(+), 4 deletions(-)

Comments

Ley Foon Tan July 6, 2018, 2:45 a.m. UTC | #1
On Thu, Jun 14, 2018 at 6:45 PM, Ley Foon Tan <ley.foon.tan@intel.com> wrote:
> Add reset ctrl to dwmmc socfpga, designware Ethernet and ns16550 serial drivers.
>
> A reset property is an optional feature, so only print out a warning and
> do not fail if a reset property is not present.
>
> If a reset property is discovered, then use it to deassert, thus bringing the
> IP out of reset.
>
> v6:
> - Include change history to patches and cover letter
> - Added Joe's Acked-by in designware emac patch.
>
> v5: https://patchwork.ozlabs.org/cover/924857/
> - Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
> - Change to use CONFIG_IS_ENABLED(DM_RESET) in reset.h
> - Added Simon's Reviewed-by in dwmmc, 16550 serial and designware emac patches.
>
> v4: https://patchwork.ozlabs.org/cover/923883/
> - Add patch to check CONFIG_SPL_RESET_SUPPORT in reset.h
>
> v3: https://patchwork.ozlabs.org/cover/910018/
> - remove #ifdef CONFIG_DM_RESET switch
> - add maintainer emails
>
> v2: https://patchwork.ozlabs.org/cover/908667/
> - remove 'return' in designware emac driver
> - keep reset control in socfpga_dw_mmc.c because it didn't call to common dwmmc probe
>   function when in SPL.
> - add reviewed-by in ns16550 patch
>
> v1: https://patchwork.ozlabs.org/cover/905519/
>
> Ley Foon Tan (5):
>   reset: Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
>   include: reset: Change to use CONFIG_IS_ENABLED(DM_RESET)
>   mmc: dwmmc: socfpga: Add reset ctrl to driver
>   serial: ns16550: Add reset ctrl to driver
>   net: designware: Add reset ctrl to driver
>
>  arch/arm/mach-stm32mp/Kconfig |  2 +-
>  common/spl/Kconfig            |  2 +-
>  drivers/Makefile              |  2 +-
>  drivers/mmc/socfpga_dw_mmc.c  | 17 +++++++++++++++++
>  drivers/net/designware.c      |  8 ++++++++
>  drivers/serial/ns16550.c      |  8 ++++++++
>  include/reset.h               |  2 +-
>  7 files changed, 37 insertions(+), 4 deletions(-)
>
> --
> 2.2.2
>

Hi Tom

Can help to merge this series of patches?

Thanks.

Regards
Ley Foon
Tom Rini July 6, 2018, 5:26 p.m. UTC | #2
On Fri, Jul 06, 2018 at 10:45:18AM +0800, Ley Foon Tan wrote:
> On Thu, Jun 14, 2018 at 6:45 PM, Ley Foon Tan <ley.foon.tan@intel.com> wrote:
> > Add reset ctrl to dwmmc socfpga, designware Ethernet and ns16550 serial drivers.
> >
> > A reset property is an optional feature, so only print out a warning and
> > do not fail if a reset property is not present.
> >
> > If a reset property is discovered, then use it to deassert, thus bringing the
> > IP out of reset.
> >
> > v6:
> > - Include change history to patches and cover letter
> > - Added Joe's Acked-by in designware emac patch.
> >
> > v5: https://patchwork.ozlabs.org/cover/924857/
> > - Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
> > - Change to use CONFIG_IS_ENABLED(DM_RESET) in reset.h
> > - Added Simon's Reviewed-by in dwmmc, 16550 serial and designware emac patches.
> >
> > v4: https://patchwork.ozlabs.org/cover/923883/
> > - Add patch to check CONFIG_SPL_RESET_SUPPORT in reset.h
> >
> > v3: https://patchwork.ozlabs.org/cover/910018/
> > - remove #ifdef CONFIG_DM_RESET switch
> > - add maintainer emails
> >
> > v2: https://patchwork.ozlabs.org/cover/908667/
> > - remove 'return' in designware emac driver
> > - keep reset control in socfpga_dw_mmc.c because it didn't call to common dwmmc probe
> >   function when in SPL.
> > - add reviewed-by in ns16550 patch
> >
> > v1: https://patchwork.ozlabs.org/cover/905519/
> >
> > Ley Foon Tan (5):
> >   reset: Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET
> >   include: reset: Change to use CONFIG_IS_ENABLED(DM_RESET)
> >   mmc: dwmmc: socfpga: Add reset ctrl to driver
> >   serial: ns16550: Add reset ctrl to driver
> >   net: designware: Add reset ctrl to driver
> >
> >  arch/arm/mach-stm32mp/Kconfig |  2 +-
> >  common/spl/Kconfig            |  2 +-
> >  drivers/Makefile              |  2 +-
> >  drivers/mmc/socfpga_dw_mmc.c  | 17 +++++++++++++++++
> >  drivers/net/designware.c      |  8 ++++++++
> >  drivers/serial/ns16550.c      |  8 ++++++++
> >  include/reset.h               |  2 +-
> >  7 files changed, 37 insertions(+), 4 deletions(-)
> >
> > --
> > 2.2.2
> >
> 
> Hi Tom
> 
> Can help to merge this series of patches?

I suspect I will take these shortly after the release, thanks for being
patient!