Commit a33733fc authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

Merge branch 'for-next' of...

Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/mzx/devfreq into pm-devfreq

Pull devfreq fixes and documentation updates for v4.19 from MyungJoo Ham.

* 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/mzx/devfreq:
  PM / devfreq: rk3399_dmc: Fix duplicated opp table on reload.
  PM / devfreq: Init user limits from OPP limits, not viceversa
  PM / devfreq: rk3399_dmc: fix spelling mistakes.
  PM / devfreq: rk3399_dmc: do not print error when get supply and clk defer.
  dt-bindings: devfreq: rk3399_dmc: move interrupts to be optional.
  PM / devfreq: rk3399_dmc: remove wait for dcf irq event.
  dt-bindings: clock: add rk3399 DDR3 standard speed bins.
  dt-bindings: devfreq: rk3399_dmc: improve binding documentation.
  PM / devfreq: use put_device() instead of kfree()
  PM / devfreq: exynos-ppmu: Delete an error message for a failed memory allocation in exynos_ppmu_probe()
parents 9d3cce1e d6e98f3e
Loading
Loading
Loading
Loading
+108 −103
Original line number Original line Diff line number Diff line
@@ -3,12 +3,8 @@
Required properties:
Required properties:
- compatible:		 Must be "rockchip,rk3399-dmc".
- compatible:		 Must be "rockchip,rk3399-dmc".
- devfreq-events:	 Node to get DDR loading, Refer to
- devfreq-events:	 Node to get DDR loading, Refer to
			 Documentation/devicetree/bindings/devfreq/
			 Documentation/devicetree/bindings/devfreq/event/
			 rockchip-dfi.txt
			 rockchip-dfi.txt
- interrupts:		 The interrupt number to the CPU. The interrupt
			 specifier format depends on the interrupt controller.
			 It should be DCF interrupts, when DDR dvfs finish,
			 it will happen.
- clocks:		 Phandles for clock specified in "clock-names" property
- clocks:		 Phandles for clock specified in "clock-names" property
- clock-names :		 The name of clock used by the DFI, must be
- clock-names :		 The name of clock used by the DFI, must be
			 "pclk_ddr_mon";
			 "pclk_ddr_mon";
@@ -17,139 +13,148 @@ Required properties:
- center-supply:	 DMC supply node.
- center-supply:	 DMC supply node.
- status:		 Marks the node enabled/disabled.
- status:		 Marks the node enabled/disabled.


Following properties are ddr timing:
Optional properties:

- interrupts:		 The CPU interrupt number. The interrupt specifier
- rockchip,dram_speed_bin :	  Value reference include/dt-bindings/clock/ddr.h,
			 format depends on the interrupt controller.
				  it select ddr3 cl-trp-trcd type, default value
			 It should be a DCF interrupt. When DDR DVFS finishes
				  "DDR3_DEFAULT".it must selected according to
			 a DCF interrupt is triggered.
				  "Speed Bin" in ddr3 datasheet, DO NOT use

				  smaller "Speed Bin" than ddr3 exactly is.
Following properties relate to DDR timing:


- rockchip,pd_idle :		  Config the PD_IDLE value, defined the power-down
- rockchip,dram_speed_bin :	  Value reference include/dt-bindings/clock/rk3399-ddr.h,
				  idle period, memories are places into power-down
				  it selects the DDR3 cl-trp-trcd type. It must be
				  mode if bus is idle for PD_IDLE DFI clocks.
				  set according to "Speed Bin" in DDR3 datasheet,

				  DO NOT use a smaller "Speed Bin" than specified
- rockchip,sr_idle :		  Configure the SR_IDLE value, defined the
				  for the DDR3 being used.
				  selfrefresh idle period, memories are places

				  into self-refresh mode if bus is idle for
