On 07/10/2013 03:01 PM, Sascha Hauer wrote: > Anyway, maybe Jean-Christophe is right and we should do this in > console_std_set by adding: > > if(flag && cdev->setbrg) > cdev->setbrg(cdev, cdev->baudrate); > > This makes sure the hardware is only ever touched when the console is > actually used. > > Sascha > Hi, Agree. The only reason not do to so was that I saw the existing code, especially the else clause in console_baudrate_set(). I just made v2 of the patch. I'm testing it right now. If it's Ok, I'll submit it. I did almost exactly this. The difference is that I do not call setbrg() again if the device is already active. I also removed setbrg() call for inactive devices from console_baudrate_set() as this would be duplicate call now. Regards, Alex _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
