[PATCH] HID: intel-thc-hid: Separate max input size control conditional list

[PATCH] HID: intel-thc-hid: Separate max input size control conditional list

[Bagas Sanjaya]

Stephen Rothwell reports htmldocs warning:

Documentation/hid/intel-thc-hid.rst:200: ERROR: Unexpected indentation. [docutils]

Separate conditional list for max input size control by a blank line
to fix the warning.

Fixes: 45e92a093099 ("HID: Intel-thc-hid: Intel-thc: Introduce max input size control")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Closes: https://lore.kernel.org/linux-next/20250611142409.7d4683b0@canb.auug.org.au/
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 Documentation/hid/intel-thc-hid.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/hid/intel-thc-hid.rst b/Documentation/hid/intel-thc-hid.rst
index 23d5110cb8710a..8b378c57b5aac0 100644
--- a/Documentation/hid/intel-thc-hid.rst
+++ b/Documentation/hid/intel-thc-hid.rst
@@ -195,6 +195,7 @@ This is a new feature introduced in Panther Lake platform, THC hardware allows d
 a max input size for RxDMA. After this max size gets set and enabled, for every input report
 packet reading, THC hardware sequencer will first read incoming input packet size, then compare
 input packet size with the given max size:
+
 - if input packet size <= max size, THC continues using input packet size to finish the reading
 - if input packet size > max size, there is potential input data crash risk during
   transferring, THC will use max size instead of input packet size for reading

base-commit: da04eb7791c461bc0f113ce96af4ed59bcc12555
-- 
An old man doll... just what I always wanted! - Clara

RE: [PATCH] HID: intel-thc-hid: Separate max input size control conditional list

[Xu, Even]

Thanks for the fix, Bagas!

Best Regards,
Even Xu

> -----Original Message-----
> From: Bagas Sanjaya <bagasdotme@gmail.com>
> Sent: Friday, June 13, 2025 9:43 AM
> To: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Linux
> Documentation <linux-doc@vger.kernel.org>; Linux Input Devices <linux-
> input@vger.kernel.org>
> Cc: Jiri Kosina <jikos@kernel.org>; Benjamin Tissoires <bentiss@kernel.org>;
> Jonathan Corbet <corbet@lwn.net>; Andy Shevchenko
> <andriy.shevchenko@linux.intel.com>; Xu, Even <even.xu@intel.com>; Han,
> Chong <chong.han@intel.com>; Bagas Sanjaya <bagasdotme@gmail.com>;
> Stephen Rothwell <sfr@canb.auug.org.au>
> Subject: [PATCH] HID: intel-thc-hid: Separate max input size control conditional
> list
> 
> Stephen Rothwell reports htmldocs warning:
> 
> Documentation/hid/intel-thc-hid.rst:200: ERROR: Unexpected indentation.
> [docutils]
> 
> Separate conditional list for max input size control by a blank line to fix the
> warning.
> 
> Fixes: 45e92a093099 ("HID: Intel-thc-hid: Intel-thc: Introduce max input size
> control")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Closes: https://lore.kernel.org/linux-
> next/20250611142409.7d4683b0@canb.auug.org.au/
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>  Documentation/hid/intel-thc-hid.rst | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/hid/intel-thc-hid.rst b/Documentation/hid/intel-thc-
> hid.rst
> index 23d5110cb8710a..8b378c57b5aac0 100644
> --- a/Documentation/hid/intel-thc-hid.rst
> +++ b/Documentation/hid/intel-thc-hid.rst
> @@ -195,6 +195,7 @@ This is a new feature introduced in Panther Lake
> platform, THC hardware allows d  a max input size for RxDMA. After this max size
> gets set and enabled, for every input report  packet reading, THC hardware
> sequencer will first read incoming input packet size, then compare  input packet
> size with the given max size:
> +
>  - if input packet size <= max size, THC continues using input packet size to finish
> the reading
>  - if input packet size > max size, there is potential input data crash risk during
>    transferring, THC will use max size instead of input packet size for reading
> 
> base-commit: da04eb7791c461bc0f113ce96af4ed59bcc12555
> --

Reviewed-by: Even Xu <even.xu@intel.com>

> An old man doll... just what I always wanted! - Clara

Re: [PATCH] HID: intel-thc-hid: Separate max input size control conditional list

[Jiri Kosina]

On Fri, 13 Jun 2025, Bagas Sanjaya wrote:

> Stephen Rothwell reports htmldocs warning:
> 
> Documentation/hid/intel-thc-hid.rst:200: ERROR: Unexpected indentation. [docutils]
> 
> Separate conditional list for max input size control by a blank line
> to fix the warning.
> 
> Fixes: 45e92a093099 ("HID: Intel-thc-hid: Intel-thc: Introduce max input size control")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Closes: https://lore.kernel.org/linux-next/20250611142409.7d4683b0@canb.auug.org.au/
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>

Applied to hid.git#for-6.17/intel-thc, thanks.

-- 
Jiri Kosina
SUSE Labs