From nobody Tue Jun 23 02:06:26 2026 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 3A745C433FE for ; Sat, 12 Mar 2022 17:31:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232322AbiCLRct (ORCPT ); Sat, 12 Mar 2022 12:32:49 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49496 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229491AbiCLRco (ORCPT ); Sat, 12 Mar 2022 12:32:44 -0500 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 6E33C6D1A4 for ; Sat, 12 Mar 2022 09:31:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1647106296; 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=OQDHbRK5h5hzHKCQDiG2Xi9MV3Kh7CjgfW1/1yG8i58=; b=Tv+uLLhEep3UjQopCNW3XhbvsxzfrUptD6b50UsSdksIZE+2tYryCto3S9s3QWk2rHIna+ 04hXDv9DWHmbd1pJ8H2kEQj92v4IaSN5Ey5nhH/Hpc4VBTO++nL4eXDjURDgwYlJ0IgqaL 2MNxQgCP4ymGga/rmLo3aoWE2i1WNWA= Received: from mail-qk1-f198.google.com (mail-qk1-f198.google.com [209.85.222.198]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-643-3qlx9y-uOxORA6moy21J0Q-1; Sat, 12 Mar 2022 12:31:35 -0500 X-MC-Unique: 3qlx9y-uOxORA6moy21J0Q-1 Received: by mail-qk1-f198.google.com with SMTP id m123-20020a375881000000b0067af33d4ac1so8468113qkb.4 for ; Sat, 12 Mar 2022 09:31:35 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=OQDHbRK5h5hzHKCQDiG2Xi9MV3Kh7CjgfW1/1yG8i58=; b=4KUgfwrXUWupWB1NIYXEuYQOxZLgnElLTou3blsEKjVDN7T9udWCSk6mxWY2z/ts7l i+pRaMUs5hUfYE/v/aLs81WhZUYHxi1W1fyH4FrDor1zNkVEv+THdq2TqLS0Q1E78Hlz wQbEyXqZSBupRuJABLDQ+xSMAChkADTf2NbAHfctoWSjZkvT3HAoByw6Npkzc+aWbKJV lq/YGUs7VPHcUtNkbVsyb4Eer8QJ9AycYUORHzhDy/FoXhmxoY200Thpjp7KkHA64knG zHW42g8kbCbkfbZ613aDEMEmS8NjD7qaOJQN/+yIE0eZSiyGPxS3ic52by7jr44ejKi6 Fu+Q== X-Gm-Message-State: AOAM533UMjIqoi7og+qgSbd+cHp40yKPs9PbXKIWhl4rVPTuN9UwoePt sPB6YqzGTWz+5rDIQoHMkSCPAXrHbCV1Za6YlmDj6MrSdLtmYCzUwD+M84rVM0Yjd2fSCfABh64 IozIbe5aLrLHRsrTi5nFkLWKq X-Received: by 2002:a05:620a:4442:b0:67d:b94a:8c6a with SMTP id w2-20020a05620a444200b0067db94a8c6amr46205qkp.569.1647106295010; Sat, 12 Mar 2022 09:31:35 -0800 (PST) X-Google-Smtp-Source: ABdhPJxtnu94rjTmOi+1VORDdqL74UhgzCuwVSAqQfaBh5XrYQnCoZ1rChJn2tfiQdAuzAwWyfgQ1Q== X-Received: by 2002:a05:620a:4442:b0:67d:b94a:8c6a with SMTP id w2-20020a05620a444200b0067db94a8c6amr46188qkp.569.1647106294786; Sat, 12 Mar 2022 09:31:34 -0800 (PST) Received: from localhost.localdomain.com (024-205-208-113.res.spectrum.com. [24.205.208.113]) by smtp.gmail.com with ESMTPSA id v3-20020a05622a188300b002e1cbca8ea4sm1161476qtc.59.2022.03.12.09.31.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 12 Mar 2022 09:31:34 -0800 (PST) From: trix@redhat.com To: mchehab@kernel.org, hverkuil-cisco@xs4all.nl, cai.huoqing@linux.dev, paskripkin@gmail.com, xose.vazquez@gmail.com Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Tom Rix Subject: [PATCH] media: stkwebcam: move stk_camera_read_reg() scratch buffer to struct stk_camera Date: Sat, 12 Mar 2022 09:30:49 -0800 Message-Id: <20220312173049.1410977-1-trix@redhat.com> X-Mailer: git-send-email 2.26.3 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Tom Rix In stk_camera_read_reg() a single byte buffer is alloc-ed and freed on every function call. Since the size is known, move the buffer to the struct stk_camera where it will be alloc-ed and freed once. Signed-off-by: Tom Rix --- drivers/media/usb/stkwebcam/stk-webcam.c | 11 ++--------- drivers/media/usb/stkwebcam/stk-webcam.h | 2 ++ 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/media/usb/stkwebcam/stk-webcam.c b/drivers/media/usb/s= tkwebcam/stk-webcam.c index 5b822214ccc5c..787edb3d47c23 100644 --- a/drivers/media/usb/stkwebcam/stk-webcam.c +++ b/drivers/media/usb/stkwebcam/stk-webcam.c @@ -150,25 +150,18 @@ int stk_camera_write_reg(struct stk_camera *dev, u16 = index, u8 value) int stk_camera_read_reg(struct stk_camera *dev, u16 index, u8 *value) { struct usb_device *udev =3D dev->udev; - unsigned char *buf; int ret; =20 - buf =3D kmalloc(sizeof(u8), GFP_KERNEL); - if (!buf) - return -ENOMEM; - ret =3D usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), 0x00, USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE, 0x00, index, - buf, + &dev->read_reg_scratch, sizeof(u8), 500); if (ret >=3D 0) - *value =3D *buf; - - kfree(buf); + *value =3D dev->read_reg_scratch; =20 if (ret < 0) return ret; diff --git a/drivers/media/usb/stkwebcam/stk-webcam.h b/drivers/media/usb/s= tkwebcam/stk-webcam.h index 14519e5308b18..136decffe9ced 100644 --- a/drivers/media/usb/stkwebcam/stk-webcam.h +++ b/drivers/media/usb/stkwebcam/stk-webcam.h @@ -105,6 +105,8 @@ struct stk_camera { struct list_head sio_avail; struct list_head sio_full; unsigned sequence; + + u8 read_reg_scratch; }; =20 #define vdev_to_camera(d) container_of(d, struct stk_camera, vdev) --=20 2.26.3