From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f43.google.com (mail-wr1-f43.google.com [209.85.221.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 771023D3D03 for ; Wed, 20 May 2026 10:12:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779271970; cv=none; b=tqcx/TBSP0jTW/SQSihkHdBhlWA0PoyawQNxExnM0P3qQjLJk1GQfovX5KSe3EXMYt/oJH3Ue4hFxpu3OMj/TRKIzRyOBMOIg/f34dLYsFuH++IfUpDZKl4KSzCdhV/cGjEi7eRKznvFBJ0fKKg5aK8unraynUIJtNvblabSHJc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779271970; c=relaxed/simple; bh=0B2p36/qTj2Fdw815lUlDWHFRk7MfZrklT7ORDPhlOI=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=ukSAMtx9/d1zVHQ/nA4IPBjPa0yKS6pgvJw7KIDjRgKv0tteku7SdRpIrYZEQwGObjv378w19YGofSXlG42s18mSQOgQJQdMXxtgEhSy2vUKSTJmZu2iPmYBk19JrQWq+9Z7JWm6IZZKVIGXaM+yqchsMz8yDc9pO5IusXVQ2Po= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OmA2MsAz; arc=none smtp.client-ip=209.85.221.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OmA2MsAz" Received: by mail-wr1-f43.google.com with SMTP id ffacd0b85a97d-44c350a5b87so2813804f8f.3 for ; Wed, 20 May 2026 03:12:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779271968; x=1779876768; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=0B2p36/qTj2Fdw815lUlDWHFRk7MfZrklT7ORDPhlOI=; b=OmA2MsAz2sR5y1NO37zy6c2vazFq9jIQ3DAafBB7de1wK3bso3yVKR3rD+KWTwClX7 NDwWrbGiTwC8TIdBBpY7hOhFZpPksgZ+QGOofg+Uv/TxA51xrHbC2bPI8Wlr0nEXm7td BAq4DFQ+dZNC2vzeHrs1Dpi8a7jQgQsm3LLUA7AmQEYAJFjYWeSaHg/ERq98KtKA3YOf pjC1v0P6A0bQBgVK6pKhewxcN4qGyl19yAOsiEwvCe+aoQJcpj8CeIqblBgN7615nMnJ gndP+vEYPeOiS6L31/DNXz1kNZZy7uttNmx1UFp5hXSKo8Xv065R7ugwnGfdiOa+xQVd KYHQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779271968; x=1779876768; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=0B2p36/qTj2Fdw815lUlDWHFRk7MfZrklT7ORDPhlOI=; b=oCIVcacpiC5XTAH4YqieTDPMuM0lUJM78J0W2vqehAtmPHV89qraICtCcfvl+L95gS G1sDzWYDKj2qhv1vA+szha5cmnl/fvjw/PFHTXM9CrzLBjIYqXXalofdDSwfzVGtTVxe 6vHYMR4Th0dFPfJfzrWz8bHbbql92MieeJk5kDM2KYm28DN+NhFk7ISZw7VQ5nAH0eS8 6+s2IR1Pw6PALkKn5sr2ML719mg5MPR8BqHcrLuBvPex+wiQU9JaXfvpFrwALvWW7ua2 2RuaTLYvt71fL4423seCEiWDMXROYTP0PgFdGKYynYXfVKZkEJ+Zyh2NOgrNHRXqDEvS 1V0w== X-Forwarded-Encrypted: i=1; AFNElJ+Q6fJ+Rjt3+hV0B01FRgH1IAgtsx5XgH9YboKM0V9TxwOHoKMhmZ4+ROFiRNpzLtxLnn9qbSh9c8d6UY4=@vger.kernel.org X-Gm-Message-State: AOJu0YwtArZiUrIWdSL9Sdg/1xCOV16JUAEk62ddRpZ86mN/oi/s/Zrp 1b+kwRosGm7hJrYs1sEp2p8M5BpVhAnVjfh6989Z5GQHA/jcC/FfTGD+ X-Gm-Gg: Acq92OFCp2MxXaPBqbQ8g0TEjGKR3CycvOy006x1mdBWZNK6sA2Hcu+5NHRZEd7JCz8 0iBInM7oXloLaVriBWnZ9aWQmzaf5V4rppRNRnPCwdN2mOmzB0EmcEDIMN+iHs+8O34FID/ytyF Brws/swzBIs9BbnDC4rcvagqN8G1pBAPwF2m8b6alNXZ26I8G5tbi6Kv+76Lzuh/SJfSOSvB/Sz WrNDMOssgPTj6Hi4qlcIhw+FmUxGrMpaUSfH/rRygp6RMvIIOxDb9nXa9dA9q5oYncPU/Pzo1/1 lfCJ+6x/J3xydGCaQj/Z0cWseMzRJrKWBf9QhZEX5xCKS+Ptj5kPHswgw/LqQuuA96gDx39eCB3 yeFVrWGWdPCYJrVCfSBuMjG2Q1/4OkabjMJ0k14KGcGlwZFSm7YuYFZqcuwhWrqzd6Csumuu5/n ByClUTbW7HV1UgSWXDN1rVDjQionjLgzPWUmigzC6sc3++axoVsC4v5ko8k3Kqt88= X-Received: by 2002:adf:f64c:0:b0:45e:73b3:4515 with SMTP id ffacd0b85a97d-45e73b34553mr17891059f8f.35.1779271967665; Wed, 20 May 2026 03:12:47 -0700 (PDT) Received: from localhost.localdomain ([2001:8a0:6c9b:a000:ed18:5665:257e:fff1]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45da0fe0fecsm48760228f8f.26.2026.05.20.03.12.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 May 2026 03:12:47 -0700 (PDT) From: Gabriel Rondon To: jic23@kernel.org Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, andy@kernel.org, nuno.sa@analog.com, dlechner@baylibre.com Subject: Re: [PATCH] iio: accel: bmc150: use guard(mutex) for mutex handling Date: Wed, 20 May 2026 11:12:44 +0100 Message-Id: <20260520101244.64434-1-grondon@gmail.com> X-Mailer: git-send-email 2.33.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Mon, 20 Apr 2026 19:15:19 +0100 Jonathan Cameron wrote: > Not sure this list is useful in the commit log. Rather verbose. Dropped the function list from the commit log in v2. > I took a quick look and can't see anything beyond an if (ret) check > in bmc150_accel_get_axis() > If it's the only one left I'd use scoped_guard() for > bmc150_accel_trigger_handler() Agreed. v2 also converts bmc150_accel_get_axis() to guard(mutex), and uses scoped_guard() for bmc150_accel_trigger_handler(), so no manual mutex_lock()/mutex_unlock() pairs remain in the touched code. > Unrelated change [int ret; movement] - here it just acts as noise > [...] don't do this sort of code movement of lines we aren't > otherwise touching. Reverted; v2 leaves the declaration order in bmc150_accel_get_temp() untouched. > return sprintf(buf, "%d\n", data->watermark); > [...] maybe also return sysfs_emit(...) is appropriate. Done for bmc150_accel_get_fifo_watermark() and bmc150_accel_get_fifo_state(); the intermediate variables are gone. These already use sysfs_emit() in the current iio tree, so v2 keeps sysfs_emit() and returns directly. > Unrelated but [...] val = min(val, BMC150_ACCEL_FIFO_LENGTH); Left out of v2 to keep this to a single logical change. Happy to send it as a separate cleanup. Thanks for the review. Gabriel