From patchwork Fri Oct 28 14:27:15 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Jason A. Donenfeld" X-Patchwork-Id: 12348 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp864283wru; Fri, 28 Oct 2022 07:30:46 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4K5Ag6og2iLwfI1Gu2me2/pKYu8n5DKpf895b0jvhqy/JhLeyh/DLpTB3K5YIzrnti6eYs X-Received: by 2002:a65:6d1b:0:b0:46a:f245:8825 with SMTP id bf27-20020a656d1b000000b0046af2458825mr46921736pgb.224.1666967446171; Fri, 28 Oct 2022 07:30:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666967446; cv=none; d=google.com; s=arc-20160816; b=xwjESR+hS6GtJ5cTA86hv+hI7ymE/Q1kFRSNhs3CxtD5Fi6eff9s0E+yh5LBzm9FX3 WnWUITkA8SNLOz/cbBXolHyku0BdqSS+YNvhqUiG4KgFNXQSPCRqf5PR/0qMRaMdIpxg 4Su4pXzs0vWk7SiJb05Qn9yOWDwoLUBeWKuj0gHUxmadVYrsMFcKKwrO3luc97B8dS+N y11ORTGvX2uKge4KTXD00QSgt+i8mN+WV4Yu03knHULxZroRWP6AmI236cLbnOcLTMZd xAJt7KPcMCzygXF6LMUGzqFJoCZvp7qNJG9PjKXn+MU0ZO4v/CEhrl0+JE01+ao3rAWg 5vwg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=gD+ZFnxbIhSu4CHS6E4Tlv4ldzK84O8SYLBYJTRnbkA=; b=hLf72s+LLWz+UPWVTkyTrvV9IioMb1Mojoq2fSIzAneGCvs774UshUpUBwfuBHIIau 4nhPD3j2qMipU7qW35cPg67Cye7aBcqJzZ3PRM3dUwKorOO33RtQ4cH4Cxrvt5CQhv8u z29ptZuQCzNxDDGfj1+OyCzZcQH3aHdd1ojMjkl3dknWOxZGd4I81NrrWpc6rWKWa8z+ VVCtPRPF6oZwW+fic4P3xITtHGytcjlyX4Fd+pHNMTXpm5YzOtxpZiX4FQfj85TgB3UD BBgW/4wC+lI7QfSbcqD85avh9pZqCnf4MOjIT/ejwZgwar2RGTYKGVE5NQuP0H+ry9fv 5Mvw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@zx2c4.com header.s=20210105 header.b=PFepB4YP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=zx2c4.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b6-20020a056a000cc600b005660f2c4b27si6164205pfv.274.2022.10.28.07.30.30; Fri, 28 Oct 2022 07:30:46 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@zx2c4.com header.s=20210105 header.b=PFepB4YP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=zx2c4.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231162AbiJ1O2Q (ORCPT + 99 others); Fri, 28 Oct 2022 10:28:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35214 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229455AbiJ1O1d (ORCPT ); Fri, 28 Oct 2022 10:27:33 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1225132D95; Fri, 28 Oct 2022 07:27:31 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id C0FD2B82971; Fri, 28 Oct 2022 14:27:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B904BC433D6; Fri, 28 Oct 2022 14:27:27 +0000 (UTC) Authentication-Results: smtp.kernel.org; dkim=pass (1024-bit key) header.d=zx2c4.com header.i=@zx2c4.com header.b="PFepB4YP" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=zx2c4.com; s=20210105; t=1666967245; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=gD+ZFnxbIhSu4CHS6E4Tlv4ldzK84O8SYLBYJTRnbkA=; b=PFepB4YPb+usJ8MA6K3aAyzI+Hyn824xAIQIehgc8or1tF8P4KqYQKHWiUzzdW4CXE0wFV rLaYP2XnSuEx7En0n5Ii0TFjHRMhRGcoUji8nXMF1/qOS/U1mt+1QUVI0qrwYXk1A1LJh3 dhvskCyJRA5TOArw3j1fkgSH6tVxxmM= Received: by mail.zx2c4.com (ZX2C4 Mail Server) with ESMTPSA id 62688d8d (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Fri, 28 Oct 2022 14:27:24 +0000 (UTC) From: "Jason A. Donenfeld" To: sparclinux@vger.kernel.org, linux-kernel@vger.kernel.org Cc: "Jason A. Donenfeld" , "David S . Miller" Subject: [PATCH] sparc: sbus: treat CPU index as integer Date: Fri, 28 Oct 2022 16:27:15 +0200 Message-Id: <20221028142715.2021166-1-Jason@zx2c4.com> MIME-Version: 1.0 X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1747942057164668582?= X-GMAIL-MSGID: =?utf-8?q?1747942057164668582?= Using a `char` to fit a CPU index is too small, and assigning -1 to it isn't correct either, since `char` is sometimes signed and sometimes not. In this case, it's been fine since this driver only works on sparc, but that will soon be changing when chars become unsigned everywhere. So instead, use a normal `int` type, which matches the `int cpu` function argument that this is being compared against. Cc: David S. Miller Signed-off-by: Jason A. Donenfeld --- DaveM - this is part of the -funsigned-char work I've been accumulating in my unsigned-char branch. If you want to take this as a fix for 6.1, go ahead. Otherwise, Linus asked me to keep the 6.2 unsigned-char patches together in my branch, so I'll take this, pending your Ack. -Jason drivers/sbus/char/envctrl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/sbus/char/envctrl.c b/drivers/sbus/char/envctrl.c index 843e830b5f87..ea914a7eaa7f 100644 --- a/drivers/sbus/char/envctrl.c +++ b/drivers/sbus/char/envctrl.c @@ -363,8 +363,8 @@ static int envctrl_read_cpu_info(int cpu, struct i2c_child_t *pchild, char mon_type, unsigned char *bufdata) { unsigned char data; - int i; - char *tbl, j = -1; + int i, j = -1; + char *tbl; /* Find the right monitor type and channel. */ for (i = 0; i < PCF8584_MAX_CHANNELS; i++) {