RE: [PATCH 2/4] irqchip: add RZ/{T2H,N2H} Interrupt Controller (ICU) driver

[Thomas Gleixner]

On Mon, Nov 24 2025 at 15:28, Cosmin-Gabriel Tanislav wrote:
>> From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
>> Sent: Monday, November 24, 2025 3:49 PM
>> 
>> On Mon, Nov 24 2025 at 12:50, Cosmin-Gabriel Tanislav wrote:
>> >> From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
>> >> Sent: Saturday, November 22, 2025 5:56 PM
>> 
>> Can you please fix your mail-client not to copy the whole header into
>> the reply?
>
> Outlook, it's unfixable. I can remove it manually each time if it's
> too much noise.

Either that or ask your colleagues how they avoid this nonsense.

>> >> > +   if (!irq_domain) {
>> >> > +           pm_runtime_put(dev);
>> >> > +           return -ENOMEM;
>> >> > +   }
>> >>
>> >> The mix of 'return $ERR' and 'return dev_err_probe()' is confusing at best.
>> >>
>> >
>> > For ENOMEM, dev_err_probe() doesn't really print anything. ENOMEM is
>> > what other drivers seem to use for a NULL irq_domain_create_hierarchy()
>> > result.
>> 
>> That's what I was missing. Now it makes sense.
>> 
> In conclusion, should I keep the bare `return -ENOMEM` in both instances?
> Just to make sure the next version is proper.

Keep the -ENOMEM.