From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Mon, 1 Oct 2007 06:57:21 -0400 Subject: [U-Boot-Users] API for serial functions In-Reply-To: <20071001104309.9753F2408A@gemini.denx.de> References: <20071001104309.9753F2408A@gemini.denx.de> Message-ID: <200710010657.22644.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Monday 01 October 2007, Wolfgang Denk wrote: > In message <200710010620.50106.vapier@gentoo.org> you wrote: > > > So I vote for not removing this transmitter empty check. > > > > i'd note that this is pretty inconsistent across different ports ... > > I agree with Stefan. Please don't change this. The current implemen- > tation (wait until trasmit has completed) is what I prefer. np ... as for non-Blackfin ports, it isnt my code so i dont care ;) > > so when do you return ? after making sure the byte has moved from the > > core to the peripheral hardware ? after making sure the byte has started > > to be shifted out from the peripheral onto the line ? after making sure > > the byte has been completed shifted onto the line ? the current Blackfin > > serial driver exhibits the last -- it waits until the whole byte has gone > > through the whole process, from core through the line > > If that's how it was implemented, then leave it that way. It's OK. there is plenty of code (on the Blackfin side) that was thrown together and left that way because "it just worked" rather than being correct ... so i dont buy the "this is how it was, so just leave it" argument but if the behavior we want is "drain the byte completely across the line before returning", that's easy enough thanks :) -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 827 bytes Desc: This is a digitally signed message part. Url : http://lists.denx.de/pipermail/u-boot/attachments/20071001/c97516f8/attachment.pgp