Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
Message-ID: <trinity-23757337-a194-45c1-864b-6f96a754fed1-1627645548572@3c-app-gmx-bs53>
From:   Lino Sanfilippo <LinoSanfilippo@gmx.de>
To:     Jarkko Sakkinen <jarkko@kernel.org>
Cc:     peterhuewe@gmx.de, jgg@ziepe.ca, linus.walleij@linaro.org,
        linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org,
        stable@vger.kernel.org
Subject: Aw: Re: [PATCH v2] tpm, tpm_tis_spi: Allow to sleep in the
 interrupt handler
Content-Type: text/plain; charset=UTF-8
Date:   Fri, 30 Jul 2021 13:45:48 +0200
Importance: normal
Sensitivity: Normal
In-Reply-To: <20210623133420.gw2lziue5nkvjtps@kernel.org>
References: <20210620023444.14684-1-LinoSanfilippo@gmx.de>
 <20210623133420.gw2lziue5nkvjtps@kernel.org>
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Jarko,

> Gesendet: Mittwoch, 23. Juni 2021 um 15:34 Uhr
> Von: "Jarkko Sakkinen" <jarkko@kernel.org>
> An: "Lino Sanfilippo" <LinoSanfilippo@gmx.de>
> Cc: peterhuewe@gmx.de, jgg@ziepe.ca, linus.walleij@linaro.org, linux-int=
egrity@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.o=
rg
> Betreff: Re: [PATCH v2] tpm, tpm_tis_spi: Allow to sleep in the interrup=
t handler
>
> On Sun, Jun 20, 2021 at 04:34:44AM +0200, Lino Sanfilippo wrote:
> > Interrupt handling at least includes reading and writing the interrupt
> > status register within the interrupt routine. For accesses over SPI a =
mutex
> > is used in the concerning functions. Since this requires a sleepable
> > context request a threaded interrupt handler for this case.
> >
> > Cc: stable@vger.kernel.org
> > Fixes: 1a339b658d9d ("tpm_tis_spi: Pass the SPI IRQ down to the driver=
")
> > Signed-off-by: Lino Sanfilippo <LinoSanfilippo@gmx.de>
>
> I'll test this after rc1 PR (I have one NUC which uses tpm_tis_spi).
>
> /Jarkko
>

any news from this patch? Did you already have the opportunity to test it?

Regards,
Lino