staging: pi433: Added information about bit_rate configuration

[PATCH] staging: pi433: Added information about bit_rate configuration

Posted by Guru Mehar Rachaputi 2 years, 7 months ago

Information in the TODO file for bit_rate configuration is
insufficient.

This patch adds information on how to approach when considering
to modify bit_rate to support upto 300kbps.

Signed-off-by: Guru Mehar Rachaputi <gurumeharrachaputi@gmail.com>
---
 drivers/staging/pi433/TODO | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/staging/pi433/TODO b/drivers/staging/pi433/TODO
index 5cf3fd99d521..daa0dbcf6d53 100644
--- a/drivers/staging/pi433/TODO
+++ b/drivers/staging/pi433/TODO
@@ -3,3 +3,6 @@
 * Some missing data (marked with ###) needs to be added in the documentation
 * Change (struct pi433_tx_cfg)->bit_rate to be a u32 so that we can support
     bit rates up to 300kbps per the spec.
+  -> This configuration needs to be moved to sysfs instead of being done through
+     IOCTL. Goind forward, we need to port userspace tools to use sysfs instead
+     of IOCTL and then we would delete IOCTL.
-- 
2.34.1


-- 
Thanks & Regards,
Guru

Re: [PATCH] staging: pi433: Added information about bit_rate configuration

Posted by Dan Carpenter 2 years, 7 months ago

On Tue, Jan 31, 2023 at 07:14:39PM +0100, Guru Mehar Rachaputi wrote:
> Information in the TODO file for bit_rate configuration is
> insufficient.
> 
> This patch adds information on how to approach when considering
> to modify bit_rate to support upto 300kbps.
> 
> Signed-off-by: Guru Mehar Rachaputi <gurumeharrachaputi@gmail.com>
> ---

Thanks!

Acked-by: Dan Carpenter <error27@gmail.com>

regards,
dan carpenter