Re: [PATCH v2 7/7] mfd: qcom-pm8008: Tolerate missing interrupt

[Jishnu Prakash]

Hi Konrad,

On 10/26/2026 7:42 PM, Konrad Dybcio wrote:
> On 10/19/26 7:01 AM, Jishnu Prakash wrote:
>> Hi Konrad,
>>
>> On 10/15/2026 5:35 PM, Konrad Dybcio wrote:
>>> On 10/14/26 12:32 PM, Jishnu Prakash wrote:
>>>> From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
> 
> [...]
> 
>> And another point we should note is, the MFD cell name
>> "qpnp-temp-alarm" used right now does not bind to our temp-alarm
>> driver and this was intentional, as explained by Johan Hovold
>> in his change (mfd: pm8008: Rework to match new DT binding),
>> in the last paragraph of the commit message:
>>
>>     Finally, note that the temperature alarm and GPIO subdrivers need some
>>     minor rework before they can be used with non-SPMI devices like the
>>     PM8008. The temperature alarm MFD cell name specifically uses a "qpnp"
>>     rather than "spmi" prefix to prevent binding until the driver has been
>>     updated.
>>
>> Since these changes are not yet done, we cannot use
>> temp-alarm for PM8008 or PM8010 over I2C for now.
> 
> I read that as one more reason to shave off said diff
> 

If you mean I should simply use the existing MFD cells instead of
adding new no-IRQ cells, that would also not work, because I
added them specifically to address an issue Sashiko found in v1:
https://lore.kernel.org/all/20260918165721.CA17C1F000FF@xxxxxxxxxxxxxxx/.

They are needed to avoid assigning IRQ resources from the MFD
cells when no parent IRQ is present.

Thanks,
Jishnu


> Konrad