mbox series

[U-Boot,0/4] Drop redundant code for eSDHC clock getting

Message ID 20191112112839.10960-1-yangbo.lu@nxp.com
Headers show
Series Drop redundant code for eSDHC clock getting | expand

Message

Yangbo Lu Nov. 12, 2019, 11:28 a.m. UTC 
This patch-set is to drop redundant code for eSDHC clock getting.
The fsl_esdhc driver is able to simply get clock from gd->arch.sdhc_clk.

The CI build link:
https://travis-ci.org/yangbolu1991/u-boot-test/builds/610688748

Yangbo Lu (4):
  mmc: fsl_esdhc: get clock directly from global data
  arm: ls1021a: drop redundant board_mmc_init()
  arm: drop eSDHC clock getting in mxc_get_clock() for layerscape
  mpc83xx: remove unused clock.h

 arch/arm/cpu/armv7/ls102xa/clock.c                 |  2 --
 .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c | 15 ----------
 .../arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c | 15 ----------
 arch/arm/include/asm/arch-fsl-layerscape/clock.h   |  2 --
 arch/arm/include/asm/arch-ls102xa/clock.h          |  1 -
 arch/powerpc/include/asm/arch-mpc83xx/clock.h      | 22 --------------
 board/freescale/ls1021aiot/ls1021aiot.c            | 15 ----------
 board/freescale/ls1021aqds/ls1021aqds.c            | 14 ---------
 board/freescale/ls1021atwr/ls1021atwr.c            | 14 ---------
 drivers/mmc/fsl_esdhc.c                            | 34 +++-------------------
 10 files changed, 4 insertions(+), 130 deletions(-)
 delete mode 100644 arch/powerpc/include/asm/arch-mpc83xx/clock.h

Comments

Yangbo Lu Nov. 19, 2019, 1:56 a.m. UTC | #1 
Any comments?
Thanks.

> -----Original Message-----
> From: Yangbo Lu <yangbo.lu@nxp.com>
> Sent: Tuesday, November 12, 2019 7:29 PM
> To: u-boot@lists.denx.de
> Cc: Peng Fan <peng.fan@nxp.com>; Feng Li <feng.li_2@nxp.com>; Alison
> Wang <alison.wang@nxp.com>; Sumit Garg <sumit.garg@nxp.com>; Priyanka
> Jain <priyanka.jain@nxp.com>; Mario Six <mario.six@gdsys.cc>; Y.b. Lu
> <yangbo.lu@nxp.com>
> Subject: [PATCH 0/4] Drop redundant code for eSDHC clock getting
> 
> This patch-set is to drop redundant code for eSDHC clock getting.
> The fsl_esdhc driver is able to simply get clock from gd->arch.sdhc_clk.
> 
> The CI build link:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci
> .org%2Fyangbolu1991%2Fu-boot-test%2Fbuilds%2F610688748&amp;data=02
> %7C01%7Cyangbo.lu%40nxp.com%7C631a406c77594a0d737808d767637ae2
> %7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637091549269378
> 396&amp;sdata=7FT0Vn3oB0dMB5K1csOnNF5O9o61QxHheVtOeBJjDJw%3D
> &amp;reserved=0
> 
> Yangbo Lu (4):
>   mmc: fsl_esdhc: get clock directly from global data
>   arm: ls1021a: drop redundant board_mmc_init()
>   arm: drop eSDHC clock getting in mxc_get_clock() for layerscape
>   mpc83xx: remove unused clock.h
> 
>  arch/arm/cpu/armv7/ls102xa/clock.c                 |  2 --
>  .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c | 15
> ----------  .../arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c | 15 ----------
>  arch/arm/include/asm/arch-fsl-layerscape/clock.h   |  2 --
>  arch/arm/include/asm/arch-ls102xa/clock.h          |  1 -
>  arch/powerpc/include/asm/arch-mpc83xx/clock.h      | 22 --------------
>  board/freescale/ls1021aiot/ls1021aiot.c            | 15 ----------
>  board/freescale/ls1021aqds/ls1021aqds.c            | 14 ---------
>  board/freescale/ls1021atwr/ls1021atwr.c            | 14 ---------
>  drivers/mmc/fsl_esdhc.c                            | 34
> +++-------------------
>  10 files changed, 4 insertions(+), 130 deletions(-)  delete mode 100644
> arch/powerpc/include/asm/arch-mpc83xx/clock.h
> 
> --
> 2.7.4
Peng Fan Nov. 21, 2019, 1:57 a.m. UTC | #2 
> Subject: [PATCH 0/4] Drop redundant code for eSDHC clock getting
> 

After CI https://travis-ci.org/MrVan/u-boot/builds/614839218
It will show in mmc/master.

Thanks,
Peng.

> This patch-set is to drop redundant code for eSDHC clock getting.
> The fsl_esdhc driver is able to simply get clock from gd->arch.sdhc_clk.
> 
> The CI build link:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-
> ci.org%2Fyangbolu1991%2Fu-boot-test%2Fbuilds%2F610688748&amp;data=
> 02%7C01%7Cpeng.fan%40nxp.com%7C631a406c77594a0d737808d767637a
> e2%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6370915492695
> 48293&amp;sdata=M3JTaPLlBN2Cq8P3Fujgkv7pnN73EYlpyMjMSObr2Ys%3D
> &amp;reserved=0
> 
> Yangbo Lu (4):
>   mmc: fsl_esdhc: get clock directly from global data
>   arm: ls1021a: drop redundant board_mmc_init()
>   arm: drop eSDHC clock getting in mxc_get_clock() for layerscape
>   mpc83xx: remove unused clock.h
> 
>  arch/arm/cpu/armv7/ls102xa/clock.c                 |  2 --
>  .../arm/cpu/armv8/fsl-layerscape/fsl_lsch2_speed.c | 15
> ----------  .../arm/cpu/armv8/fsl-layerscape/fsl_lsch3_speed.c | 15 ----------
>  arch/arm/include/asm/arch-fsl-layerscape/clock.h   |  2 --
>  arch/arm/include/asm/arch-ls102xa/clock.h          |  1 -
>  arch/powerpc/include/asm/arch-mpc83xx/clock.h      | 22 --------------
>  board/freescale/ls1021aiot/ls1021aiot.c            | 15 ----------
>  board/freescale/ls1021aqds/ls1021aqds.c            | 14 ---------
>  board/freescale/ls1021atwr/ls1021atwr.c            | 14 ---------
>  drivers/mmc/fsl_esdhc.c                            | 34
> +++-------------------
>  10 files changed, 4 insertions(+), 130 deletions(-)  delete mode 100644
> arch/powerpc/include/asm/arch-mpc83xx/clock.h
> 
> --
> 2.7.4