linux-mediatek.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [RFC] mfd: mt6358: Using a single IRQ array instead of two companion arrays?
@ 2025-11-19 15:52 Markus Elfring
  2025-11-20 10:05 ` Lee Jones
  0 siblings, 1 reply; 3+ messages in thread
From: Markus Elfring @ 2025-11-19 15:52 UTC (permalink / raw)
  To: linux-mediatek, linux-arm-kernel, Angelo Gioacchino Del Regno,
	Hsin-Hsiung Wang, Lee Jones, Matthias Brugger
  Cc: LKML

Hello,

I have taken another look also at the implementation of the function “mt6358_irq_init”.
https://elixir.bootlin.com/linux/v6.18-rc6/source/drivers/mfd/mt6358-irq.c#L227-L293

I see that there are two arrays allocated so far.
How do you think about the possibility to use a single array with the needed data instead?

Regards,
Markus


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC] mfd: mt6358: Using a single IRQ array instead of two companion arrays?
  2025-11-19 15:52 [RFC] mfd: mt6358: Using a single IRQ array instead of two companion arrays? Markus Elfring
@ 2025-11-20 10:05 ` Lee Jones
  2025-11-20 10:51   ` Markus Elfring
  0 siblings, 1 reply; 3+ messages in thread
From: Lee Jones @ 2025-11-20 10:05 UTC (permalink / raw)
  To: Markus Elfring
  Cc: linux-mediatek, linux-arm-kernel, Angelo Gioacchino Del Regno,
	Hsin-Hsiung Wang, Matthias Brugger, LKML

On Wed, 19 Nov 2025, Markus Elfring wrote:

> Hello,
> 
> I have taken another look also at the implementation of the function “mt6358_irq_init”.
> https://elixir.bootlin.com/linux/v6.18-rc6/source/drivers/mfd/mt6358-irq.c#L227-L293
> 
> I see that there are two arrays allocated so far.
> How do you think about the possibility to use a single array with the needed data instead?

It'll be easier if you just submit the patch.

-- 
Lee Jones [李琼斯]


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC] mfd: mt6358: Using a single IRQ array instead of two companion arrays?
  2025-11-20 10:05 ` Lee Jones
@ 2025-11-20 10:51   ` Markus Elfring
  0 siblings, 0 replies; 3+ messages in thread
From: Markus Elfring @ 2025-11-20 10:51 UTC (permalink / raw)
  To: Lee Jones, linux-mediatek, linux-arm-kernel,
	Angelo Gioacchino Del Regno, Hsin-Hsiung Wang, Matthias Brugger
  Cc: LKML

>> I have taken another look also at the implementation of the function “mt6358_irq_init”.
>> https://elixir.bootlin.com/linux/v6.18-rc6/source/drivers/mfd/mt6358-irq.c#L227-L293
>>
>> I see that there are two arrays allocated so far.
>> How do you think about the possibility to use a single array with the needed data instead?
> 
> It'll be easier if you just submit the patch.

I would appreciate to clarify before if the adjustment of array usage
can influence software run time characteristics in desirable directions here.
https://elixir.bootlin.com/linux/v6.18-rc6/source/include/linux/mfd/mt6358/core.h#L19-L26

Regards,
Markus


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-11-20 10:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-19 15:52 [RFC] mfd: mt6358: Using a single IRQ array instead of two companion arrays? Markus Elfring
2025-11-20 10:05 ` Lee Jones
2025-11-20 10:51   ` Markus Elfring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).