+Sam,
On 10/18/21 10:42, Greg KH wrote:
On Mon, Oct 18, 2021 at 01:50:00PM +0530, Shubhrajyoti Datta wrote:
Update the default number of uarts.
The commit b44b96a060f3 changed the default from 16 to 1 change it back to 16.
Fixes: b44b96a060f3 (uartlite: Adding a kernel parameter for the number of uartlites)
No extra line here please.
Also, your subject is odd, please make it more obvious what is
happening.
But nost importantly, _WHY_ are you making this change? What happened
since that commit was made that now we should have the default be 16?
Previously that commit said the default should NOT be 16, and here you
are changing it back. What changed to require this?
"The uartlite is used by FPGAs that support a basically unlimited number
of uarts so limiting it at 16 dosn't make sense as users might need more
than that."
He said that doesn't make sense to limit it to 16 and limit it to 256 by
Kconfig range and set default to 1. Where it is also said that number
should be unlimited.
And also no information why he setup default to 1 where it should be
unlimited. I would say high number should be fine for the most of cases.
I have never seen any design with 256 uarts on it.
I think we could potentially find real maximum number for looking at the
biggest fpga and setup this number but from my perspective 256 is big
enough for a range. But default 1 sounds weird. And origin 16 written in
driver should be quite reasonable default to cover 99% cases.
Thanks,
Michal
