On 1/20/20 6:46 AM, Peter Maydell wrote:
On Mon, 20 Jan 2020 at 14:30, Guenter Roeck <li...@roeck-us.net> wrote:
On 1/20/20 5:35 AM, Peter Maydell wrote:
As the comment in or-irq.h notes, we can safely simply bump the
#define value without breaking anything if you need more input
OR lines than 32.
Yes, I noticed the comment, and I did that initially, but then
I noticed the complexity of actually doing it in the code
increasing it from 16 to 32, and decided I better leave it alone.
Yeah, the conversion from 16 to 32 was hairy because our
initial implementation made the migration-compatibility
awkward. When I wrote that conversion I was careful to
avoid creating a similar problem for my future self if
I needed to bump the value again :-)
Ok, with that in mind I'll add a patch increasing the limit.
I'll set it to 48. Then we can discuss if the new limit is
sufficient/acceptable ;-).
Thanks,
Guenter
