BBB and BBAI compatible BBORG motorCape overlay #188
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,8 @@ | ||
| /* | ||
| * Copyright (C) 2020 Deepak Khatri <[email protected]> | ||
| * | ||
| * Based on BBORG_MOTOR-00A2.dts from Robert Nelson for kernel <4.14 | ||
| * Note: Requires Compaibility code from BeagleBoard-DeviceTrees/pull/17 | ||
| * | ||
| * This program is free software; you can redistribute it and/or modify | ||
| * it under the terms of the GNU General Public License version 2 as | ||
|
|
@@ -9,135 +12,97 @@ | |
| /dts-v1/; | ||
| /plugin/; | ||
|
|
||
| /* | ||
| * Helper to show loaded overlays under: /proc/device-tree/chosen/overlays/ | ||
| */ | ||
| &{/chosen} { | ||
| overlays { | ||
| BBORG_MOTOR-00A2 = __TIMESTAMP__; | ||
| }; | ||
| }; | ||
|
|
||
| /* | ||
| * Free up the pins used by the cape from the pinmux helpers. | ||
| */ | ||
| &ocp { | ||
| P8_13_pinmux { status = "disabled"; }; /* motor3 speed */ | ||
| P8_14_pinmux { status = "disabled"; }; /* motor3 direction */ | ||
| P8_16_pinmux { status = "disabled"; }; /* motor2 direction */ | ||
| P8_18_pinmux { status = "disabled"; }; /* motor1 direction */ | ||
| P8_19_pinmux { status = "disabled"; }; /* motor4 speed */ | ||
| P8_26_pinmux { status = "disabled"; }; /* motor4 direction */ | ||
| P9_14_pinmux { status = "disabled"; }; /* motor2 speed */ | ||
| P9_16_pinmux { status = "disabled"; }; /* motor1 speed */ | ||
| }; | ||
|
|
||
| /* | ||
| * Enable and Update the default state of the pins | ||
| */ | ||
| &ocp { | ||
| P8_13_pinmux { status = "okay"; pinctrl-0 = <&P8_13_pwm_pin>; }; /* motor3 speed */ | ||
|
There was a problem hiding this comment. Given this approach, I don't think disabling the pinmux helper is needed. As only loosely mentioned in the comment, what you are doing here is changing the default mode with the pinmux helper, rather than using the driver to simply define the mode. This may be an OK approach, but it is a bit error-prone if someone messes with the pinmux helper. This change needs to be a bit better documented for users in some kind of changelog. There was a problem hiding this comment. Yes, you are right we don't need those disabling I will try things without it. There was a much bigger problem in the previous implementation method. we were limiting users to use labels we defined for the buses. With this approach and good documentation, the user can create new overlays without chaing in the compatibility code. I will write the documentation soon :) |
||
| P8_14_pinmux { status = "okay"; pinctrl-0 = <&P8_14_gpio_pin>; }; /* motor3 direction */ | ||
| P8_16_pinmux { status = "okay"; pinctrl-0 = <&P8_16_gpio_pin>; }; /* motor2 direction */ | ||
| P8_18_pinmux { status = "okay"; pinctrl-0 = <&P8_18_gpio_pin>; }; /* motor1 direction */ | ||
| P8_19_pinmux { status = "okay"; pinctrl-0 = <&P8_19_pwm_pin>; }; /* motor4 speed */ | ||
| P8_26_pinmux { status = "okay"; pinctrl-0 = <&P8_26_gpio_pin>; }; /* motor4 direction */ | ||
| P9_14_pinmux { status = "okay"; pinctrl-0 = <&P9_14_pwm_pin>; }; /* motor2 speed */ | ||
| P9_16_pinmux { status = "okay"; pinctrl-0 = <&P9_16_pwm_pin>; }; /* motor1 speed */ | ||
| }; | ||
|
|
||
| // Motor 1, 2 | ||
| &bone_pwmss_1 { | ||
| status = "okay"; | ||
| }; | ||
|
|
||
| &bone_pwm_1 { | ||
| status = "okay"; | ||
| }; | ||
|
|
||
| // Motor 3, 4 | ||
| &bone_pwmss_2 { | ||
| status = "okay"; | ||
| }; | ||
|
|
||
| &bone_pwm_2 { | ||
| status = "okay"; | ||
| }; | ||
|
|
||
|
|
||
| /* | ||
| * Easy Motor direction control through sysfs (/sys/class/leds/motor*) using gpio-leds driver | ||
| * See these files for the led_P8_#/led_P9_# definition | ||
|
There was a problem hiding this comment. Interesting to carry these led definitions at the "bus" level. I suppose it works if they are useful enough. We need a sane way to document helpers that are exposed by the "bus". Please update the wiki page specification with usage guidelines for what interfaces are exposed for overlays to use. |
||
| * https://github.com/beagleboard/BeagleBoard-DeviceTrees/src/arm/bbai-bone-buses.dtsi | ||
| * https://github.com/beagleboard/BeagleBoard-DeviceTrees/src/arm/bbb-bone-buses.dtsi | ||
| * | ||
| */ | ||
| &{/} { | ||
| leds { | ||
| // Motor 1 | ||
| led_P8_18 { | ||
| status = "okay"; | ||
| label = "motor1"; | ||
| default-state = "off"; | ||
| }; | ||
|
|
||
| // Motor 2 | ||
| led_P8_16 { | ||
| status = "okay"; | ||
| label = "motor2"; | ||
| default-state = "off"; | ||
| }; | ||
|
|
||
| // Motor 3 | ||
| led_P8_14 { | ||
| status = "okay"; | ||
| label = "motor3"; | ||
| default-state = "off"; | ||
| }; | ||
|
|
||
| // Motor 4 | ||
| led_P8_26 { | ||
| status = "okay"; | ||
| label = "motor4"; | ||
| default-state = "off"; | ||
| }; | ||
| }; | ||
| }; | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right approach is to add your copyright, not remove someone else's.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new code is very different from the old code that's why I dropped that. I will do that if you say so, talked to ds2 he suggested to include the comment you are seeing.