On 04/25/2016 08:03 AM, Rob Herring wrote: > On Fri, Apr 22, 2016 at 11:40:34AM -0500, Franklin S Cooper Jr wrote: >> Switch to a new ECAP and EPWM bindings that doesn't depend on hwmod to >> provide the various required clocks. >> >> For AM437 and AM335x, add the required clocks explicitly to DT. The >> hwmod entries for ECAP and EPWM will be removed and this will prevent >> anything from breaking. >> >> Signed-off-by: Franklin S Cooper Jr <fcooper@xxxxxx> >> --- >> .../devicetree/bindings/pwm/pwm-tiecap.txt | 12 +++-- >> .../devicetree/bindings/pwm/pwm-tiehrpwm.txt | 12 +++-- >> arch/arm/boot/dts/am33xx.dtsi | 30 +++++++++--- >> arch/arm/boot/dts/am4372.dtsi | 54 ++++++++++++++++++---- >> arch/arm/boot/dts/da850.dtsi | 15 ++++-- >> 5 files changed, 95 insertions(+), 28 deletions(-) > > [...] > >> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi >> index 12fcde4..e858460 100644 >> --- a/arch/arm/boot/dts/am4372.dtsi >> +++ b/arch/arm/boot/dts/am4372.dtsi >> @@ -672,18 +672,26 @@ >> status = "disabled"; >> >> ecap0: ecap@48300100 { >> - compatible = "ti,am4372-ecap","ti,am33xx-ecap"; >> + compatible = "ti,am4372-ecap", > > This doesn't appear to be documented. > >> + "ti,am3352-ecap", >> + "ti,am33xx-ecap"; >> #pwm-cells = <3>; >> reg = <0x48300100 0x80>; >> ti,hwmods = "ecap0"; >> + clocks = <&l4ls_gclk>; >> + clock-names = "fck"; >> status = "disabled"; >> }; >> >> ehrpwm0: pwm@48300200 { >> - compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm"; >> + compatible = "ti,am4372-ehrpwm", > > ditto. Will fix > >> + "ti,am3352-ehrpwm", >> + "ti,am33xx-ehrpwm"; >> #pwm-cells = <3>; >> reg = <0x48300200 0x80>; >> ti,hwmods = "ehrpwm0"; >> + clocks = <&ehrpwm0_tbclk>, <&l4ls_gclk>; >> + clock-names = "tbclk", "fck"; >> status = "disabled"; >> }; >> }; -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html