Re: [RFC v2 0/3] OMAP3: convert opp-v1 to opp-v2 and read speed binned / 720MHz grade bits

From: Viresh Kumar
Date: Thu Sep 05 2019 - 01:03:08 EST


On 04-09-19, 10:53, H. Nikolaus Schaller wrote:
> Changes V2:
> * merge separate patch to remove opp-v1 table from n950-n9 into
> the general omap3xxx.dtsi patch
> * add legacy compatibility to ti,omap3430 and ti,omap3630 for
> the ti-cpufreq driver
> * make driver and omap3xxx.dtsi patches pass checkpatch
> * add bulk patch to explicitly define compatibility to ti,omap3430
> and ti,omap36xx in addition to ti,omap3 of all in-tree boards
> where it was missing
>
> RFC V1 2019-09-02 12:55:55:
>
> This patch set converts the opp tables to opp-v2 format
> and extends the ti-cpufreq to support omap3.
>
> It adds 720 MHz (omap34xx) and 1 GHz (omap36xx) OPPs but
> tells the ti-cpufreq driver to disable them if the speed
> binned / 720MHz grade eFuse bits indicate that the chip
> is not rated for that speed.
>
> It has been tested (for chip variant detection, not reliability
> of the high speed OPPs) on:
>
> * BeagleBoard C2 (omap3430 600MHz)
> * BeagleBoard XM B (dm3730 800MHz)
> * GTA04A4 (dm3730 800MHz)
> * GTA04A5 (dm3730 1GHz)
>
>
> H. Nikolaus Schaller (3):
> cpufreq: ti-cpufreq: add support for omap34xx and omap36xx
> ARM: dts: replace opp-v1 tables by opp-v2 for omap34xx and omap36xx
> ARM: dts: omap3: bulk convert compatible to be explicitly ti,omap3430
> or ti,omap36xx
>
> arch/arm/boot/dts/am3517_mt_ventoux.dts | 2 +-
> .../boot/dts/logicpd-som-lv-35xx-devkit.dts | 2 +-
> .../boot/dts/logicpd-som-lv-37xx-devkit.dts | 2 +-
> .../boot/dts/logicpd-torpedo-35xx-devkit.dts | 2 +-
> .../boot/dts/logicpd-torpedo-37xx-devkit.dts | 2 +-
> arch/arm/boot/dts/omap3-beagle.dts | 2 +-
> arch/arm/boot/dts/omap3-cm-t3530.dts | 2 +-
> arch/arm/boot/dts/omap3-devkit8000-lcd43.dts | 2 +-
> arch/arm/boot/dts/omap3-devkit8000-lcd70.dts | 2 +-
> arch/arm/boot/dts/omap3-devkit8000.dts | 2 +-
> arch/arm/boot/dts/omap3-evm-37xx.dts | 2 +-
> arch/arm/boot/dts/omap3-ha-lcd.dts | 2 +-
> arch/arm/boot/dts/omap3-ha.dts | 2 +-
> arch/arm/boot/dts/omap3-ldp.dts | 2 +-
> arch/arm/boot/dts/omap3-n950-n9.dtsi | 7 --
> arch/arm/boot/dts/omap3-sbc-t3530.dts | 2 +-
> arch/arm/boot/dts/omap3-thunder.dts | 2 +-
> arch/arm/boot/dts/omap3430-sdp.dts | 2 +-
> arch/arm/boot/dts/omap34xx.dtsi | 65 ++++++++++++--
> arch/arm/boot/dts/omap36xx.dtsi | 53 +++++++++--
> drivers/cpufreq/cpufreq-dt-platdev.c | 2 +-
> drivers/cpufreq/ti-cpufreq.c | 87 ++++++++++++++++++-
> 22 files changed, 204 insertions(+), 44 deletions(-)

Most of the stuff looks fine to me here. I will pick the patches when
the SoC maintainers provide an Ack.

--
viresh