diff mbox series

HID: lenovo: Fix to ensure the data as __le32 instead of u32

Message ID 20250114004159.102555-1-vishnuocv@gmail.com
State New
Headers show
Series HID: lenovo: Fix to ensure the data as __le32 instead of u32 | expand

Commit Message

Vishnu Sankar Jan. 14, 2025, 12:41 a.m. UTC 
Ensure that data is treated as __le32 instead of u32 before
applying le32_to_cpu.
This patch fixes the sparse warning "sparse: cast to restricted __le32".

Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
Signed-off-by: Vishnu Sankar <vsankar@lenovo.com>
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202501101635.qJrwAOwf-lkp@intel.com/
---
 drivers/hid/hid-lenovo.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jiri Kosina March 4, 2025, 8:13 p.m. UTC | #1 
On Tue, 14 Jan 2025, Vishnu Sankar wrote:

> Ensure that data is treated as __le32 instead of u32 before
> applying le32_to_cpu.
> This patch fixes the sparse warning "sparse: cast to restricted __le32".
> 
> Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
> Signed-off-by: Vishnu Sankar <vsankar@lenovo.com>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202501101635.qJrwAOwf-lkp@intel.com/

Sorry for the delay, this fell in between cracks. Now applied.
Vishnu Sankar March 5, 2025, 12:08 a.m. UTC | #2 
Thank you Jiri.

On Wed, Mar 5, 2025 at 5:13 AM Jiri Kosina <jikos@kernel.org> wrote:
>
> On Tue, 14 Jan 2025, Vishnu Sankar wrote:
>
> > Ensure that data is treated as __le32 instead of u32 before
> > applying le32_to_cpu.
> > This patch fixes the sparse warning "sparse: cast to restricted __le32".
> >
> > Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
> > Signed-off-by: Vishnu Sankar <vsankar@lenovo.com>
> > Reported-by: kernel test robot <lkp@intel.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202501101635.qJrwAOwf-lkp@intel.com/
>
> Sorry for the delay, this fell in between cracks. Now applied.
>
> --
> Jiri Kosina
> SUSE Labs
>
diff mbox series

Patch

diff --git a/drivers/hid/hid-lenovo.c b/drivers/hid/hid-lenovo.c
index 4d00bc4d656e..0d9792750dcf 100644
--- a/drivers/hid/hid-lenovo.c
+++ b/drivers/hid/hid-lenovo.c
@@ -783,7 +783,7 @@  static int lenovo_raw_event(struct hid_device *hdev,
 	if (unlikely((hdev->product == USB_DEVICE_ID_LENOVO_X12_TAB
 			|| hdev->product == USB_DEVICE_ID_LENOVO_X12_TAB2)
 			&& size >= 3 && report->id == 0x03))
-		return lenovo_raw_event_TP_X12_tab(hdev, le32_to_cpu(*(u32 *)data));
+		return lenovo_raw_event_TP_X12_tab(hdev, le32_to_cpu(*(__le32 *)data));
 
 	return 0;
 }