From mboxrd@z Thu Jan 1 00:00:00 1970 From: Detlev Zundel Date: Fri, 09 Sep 2011 14:09:43 +0200 Subject: [U-Boot] [PATCH v3] ns16550: change to allow 32 bit access to registers In-Reply-To: <20110907212224.44D6E140875D@gemini.denx.de> (Wolfgang Denk's message of "Wed, 07 Sep 2011 23:22:24 +0200") References: <1314877212-31552-1-git-send-email-fovsoft@gmail.com> <20110907212224.44D6E140875D@gemini.denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, > Dear Dave Aldridge, > > In message <1314877212-31552-1-git-send-email-fovsoft@gmail.com> you wrote: >> If CONFIG_SYS_NS16550_MEM32 is defined then 32 bit memory >> mapped access will be used to read/write the uart registers. >> >> This is especially useful for SoC devices that implement 16550 >> compatible uarts but that have peripheral access width constraints. >> >> Signed-off-by: Dave Aldridge > ... > >> --- a/drivers/serial/ns16550.c >> +++ b/drivers/serial/ns16550.c >> @@ -19,6 +19,12 @@ >> #ifdef CONFIG_SYS_NS16550_PORT_MAPPED >> #define serial_out(x,y) outb(x,(ulong)y) >> #define serial_in(y) inb((ulong)y) >> +#elif defined(CONFIG_SYS_NS16550_MEM32) && (CONFIG_SYS_NS16550_REG_SIZE > 0) >> +#define serial_out(x,y) out_be32(y,x) >> +#define serial_in(y) in_be32(y) >> +#elif defined(CONFIG_SYS_NS16550_MEM32) && (CONFIG_SYS_NS16550_REG_SIZE < 0) >> +#define serial_out(x,y) out_le32(y,x) >> +#define serial_in(y) in_le32(y) > > Sorry for the dumb question, but in which way is REG_SIZE > 0 or > REG_SIZE < 0 connected to the endianess of the target system? > > My understanding is that this only defines how byte wide registers > need to be padded, i. e. wether they are connected to the highest or > to the lowest byte lane. THis has nothing to do with the endianess of > the system, and it appears wrong to me, to imply such a relation here. > > Detlev, what do you think? I _think_ that if we are concerned with the question of where a one-byte entity is placed in relation to its embracing 32-bit unit, then this is the definiton of endianness, right? Actually the endianness of the UART, and this is exactly what the interpretation of the sign of the REG_SIZE macro is. So if the patch works for Dave, he gets my Acked-by: Detlev Zundel on the latter version. Cheers Detlev -- The Buddha, the Godhead, resides quite as comfortably in the circuits of a digital computer or the gears of a cycle transmission as he does at the top of a mountain or in the petals of a flower. To think otherwise is to demean the Buddha - which is to demean oneself. -- Robert M. Pirsig -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de