From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f178.google.com (mail-pf1-f178.google.com [209.85.210.178]) (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 22DE52D594F for ; Mon, 5 Jan 2026 19:58:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767643116; cv=none; b=sRXqdGF1XpAfjVOc6n23130q0hDF8oBohgFcmoqgeovdQ7RxlclII9EDc93JuvIsosHtDaNcJO8aF0JOXv8h/5Cdq/mu1IGIcrzL5Fh39Zkbr93NknNLFOmoxOS8MJhqt7KE5rZewpEe8RzhNlsvGIHFynOS3EzLxZZ25Dt40Pc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767643116; c=relaxed/simple; bh=M+mgHPLZQJfA6q4BXgM2dcEStPY5JylfnRcjT3LMCio=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=PqAAwY9bwMN9wrW2HX4jbvxNTZUnWp9gP5dwvV2dahl/9nmfTqV4NvA0TWG+LJU9qIDe8rAkoNLO7sFGg/HIbxT2k9xGUgrqHKDCuBhDEetfHWeSesVZgg/Rky3r8z6YqVLFFiE/78kUYLs1ZxfhqzjUemQwIKIHcBnNdFi1Udo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=vcO5bnn9; arc=none smtp.client-ip=209.85.210.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="vcO5bnn9" Received: by mail-pf1-f178.google.com with SMTP id d2e1a72fcca58-7e1651ae0d5so286570b3a.1 for ; Mon, 05 Jan 2026 11:58:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1767643114; x=1768247914; 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=F8DuddcYEOiefRlBzZKrtL15qZCqx9WVAFvprpXIUBQ=; b=vcO5bnn9SsBClh9izXYYaeDcnpJzoNDUo291b5h4iEgkijqWxnIzAkzw+tOc60kdSh WDkEpMc3UrGa6GawRNjBs2dVuu8cICIjVq5XHOgQb8dEawwaCYXsUV2PvWZM+UkcqRzt vV6Q/UyaGcLM/yqH6Q1wbMGdA96+/dQt4ciEepT1JkMhxsjftOh5oJ8RHEDPH+eRvTam qsQpgLWQogc2EyHwPj8W37y9RO58JRDufrlNVQ7LP1qA+O4ZTdWHt1Jh+h8VsI5s6LQa FBwP6CIMJAnOS3WkqQZl723DNt2dsRJl1kM0tJKq3pWeeW0CRRwmE1xTMmFEioL7Fk8z CP7A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1767643114; x=1768247914; 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=F8DuddcYEOiefRlBzZKrtL15qZCqx9WVAFvprpXIUBQ=; b=dzXHKqSCxq6tzPsOhOdettqK3JBiWfOuMjsX4E4fMchnmbpfKVeB+QDFzq/3egvrDA Zc1FGICqCanpRtd6A+jJJOPeMPHzCC6NYbfdUuG4fpqLbseBKdHJRA6PrOpTCCMqfiTF Ok5QSR22YUxkV7SyEIE8nWpL/AyL18lDldChyot5aOMWZFO02TgxkcFRN71cvSNX9PmU nO2PHpXAmPBAYHw7y6iKRUnxgDWaypwvbXn4bva6IjBquEyQhZ5Mytaf/ataknoEV23Q ZKAMSUY+8DVt7z62qd1H0+/bA/dg3eZVZ1qrL0a2O5Hbjf6lrOpWAMRGLysXhIq9Omce x5YQ== X-Gm-Message-State: AOJu0YxvAi9xgzTXa2z9yquWV9hKSB5890bbcbEriASHQ3baMRyDdVWq gb8B9rhwEfO76J5B0Gxn+7BRW6JUvnXZyh11M+JRR42vZ8tzuoXPAaPdLVooQcts0Q== X-Gm-Gg: AY/fxX4GzxxLlNkZ4JNfURYVpw2Whreu/nvGFhTcCgW0TunqumausvIOorRe5UG8kAW 9xPW4H/rOkGz9QAVsj4xyAjHTnJC3g7BnThcxgJ6K/+M4APYyq9rAUdHAfgiG3DOohkTZQ88mo2 KpcwyjacZVTYfKnCXTpvPDNrUYGAezZ+gfprsEH872ugGdvTCNN95ApEftLUhAIbURqa6nxoj5U b6ytAhiyWxXZZPnQx6F0/MtLw1htGD7vk7Qp4edAhPJl2MqPg3Qe5Sg1Ca0Xqks5Pc0tALbvU1O 9K9tf77EOPeb/2WeftSJDmxwwYOG5SwE5q0BEBBXUmbjA88IkCApZCt2jSSJbGjxDP4TJ8tHNT+ b8MWojVUdxYZPCtU/vC/oIE3ULBpzPFLJhKGCg8gmlBHMlhNT1BNztdrg7YW58C0LE0KsBQYxNq r8VN/N8sTsWPMHuTTsmdNPLVgTFLykJ5FdUjOXTS9uTxuX9ZSXDKuwqs6F3vRj3M9QMcKhrE+yp ei6gIEz0J7kNl1fWFlxKrve X-Google-Smtp-Source: AGHT+IFYi37OgBvlreLAlI6opJMukcl64WhdUNqoIdTxzRCBTYpyF7jQRnopByzdWT0cS1RWXEtYww== X-Received: by 2002:a05:7022:2215:b0:11b:c86b:386a with SMTP id a92af1059eb24-121f18a3909mr496909c88.5.1767643111325; Mon, 05 Jan 2026 11:58:31 -0800 (PST) Received: from ?IPV6:2a00:79e0:2e7c:8:7c3d:6070:7e67:29c5? ([2a00:79e0:2e7c:8:7c3d:6070:7e67:29c5]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-121f12209efsm1739565c88.3.2026.01.05.11.58.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 05 Jan 2026 11:58:30 -0800 (PST) Message-ID: Date: Mon, 5 Jan 2026 11:58:29 -0800 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 3/5] mfd: max77759: add register bitmasks and modify irq configs for charger To: =?UTF-8?Q?Andr=C3=A9_Draszik?= , Sebastian Reichel , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Lee Jones , Greg Kroah-Hartman , Badhri Jagan Sridharan , Heikki Krogerus , Peter Griffin , Tudor Ambarus , Alim Akhtar Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, RD Babiera , Kyle Tso References: <20251227-max77759-charger-v3-0-54e664f5ca92@google.com> <20251227-max77759-charger-v3-3-54e664f5ca92@google.com> Content-Language: en-US From: Amit Sunil Dhamne In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 1/5/26 8:45 AM, André Draszik wrote: > On Sat, 2025-12-27 at 00:04 +0000, Amit Sunil Dhamne via B4 Relay wrote: >> From: Amit Sunil Dhamne >> >> Add register bitmasks for charger function. >> In addition split the charger IRQs further such that each bit represents >> an IRQ downstream of charger regmap irq chip. In addition populate the >> ack_base to offload irq ack to the regmap irq chip framework. >> >> Signed-off-by: Amit Sunil Dhamne >> --- >>  drivers/mfd/max77759.c       |  91 +++++++++++++++++-- >>  include/linux/mfd/max77759.h | 202 ++++++++++++++++++++++++++++++++++++------- >>  2 files changed, 256 insertions(+), 37 deletions(-) >> >> [...] >> >> diff --git a/include/linux/mfd/max77759.h b/include/linux/mfd/max77759.h >> index c6face34e385..e674a519e782 100644 >> --- a/include/linux/mfd/max77759.h >> +++ b/include/linux/mfd/max77759.h >> @@ -59,35 +59,65 @@ >>  #define MAX77759_MAXQ_REG_AP_DATAIN0            0xb1 >>  #define MAX77759_MAXQ_REG_UIC_SWRST             0xe0 >> >> -#define MAX77759_CHGR_REG_CHG_INT               0xb0 >> -#define MAX77759_CHGR_REG_CHG_INT2              0xb1 >> -#define MAX77759_CHGR_REG_CHG_INT_MASK          0xb2 >> -#define MAX77759_CHGR_REG_CHG_INT2_MASK         0xb3 >> -#define MAX77759_CHGR_REG_CHG_INT_OK            0xb4 >> -#define MAX77759_CHGR_REG_CHG_DETAILS_00        0xb5 >> -#define MAX77759_CHGR_REG_CHG_DETAILS_01        0xb6 >> -#define MAX77759_CHGR_REG_CHG_DETAILS_02        0xb7 >> -#define MAX77759_CHGR_REG_CHG_DETAILS_03        0xb8 >> -#define MAX77759_CHGR_REG_CHG_CNFG_00           0xb9 >> -#define MAX77759_CHGR_REG_CHG_CNFG_01           0xba >> -#define MAX77759_CHGR_REG_CHG_CNFG_02           0xbb >> -#define MAX77759_CHGR_REG_CHG_CNFG_03           0xbc >> -#define MAX77759_CHGR_REG_CHG_CNFG_04           0xbd >> -#define MAX77759_CHGR_REG_CHG_CNFG_05           0xbe >> -#define MAX77759_CHGR_REG_CHG_CNFG_06           0xbf >> -#define MAX77759_CHGR_REG_CHG_CNFG_07           0xc0 >> -#define MAX77759_CHGR_REG_CHG_CNFG_08           0xc1 >> -#define MAX77759_CHGR_REG_CHG_CNFG_09           0xc2 >> -#define MAX77759_CHGR_REG_CHG_CNFG_10           0xc3 >> -#define MAX77759_CHGR_REG_CHG_CNFG_11           0xc4 >> -#define MAX77759_CHGR_REG_CHG_CNFG_12           0xc5 >> -#define MAX77759_CHGR_REG_CHG_CNFG_13           0xc6 >> -#define MAX77759_CHGR_REG_CHG_CNFG_14           0xc7 >> -#define MAX77759_CHGR_REG_CHG_CNFG_15           0xc8 >> -#define MAX77759_CHGR_REG_CHG_CNFG_16           0xc9 >> -#define MAX77759_CHGR_REG_CHG_CNFG_17           0xca >> -#define MAX77759_CHGR_REG_CHG_CNFG_18           0xcb >> -#define MAX77759_CHGR_REG_CHG_CNFG_19           0xcc >> +#define MAX77759_CHGR_REG_CHG_INT                      0xb0 >> +#define   MAX77759_CHGR_REG_CHG_INT_AICL               BIT(7) >> +#define   MAX77759_CHGR_REG_CHG_INT_CHGIN              BIT(6) >> +#define   MAX77759_CHGR_REG_CHG_INT_WCIN               BIT(5) >> +#define   MAX77759_CHGR_REG_CHG_INT_CHG                BIT(4) >> +#define   MAX77759_CHGR_REG_CHG_INT_BAT                BIT(3) >> +#define   MAX77759_CHGR_REG_CHG_INT_INLIM              BIT(2) >> +#define   MAX77759_CHGR_REG_CHG_INT_THM2               BIT(1) >> +#define   MAX77759_CHGR_REG_CHG_INT_BYP                BIT(0) >> +#define MAX77759_CHGR_REG_CHG_INT2                     0xb1 >> +#define   MAX77759_CHGR_REG_CHG_INT2_INSEL             BIT(7) >> +#define   MAX77759_CHGR_REG_CHG_INT2_SYS_UVLO1         BIT(6) >> +#define   MAX77759_CHGR_REG_CHG_INT2_SYS_UVLO2         BIT(5) >> +#define   MAX77759_CHGR_REG_CHG_INT2_BAT_OILO          BIT(4) >> +#define   MAX77759_CHGR_REG_CHG_INT2_CHG_STA_CC        BIT(3) >> +#define   MAX77759_CHGR_REG_CHG_INT2_CHG_STA_CV        BIT(2) >> +#define   MAX77759_CHGR_REG_CHG_INT2_CHG_STA_TO        BIT(1) >> +#define   MAX77759_CHGR_REG_CHG_INT2_CHG_STA_DONE      BIT(0) >> +#define MAX77759_CHGR_REG_CHG_INT_MASK                 0xb2 >> +#define MAX77759_CHGR_REG_CHG_INT2_MASK                0xb3 >> +#define MAX77759_CHGR_REG_CHG_INT_OK                   0xb4 >> +#define MAX77759_CHGR_REG_CHG_DETAILS_00               0xb5 >> +#define   MAX77759_CHGR_REG_CHG_DETAILS_OO_CHGIN_DTLS  GENMASK(6, 5) >> +#define MAX77759_CHGR_REG_CHG_DETAILS_01               0xb6 >> +#define   MAX77759_CHGR_REG_CHG_DETAILS_01_BAT_DTLS    GENMASK(6, 4) >> +#define   MAX77759_CHGR_REG_CHG_DETAILS_01_CHG_DTLS    GENMASK(3, 0) >> +#define MAX77759_CHGR_REG_CHG_DETAILS_02               0xb7 >> +#define   MAX77759_CHGR_REG_CHG_DETAILS_02_CHGIN_STS   BIT(5) >> +#define MAX77759_CHGR_REG_CHG_DETAILS_03               0xb8 >> +#define MAX77759_CHGR_REG_CHG_CNFG_00                  0xb9 >> +#define   MAX77759_CHGR_REG_CHG_CNFG_00_MODE           GENMASK(3, 0) >> +#define MAX77759_CHGR_REG_CHG_CNFG_01                  0xba >> +#define MAX77759_CHGR_REG_CHG_CNFG_02                  0xbb >> +#define   MAX77759_CHGR_REG_CHG_CNFG_02_CHGCC        GENMASK(5, 0) > Small nit - there seems to be a stray TAB in this line. Will fix it in the next revision. BR, Amit > > Other than that: > Reviewed-by: André Draszik