On 12/17/07, Peter Korsgaard <[EMAIL PROTECTED]> wrote: > >>>>> "Grant" == Grant Likely <[EMAIL PROTECTED]> writes: > > Hi, > > >> > 3) Uartlite requires a port-number property for the console to work. > >> > >> Why? In general we try to avoid magical sequence numbers - cell-index > >> should *only* be used when it's needed to index or program some shared > >> register. > > Grant> Aliases is probably the correct construct for this. > Grant> port-number was a bad idea and I never should have gone that > Grant> direction. > > Perhaps it would make more sense to add a _find_match_or_unused(port) > like 8250.c and make the initialization order define the index instead > of explicitly setting it?
aliases I think is the better approach as it is an already established convention. enumeration based on init order is fragile and there is no guarantee that order will not change in the future. > > Now we're at it - Why isn't the uartlite of glue handled by of_serial.c? The of_serial.c glue is IMHO too complex for the uartlite case. I saw no advantage provided by the added complexity. (ulite_of_probe() is all of 12 lines of code). Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. [EMAIL PROTECTED] (403) 399-0195 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
