Received: by 2002:a05:7412:cfc7:b0:fc:a2b0:25d7 with SMTP id by7csp1675657rdb; Tue, 20 Feb 2024 03:51:07 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWF2htYDJmySWLPxv8ux12CceLSpQHaoM8jtprN7XZFLV/FsqUwZu149wqzrQpWX/7PxWZ/tX2WQNSWkiI9ybAcRXdYm6qG3qrAdvyZYQ== X-Google-Smtp-Source: AGHT+IGnSPjGBLRd9LAvWbsG6pc5EGRpA13zDdmk9/LCRI0ln2sdw4EKuBueNUanXj9uPL8ccRDK X-Received: by 2002:a05:6871:713:b0:21e:6670:b84a with SMTP id f19-20020a056871071300b0021e6670b84amr11505949oap.5.1708429867128; Tue, 20 Feb 2024 03:51:07 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708429867; cv=pass; d=google.com; s=arc-20160816; b=eVkeRFAcahMhx1vnEjmxhkeqfRlTAwktBC1BVbNzMBS9MLSKuw9q3dCgaUbJyPGUMN zPJnm48yaq0G1Hk6b3wR4nmoId0hLqTmId6yWtRWviQ1j2RiAfduiasqWwAO8MVVhraA msIN7yVWjN+Zt5+MqTSnZFVV5fZsZ60XBeiqVXDf7BoSWQIpBaa/CbwC+Uphq4gqmDIU NKxJgvIswtdp4muY6PaLs3a7qhFoH60f4Z4qNGkAg8B0r+Ip9frUQpZdsNpcnEJIv3C6 GS0DVLpCYzaiy4QEQxP8f6emJekOoV/w6MULtysLTri9QJ02+tZags4yToZNFfo/RFnh e/BQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:dkim-signature; bh=OyeFWWK1ZILZ0GtHtp44q7Pr25xJ4Cjvt/OcbzpZ34E=; fh=pgsUPk0EP/zS2r9GJDKi9J1Iynk9J1EPwC08RgzrUQ0=; b=CE/2O/bzZHZKNsX9un+cI7hJukKypFnRzy0vFfWrMOonDZl7u/WF14D6WtrjelwFdL 7W943L1eh8n+2sO+77QEPcgH2sl9f5VlE0FMJYhjUWhSDNi6/uZdCWNeNvZQnAsK7i9V IVKZG7TkI4mZG4NU/SRBtYZeBi11zcZ2As9zQjT/2pVzpyq5eob9IC3ktUzAsnF4MLJ/ IpOXPWqWcdaZfnkLLsFR6erKKpEPKst7Wf5QBr0R8oAgNOTu3woKlpVpNuJGzvA/LkHJ cZOUvb4kK8emtnkUyTPvmhqsepDboJSvRwdctP68R0iPD+3QIZAqH2eHiLWBe+aizrIY 48SA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=F83UHyPF; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-72940-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-72940-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id e12-20020a65678c000000b005dca7d41860si6157935pgr.418.2024.02.20.03.51.06 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 20 Feb 2024 03:51:07 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-72940-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=F83UHyPF; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-72940-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-72940-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 83B1D2836BA for ; Tue, 20 Feb 2024 11:50:22 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4465B67A1A; Tue, 20 Feb 2024 11:50:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="F83UHyPF" Received: from mail-ed1-f50.google.com (mail-ed1-f50.google.com [209.85.208.50]) (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 A734667A0F for ; Tue, 20 Feb 2024 11:50:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708429819; cv=none; b=hkDsDoYf7GALRJosBXTm9wU2qm6a20J5MeGdSfOOeiHaUj9wDn3KMmIFUhQp4VbKf7DkpI1kCXMI7KG2pJjaLkO3xgt34PL3R0Arz9L0GKCQocGQEwdXH1DNjZE2oYKxgj3DLW7TCnUAsjY6NCc/ECI4j+XtBD77n+pPX+R22YQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708429819; c=relaxed/simple; bh=ZPJxtFrZyxgBNUAQe6H2xk0BZZEMxsLG6V/Dz7Hto44=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=KejxxODlobE1Hf2HnuJ38TWHnj7pvh+XWb7AKplWhCTFYR30yFIdHGhu3jnaQjunTFiuaPnxD9ocpVgNxMnGVWIgrKJ9l3UDXIUU1mkc3w94r32hyNvVak0Fhr0rEsZPTnxJfqLxaSgnxudFM/9zGUCHGkJRJfM6XfIG9tHLxxU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=F83UHyPF; arc=none smtp.client-ip=209.85.208.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-ed1-f50.google.com with SMTP id 4fb4d7f45d1cf-55a5e7fa471so5576059a12.1 for ; Tue, 20 Feb 2024 03:50:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1708429816; x=1709034616; 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=OyeFWWK1ZILZ0GtHtp44q7Pr25xJ4Cjvt/OcbzpZ34E=; b=F83UHyPFqARWoYpjDgp0g/cRegsLHC/qRtasLuMp/kUGevDhz1T4JQMSSrnyX+AcLx PIHbFhdbfvd5A/xoETyXqigRtjoojKzMQg75ZKdKBnc7aHzjDiNE2Pma/6z7pJ0ZRjZN Qmam9EmftkjShBqy+oBjueN3x7PITMOf8VXl2rUIqtRdzDR73WsUNDbbBI61bSW+Z/Zl 3MyulL1wKRYW1c2cdLfkRJuEKBWI1X7tGTGLS9flJjQs3ccyFIf3TxKHp87bmHag82PH P++w8sQJCSrM2HWx/k1KWx57TjR7KJF8am8hgXNyLtcDt5S2ibJCQjXj2F6kfWJbVPtC 1uxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708429816; x=1709034616; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=OyeFWWK1ZILZ0GtHtp44q7Pr25xJ4Cjvt/OcbzpZ34E=; b=BWpMb1+IefKm2lQ2FbQD5tJJHmNIzm9ORzyGQ1XnYzHIGswmI4cmmn5IK+qTKzsJ1q Y2XiCKpHnq9BQJEkr63KZTXj2ockF5NXzkpXDlhbJETDc7TKpinUmZTvWJtBseFsxcU9 FTiMNciVqDxRoKz0XuQHimZoD0/TEL4aYhGL5+/CXBtf8EnqGW3jyUhpppyJB4CSoUo+ zQIAB49mWQm/wanpQlpgijqHlKL4V1OK9hRKXCI0vdkI4B0oS7tP3xGubokbeu0ftNEE 1rxDNUwHul33e+iJ0Qax+3tJefW0bImDSXVI8hknsmlsY6UlRjFdt1dKU2Xu+JWCDaqn mUnA== X-Forwarded-Encrypted: i=1; AJvYcCUPfEbj/ieT3i5W+yM/BHYyIgWE2c5XZAiX3VQFkqlFAst15Ksg/Oi5z2sGWqE2SS6T+25l6fbtINanA7eYuCbit1PHTVYpPhzQkrun X-Gm-Message-State: AOJu0Yx0sVeomIeht6mdWBY8ttuJdkSWpJ4XS0sN9tSbG5P93bwVMVyp 5aMX6BNnksXH211t9cyb2lK42GAXcffYj7ygqMXO12yZ2oaMg83aiduyuYs4MtM= X-Received: by 2002:a05:6402:5162:b0:564:7921:37c7 with SMTP id d2-20020a056402516200b00564792137c7mr3904110ede.1.1708429815948; Tue, 20 Feb 2024 03:50:15 -0800 (PST) Received: from gpeter-l.lan (host-92-18-74-232.as13285.net. [92.18.74.232]) by smtp.gmail.com with ESMTPSA id u16-20020aa7db90000000b005645c4af6c7sm2317832edt.13.2024.02.20.03.50.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 20 Feb 2024 03:50:15 -0800 (PST) From: Peter Griffin To: arnd@arndb.de, lee@kernel.org, thor.thayer@linux.intel.com Cc: kernel-team@android.com, krzysztof.kozlowski@linaro.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, pankaj.dubey@samsung.com, tudor.ambarus@linaro.org, andre.draszik@linaro.org, saravanak@google.com, willmcvicker@google.com, Peter Griffin Subject: [PATCH 0/3] mfd syscon and altera-sysmgr cleanup Date: Tue, 20 Feb 2024 11:50:09 +0000 Message-ID: <20240220115012.471689-1-peter.griffin@linaro.org> X-Mailer: git-send-email 2.44.0.rc0.258.g7320e95886-goog Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Hi, Whilst implementing a driver similar to altera-sysmgr for exynos it was noticed during the review feedback that we should only call of_node_put() if the property is provided otherwise nothing has taken a reference on the node. Both syscon and altera-sysmgr also have the same issue which this series fixes. Another piece of review feedback was not to use extern keyword in the header file, so I also cleaned that up whilst being here. regards, Peter. Peter Griffin (3): mfd: syscon: call of_node_put() only when of_parse_phandle() takes a ref mfd: syscon: remove extern from function prototypes mfd: altera-sysmgr: call of_node_put() only when of_parse_phandle() takes a ref drivers/mfd/altera-sysmgr.c | 4 +++- drivers/mfd/syscon.c | 4 +++- include/linux/mfd/syscon.h | 25 +++++++++++-------------- 3 files changed, 17 insertions(+), 16 deletions(-) -- 2.44.0.rc0.258.g7320e95886-goog