mlyszczek commented on code in PR #16714: URL: https://github.com/apache/nuttx/pull/16714#discussion_r2201998811
########## drivers/input/sbutton.c: ########## @@ -0,0 +1,269 @@ +/**************************************************************************** + * drivers/input/sbutton.c + * + * SPDX-License-Identifier: Apache-2.0 + * + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor license agreements. See the NOTICE file distributed with + * this work for additional information regarding copyright ownership. The + * ASF licenses this file to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance with the + * License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT + * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the + * License for the specific language governing permissions and limitations + * under the License. + * + ****************************************************************************/ + +/**************************************************************************** + * Included Files + ****************************************************************************/ + +#include <nuttx/config.h> + +#include <stdbool.h> +#include <stdio.h> +#include <assert.h> +#include <debug.h> +#include <errno.h> +#include <poll.h> +#include <fcntl.h> + +#include <nuttx/input/sbutton.h> +#include <nuttx/fs/fs.h> +#include <nuttx/clock.h> +#include <nuttx/ascii.h> +#include <nuttx/kmalloc.h> +#include <nuttx/mutex.h> +#include <nuttx/wqueue.h> +#include <nuttx/semaphore.h> +#include <nuttx/input/keyboard.h> +#include <nuttx/input/kbd_codec.h> + +/**************************************************************************** + * Pre-processor Definitions + ****************************************************************************/ + +/* This format is used to construct the /dev/kbd[n] device driver path. It + * defined here so that it will be used consistently in all places. + */ + +#define DEV_FORMAT "/dev/kbd%d" +#define DEV_NAMELEN 11 + +#define KEY_PRESS true +#define KEY_RELEASE false + +#define KEY_THRESH_MS 500 + +/**************************************************************************** + * Private Types + ****************************************************************************/ + +struct sbutton_dev_s +{ + FAR const struct sbutton_config_s *config; /* Board configuration data */ + + mutex_t lock; /* Exclusive access to dev */ + clock_t start; /* Clock tick when the key was pressed */ + clock_t end; /* Clock tick when the key was released */ + bool pressed; /* Keep previous status of button */ + struct work_s work; /* Supports the interrupt handling "bottom half" */ + + /* Keyboard lowerhalf of the registered keyboard */ + + struct keyboard_lowerhalf_s lower; +}; + +/**************************************************************************** + * Static Function Prototypes + ****************************************************************************/ + +static int sbutton_interrupt(int irq, FAR void *context, FAR void *arg); +static void sbutton_worker(FAR void *arg); +static void sbutton_putbuffer(FAR struct sbutton_dev_s *priv, + uint8_t keycode); + +/**************************************************************************** + * Private Data + ****************************************************************************/ + +/**************************************************************************** + * Private Functions + ****************************************************************************/ + +/**************************************************************************** + * Name: sbutton_worker + ****************************************************************************/ + +static void sbutton_worker(FAR void *arg) +{ + FAR struct sbutton_dev_s *priv = (FAR struct sbutton_dev_s *)arg; + uint8_t state; + int ret; + + ret = nxmutex_lock(&priv->lock); + if (ret < 0) + { + return; + } + + /* Read the status of the button */ + + state = priv->config->pin_status(); + + /* If the user just pressed the button, start measuring */ + + if (state == KEY_PRESS && !priv->pressed) + { + iinfo("Button pressed\n"); + + priv->pressed = true; + priv->start = clock_systime_ticks(); + } + else + { + if (state == KEY_RELEASE && priv->pressed) + { + uint32_t elapsed; + + iinfo("Button released\n"); + + priv->pressed = false; + priv->end = clock_systime_ticks(); + + elapsed = priv->end - priv->start; + if (elapsed < MSEC2TICK(KEY_THRESH_MS)) + { + iinfo("KEY_1\n"); + keyboard_event(&priv->lower, ASCII_A, KEYBOARD_PRESS); + } + else + { + iinfo("KEY_2\n"); + keyboard_event(&priv->lower, ASCII_B, KEYBOARD_PRESS); Review Comment: Maybe it would be good to let end user decide which key to trigger with long and short presses? Best would be to let user decide to use any keyboard keycode. If not that, at least let him swap behavior. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org