From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 96FA8C74A5B for ; Sat, 18 Mar 2023 16:26:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229705AbjCRQ0P (ORCPT ); Sat, 18 Mar 2023 12:26:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49882 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229575AbjCRQ0N (ORCPT ); Sat, 18 Mar 2023 12:26:13 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8C862A279 for ; Sat, 18 Mar 2023 09:25:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1679156724; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RMLleG2DXcNogGAuAWxYrgTG9bG50WKJ62cIAjwjdNE=; b=B7UJAcBAjpdi1GSanRznzEFjKgfyeT4DDcUt8RlRgnDfRoJr4pP0PlUpv55NaQJ16fXKeT fnoQI2H81rmtG4l1WoZwS6hTgYIaaWO66NDT8yQHLJN6RLQtyvGB+MaGBOoRBSRjebpa1I ahODhs1voZTDkr84NVeMCCCwSTs9gJA= Received: from mail-ed1-f69.google.com (mail-ed1-f69.google.com [209.85.208.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-244-ulJR2XwqN560UVv2zOruaQ-1; Sat, 18 Mar 2023 12:25:23 -0400 X-MC-Unique: ulJR2XwqN560UVv2zOruaQ-1 Received: by mail-ed1-f69.google.com with SMTP id er23-20020a056402449700b004fed949f808so11850235edb.20 for ; Sat, 18 Mar 2023 09:25:22 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679156722; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=RMLleG2DXcNogGAuAWxYrgTG9bG50WKJ62cIAjwjdNE=; b=7XcytpSPA0FCiXZVyLtxIP/N59xkuuzattnUnyNXr7LSyBADROcwW+11Ybx5T5S6fy biECyW9ht2Mg0ex+YZHHOJOm8naho3ceakKDXeSGiT4QV7yzemZlCpPYUAa70HL6z3Ok rnoiAah31TxCCT7N9L2guYKirmSjN1yQ19vFdmUDmahZAi5fdIc5GsiPeZYcyfnfTnDr g+8NGiJB83HbtGB8P/Lu3xy/tBbmLUgxgh6Rr5+q8SURX7Vf+qm43wvK8XK037gyGp0T ROD6ZltshnTE3OtKhfeyjS6Pxs/ulQJd1hZTXVL35+fYo6oHeg8o40p6gVwdU9G5ZF4t +NUQ== X-Gm-Message-State: AO0yUKVhXxAPXJGUowFlxUVcE6pmyQ1xwjoAMf1QSYFfFkGQTEoI//yb asijfckLQv3c5gb+8fATE0otAYGDOZ1QaPKAc4rVA9HZOv+IMTDFv16R5k2iccv0FCc9DPnxsD0 0S4R46yKNrwxFTyluHX5IQPjB X-Received: by 2002:a17:907:8690:b0:933:3a65:67ed with SMTP id qa16-20020a170907869000b009333a6567edmr2477140ejc.75.1679156721977; Sat, 18 Mar 2023 09:25:21 -0700 (PDT) X-Google-Smtp-Source: AK7set/0JZzssF+//gwBugYtJOfPoHdL9gBfMtdVZPgFd+DEMg4E7q1aCuABo4Gc7m0xJc3btkS1Og== X-Received: by 2002:a17:907:8690:b0:933:3a65:67ed with SMTP id qa16-20020a170907869000b009333a6567edmr2477124ejc.75.1679156721688; Sat, 18 Mar 2023 09:25:21 -0700 (PDT) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id qq24-20020a17090720d800b008df7d2e122dsm2318909ejb.45.2023.03.18.09.25.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 18 Mar 2023 09:25:21 -0700 (PDT) Message-ID: Date: Sat, 18 Mar 2023 17:25:20 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: [PATCH] input: alps: fix compatibility with -funsigned-char To: msizanoen , =?UTF-8?Q?Pali_Roh=c3=a1r?= , Dmitry Torokhov Cc: stable@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org References: <20230318144206.14309-1-msizanoen@qtmlabs.xyz> Content-Language: en-US, nl From: Hans de Goede In-Reply-To: <20230318144206.14309-1-msizanoen@qtmlabs.xyz> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 3/18/23 15:42, msizanoen wrote: > The AlpsPS/2 code previously relied on the assumption that `char` is a > signed type, which was true on x86 platforms (the only place where this > driver is used) before kernel 6.2. However, on 6.2 and later, this > assumption is broken due to the introduction of -funsigned-char as a new > global compiler flag. > > Fix this by explicitly specifying the signedness of `char` when sign > extending the values received from the device. > > Fixes: f3f33c677699 ("Input: alps - Rushmore and v7 resolution support") > Cc: stable@vger.kernel.org > Signed-off-by: msizanoen Thanks, patch looks good to me: Reviewed-by: Hans de Goede Regards, Hans > --- > drivers/input/mouse/alps.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c > index 989228b5a0a4..1c570d373b30 100644 > --- a/drivers/input/mouse/alps.c > +++ b/drivers/input/mouse/alps.c > @@ -2294,20 +2294,20 @@ static int alps_get_v3_v7_resolution(struct psmouse *psmouse, int reg_pitch) > if (reg < 0) > return reg; > > - x_pitch = (char)(reg << 4) >> 4; /* sign extend lower 4 bits */ > + x_pitch = (signed char)(reg << 4) >> 4; /* sign extend lower 4 bits */ > x_pitch = 50 + 2 * x_pitch; /* In 0.1 mm units */ > > - y_pitch = (char)reg >> 4; /* sign extend upper 4 bits */ > + y_pitch = (signed char)reg >> 4; /* sign extend upper 4 bits */ > y_pitch = 36 + 2 * y_pitch; /* In 0.1 mm units */ > > reg = alps_command_mode_read_reg(psmouse, reg_pitch + 1); > if (reg < 0) > return reg; > > - x_electrode = (char)(reg << 4) >> 4; /* sign extend lower 4 bits */ > + x_electrode = (signed char)(reg << 4) >> 4; /* sign extend lower 4 bits */ > x_electrode = 17 + x_electrode; > > - y_electrode = (char)reg >> 4; /* sign extend upper 4 bits */ > + y_electrode = (signed char)reg >> 4; /* sign extend upper 4 bits */ > y_electrode = 13 + y_electrode; > > x_phys = x_pitch * (x_electrode - 1); /* In 0.1 mm units */