- rockchip,pd_idle :		  Configure the PD_IDLE value. Defines the
				  SR_IDLE*1024 DFI clocks (DFI clocks freq is
				  power-down idle period in which memories are
				  half of dram's clocks), defaule value is "0".
				  placed into power-down mode if bus is idle

				  for PD_IDLE DFI clock cycles.
- rockchip,sr_mc_gate_idle :	  Defined the self-refresh with memory and

				  controller clock gating idle period, memories
- rockchip,sr_idle :		  Configure the SR_IDLE value. Defines the
				  are places into self-refresh mode and memory
				  self-refresh idle period in which memories are
				  controller clock arg gating if bus is idle for
				  placed into self-refresh mode if bus is idle
				  sr_mc_gate_idle*1024 DFI clocks.
				  for SR_IDLE * 1024 DFI clock cycles (DFI

				  clocks freq is half of DRAM clock), default
- rockchip,srpd_lite_idle :	  Defined the self-refresh power down idle
				  value is "0".
				  period, memories are places into self-refresh

				  power down mode if bus is idle for
- rockchip,sr_mc_gate_idle :	  Defines the memory self-refresh and controller
				  srpd_lite_idle*1024 DFI clocks. This parameter
				  clock gating idle period. Memories are placed
				  is for LPDDR4 only.
				  into self-refresh mode and memory controller

				  clock arg gating started if bus is idle for
- rockchip,standby_idle :	  Defined the standby idle period, memories are
				  sr_mc_gate_idle*1024 DFI clock cycles.
				  places into self-refresh than controller, pi,

				  phy and dram clock will gating if bus is idle
- rockchip,srpd_lite_idle :	  Defines the self-refresh power down idle
				  for standby_idle * DFI clocks.
				  period in which memories are placed into

				  self-refresh power down mode if bus is idle
- rockchip,dram_dll_disb_freq :  It's defined the DDR3 dll bypass frequency in
				  for srpd_lite_idle * 1024 DFI clock cycles.
				  MHz, when ddr freq less than DRAM_DLL_DISB_FREQ,
				  This parameter is for LPDDR4 only.
				  ddr3 dll will bypssed note: if dll was bypassed,

				  the odt also stop working.
- rockchip,standby_idle :	  Defines the standby idle period in which

				  memories are placed into self-refresh mode.
- rockchip,phy_dll_disb_freq :	  Defined the PHY dll bypass frequency in
				  The controller, pi, PHY and DRAM clock will
				  MHz (Mega Hz), when ddr freq less than
				  be gated if bus is idle for standby_idle * DFI
				  DRAM_DLL_DISB_FREQ, phy dll will bypssed.
				  clock cycles.
				  note: phy dll and phy odt are independent.


- rockchip,dram_dll_dis_freq :	  Defines the DDR3 DLL bypass frequency in MHz.
- rockchip,ddr3_odt_disb_freq :  When dram type is DDR3, this parameter defined
				  When DDR frequency is less than DRAM_DLL_DISB_FREQ,
				  the odt disable frequency in MHz (Mega Hz),
				  DDR3 DLL will be bypassed. Note: if DLL was bypassed,
				  when ddr frequency less then ddr3_odt_disb_freq,
				  the odt will also stop working.
				  the odt on dram side and controller side are

- rockchip,phy_dll_dis_freq :	  Defines the PHY dll bypass frequency in
				  MHz (Mega Hz). When DDR frequency is less than
				  DRAM_DLL_DISB_FREQ, PHY DLL will be bypassed.
				  Note: PHY DLL and PHY ODT are independent.

- rockchip,ddr3_odt_dis_freq :	  When the DRAM type is DDR3, this parameter defines
				  the ODT disable frequency in MHz (Mega Hz).
				  when the DDR frequency is  less then ddr3_odt_dis_freq,
				  the ODT on the DRAM side and controller side are
				  both disabled.
				  both disabled.


- rockchip,ddr3_drv :		  When dram type is DDR3, this parameter define
- rockchip,ddr3_drv :		  When the DRAM type is DDR3, this parameter defines
				  the dram side driver stength in ohm, default
				  the DRAM side driver strength in ohms. Default
				  value is DDR3_DS_40ohm.
				  value is DDR3_DS_40ohm.


- rockchip,ddr3_odt :		  When dram type is DDR3, this parameter define
- rockchip,ddr3_odt :		  When the DRAM type is DDR3, this parameter defines
				  the dram side ODT stength in ohm, default value
				  the DRAM side ODT strength in ohms. Default value
				  is DDR3_ODT_120ohm.
				  is DDR3_ODT_120ohm.


- rockchip,phy_ddr3_ca_drv :	  When dram type is DDR3, this parameter define
- rockchip,phy_ddr3_ca_drv :	  When the DRAM type is DDR3, this parameter defines
				  the phy side CA line (incluing command line,
				  the phy side CA line (incluing command line,
				  address line and clock line) driver strength.
				  address line and clock line) driver strength.
				  Default value is PHY_DRV_ODT_40.
				  Default value is PHY_DRV_ODT_40.


- rockchip,phy_ddr3_dq_drv :	  When dram type is DDR3, this parameter define
- rockchip,phy_ddr3_dq_drv :	  When the DRAM type is DDR3, this parameter defines
				  the phy side DQ line(incluing DQS/DQ/DM line)
				  the PHY side DQ line (including DQS/DQ/DM line)
				  driver strength. default value is PHY_DRV_ODT_40.
				  driver strength. Default value is PHY_DRV_ODT_40.


- rockchip,phy_ddr3_odt : 	  When dram type is DDR3, this parameter define the
- rockchip,phy_ddr3_odt : 	  When the DRAM type is DDR3, this parameter defines
				  phy side odt strength, default value is
				  the PHY side ODT strength. Default value is
				  PHY_DRV_ODT_240.
				  PHY_DRV_ODT_240.


- rockchip,lpddr3_odt_disb_freq : When dram type is LPDDR3, this parameter defined
- rockchip,lpddr3_odt_dis_freq : When the DRAM type is LPDDR3, this parameter defines
				  then odt disable frequency in MHz (Mega Hz),
				  then ODT disable frequency in MHz (Mega Hz).
				  when ddr frequency less then ddr3_odt_disb_freq,
				  When DDR frequency is less then ddr3_odt_dis_freq,
				  the odt on dram side and controller side are
				  the ODT on the DRAM side and controller side are
				  both disabled.
				  both disabled.


- rockchip,lpddr3_drv :	  When dram type is LPDDR3, this parameter define
- rockchip,lpddr3_drv :		  When the DRAM type is LPDDR3, this parameter defines
				  the dram side driver stength in ohm, default
				  the DRAM side driver strength in ohms. Default
				  value is LP3_DS_34ohm.
				  value is LP3_DS_34ohm.


- rockchip,lpddr3_odt :	  When dram type is LPDDR3, this parameter define
- rockchip,lpddr3_odt :		  When the DRAM type is LPDDR3, this parameter defines
				  the dram side ODT stength in ohm, default value
				  the DRAM side ODT strength in ohms. Default value
				  is LP3_ODT_240ohm.
				  is LP3_ODT_240ohm.


- rockchip,phy_lpddr3_ca_drv :	  When dram type is LPDDR3, this parameter define
- rockchip,phy_lpddr3_ca_drv :	  When the DRAM type is LPDDR3, this parameter defines
				  the phy side CA line(incluing command line,
				  the PHY side CA line (including command line,
				  address line and clock line) driver strength.
				  address line and clock line) driver strength.
				  default value is PHY_DRV_ODT_40.
				  Default value is PHY_DRV_ODT_40.


- rockchip,phy_lpddr3_dq_drv :	  When dram type is LPDDR3, this parameter define
- rockchip,phy_lpddr3_dq_drv :	  When the DRAM type is LPDDR3, this parameter defines
				  the phy side DQ line(incluing DQS/DQ/DM line)
				  the PHY side DQ line (including DQS/DQ/DM line)
				  driver strength. default value is
				  driver strength. Default value is
				  PHY_DRV_ODT_40.
				  PHY_DRV_ODT_40.


- rockchip,phy_lpddr3_odt : 	  When dram type is LPDDR3, this parameter define
- rockchip,phy_lpddr3_odt : 	  When dram type is LPDDR3, this parameter define
				  the phy side odt strength, default value is
				  the phy side odt strength, default value is
				  PHY_DRV_ODT_240.
				  PHY_DRV_ODT_240.


- rockchip,lpddr4_odt_disb_freq : When dram type is LPDDR4, this parameter
- rockchip,lpddr4_odt_dis_freq : When the DRAM type is LPDDR4, this parameter
				  defined the odt disable frequency in
				  defines the ODT disable frequency in
				  MHz (Mega Hz), when ddr frequency less then
				  MHz (Mega Hz). When the DDR frequency is less then
				  ddr3_odt_disb_freq, the odt on dram side and
				  ddr3_odt_dis_freq, the ODT on the DRAM side and
				  controller side are both disabled.
				  controller side are both disabled.


- rockchip,lpddr4_drv :	  When dram type is LPDDR4, this parameter define
- rockchip,lpddr4_drv :		  When the DRAM type is LPDDR4, this parameter defines
				  the dram side driver stength in ohm, default
				  the DRAM side driver strength in ohms. Default
				  value is LP4_PDDS_60ohm.
				  value is LP4_PDDS_60ohm.


- rockchip,lpddr4_dq_odt : 	  When dram type is LPDDR4, this parameter define
- rockchip,lpddr4_dq_odt : 	  When the DRAM type is LPDDR4, this parameter defines
				  the dram side ODT on dqs/dq line stength in ohm,
				  the DRAM side ODT on DQS/DQ line strength in ohms.
				  default value is LP4_DQ_ODT_40ohm.
				  Default value is LP4_DQ_ODT_40ohm.


- rockchip,lpddr4_ca_odt :	  When dram type is LPDDR4, this parameter define
- rockchip,lpddr4_ca_odt :	  When the DRAM type is LPDDR4, this parameter defines
				  the dram side ODT on ca line stength in ohm,
				  the DRAM side ODT on CA line strength in ohms.
				  default value is LP4_CA_ODT_40ohm.
				  Default value is LP4_CA_ODT_40ohm.


- rockchip,phy_lpddr4_ca_drv :	  When dram type is LPDDR4, this parameter define
- rockchip,phy_lpddr4_ca_drv :	  When the DRAM type is LPDDR4, this parameter defines
				  the phy side  CA line(incluing command address
				  the PHY side CA line (including command address
				  line) driver strength. default value is
				  line) driver strength. Default value is
				  PHY_DRV_ODT_40.
				  PHY_DRV_ODT_40.


- rockchip,phy_lpddr4_ck_cs_drv : When dram type is LPDDR4, this parameter define
- rockchip,phy_lpddr4_ck_cs_drv : When the DRAM type is LPDDR4, this parameter defines
				  the phy side clock line and cs line driver
				  the PHY side clock line and CS line driver
				  strength. default value is PHY_DRV_ODT_80.
				  strength. Default value is PHY_DRV_ODT_80.


- rockchip,phy_lpddr4_dq_drv :	  When dram type is LPDDR4, this parameter define
- rockchip,phy_lpddr4_dq_drv :	  When the DRAM type is LPDDR4, this parameter defines
				  the phy side DQ line(incluing DQS/DQ/DM line)
				  the PHY side DQ line (including DQS/DQ/DM line)
				  driver strength. default value is PHY_DRV_ODT_80.
				  driver strength. Default value is PHY_DRV_ODT_80.


- rockchip,phy_lpddr4_odt :	  When dram type is LPDDR4, this parameter define
- rockchip,phy_lpddr4_odt :	  When the DRAM type is LPDDR4, this parameter defines
				  the phy side odt strength, default value is
				  the PHY side ODT strength. Default value is
				  PHY_DRV_ODT_60.
				  PHY_DRV_ODT_60.


Example:
Example:
+9 −7
Original line number Original line Diff line number Diff line
@@ -604,28 +604,29 @@ struct devfreq *devfreq_add_device(struct device *dev,
		mutex_lock(&devfreq->lock);
		mutex_lock(&devfreq->lock);
	}
	}


	devfreq->min_freq = find_available_min_freq(devfreq);
	devfreq->scaling_min_freq = find_available_min_freq(devfreq);
	if (!devfreq->min_freq) {
	if (!devfreq->scaling_min_freq) {
		mutex_unlock(&devfreq->lock);
		mutex_unlock(&devfreq->lock);
		err = -EINVAL;
		err = -EINVAL;
		goto err_dev;
		goto err_dev;
	}
	}
	devfreq->scaling_min_freq = devfreq->min_freq;
	devfreq->min_freq = devfreq->scaling_min_freq;


	devfreq->max_freq = find_available_max_freq(devfreq);
	devfreq->scaling_max_freq = find_available_max_freq(devfreq);
	if (!devfreq->max_freq) {
	if (!devfreq->scaling_max_freq) {
		mutex_unlock(&devfreq->lock);
		mutex_unlock(&devfreq->lock);
		err = -EINVAL;
		err = -EINVAL;
		goto err_dev;
		goto err_dev;
	}
	}
	devfreq->scaling_max_freq = devfreq->max_freq;
	devfreq->max_freq = devfreq->scaling_max_freq;


	dev_set_name(&devfreq->dev, "devfreq%d",
	dev_set_name(&devfreq->dev, "devfreq%d",
				atomic_inc_return(&devfreq_no));
				atomic_inc_return(&devfreq_no));
	err = device_register(&devfreq->dev);
	err = device_register(&devfreq->dev);
	if (err) {
	if (err) {
		mutex_unlock(&devfreq->lock);
		mutex_unlock(&devfreq->lock);
		goto err_dev;
		put_device(&devfreq->dev);
		goto err_out;
	}
	}


	devfreq->trans_table =
	devfreq->trans_table =
