JamesGKent / rotary_volume

kernel module to use rotary encoder as a volume control
GNU General Public License v2.0
13 stars 6 forks source link

Support Push-Button-Press #1

Open laclaro opened 5 years ago

laclaro commented 5 years ago

Hi,

the most common rotary-encoder KY-040 support also button-press. Could you add this functionality to your code as well? It works great!

Best,

Henning

laclaro commented 5 years ago

Worked it out for myself. https://github.com/MiczFlor/RPi-Jukebox-RFID/issues/267#issuecomment-457958959

Is it possible to change the GPIOs used?

JamesGKent commented 5 years ago

Hi there, glad to see someone getting use of this!

I believe that to set the GPIO pins you can change the dtoverlay line:

dtoverlay=rotary-encoder:relative_axis=1,linux_axis=9,gpio_a=5,gpio_b=7

for example, this is down to the rotary-encoder module and I'm struggling to confirm the documentation at the moment.

also I have to ask, given this module is supposed to generate keyboard volume up and down button presses, is there a reason you've wrapped it with a python script? or can that Jukebox software not catch those directly?

also for the buttons there is already the gpio-keys module: https://www.kernel.org/doc/Documentation/devicetree/bindings/input/gpio-keys.txt

that could create button presses (play/pause button?) when a gpio key is pressed.