2022-02-26 05:40:28

by Paulo Miguel Almeida

[permalink] [raw]
Subject: [PATCH] staging: pi433: remove TODOs-related item from the TODO file

All occurences of code annotated with TODO in this driver have been
addressed in the last couple of years by the community. The last
occurrence of the word "TODO" was in the form of a multi-line comment in
the header file intended to be exposed to userspace once this driver
moves away from the staging folder.

This patch removes that TODO as well as the TODO-related item from the
driver's TODO file

Signed-off-by: Paulo Miguel Almeida <[email protected]>
---
Meta-comments:
- in the include/linux folder I've seen many standards such as:
-> Just the explanation of that that file is used for (I choose that)
-> name_of_file.h - brief explanation
-> include/linux/name_of_file.h - brief explanation

Since there was not clear pattern I chose the first one. Let me know
if anyone favours any other approach.

---
drivers/staging/pi433/TODO | 1 -
drivers/staging/pi433/pi433_if.h | 2 --
2 files changed, 3 deletions(-)

diff --git a/drivers/staging/pi433/TODO b/drivers/staging/pi433/TODO
index 8d0f1b57961a..5cf3fd99d521 100644
--- a/drivers/staging/pi433/TODO
+++ b/drivers/staging/pi433/TODO
@@ -1,4 +1,3 @@
-* still TODOs, annotated in the code
* currently the code introduces new IOCTLs. I'm afraid this is a bad idea.
-> Replace this with another interface, hints are welcome!
* Some missing data (marked with ###) needs to be added in the documentation
diff --git a/drivers/staging/pi433/pi433_if.h b/drivers/staging/pi433/pi433_if.h
index 855f0bebdc1c..25ee0b77a32c 100644
--- a/drivers/staging/pi433/pi433_if.h
+++ b/drivers/staging/pi433/pi433_if.h
@@ -1,7 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0+ */
/*
- * include/linux/TODO
- *
* userspace interface for pi433 radio module
*
* Pi433 is a 433MHz radio module for the Raspberry Pi.
--
2.34.1