Re: [PATCH 3/7] mfd: db8500-prcmu: Remove unused *prcmu_set_ddr_opp() calls

[Ulf Hansson]

On 15 September 2016 at 12:45, Lee Jones <lee.jones@xxxxxxxxxx> wrote:
> There are no call sites for these functions.  Strip them out.
>
> Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
> ---
>  drivers/mfd/db8500-prcmu.c       | 19 -------------------
>  include/linux/mfd/db8500-prcmu.h |  6 ------
>  include/linux/mfd/dbx500-prcmu.h |  9 ---------
>  3 files changed, 34 deletions(-)
>
> diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
> index 388e268..ca38a6a 100644
> --- a/drivers/mfd/db8500-prcmu.c
> +++ b/drivers/mfd/db8500-prcmu.c
> @@ -938,25 +938,6 @@ int db8500_prcmu_get_ddr_opp(void)
>         return readb(PRCM_DDR_SUBSYS_APE_MINBW);
>  }
>
> -/**
> - * db8500_set_ddr_opp - set the appropriate DDR OPP
> - * @opp: The new DDR operating point to which transition is to be made
> - * Returns: 0 on success, non-zero on failure
> - *
> - * This function sets the operating point of the DDR.
> - */
> -static bool enable_set_ddr_opp;
> -int db8500_prcmu_set_ddr_opp(u8 opp)
> -{
> -       if (opp < DDR_100_OPP || opp > DDR_25_OPP)
> -               return -EINVAL;
> -       /* Changing the DDR OPP can hang the hardware pre-v21 */
> -       if (enable_set_ddr_opp)
> -               writeb(opp, PRCM_DDR_SUBSYS_APE_MINBW);
> -
> -       return 0;
> -}
> -

This function was actually used in the vendor tree to deal with DVFS,
implemented a system wide PM QoS thing.
I guess those hacks never made it upstream. :-)

>  /* Divide the frequency of certain clocks by 2 for APE_50_PARTLY_25_OPP. */
>  static void request_even_slower_clocks(bool enable)
>  {
> diff --git a/include/linux/mfd/db8500-prcmu.h b/include/linux/mfd/db8500-prcmu.h
> index 0bd6944..7ba67b5 100644
> --- a/include/linux/mfd/db8500-prcmu.h
> +++ b/include/linux/mfd/db8500-prcmu.h
> @@ -538,7 +538,6 @@ int db8500_prcmu_get_arm_opp(void);
>  int db8500_prcmu_set_ape_opp(u8 opp);
>  int db8500_prcmu_get_ape_opp(void);
>  int db8500_prcmu_request_ape_opp_100_voltage(bool enable);
> -int db8500_prcmu_set_ddr_opp(u8 opp);
>  int db8500_prcmu_get_ddr_opp(void);
>
>  u32 db8500_prcmu_read(unsigned int reg);
> @@ -594,11 +593,6 @@ static inline int prcmu_release_usb_wakeup_state(void)
>         return 0;
>  }
>
> -static inline int db8500_prcmu_set_ddr_opp(u8 opp)
> -{
> -       return 0;
> -}
> -
>  static inline int db8500_prcmu_get_ddr_opp(void)
>  {
>         return DDR_100_OPP;
> diff --git a/include/linux/mfd/dbx500-prcmu.h b/include/linux/mfd/dbx500-prcmu.h
> index 5d37460..2e2c6a6 100644
> --- a/include/linux/mfd/dbx500-prcmu.h
> +++ b/include/linux/mfd/dbx500-prcmu.h
> @@ -269,10 +269,6 @@ unsigned long prcmu_clock_rate(u8 clock);
>  long prcmu_round_clock_rate(u8 clock, unsigned long rate);
>  int prcmu_set_clock_rate(u8 clock, unsigned long rate);
>
> -static inline int prcmu_set_ddr_opp(u8 opp)
> -{
> -       return db8500_prcmu_set_ddr_opp(opp);
> -}
>  static inline int prcmu_get_ddr_opp(void)
>  {
>         return db8500_prcmu_get_ddr_opp();
> @@ -489,11 +485,6 @@ static inline int prcmu_get_arm_opp(void)
>         return ARM_100_OPP;
>  }
>
> -static inline int prcmu_set_ddr_opp(u8 opp)
> -{
> -       return 0;
> -}
> -
>  static inline int prcmu_get_ddr_opp(void)
>  {
>         return DDR_100_OPP;
> --
> 2.9.3
>

If not too late:

Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>

Kind regards
Uffe