From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f43.google.com (mail-ed1-f43.google.com [209.85.208.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 C61B426561E for ; Sat, 24 Jan 2026 20:23:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769286222; cv=none; b=UC3SldTu012tUJaeLRuCBsd0+rqN8Eo7Xwkc+RZ9huKyvvDjG4C8LbND3yPsFNmbrA3rr53X7ZJt61+knrAIx9tx/zFiiAss6mi+FEZoZdlHVfArDaKl5Ue7ULUpYrHpCFWBgwBsiIjImrrGb/msKUGTji5NW3i2uRMGAi517sU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769286222; c=relaxed/simple; bh=K9T/oFt08Y6R26vl8medrow+Qm7xgj19wlgomw+eLtw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZOV5l3PkJbNnJ/AHIK+RHF3QSCsEHEGaL/BqElcuta2HaLahX72YPkOaEZBsfs7xj+wgetyp8nyZZtvq/YQ/AvbHjcd2jHVWY3353F436ko6uKbXHlZMPU5RclRhX7fHQqgogNcBbohPsabXbQUixpol5gWnzoxE6EAHAzNPmow= 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=FNL3wrtM; arc=none smtp.client-ip=209.85.208.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="FNL3wrtM" Received: by mail-ed1-f43.google.com with SMTP id 4fb4d7f45d1cf-658072a4e56so6679971a12.0 for ; Sat, 24 Jan 2026 12:23:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1769286219; x=1769891019; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Gzxkm0h8XEMj0XxTsj3iPE14mhoIpF5KuL5jI5jwGiU=; b=FNL3wrtMkSauYix9WHBd1nSjLgpVxg2CfUhl/u8TCRW3DuxwNkZztRuAjUKIjpm6ub Vv9yN19DSpj1AYnFr62OxMWjyy7Wpmr5jebJR+rzMQbZCWAkfJSrhOFWi1I1EUeMiBJn yeHLK0XB5dT/NrQSB3nEXu1GNrnWFrRzu50bH4a3B2ADRBEg2Ad3+neX7eDVUCyBGyT4 uHLZcCReHmubZRmuesI6vTI7Q/mggcUnnpDtBC8P6L5ZoyC0UvvL8DcfrY7KdW/t7+x3 NoVRklV+848cIu5PgrYOuPrcoDyAOuQaknhiSv+/Y4og0/+0ibsM4LCPSuLLjYzyyOYN PC4w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1769286219; x=1769891019; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Gzxkm0h8XEMj0XxTsj3iPE14mhoIpF5KuL5jI5jwGiU=; b=mH/PT2FH5mYIIJjME11kXVYWjWYWLILzpn3lHZUYKaSjQEQ8drcnk21Qc/b+rSMs85 gU0MVcDglOWsKtjfoFgTa5/9H+bBha64yM3zKPXHMJuyI/Keuy/bL6wx3gHqTzDp0RmJ fL48btT/gI4KZlGYvVSQIrHpYizrqemp4oayfyaSmFwHRBq/ZHbIaQMuLmolXlCHBnqN wzYy8HOUpC8nWxDjOvllykan8GvjuUQzuVJu/Vq8DVVEIvY9AgL1wIAFgCLjKpRjCDZe dIkJcKgZx6FXLp/BxMmynwAKcNAjI+/HTPlv4zF/bCu685xwosfK7e4BWFBTLJvTLQqZ QEaA== X-Gm-Message-State: AOJu0YxseizwalOMdNGofYB2omXuVFGjJvvQopErciaC/D/tcDVomb/b w5Bkk7kX9vBoBGt2J9ph1FG14o9fUzURGMqBKDuAR+GBdtn1KBTKd2qo X-Gm-Gg: AZuq6aJE2MVKCcZ22cPaTUm/k0XL3cNzF83UePEBgsOlqSbTLXY5TxpyRX0QDcr+ZZo U+ba714VvxuGLKOTzIrWqpaQrnUO/lrVG0jV10gjDLbVIj9VBKRes7/PQYsPxuKq0DPe7+/0+23 P9vIMZsTFoQgc4oUFR3Ao+NgTQRmZhQlpxNpt+qJomI7Tp0FHiubcujG31nW3egOeGfNDAc1sgh mJ4jWPD/LWrSIwJTOKle6uh83HIkpzv3K7RsDV5zKiZ+Ux67Fionw4KmkX84hGUTSH/Zz2D4wLY y776rP7PSeCrxFtQ5QuH0bvNJUGgNBnPuUnREeJyQv8N82uihMCxSex+jcE1jAXcvtznTwFRcOM CTKT1y0XqC7XMMN/D+O5E3TqDQUxb8HEvsk246FA1aE2g95iqKd2bMWoLeb0o+p1P5b3odISaag jesR+9SgbiawzbNY71nwVhjr+jVfOpNVyjx4MnVErSApz+Q/csuoPW9a9F7GUxsj/oWund8h+Xc vW7CUuyb/uYUtQG6qRNv3LvyesCEnet6bJos45XarhtIjdSKZwmurv7aQaffeWHfl/2ao3I6+w= X-Received: by 2002:a05:6402:26c4:b0:64b:7dd2:6ba4 with SMTP id 4fb4d7f45d1cf-6584874e648mr4167820a12.4.1769286218916; Sat, 24 Jan 2026 12:23:38 -0800 (PST) Received: from ?IPV6:2a01:114f:470e:6200:4c6a:9244:66ab:7c8e? (2a01-114f-470e-6200-4c6a-9244-66ab-7c8e.ea.ipv6.supernova.orange.pl. [2a01:114f:470e:6200:4c6a:9244:66ab:7c8e]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6584b96cae0sm2579920a12.33.2026.01.24.12.23.37 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 24 Jan 2026 12:23:38 -0800 (PST) Message-ID: <2153fb6d-17d3-4cec-b348-894599743b93@gmail.com> Date: Sat, 24 Jan 2026 21:23:36 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3] mfd: bcm590xx: Add support for interrupt handling To: Lee Jones Cc: linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, Stanislav Jakubek References: <20251013-bcm590xx-irq-v3-1-0ceb060d2ee8@gmail.com> <20251023130335.GM475031@google.com> Content-Language: en-US From: Artur Weber In-Reply-To: <20251023130335.GM475031@google.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sorry for the late reply to a 3-month-old review, but I missed this comment: On 23.10.2025 15:03, Lee Jones wrote: > On Mon, 13 Oct 2025, Artur Weber wrote: >> >> +static bool bcm590xx_volatile_pri(struct device *dev, unsigned int reg) > > If I've asked a question or showed uncertainty about something, it > usually means that changes need to be made. Asking what "pri" meant > wasn't a one time thing. It shows that something is not clear and if > I'm asking, others will wonder too. > > Can we change 'sec' to 'secondary' and 'pri' to 'primary' please? That function was named for consistency with the other uses of "pri" and "sec" in the code; this function is passed to a field in the struct "bcm590xx_regmap_config_pri". (Admittedly, "bcm590xx_regmap_volatile_pri" would be a more accurate function name.) I understand that the pri/sec naming could be confusing though. Should I update the entire driver to use primary/secondary instead, or just this one function? Or just the regmap_config? The regmap_pri and regmap_sec names are also used in the bcm590xx struct which is passed to other drivers (currently only the regulator driver), changing those would also involve changing that driver, but that's fine by me. Best regards Artur