[PATCH] staging: fbtft: Update RA8875 Kconfig help description

[PATCH] staging: fbtft: Update RA8875 Kconfig help description

[Adam Azuddin]

The current description is too brief. Update the description to
include the manufacturer (RAiO) and the supported resolution
(up to 800x480 pixels) to help users identify the correct driver.

Signed-off-by: Adam Azuddin <azuddinadam@gmail.com>
---
 drivers/staging/fbtft/Kconfig | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/fbtft/Kconfig b/drivers/staging/fbtft/Kconfig
index 578412a2f379..92943564cb91 100644
--- a/drivers/staging/fbtft/Kconfig
+++ b/drivers/staging/fbtft/Kconfig
@@ -86,7 +86,11 @@ config FB_TFT_PCD8544
 config FB_TFT_RA8875
 	tristate "FB driver for the RA8875 LCD Controller"
 	help
-	  Generic Framebuffer support for RA8875
+	  This enables generic framebuffer support for the RAiO RA8875
+	  display controller. The controller is intended for medium size text/graphic
+	  mixed displays with a resolution of up to 800x480 pixels.
+
+	  Say Y if you have such a display that utilizes this controller.
 
 config FB_TFT_S6D02A1
 	tristate "FB driver for the S6D02A1 LCD Controller"
-- 
2.53.0

Re: [PATCH] staging: fbtft: Update RA8875 Kconfig help description

[Andy Shevchenko]

On Wed, Mar 4, 2026 at 11:39 AM Adam Azuddin <azuddinadam@gmail.com> wrote:
>
> The current description is too brief. Update the description to
> include the manufacturer (RAiO) and the supported resolution
> (up to 800x480 pixels) to help users identify the correct driver.

While this is a good intention, I don't know if it's a good point at
the end as the idea is to have a proper driver to be located under
drivers/gpu/drm/.
Reviewed-by: Andy Shevchenko <andy@kernel.org>
in case Greg wants to take it.

-- 
With Best Regards,
Andy Shevchenko

Claude review: staging: fbtft: Update RA8875 Kconfig help description

[Claude Code Review Bot]

Overall Series Review

Subject: staging: fbtft: Update RA8875 Kconfig help description
Author: Adam Azuddin <azuddinadam@gmail.com>
Patches: 2
Reviewed: 2026-03-05T13:39:12.517041

---

This is a single, trivial documentation-only patch that improves the Kconfig help text for the RA8875 LCD controller driver in `drivers/staging/fbtft/`. The change is straightforward and follows an existing pattern already used by the ST7789V entry in the same file. No functional code is modified.

**Verdict:** Looks good. Minor nit below.

---
Generated by Claude Code Patch Reviewer

Claude review: staging: fbtft: Update RA8875 Kconfig help description

[Claude Code Review Bot]

Patch Review

**Positive observations:**

- The new description closely follows the pattern established by `FB_TFT_ST7789V` (lines 145-149 of `Kconfig`), which uses the same "This enables generic framebuffer support for the..." / "Say Y if you have such a display..." structure. Good consistency.
- Adding the manufacturer name (RAiO) and resolution (800x480) is genuinely useful information for users configuring their kernel.
- The commit message is clear and describes the motivation well.

**Minor nit:**

The second line of the new help text is quite long:

```
+	  display controller. The controller is intended for medium size text/graphic
```

This is 80 characters of content (after the tab+2-spaces indentation). While the kernel's Kconfig style doesn't enforce a strict line length, it would read slightly better as:

```
	  display controller. The controller is intended for medium-size
	  text/graphic mixed displays with a resolution of up to
	  800x480 pixels.
```

Also, "medium size" should arguably be hyphenated as "medium-size" since it's a compound adjective modifying "displays." This matches how the ST7789V entry uses "small color displays" (where "small" is a single adjective and doesn't need a hyphen).

**Overall:** This is a clean, well-motivated documentation improvement. No blocking issues.

---
Generated by Claude Code Patch Reviewer