mraa icon indicating copy to clipboard operation
mraa copied to clipboard

Fix for PWM on Beaglebone black with a kernel > 3.8.13

Open SyrianSpock opened this issue 9 years ago • 9 comments

Tested on hardware (Beaglebone black Rev B6)

SyrianSpock avatar Sep 05 '15 23:09 SyrianSpock

Thanks for the contribution! Is this a debian thing or global? I've never seen such a thing and edison for example has a 3.10 kernel? I'd like to understand if this is a beaglebone thing or general? Quark also uses a 3.8.x kernel and doesn't have this.

arfoll avatar Sep 07 '15 09:09 arfoll

I have only seen this on the Beaglebone black with Debian and Ubuntu. I don't know if the Beaglebone is the only board concerned or if it's an Linux on ARM thingy, sorry I don't have an answer.

SyrianSpock avatar Sep 07 '15 11:09 SyrianSpock

Ok I unfortunately don't have a BBB, @tingleby do you think you could take a look? My only arm platform is an rpi2 and that doesn't have pwm (least not natively)

arfoll avatar Sep 08 '15 10:09 arfoll

@cosscat is this something you could help with if you have a board?

arfoll avatar Sep 08 '15 10:09 arfoll

I don't have BBB either, but I've found this nice description: https://github.com/szmoore/MCTX3420/wiki/Software:-PWM-Control and looks like the standard sysfs location is actually just a secondary mean to use PWMs on BBB and the one that doesn't follow the kernel documentation fully (dropping the pwmchip piece).

alext-mkrs avatar Oct 09 '15 20:10 alext-mkrs

I believe this should rather be reworked using _replace approach, as this is specific to one board we support. @SyrianSpock, do you think you'd want to do that?

alext-mkrs avatar Feb 07 '16 08:02 alext-mkrs

@alext-mkrs Sure, I'll do it ASAP.

SyrianSpock avatar Feb 07 '16 10:02 SyrianSpock

Thanks and no rush :smile: I just believe this is a better way to go and looking at the code and @arfoll's comments I think it will increase its chances for merging.

alext-mkrs avatar Feb 07 '16 12:02 alext-mkrs

So @SyrianSpock, would you be willing to modify this one as requested?

alext-mkrs avatar May 06 '17 13:05 alext-mkrs