Re: [PATCH] EDAC/dmc520: Don't print an error for each unconfigured interrupt line

[Borislav Petkov]

On Mon, Apr 18, 2022 at 03:40:29PM -0500, Tyler Hicks wrote:
> > The API argument seems to have fizzled out in v2:
> > 
> >  https://lore.kernel.org/lkml/20220212201631.12648-1-s.shtylyov@xxxxxx/

I don't see those two upstream yet, on a quick glance. Perhaps in Greg's tree?

Greg, what's the latest with that platform_get_*_optional() fun?

Also, the second of those two patches above has:

+ * Return: non-zero IRQ number on success, 0 if IRQ wasn't found, negative error
+ * number on failure.
  */
 int platform_get_irq_byname_optional(struct platform_device *dev,

and your patch does:

+		irq = platform_get_irq_byname_optional(pdev, dmc520_irq_configs[idx].name);
 		irqs[idx] = irq;

so on failure, it would still write the negative error value in
irqs[idx].

How can that be right?

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette