Re: [PATCH -next] soc/tegra: fuse: don't return -ENOMEM when allocate lookups failed

From: Thierry Reding
Date: Tue Jun 01 2021 - 13:00:34 EST


On Mon, Apr 12, 2021 at 10:05:27PM +0800, Yang Yingliang wrote:
> fuse->base can not be unmapped if allocate lookups failed in
> tegra_init_fuse(), because it is an early_initcall, the driver
> will be loaded anyway and fuse->base will be accessed by other
> functions later, so remove the return -ENOMEM after allocating
> lookups failed to make less confusing.
>
> Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx>
> ---
> drivers/soc/tegra/fuse/fuse-tegra.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)

Applied, though I did drop the error message. Out-of-memory situations
are noisy anyway, so the extra error message doesn't add anything.

Thanks,
Thierry

Attachment: signature.asc
Description: PGP signature