@@ -672,6 +673,7 @@ err_init:
	mutex_unlock(&devfreq_list_lock);
	mutex_unlock(&devfreq_list_lock);


	device_unregister(&devfreq->dev);
	device_unregister(&devfreq->dev);
	devfreq = NULL;
err_dev:
err_dev:
	if (devfreq)
	if (devfreq)
		kfree(devfreq);
		kfree(devfreq);
+2 −4
Original line number Original line Diff line number Diff line
@@ -627,11 +627,9 @@ static int exynos_ppmu_probe(struct platform_device *pdev)


	size = sizeof(struct devfreq_event_dev *) * info->num_events;
	size = sizeof(struct devfreq_event_dev *) * info->num_events;
	info->edev = devm_kzalloc(&pdev->dev, size, GFP_KERNEL);
	info->edev = devm_kzalloc(&pdev->dev, size, GFP_KERNEL);
	if (!info->edev) {
	if (!info->edev)
		dev_err(&pdev->dev,
			"failed to allocate memory devfreq-event devices\n");
		return -ENOMEM;
		return -ENOMEM;
	}

	edev = info->edev;
	edev = info->edev;
	platform_set_drvdata(pdev, info);
	platform_set_drvdata(pdev, info);


+40 −62
Original line number Original line Diff line number Diff line
@@ -68,15 +68,6 @@ struct rk3399_dmcfreq {
	struct devfreq_event_dev *edev;
	struct devfreq_event_dev *edev;
	struct mutex lock;
	struct mutex lock;
	struct dram_timing timing;
	struct dram_timing timing;

	/*
	 * DDR Converser of Frequency (DCF) is used to implement DDR frequency
	 * conversion without the participation of CPU, we will implement and
	 * control it in arm trust firmware.
	 */
	wait_queue_head_t	wait_dcf_queue;
	int irq;
	int wait_dcf_flag;
	struct regulator *vdd_center;
	struct regulator *vdd_center;
	unsigned long rate, target_rate;
	unsigned long rate, target_rate;
	unsigned long volt, target_volt;
	unsigned long volt, target_volt;
@@ -112,30 +103,21 @@ static int rk3399_dmcfreq_target(struct device *dev, unsigned long *freq,
		err = regulator_set_voltage(dmcfreq->vdd_center, target_volt,
		err = regulator_set_voltage(dmcfreq->vdd_center, target_volt,
					    target_volt);
					    target_volt);
		if (err) {
		if (err) {
			dev_err(dev, "Cannot to set voltage %lu uV\n",
			dev_err(dev, "Cannot set voltage %lu uV\n",
				target_volt);
				target_volt);
			goto out;
			goto out;
		}
		}
	}
	}
	dmcfreq->wait_dcf_flag = 1;


	err = clk_set_rate(dmcfreq->dmc_clk, target_rate);
	err = clk_set_rate(dmcfreq->dmc_clk, target_rate);
	if (err) {
	if (err) {
		dev_err(dev, "Cannot to set frequency %lu (%d)\n",
		dev_err(dev, "Cannot set frequency %lu (%d)\n", target_rate,
			target_rate, err);
			err);
		regulator_set_voltage(dmcfreq->vdd_center, dmcfreq->volt,
		regulator_set_voltage(dmcfreq->vdd_center, dmcfreq->volt,
				      dmcfreq->volt);
				      dmcfreq->volt);
		goto out;
		goto out;
	}
	}


	/*
	 * Wait until bcf irq happen, it means freq scaling finish in
	 * arm trust firmware, use 100ms as timeout time.
	 */
	if (!wait_event_timeout(dmcfreq->wait_dcf_queue,
				!dmcfreq->wait_dcf_flag, HZ / 10))
		dev_warn(dev, "Timeout waiting for dcf interrupt\n");

	/*
	/*
	 * Check the dpll rate,
	 * Check the dpll rate,
	 * There only two result we will get,
	 * There only two result we will get,
@@ -146,8 +128,8 @@ static int rk3399_dmcfreq_target(struct device *dev, unsigned long *freq,


	/* If get the incorrect rate, set voltage to old value. */
	/* If get the incorrect rate, set voltage to old value. */
	if (dmcfreq->rate != target_rate) {
	if (dmcfreq->rate != target_rate) {
		dev_err(dev, "Get wrong ddr frequency, Request frequency %lu,\
		dev_err(dev, "Got wrong frequency, Request %lu, Current %lu\n",
			Current frequency %lu\n", target_rate, dmcfreq->rate);
			target_rate, dmcfreq->rate);
		regulator_set_voltage(dmcfreq->vdd_center, dmcfreq->volt,
		regulator_set_voltage(dmcfreq->vdd_center, dmcfreq->volt,
				      dmcfreq->volt);
				      dmcfreq->volt);
		goto out;
		goto out;
@@ -155,7 +137,7 @@ static int rk3399_dmcfreq_target(struct device *dev, unsigned long *freq,
		err = regulator_set_voltage(dmcfreq->vdd_center, target_volt,
		err = regulator_set_voltage(dmcfreq->vdd_center, target_volt,
					    target_volt);
					    target_volt);
	if (err)
	if (err)
		dev_err(dev, "Cannot to set vol %lu uV\n", target_volt);
		dev_err(dev, "Cannot set voltage %lu uV\n", target_volt);


	dmcfreq->rate = target_rate;
	dmcfreq->rate = target_rate;
	dmcfreq->volt = target_volt;
	dmcfreq->volt = target_volt;
@@ -241,22 +223,6 @@ static __maybe_unused int rk3399_dmcfreq_resume(struct device *dev)
static SIMPLE_DEV_PM_OPS(rk3399_dmcfreq_pm, rk3399_dmcfreq_suspend,
static SIMPLE_DEV_PM_OPS(rk3399_dmcfreq_pm, rk3399_dmcfreq_suspend,
			 rk3399_dmcfreq_resume);
			 rk3399_dmcfreq_resume);


static irqreturn_t rk3399_dmc_irq(int irq, void *dev_id)
{
	struct rk3399_dmcfreq *dmcfreq = dev_id;
	struct arm_smccc_res res;

	dmcfreq->wait_dcf_flag = 0;
	wake_up(&dmcfreq->wait_dcf_queue);

	/* Clear the DCF interrupt */
	arm_smccc_smc(ROCKCHIP_SIP_DRAM_FREQ, 0, 0,
		      ROCKCHIP_SIP_CONFIG_DRAM_CLR_IRQ,
		      0, 0, 0, 0, &res);

	return IRQ_HANDLED;
}

static int of_get_ddr_timings(struct dram_timing *timing,
static int of_get_ddr_timings(struct dram_timing *timing,
			      struct device_node *np)
			      struct device_node *np)
{
{
@@ -330,16 +296,10 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev)
	struct device *dev = &pdev->dev;
	struct device *dev = &pdev->dev;
	struct device_node *np = pdev->dev.of_node;
	struct device_node *np = pdev->dev.of_node;
	struct rk3399_dmcfreq *data;
	struct rk3399_dmcfreq *data;
	int ret, irq, index, size;
	int ret, index, size;
	uint32_t *timing;
	uint32_t *timing;
	struct dev_pm_opp *opp;
	struct dev_pm_opp *opp;


	irq = platform_get_irq(pdev, 0);
	if (irq < 0) {
		dev_err(&pdev->dev,
			"Cannot get the dmc interrupt resource: %d\n", irq);
		return irq;
	}
	data = devm_kzalloc(dev, sizeof(struct rk3399_dmcfreq), GFP_KERNEL);
	data = devm_kzalloc(dev, sizeof(struct rk3399_dmcfreq), GFP_KERNEL);
	if (!data)
	if (!data)
		return -ENOMEM;
		return -ENOMEM;
@@ -348,27 +308,22 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev)


	data->vdd_center = devm_regulator_get(dev, "center");
	data->vdd_center = devm_regulator_get(dev, "center");
	if (IS_ERR(data->vdd_center)) {
	if (IS_ERR(data->vdd_center)) {
		if (PTR_ERR(data->vdd_center) == -EPROBE_DEFER)
			return -EPROBE_DEFER;

		dev_err(dev, "Cannot get the regulator \"center\"\n");
		dev_err(dev, "Cannot get the regulator \"center\"\n");
		return PTR_ERR(data->vdd_center);
		return PTR_ERR(data->vdd_center);
	}
	}


	data->dmc_clk = devm_clk_get(dev, "dmc_clk");
	data->dmc_clk = devm_clk_get(dev, "dmc_clk");
	if (IS_ERR(data->dmc_clk)) {
	if (IS_ERR(data->dmc_clk)) {
		if (PTR_ERR(data->dmc_clk) == -EPROBE_DEFER)
			return -EPROBE_DEFER;

		dev_err(dev, "Cannot get the clk dmc_clk\n");
		dev_err(dev, "Cannot get the clk dmc_clk\n");
		return PTR_ERR(data->dmc_clk);
		return PTR_ERR(data->dmc_clk);
	};
	};


	data->irq = irq;
	ret = devm_request_irq(dev, irq, rk3399_dmc_irq, 0,
			       dev_name(dev), data);
	if (ret) {
		dev_err(dev, "Failed to request dmc irq: %d\n", ret);
		return ret;
	}

	init_waitqueue_head(&data->wait_dcf_queue);
	data->wait_dcf_flag = 0;

	data->edev = devfreq_event_get_edev_by_phandle(dev, 0);
	data->edev = devfreq_event_get_edev_by_phandle(dev, 0);
	if (IS_ERR(data->edev))
	if (IS_ERR(data->edev))
		return -EPROBE_DEFER;
		return -EPROBE_DEFER;
@@ -420,8 +375,10 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev)
	data->rate = clk_get_rate(data->dmc_clk);
	data->rate = clk_get_rate(data->dmc_clk);


	opp = devfreq_recommended_opp(dev, &data->rate, 0);
	opp = devfreq_recommended_opp(dev, &data->rate, 0);
	if (IS_ERR(opp))
	if (IS_ERR(opp)) {
		return PTR_ERR(opp);
		ret = PTR_ERR(opp);
		goto err_free_opp;
	}


	data->rate = dev_pm_opp_get_freq(opp);
	data->rate = dev_pm_opp_get_freq(opp);
	data->volt = dev_pm_opp_get_voltage(opp);
	data->volt = dev_pm_opp_get_voltage(opp);
@@ -433,13 +390,33 @@ static int rk3399_dmcfreq_probe(struct platform_device *pdev)
					   &rk3399_devfreq_dmc_profile,
					   &rk3399_devfreq_dmc_profile,
					   DEVFREQ_GOV_SIMPLE_ONDEMAND,
					   DEVFREQ_GOV_SIMPLE_ONDEMAND,
					   &data->ondemand_data);
					   &data->ondemand_data);
	if (IS_ERR(data->devfreq))
	if (IS_ERR(data->devfreq)) {
		return PTR_ERR(data->devfreq);
		ret = PTR_ERR(data->devfreq);
		goto err_free_opp;
	}

	devm_devfreq_register_opp_notifier(dev, data->devfreq);
	devm_devfreq_register_opp_notifier(dev, data->devfreq);


	data->dev = dev;
	data->dev = dev;
	platform_set_drvdata(pdev, data);
	platform_set_drvdata(pdev, data);


	return 0;

