From nobody Thu Nov 28 06:43:40 2024 Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) (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 CFF812232A for ; Thu, 3 Oct 2024 05:34:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727933690; cv=none; b=TkVElWg3XeM+XSYgHBp9PU0znlywZHvTJpMwRLPX7oqg1Er6LLPuzEv2SD6ILOX7VbAIoEGg1dvxdeQm0htHRDJHC7gQp5HX++LoQgI2LyKXGGp8f1hf1krpZMYHFwSWm069kIVkkV0264z4uf1kyBn9m0Yeo9AThUOfvTLAUPc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727933690; c=relaxed/simple; bh=DpD0ZTxECxdClwsVU9a7YpcJ/ouW8aR7Po3GmboT/XM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=plsf6taZgsV6XvNN9FOcQuUhxvNjRJcaq4k8l/mc81m/m1v4AIvFk8hYhfPfhZbK6+O6bxch+AFxvLjDfBQS1Tv77FqX3lb3soFr9IY0SegNdRgV3XatMfzT9H1uLK2p4i+HTiN5OLH8RCF7lcdkvs+Q9zlhESEDLjkWyLgU22E= 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=g3bVcmK4; arc=none smtp.client-ip=209.85.167.49 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="g3bVcmK4" Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-5398e33155fso591841e87.3 for ; Wed, 02 Oct 2024 22:34:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1727933686; x=1728538486; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=SBf6EUOmCkG/NS6S8uWqcuN3R6dVhBCDEUMMokjbFeU=; b=g3bVcmK4Pc68+WBTIEH38g3WDIy1Z9leniUZZpODlfboLICqUojvRU9VfReIIYZuy+ gmYVQNlYZ493S9qsOIJ0yK8yoA1JEitg2QJxvsVd25Lmrg2qbm8tyPxWquTAs2kGEqRH REc0XE2yI0H3Yx6H0h1un6gR3FPRUEAmZmUoFUcNMgIRz4gh3qKfuw3Bbw6xPzLorh+U ZJbEWB2P7Byzl9yFi4ZK8wj8h7zGbCRqCNbO9J3gWvI5F1ey3yVAmD7YQVicfDFJozo7 55PK6BDQbTcekkJFB5g9AZYg7Q+bHya1wcqAl5SyL72zdojM1i+m9RWxs/6AMb8P/ByN lwmA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727933686; x=1728538486; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=SBf6EUOmCkG/NS6S8uWqcuN3R6dVhBCDEUMMokjbFeU=; b=s9xdYsiSdeZFMiQs/vUjBAwpTr/nRK9uLLpXKjMNZbw3G1tDBh+tJJzvps/hwUyLC3 EPfit8ZGWkMBQgu8KsYj0ME6aYIXc292z+S6yXev/KZPwmcAvxpcUOVA0bM51DmoIi6J lIPLxg4IPPoigaf+idwf4m0PwQI0Vu+gSSWLfc36z+sce6c4iedK85NbB8TC2Da9w5cO fqX7FDUXKC5HVPVLMEckL4WC2K4rZYLeX3nZBS1H80xSsPMVHgVseJGXwrIZRfdxVPmO 0hp9Lfoj13b6SEoZqTBZx9F3WlSIzpEsnpAEXoz4uJ3pxLBuYliwJA7bn41wKbLiGe0N AWTA== X-Forwarded-Encrypted: i=1; AJvYcCUqWmQk7EgJOShynQslzJCk4apUWb3HEU5ebzdxspuNIfrS4BJQ8gKGGqx/5BtvXSmBbdOX6UKh8dbJhB0=@vger.kernel.org X-Gm-Message-State: AOJu0YyKHnjITsDz58gMtKLV4BnhinCwwdXprfKRRJy0Kbruy9LJ4Wq2 dAJNE+ajKoX/YPisOMSYwZF7wVuc0qNmCE1azguiRYnHHa34C3GlFcM+tg== X-Google-Smtp-Source: AGHT+IEi3qBsvHELHTdptRfXnm1MM/7U9xQna+Dpmq01T20xQYanoTxPeCk/iYEIHVl16knNG2G/og== X-Received: by 2002:a05:6512:12d3:b0:52e:76d5:9504 with SMTP id 2adb3069b0e04-539a06588cfmr3630072e87.3.1727933685492; Wed, 02 Oct 2024 22:34:45 -0700 (PDT) Received: from mva-rohm ([2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-539a82567fasm80752e87.80.2024.10.02.22.34.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Oct 2024 22:34:43 -0700 (PDT) Date: Thu, 3 Oct 2024 08:34:38 +0300 From: Matti Vaittinen To: Matti Vaittinen , Matti Vaittinen Cc: Matti Vaittinen , Lee Jones , Liam Girdwood , Mark Brown , linux-kernel@vger.kernel.org Subject: [PATCH v2 1/2] mfd: bd96801: Add ERRB IRQ Message-ID: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="Fb5sccfyGGIgPt4c" Content-Disposition: inline In-Reply-To: --Fb5sccfyGGIgPt4c Content-Disposition: inline Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" The ROHM BD96801 "scalable PMIC" provides two physical IRQs. The ERRB handling can in many cases be omitted because it is used to inform fatal IRQs, which usually kill the power from the SOC. There may however be use-cases where the SOC has a 'back-up' emergency power source which allows some very short time of operation to try to gracefully shut down sensitive hardware. Furthermore, it is possible the processor controlling the PMIC is not powered by the PMIC. In such cases handling the ERRB IRQs may be beneficial. Add support for ERRB IRQs. Signed-off-by: Matti Vaittinen --- Revision history: New series (only ERRB addition) v2: - Suffle local variables in probe() - Unify and improve the error prints when adding regmap IRQ chip for INTB or ERRB fails. v1: - use devm allocation for regulator_res - use goto skip_errb instead of an if (errb) - constify immutable structs Old series (All BD96801 functionality + irqdomain and regmap changes) v2 =3D> v3: - No changes v1 =3D> v2: - New patch --- drivers/mfd/rohm-bd96801.c | 275 ++++++++++++++++++++++++++++++++----- 1 file changed, 241 insertions(+), 34 deletions(-) diff --git a/drivers/mfd/rohm-bd96801.c b/drivers/mfd/rohm-bd96801.c index 714f08ed544b..141eb3d9394b 100644 --- a/drivers/mfd/rohm-bd96801.c +++ b/drivers/mfd/rohm-bd96801.c @@ -5,13 +5,9 @@ * ROHM BD96801 PMIC driver * * This version of the "BD86801 scalable PMIC"'s driver supports only very - * basic set of the PMIC features. Most notably, there is no support for - * the ERRB interrupt and the configurations which should be done when the - * PMIC is in STBY mode. - * - * Supporting the ERRB interrupt would require dropping the regmap-IRQ - * usage or working around (or accepting a presense of) a naming conflict - * in debugFS IRQs. + * basic set of the PMIC features. + * Most notably, there is no support for the configurations which should + * be done when the PMIC is in STBY mode. * * Being able to reliably do the configurations like changing the * regulator safety limits (like limits for the over/under -voltages, over @@ -23,16 +19,14 @@ * be the need to configure these safety limits. Hence it's not simple to * come up with a generic solution. * - * Users who require the ERRB handling and STBY state configurations can - * have a look at the original RFC: + * Users who require the STBY state configurations can have a look at the + * original RFC: * https://lore.kernel.org/all/cover.1712920132.git.mazziesaccount@gmail.c= om/ - * which implements a workaround to debugFS naming conflict and some of - * the safety limit configurations - but leaves the state change handling - * and synchronization to be implemented. + * which implements some of the safety limit configurations - but leaves t= he + * state change handling and synchronization to be implemented. * * It would be great to hear (and receive a patch!) if you implement the - * STBY configuration support or a proper fix to the debugFS naming - * conflict in your downstream driver ;) + * STBY configuration support or a proper fix in your downstream driver ;) */ =20 #include @@ -46,6 +40,64 @@ #include #include =20 +static const struct resource regulator_errb_irqs[] =3D { + DEFINE_RES_IRQ_NAMED(BD96801_OTP_ERR_STAT, "bd96801-otp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_DBIST_ERR_STAT, "bd96801-dbist-err"), + DEFINE_RES_IRQ_NAMED(BD96801_EEP_ERR_STAT, "bd96801-eep-err"), + DEFINE_RES_IRQ_NAMED(BD96801_ABIST_ERR_STAT, "bd96801-abist-err"), + DEFINE_RES_IRQ_NAMED(BD96801_PRSTB_ERR_STAT, "bd96801-prstb-err"), + DEFINE_RES_IRQ_NAMED(BD96801_DRMOS1_ERR_STAT, "bd96801-drmoserr1"), + DEFINE_RES_IRQ_NAMED(BD96801_DRMOS2_ERR_STAT, "bd96801-drmoserr2"), + DEFINE_RES_IRQ_NAMED(BD96801_SLAVE_ERR_STAT, "bd96801-slave-err"), + DEFINE_RES_IRQ_NAMED(BD96801_VREF_ERR_STAT, "bd96801-vref-err"), + DEFINE_RES_IRQ_NAMED(BD96801_TSD_ERR_STAT, "bd96801-tsd"), + DEFINE_RES_IRQ_NAMED(BD96801_UVLO_ERR_STAT, "bd96801-uvlo-err"), + DEFINE_RES_IRQ_NAMED(BD96801_OVLO_ERR_STAT, "bd96801-ovlo-err"), + DEFINE_RES_IRQ_NAMED(BD96801_OSC_ERR_STAT, "bd96801-osc-err"), + DEFINE_RES_IRQ_NAMED(BD96801_PON_ERR_STAT, "bd96801-pon-err"), + DEFINE_RES_IRQ_NAMED(BD96801_POFF_ERR_STAT, "bd96801-poff-err"), + DEFINE_RES_IRQ_NAMED(BD96801_CMD_SHDN_ERR_STAT, "bd96801-cmd-shdn-err"), + + DEFINE_RES_IRQ_NAMED(BD96801_INT_PRSTB_WDT_ERR, "bd96801-prstb-wdt-err"), + DEFINE_RES_IRQ_NAMED(BD96801_INT_CHIP_IF_ERR, "bd96801-chip-if-err"), + DEFINE_RES_IRQ_NAMED(BD96801_INT_SHDN_ERR_STAT, "bd96801-int-shdn-err"), + + DEFINE_RES_IRQ_NAMED(BD96801_BUCK1_PVIN_ERR_STAT, "bd96801-buck1-pvin-err= "), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK1_OVP_ERR_STAT, "bd96801-buck1-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK1_UVP_ERR_STAT, "bd96801-buck1-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK1_SHDN_ERR_STAT, "bd96801-buck1-shdn-err= "), + + DEFINE_RES_IRQ_NAMED(BD96801_BUCK2_PVIN_ERR_STAT, "bd96801-buck2-pvin-err= "), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK2_OVP_ERR_STAT, "bd96801-buck2-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK2_UVP_ERR_STAT, "bd96801-buck2-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK2_SHDN_ERR_STAT, "bd96801-buck2-shdn-err= "), + + DEFINE_RES_IRQ_NAMED(BD96801_BUCK3_PVIN_ERR_STAT, "bd96801-buck3-pvin-err= "), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK3_OVP_ERR_STAT, "bd96801-buck3-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK3_UVP_ERR_STAT, "bd96801-buck3-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK3_SHDN_ERR_STAT, "bd96801-buck3-shdn-err= "), + + DEFINE_RES_IRQ_NAMED(BD96801_BUCK4_PVIN_ERR_STAT, "bd96801-buck4-pvin-err= "), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK4_OVP_ERR_STAT, "bd96801-buck4-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK4_UVP_ERR_STAT, "bd96801-buck4-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_BUCK4_SHDN_ERR_STAT, "bd96801-buck4-shdn-err= "), + + DEFINE_RES_IRQ_NAMED(BD96801_LDO5_PVIN_ERR_STAT, "bd96801-ldo5-pvin-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO5_OVP_ERR_STAT, "bd96801-ldo5-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO5_UVP_ERR_STAT, "bd96801-ldo5-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO5_SHDN_ERR_STAT, "bd96801-ldo5-shdn-err"), + + DEFINE_RES_IRQ_NAMED(BD96801_LDO6_PVIN_ERR_STAT, "bd96801-ldo6-pvin-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO6_OVP_ERR_STAT, "bd96801-ldo6-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO6_UVP_ERR_STAT, "bd96801-ldo6-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO6_SHDN_ERR_STAT, "bd96801-ldo6-shdn-err"), + + DEFINE_RES_IRQ_NAMED(BD96801_LDO7_PVIN_ERR_STAT, "bd96801-ldo7-pvin-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO7_OVP_ERR_STAT, "bd96801-ldo7-ovp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO7_UVP_ERR_STAT, "bd96801-ldo7-uvp-err"), + DEFINE_RES_IRQ_NAMED(BD96801_LDO7_SHDN_ERR_STAT, "bd96801-ldo7-shdn-err"), +}; + static const struct resource regulator_intb_irqs[] =3D { DEFINE_RES_IRQ_NAMED(BD96801_TW_STAT, "bd96801-core-thermal"), =20 @@ -90,20 +142,14 @@ static const struct resource regulator_intb_irqs[] =3D= { DEFINE_RES_IRQ_NAMED(BD96801_LDO7_UVD_STAT, "bd96801-ldo7-undervolt"), }; =20 -static const struct resource wdg_intb_irqs[] =3D { - DEFINE_RES_IRQ_NAMED(BD96801_WDT_ERR_STAT, "bd96801-wdg"), +enum { + WDG_CELL =3D 0, + REGULATOR_CELL, }; =20 static struct mfd_cell bd96801_cells[] =3D { - { - .name =3D "bd96801-wdt", - .resources =3D wdg_intb_irqs, - .num_resources =3D ARRAY_SIZE(wdg_intb_irqs), - }, { - .name =3D "bd96801-regulator", - .resources =3D regulator_intb_irqs, - .num_resources =3D ARRAY_SIZE(regulator_intb_irqs), - }, + [WDG_CELL] =3D { .name =3D "bd96801-wdt", }, + [REGULATOR_CELL] =3D { .name =3D "bd96801-regulator", }, }; =20 static const struct regmap_range bd96801_volatile_ranges[] =3D { @@ -128,6 +174,91 @@ static const struct regmap_access_table volatile_regs = =3D { .n_yes_ranges =3D ARRAY_SIZE(bd96801_volatile_ranges), }; =20 +/* + * For ERRB we need main register bit mapping as bit(0) indicates active I= RQ + * in one of the first 3 sub IRQ registers, For INTB we can use default 1 = to 1 + * mapping. + */ +static unsigned int bit0_offsets[] =3D {0, 1, 2}; /* System stat, 3 regist= ers */ +static unsigned int bit1_offsets[] =3D {3}; /* Buck 1 stat */ +static unsigned int bit2_offsets[] =3D {4}; /* Buck 2 stat */ +static unsigned int bit3_offsets[] =3D {5}; /* Buck 3 stat */ +static unsigned int bit4_offsets[] =3D {6}; /* Buck 4 stat */ +static unsigned int bit5_offsets[] =3D {7}; /* LDO 5 stat */ +static unsigned int bit6_offsets[] =3D {8}; /* LDO 6 stat */ +static unsigned int bit7_offsets[] =3D {9}; /* LDO 7 stat */ + +static const struct regmap_irq_sub_irq_map errb_sub_irq_offsets[] =3D { + REGMAP_IRQ_MAIN_REG_OFFSET(bit0_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit1_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit2_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit3_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit4_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit5_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit6_offsets), + REGMAP_IRQ_MAIN_REG_OFFSET(bit7_offsets), +}; + +static const struct regmap_irq bd96801_errb_irqs[] =3D { + /* Reg 0x52 Fatal ERRB1 */ + REGMAP_IRQ_REG(BD96801_OTP_ERR_STAT, 0, BD96801_OTP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_DBIST_ERR_STAT, 0, BD96801_DBIST_ERR_MASK), + REGMAP_IRQ_REG(BD96801_EEP_ERR_STAT, 0, BD96801_EEP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_ABIST_ERR_STAT, 0, BD96801_ABIST_ERR_MASK), + REGMAP_IRQ_REG(BD96801_PRSTB_ERR_STAT, 0, BD96801_PRSTB_ERR_MASK), + REGMAP_IRQ_REG(BD96801_DRMOS1_ERR_STAT, 0, BD96801_DRMOS1_ERR_MASK), + REGMAP_IRQ_REG(BD96801_DRMOS2_ERR_STAT, 0, BD96801_DRMOS2_ERR_MASK), + REGMAP_IRQ_REG(BD96801_SLAVE_ERR_STAT, 0, BD96801_SLAVE_ERR_MASK), + /* 0x53 Fatal ERRB2 */ + REGMAP_IRQ_REG(BD96801_VREF_ERR_STAT, 1, BD96801_VREF_ERR_MASK), + REGMAP_IRQ_REG(BD96801_TSD_ERR_STAT, 1, BD96801_TSD_ERR_MASK), + REGMAP_IRQ_REG(BD96801_UVLO_ERR_STAT, 1, BD96801_UVLO_ERR_MASK), + REGMAP_IRQ_REG(BD96801_OVLO_ERR_STAT, 1, BD96801_OVLO_ERR_MASK), + REGMAP_IRQ_REG(BD96801_OSC_ERR_STAT, 1, BD96801_OSC_ERR_MASK), + REGMAP_IRQ_REG(BD96801_PON_ERR_STAT, 1, BD96801_PON_ERR_MASK), + REGMAP_IRQ_REG(BD96801_POFF_ERR_STAT, 1, BD96801_POFF_ERR_MASK), + REGMAP_IRQ_REG(BD96801_CMD_SHDN_ERR_STAT, 1, BD96801_CMD_SHDN_ERR_MASK), + /* 0x54 Fatal INTB shadowed to ERRB */ + REGMAP_IRQ_REG(BD96801_INT_PRSTB_WDT_ERR, 2, BD96801_INT_PRSTB_WDT_ERR_MA= SK), + REGMAP_IRQ_REG(BD96801_INT_CHIP_IF_ERR, 2, BD96801_INT_CHIP_IF_ERR_MASK), + REGMAP_IRQ_REG(BD96801_INT_SHDN_ERR_STAT, 2, BD96801_INT_SHDN_ERR_MASK), + /* Reg 0x55 BUCK1 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_BUCK1_PVIN_ERR_STAT, 3, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK1_OVP_ERR_STAT, 3, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK1_UVP_ERR_STAT, 3, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK1_SHDN_ERR_STAT, 3, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x56 BUCK2 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_BUCK2_PVIN_ERR_STAT, 4, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK2_OVP_ERR_STAT, 4, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK2_UVP_ERR_STAT, 4, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK2_SHDN_ERR_STAT, 4, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x57 BUCK3 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_BUCK3_PVIN_ERR_STAT, 5, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK3_OVP_ERR_STAT, 5, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK3_UVP_ERR_STAT, 5, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK3_SHDN_ERR_STAT, 5, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x58 BUCK4 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_BUCK4_PVIN_ERR_STAT, 6, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK4_OVP_ERR_STAT, 6, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK4_UVP_ERR_STAT, 6, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_BUCK4_SHDN_ERR_STAT, 6, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x59 LDO5 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_LDO5_PVIN_ERR_STAT, 7, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO5_OVP_ERR_STAT, 7, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO5_UVP_ERR_STAT, 7, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO5_SHDN_ERR_STAT, 7, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x5a LDO6 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_LDO6_PVIN_ERR_STAT, 8, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO6_OVP_ERR_STAT, 8, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO6_UVP_ERR_STAT, 8, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO6_SHDN_ERR_STAT, 8, BD96801_OUT_SHDN_ERR_MASK), + /* Reg 0x5b LDO7 ERR IRQs */ + REGMAP_IRQ_REG(BD96801_LDO7_PVIN_ERR_STAT, 9, BD96801_OUT_PVIN_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO7_OVP_ERR_STAT, 9, BD96801_OUT_OVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO7_UVP_ERR_STAT, 9, BD96801_OUT_UVP_ERR_MASK), + REGMAP_IRQ_REG(BD96801_LDO7_SHDN_ERR_STAT, 9, BD96801_OUT_SHDN_ERR_MASK), +}; + static const struct regmap_irq bd96801_intb_irqs[] =3D { /* STATUS SYSTEM INTB */ REGMAP_IRQ_REG(BD96801_TW_STAT, 0, BD96801_TW_STAT_MASK), @@ -176,8 +307,25 @@ static const struct regmap_irq bd96801_intb_irqs[] =3D= { REGMAP_IRQ_REG(BD96801_LDO7_UVD_STAT, 7, BD96801_LDO_UVD_STAT_MASK), }; =20 -static struct regmap_irq_chip bd96801_irq_chip_intb =3D { +static const struct regmap_irq_chip bd96801_irq_chip_errb =3D { + .name =3D "bd96801-irq-errb", + .domain_suffix =3D "errb", + .main_status =3D BD96801_REG_INT_MAIN, + .num_main_regs =3D 1, + .irqs =3D &bd96801_errb_irqs[0], + .num_irqs =3D ARRAY_SIZE(bd96801_errb_irqs), + .status_base =3D BD96801_REG_INT_SYS_ERRB1, + .mask_base =3D BD96801_REG_MASK_SYS_ERRB, + .ack_base =3D BD96801_REG_INT_SYS_ERRB1, + .init_ack_masked =3D true, + .num_regs =3D 10, + .irq_reg_stride =3D 1, + .sub_reg_offsets =3D &errb_sub_irq_offsets[0], +}; + +static const struct regmap_irq_chip bd96801_irq_chip_intb =3D { .name =3D "bd96801-irq-intb", + .domain_suffix =3D "intb", .main_status =3D BD96801_REG_INT_MAIN, .num_main_regs =3D 1, .irqs =3D &bd96801_intb_irqs[0], @@ -199,11 +347,15 @@ static const struct regmap_config bd96801_regmap_conf= ig =3D { =20 static int bd96801_i2c_probe(struct i2c_client *i2c) { - struct regmap_irq_chip_data *intb_irq_data; + struct regmap_irq_chip_data *intb_irq_data, *errb_irq_data; + struct irq_domain *intb_domain, *errb_domain; const struct fwnode_handle *fwnode; - struct irq_domain *intb_domain; + struct resource *regulator_res; + struct resource wdg_irq; struct regmap *regmap; - int ret, intb_irq; + int intb_irq, errb_irq, num_intb, num_errb =3D 0; + int num_regu_irqs, wdg_irq_no; + int i, ret; =20 fwnode =3D dev_fwnode(&i2c->dev); if (!fwnode) @@ -213,6 +365,23 @@ static int bd96801_i2c_probe(struct i2c_client *i2c) if (intb_irq < 0) return dev_err_probe(&i2c->dev, intb_irq, "INTB IRQ not configured\n"); =20 + num_intb =3D ARRAY_SIZE(regulator_intb_irqs); + + /* ERRB may be omitted if processor is powered by the PMIC */ + errb_irq =3D fwnode_irq_get_byname(fwnode, "errb"); + if (errb_irq < 0) + errb_irq =3D 0; + + if (errb_irq) + num_errb =3D ARRAY_SIZE(regulator_errb_irqs); + + num_regu_irqs =3D num_intb + num_errb; + + regulator_res =3D devm_kcalloc(&i2c->dev, num_regu_irqs, + sizeof(*regulator_res), GFP_KERNEL); + if (!regulator_res) + return -ENOMEM; + regmap =3D devm_regmap_init_i2c(i2c, &bd96801_regmap_config); if (IS_ERR(regmap)) return dev_err_probe(&i2c->dev, PTR_ERR(regmap), @@ -230,12 +399,50 @@ static int bd96801_i2c_probe(struct i2c_client *i2c) =20 intb_domain =3D regmap_irq_get_domain(intb_irq_data); =20 - ret =3D devm_mfd_add_devices(&i2c->dev, PLATFORM_DEVID_AUTO, - bd96801_cells, - ARRAY_SIZE(bd96801_cells), NULL, 0, - intb_domain); + /* + * MFD core code is built to handle only one IRQ domain. BD96801 + * has two domains so we do IRQ mapping here and provide the + * already mapped IRQ numbers to sub-devices. + */ + for (i =3D 0; i < num_intb; i++) { + struct resource *res =3D ®ulator_res[i]; + + *res =3D regulator_intb_irqs[i]; + res->start =3D res->end =3D irq_create_mapping(intb_domain, + res->start); + } + + wdg_irq_no =3D irq_create_mapping(intb_domain, BD96801_WDT_ERR_STAT); + wdg_irq =3D DEFINE_RES_IRQ_NAMED(wdg_irq_no, "bd96801-wdg"); + bd96801_cells[WDG_CELL].resources =3D &wdg_irq; + bd96801_cells[WDG_CELL].num_resources =3D 1; + + if (!num_errb) + goto skip_errb; + + ret =3D devm_regmap_add_irq_chip(&i2c->dev, regmap, errb_irq, IRQF_ONESHO= T, + 0, &bd96801_irq_chip_errb, &errb_irq_data); + if (ret) + return dev_err_probe(&i2c->dev, ret, + "Failed to add ERRB IRQ chip\n"); + + errb_domain =3D regmap_irq_get_domain(errb_irq_data); + + for (i =3D 0; i < num_errb; i++) { + struct resource *res =3D ®ulator_res[num_intb + i]; + + *res =3D regulator_errb_irqs[i]; + res->start =3D res->end =3D irq_create_mapping(errb_domain, res->start); + } + +skip_errb: + bd96801_cells[REGULATOR_CELL].resources =3D regulator_res; + bd96801_cells[REGULATOR_CELL].num_resources =3D num_regu_irqs; + + ret =3D devm_mfd_add_devices(&i2c->dev, PLATFORM_DEVID_AUTO, bd96801_cell= s, + ARRAY_SIZE(bd96801_cells), NULL, 0, NULL); if (ret) - dev_err(&i2c->dev, "Failed to create subdevices\n"); + dev_err_probe(&i2c->dev, ret, "Failed to create subdevices\n"); =20 return ret; } --=20 2.46.2 --Fb5sccfyGGIgPt4c Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEIx+f8wZb28fLKEhTeFA3/03aocUFAmb+LO4ACgkQeFA3/03a ocUJmAgAn8gNNdZaA0d3Ei+AG7a2lsYFLFeroqSOhnkBu1wQHF0KBlOoSyTRkD89 BUrbhe584hIvLjgQqdJeW7UoF7f932ldJcECkF3oIRC3l+eW0Ja7q8y9VIBLNsl7 AC+BNOpduSDTHpPQATJuskugbvNDFpDk+Y+Uq6k+F/jH1nyvARRFFlQTSJh9OGSH OlwUxgddHRokanTR/juOeo8i0BLiZHQsgQ8mUnE+6nFbe/gjs2rp0Z6cTgD2fnod 6y7aFXv11ylrcRq9Aj645f1Wq+g6CFTfPw2Ns7Wq2oCUWlYh6lo2cuiScaQuMheo JKMO+P1yhHk3kREwxpC8OWBfJdXP9w== =lLEf -----END PGP SIGNATURE----- --Fb5sccfyGGIgPt4c-- From nobody Thu Nov 28 06:43:40 2024 Received: from mail-lj1-f181.google.com (mail-lj1-f181.google.com [209.85.208.181]) (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 143B42110E for ; Thu, 3 Oct 2024 05:35:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727933711; cv=none; b=iZpBe+7FYffxKqEwg/qs+T902rcMF2sCWcOxn2WUX3aIFMdreR2P5+WgHEPOIcWLLZap5f1dcwu4IHl+JWumC4nNyGsF4HfnrJpwXwjjzYe2iWVHTyBS8kstl5TCTky5XicvNwXhD0whzaGkZx3C7P1e65Xzwa3xuRETA6s1raM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727933711; c=relaxed/simple; bh=3IlHvMzfXs7DVVZZJeCQMP0OmMSJFsoLjhUdtfWr4zg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ssIJS1Nlfj40RZt3lgTzOTl16idSibFEXXyrCJ7a0eHlcW4Wc2fGKPSkCAVgZN8V7xTL6Cb3TYyJuO52SqcU0yZQhIniDklozPvF7G2hgURDiUrH6CKlghPRwfajMkYpRGb7MrnWYduyxHXPoO3780SZyeo3uFrXNIx1dFzV+cY= 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=UkgBUiMA; arc=none smtp.client-ip=209.85.208.181 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="UkgBUiMA" Received: by mail-lj1-f181.google.com with SMTP id 38308e7fff4ca-2fadc95ccfcso5861651fa.1 for ; Wed, 02 Oct 2024 22:35:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1727933707; x=1728538507; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=Y+bONBH8vRb45l8ZwAcQrWMX7Zp4OI2J4UYOF4hZhTE=; b=UkgBUiMAla+Zqk120n4DUkVOYH8LqfsfDCltCUc1Emfzp1wuVX6Hdw4PJyGYuC44hZ qEYOumHn5lu89VU2M8RFynWWtD6qqpymJnfuJS2waYQgEp76BzSgIriIIT+EiPieAM9v ixelx4VlyVWRM4ydtIA6XJebf2qmOvxiCNmh+orTAB5OhWn5ZM9cB7IhVKpACfC20XrR hzpGQ6IVLR1l8c+PNCiXN85lwo0va2qtq0rjyNrqQRZNw+tZz8lAXnsOPYd5epV3dFDV mTdKezHNfiT8J8IzO1MiqZXrM4T+klQGYKR/VnAePf5tzL2NfWwcPgrXB+seM3cuaynF PSZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727933707; x=1728538507; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Y+bONBH8vRb45l8ZwAcQrWMX7Zp4OI2J4UYOF4hZhTE=; b=ISVAeocLs5ePk4PMtPZ/XOg9y6gPOAp3Hzgbb750ymlaUfi6neaLPxHxlV/+KPycWy JjFz+3qlHvronohur16OikldjSkMwedyU8dlPAiNXs2K/Wzpy5WjSmXrPoqnSoeydpTR rnsNwsjOuZv6WXnVgLjAqWi+DG9UENGLKApd4WKa0GhkSZz2uDVYIjW/2nkUjbtCjhwE 1eowPjUKQ9r+ds7JuNVV27U6hr6zTyGJuqP4ntL9nv+QMZno9D13LkWe5Mb/pLIKYya3 wYxB5m7jYSqppT1xantn+UXIxzEduwdU/ZGxhq2cp3YQft9QP+jbVGugL/ayx6oqHEyz AuEQ== X-Forwarded-Encrypted: i=1; AJvYcCXCLDp0PZWkrTqJX6b2EiPkDmXHfpcLQshNAVrqdBY9h5bsCcgA17Mac1bYVy48gb8joUlc7Q4tljt9JpM=@vger.kernel.org X-Gm-Message-State: AOJu0YzSPpdUlSrsUHcSCoCPAgUlbt+Q1/4Nv8Ousb+N3FTTACb6oIdr bUCaoFFyQIqpqYbhpLY5Yi1KuwoWKH5iFSDDhERe07DHnHwOoZHh X-Google-Smtp-Source: AGHT+IFwd4okNYyype2THoni3IobGnrJuAueiDpRaPgkciz1GtILj6y3uWZvyojSryLOTHLv0zyL6Q== X-Received: by 2002:a2e:751:0:b0:2fa:dce8:7387 with SMTP id 38308e7fff4ca-2fae107f75bmr26902041fa.32.1727933706770; Wed, 02 Oct 2024 22:35:06 -0700 (PDT) Received: from mva-rohm ([2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703]) by smtp.gmail.com with ESMTPSA id 38308e7fff4ca-2faeccb1bfdsm1074751fa.138.2024.10.02.22.35.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Oct 2024 22:35:04 -0700 (PDT) Date: Thu, 3 Oct 2024 08:34:58 +0300 From: Matti Vaittinen To: Matti Vaittinen , Matti Vaittinen Cc: Matti Vaittinen , Lee Jones , Liam Girdwood , Mark Brown , linux-kernel@vger.kernel.org Subject: [PATCH v2 2/2] regulator: bd96801: Add ERRB IRQ Message-ID: <9a32fc1a5210195551ad6df1cd1242235c280a04.1727931468.git.mazziesaccount@gmail.com> References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="5PtqFW143PZN+gKM" Content-Disposition: inline In-Reply-To: --5PtqFW143PZN+gKM Content-Disposition: inline Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" The ROHM BD96801 "scalable PMIC" provides two physical IRQs. The ERRB handling can in many cases be omitted because it is used to inform fatal IRQs, which usually kill the power from the SOC. There may however be use-cases where the SOC has a 'back-up' emergency power source which allows some very short time of operation to try to gracefully shut down sensitive hardware. Furthermore, it is possible the processor controlling the PMIC is not powered by the PMIC. In such cases handling the ERRB IRQs may be beneficial. Add support for ERRB IRQs. Signed-off-by: Matti Vaittinen Reviewed-by: Mark Brown --- Revision history: New series (only ERRB addition) v1 =3D> - No changes Old series (All BD96801 functionality + irqdomain and regmap changes) v2 =3D>: - No changes v1 =3D> v2: - New patch --- drivers/regulator/bd96801-regulator.c | 130 +++++++++++++++++++++++--- 1 file changed, 115 insertions(+), 15 deletions(-) diff --git a/drivers/regulator/bd96801-regulator.c b/drivers/regulator/bd96= 801-regulator.c index 9876cc05867e..3a9d772491a8 100644 --- a/drivers/regulator/bd96801-regulator.c +++ b/drivers/regulator/bd96801-regulator.c @@ -5,12 +5,7 @@ /* * This version of the "BD86801 scalable PMIC"'s driver supports only very * basic set of the PMIC features. Most notably, there is no support for - * the ERRB interrupt and the configurations which should be done when the - * PMIC is in STBY mode. - * - * Supporting the ERRB interrupt would require dropping the regmap-IRQ - * usage or working around (or accepting a presense of) a naming conflict - * in debugFS IRQs. + * the configurations which should be done when the PMIC is in STBY mode. * * Being able to reliably do the configurations like changing the * regulator safety limits (like limits for the over/under -voltages, over @@ -22,16 +17,14 @@ * be the need to configure these safety limits. Hence it's not simple to * come up with a generic solution. * - * Users who require the ERRB handling and STBY state configurations can - * have a look at the original RFC: + * Users who require the STBY state configurations can have a look at the + * original RFC: * https://lore.kernel.org/all/cover.1712920132.git.mazziesaccount@gmail.c= om/ - * which implements a workaround to debugFS naming conflict and some of - * the safety limit configurations - but leaves the state change handling - * and synchronization to be implemented. + * which implements some of the safety limit configurations - but leaves t= he + * state change handling and synchronization to be implemented. * * It would be great to hear (and receive a patch!) if you implement the - * STBY configuration support or a proper fix to the debugFS naming - * conflict in your downstream driver ;) + * STBY configuration support in your downstream driver ;) */ =20 #include @@ -728,6 +721,95 @@ static int initialize_pmic_data(struct device *dev, return 0; } =20 +static int bd96801_map_event_all(int irq, struct regulator_irq_data *rid, + unsigned long *dev_mask) +{ + int i; + + for (i =3D 0; i < rid->num_states; i++) { + rid->states[i].notifs =3D REGULATOR_EVENT_FAIL; + rid->states[i].errors =3D REGULATOR_ERROR_FAIL; + *dev_mask |=3D BIT(i); + } + + return 0; +} + +static int bd96801_rdev_errb_irqs(struct platform_device *pdev, + struct regulator_dev *rdev) +{ + int i; + void *retp; + static const char * const single_out_errb_irqs[] =3D { + "bd96801-%s-pvin-err", "bd96801-%s-ovp-err", + "bd96801-%s-uvp-err", "bd96801-%s-shdn-err", + }; + + for (i =3D 0; i < ARRAY_SIZE(single_out_errb_irqs); i++) { + struct regulator_irq_desc id =3D { + .map_event =3D bd96801_map_event_all, + .irq_off_ms =3D 1000, + }; + struct regulator_dev *rdev_arr[1]; + char tmp[255]; + int irq; + + snprintf(tmp, 255, single_out_errb_irqs[i], rdev->desc->name); + tmp[254] =3D 0; + id.name =3D tmp; + + irq =3D platform_get_irq_byname(pdev, tmp); + if (irq < 0) + continue; + + rdev_arr[0] =3D rdev; + retp =3D devm_regulator_irq_helper(&pdev->dev, &id, irq, 0, + REGULATOR_ERROR_FAIL, NULL, + rdev_arr, 1); + if (IS_ERR(retp)) + return PTR_ERR(retp); + + } + return 0; +} + +static int bd96801_global_errb_irqs(struct platform_device *pdev, + struct regulator_dev **rdev, int num_rdev) +{ + int i, num_irqs; + void *retp; + static const char * const global_errb_irqs[] =3D { + "bd96801-otp-err", "bd96801-dbist-err", "bd96801-eep-err", + "bd96801-abist-err", "bd96801-prstb-err", "bd96801-drmoserr1", + "bd96801-drmoserr2", "bd96801-slave-err", "bd96801-vref-err", + "bd96801-tsd", "bd96801-uvlo-err", "bd96801-ovlo-err", + "bd96801-osc-err", "bd96801-pon-err", "bd96801-poff-err", + "bd96801-cmd-shdn-err", "bd96801-int-shdn-err" + }; + + num_irqs =3D ARRAY_SIZE(global_errb_irqs); + for (i =3D 0; i < num_irqs; i++) { + int irq; + struct regulator_irq_desc id =3D { + .name =3D global_errb_irqs[i], + .map_event =3D bd96801_map_event_all, + .irq_off_ms =3D 1000, + }; + + irq =3D platform_get_irq_byname(pdev, global_errb_irqs[i]); + if (irq < 0) + continue; + + retp =3D devm_regulator_irq_helper(&pdev->dev, &id, irq, 0, + REGULATOR_ERROR_FAIL, NULL, + rdev, num_rdev); + if (IS_ERR(retp)) + return PTR_ERR(retp); + } + + return 0; +} + static int bd96801_rdev_intb_irqs(struct platform_device *pdev, struct bd96801_pmic_data *pdata, struct bd96801_irqinfo *iinfo, @@ -783,11 +865,10 @@ static int bd96801_rdev_intb_irqs(struct platform_dev= ice *pdev, return 0; } =20 - - static int bd96801_probe(struct platform_device *pdev) { struct regulator_dev *ldo_errs_rdev_arr[BD96801_NUM_LDOS]; + struct regulator_dev *all_rdevs[BD96801_NUM_REGULATORS]; struct bd96801_regulator_data *rdesc; struct regulator_config config =3D {}; int ldo_errs_arr[BD96801_NUM_LDOS]; @@ -795,6 +876,7 @@ static int bd96801_probe(struct platform_device *pdev) int temp_notif_ldos =3D 0; struct device *parent; int i, ret; + bool use_errb; void *retp; =20 parent =3D pdev->dev.parent; @@ -819,6 +901,13 @@ static int bd96801_probe(struct platform_device *pdev) config.regmap =3D pdata->regmap; config.dev =3D parent; =20 + ret =3D of_property_match_string(pdev->dev.parent->of_node, + "interrupt-names", "errb"); + if (ret < 0) + use_errb =3D false; + else + use_errb =3D true; + ret =3D bd96801_walk_regulator_dt(&pdev->dev, pdata->regmap, rdesc, BD96801_NUM_REGULATORS); if (ret) @@ -837,6 +926,7 @@ static int bd96801_probe(struct platform_device *pdev) rdesc[i].desc.name); return PTR_ERR(rdev); } + all_rdevs[i] =3D rdev; /* * LDOs don't have own temperature monitoring. If temperature * notification was requested for this LDO from DT then we will @@ -856,6 +946,12 @@ static int bd96801_probe(struct platform_device *pdev) if (ret) return ret; } + /* Register per regulator ERRB notifiers */ + if (use_errb) { + ret =3D bd96801_rdev_errb_irqs(pdev, rdev); + if (ret) + return ret; + } } if (temp_notif_ldos) { int irq; @@ -877,6 +973,10 @@ static int bd96801_probe(struct platform_device *pdev) return PTR_ERR(retp); } =20 + if (use_errb) + return bd96801_global_errb_irqs(pdev, all_rdevs, + ARRAY_SIZE(all_rdevs)); + return 0; } =20 --=20 2.46.2 --5PtqFW143PZN+gKM Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEIx+f8wZb28fLKEhTeFA3/03aocUFAmb+LQIACgkQeFA3/03a ocVcdAgAxAdHTRRuExhAVmc60iG+jHPa5xirsylY2S+AcQNhJh5OqiPXNy5HwqCe 1/21unQPHLnjXCsHs7pMUbO5JSuBUvBstjVTSXguHo4wAPEyFHiEmaSy96P60dZp vchpLLaqCh5N8MggoLcibB1WQ0KHqW0HGEFkbWTCHyfF3iyjL60D4vvpB9gzrWmb KrCPjJT+tdLCZQVGW1KiYwq5hncc7J0tdi/UjdljVSCRpdrAVSX8ECH2Xu7+dkLO aI4iiXXlEsKfwPW973v164MdNiYCKCH6+WbQI/+gCcPqu9eyPgyCz4lBwFDqghhY Wxm1CPwAwsi81X5KbImUA/d3S8uVtw== =rQQy -----END PGP SIGNATURE----- --5PtqFW143PZN+gKM--