err_free_opp:
	dev_pm_opp_of_remove_table(&pdev->dev);
	return ret;
}

static int rk3399_dmcfreq_remove(struct platform_device *pdev)
{
	struct rk3399_dmcfreq *dmcfreq = dev_get_drvdata(&pdev->dev);

	/*
	 * Before remove the opp table we need to unregister the opp notifier.
	 */
	devm_devfreq_unregister_opp_notifier(dmcfreq->dev, dmcfreq->devfreq);
	dev_pm_opp_of_remove_table(dmcfreq->dev);

	return 0;
	return 0;
}
}


@@ -451,6 +428,7 @@ MODULE_DEVICE_TABLE(of, rk3399dmc_devfreq_of_match);


static struct platform_driver rk3399_dmcfreq_driver = {
static struct platform_driver rk3399_dmcfreq_driver = {
	.probe	= rk3399_dmcfreq_probe,
	.probe	= rk3399_dmcfreq_probe,
	.remove = rk3399_dmcfreq_remove,
	.driver = {
	.driver = {
		.name	= "rk3399-dmc-freq",
		.name	= "rk3399-dmc-freq",
		.pm	= &rk3399_dmcfreq_pm,
		.pm	= &rk3399_dmcfreq_pm,
+56 −0
Original line number Original line Diff line number Diff line
/* SPDX-License-Identifier: (GPL-2.0+ OR MIT) */

#ifndef DT_BINDINGS_DDR_H
#define DT_BINDINGS_DDR_H

/*
 * DDR3 SDRAM Standard Speed Bins include tCK, tRCD, tRP, tRAS and tRC for
 * each corresponding bin.
 */

/* DDR3-800 (5-5-5) */
#define DDR3_800D	0
/* DDR3-800 (6-6-6) */
#define DDR3_800E	1
/* DDR3-1066 (6-6-6) */
#define DDR3_1066E	2
/* DDR3-1066 (7-7-7) */
#define DDR3_1066F	3
/* DDR3-1066 (8-8-8) */
#define DDR3_1066G	4
/* DDR3-1333 (7-7-7) */
#define DDR3_1333F	5
/* DDR3-1333 (8-8-8) */
#define DDR3_1333G	6
/* DDR3-1333 (9-9-9) */
#define DDR3_1333H	7
/* DDR3-1333 (10-10-10) */
#define DDR3_1333J 	8
/* DDR3-1600 (8-8-8) */
#define DDR3_1600G	9
/* DDR3-1600 (9-9-9) */
#define DDR3_1600H	10
/* DDR3-1600 (10-10-10) */
#define DDR3_1600J	11
/* DDR3-1600 (11-11-11) */
#define DDR3_1600K	12
/* DDR3-1600 (10-10-10) */
#define DDR3_1866J	13
/* DDR3-1866 (11-11-11) */
#define DDR3_1866K	14
/* DDR3-1866 (12-12-12) */
#define DDR3_1866L	15
/* DDR3-1866 (13-13-13) */
#define DDR3_1866M	16
/* DDR3-2133 (11-11-11) */
#define DDR3_2133K	17
/* DDR3-2133 (12-12-12) */
#define DDR3_2133L	18
/* DDR3-2133 (13-13-13) */
#define DDR3_2133M	19
/* DDR3-2133 (14-14-14) */
#define DDR3_2133N	20
/* DDR3 ATF default */
#define DDR3_DEFAULT	21

#endif