On Mon, 26 Aug 2013 09:20:18 -0400
Jason Cooper <jason@xxxxxxxxxxxxxx> wrote:
> Hi Antony,
>
> On Mon, Aug 26, 2013 at 08:57:10AM +0400, Antony Pavlov wrote:
> > TODO: the clocksource driver need improvement!
> >
> > Signed-off-by: Antony Pavlov <antonynpavlov@xxxxxxxxx>
> > ---
> > arch/arm/Kconfig | 9 ++++
> > arch/arm/Makefile | 1 +
> > arch/arm/dts/digic4.dtsi | 24 +++++++++++
> > arch/arm/mach-digic/Kconfig | 11 +++++
> > arch/arm/mach-digic/Makefile | 2 +
> > arch/arm/mach-digic/core.c | 24 +++++++++++
> > arch/arm/mach-digic/csrc-timer.c | 67 +++++++++++++++++++++++++++++
> > arch/arm/mach-digic/include/mach/debug_ll.h | 40 +++++++++++++++++
> > arch/arm/mach-digic/include/mach/digic4.h | 29 +++++++++++++
> > arch/arm/mach-digic/include/mach/gpio.h | 1 +
> > 10 files changed, 208 insertions(+)
> > create mode 100644 arch/arm/dts/digic4.dtsi
> > create mode 100644 arch/arm/mach-digic/Kconfig
> > create mode 100644 arch/arm/mach-digic/Makefile
> > create mode 100644 arch/arm/mach-digic/core.c
> > create mode 100644 arch/arm/mach-digic/csrc-timer.c
> > create mode 100644 arch/arm/mach-digic/include/mach/debug_ll.h
> > create mode 100644 arch/arm/mach-digic/include/mach/digic4.h
> > create mode 100644 arch/arm/mach-digic/include/mach/gpio.h
> >
> ...
> > diff --git a/arch/arm/mach-digic/include/mach/debug_ll.h b/arch/arm/mach-digic/include/mach/debug_ll.h
> > new file mode 100644
> > index 0000000..5f4579e
> > --- /dev/null
> > +++ b/arch/arm/mach-digic/include/mach/debug_ll.h
> > @@ -0,0 +1,40 @@
> > +/*
> > + * Copyright (C) 2013 Antony Pavlov <antonynpavlov@xxxxxxxxx>
> > + *
> > + * This file is part of barebox.
> > + * See file CREDITS for list of people who contributed to this project.
> > + *
> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License version 2
> > + * as published by the Free Software Foundation.
> > + *
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > + * GNU General Public License for more details.
> > + *
> > + */
> > +
> > +#ifndef __MACH_DEBUG_LL_H__
> > +#define __MACH_DEBUG_LL_H__
> > +
> > +#include <io.h>
> > +#include <mach/digic4.h>
> > +
> > +#define DEBUG_LL_UART DIGIC4_UART
> > +
> > +/* Serial interface registers */
> > +#define DEBUG_LL_UART_TX (DEBUG_LL_UART + 0x0)
> > +#define DEBUG_LL_UART_ST (DEBUG_LL_UART + 0x14)
> > + #define UART_ST_TX_RDY 2
>
> leading whitespace, and perhaps use BIT() here.
The leading whitespace is put here intentionally to distinguish register address macros
and bit fields macros.
> > +
> > +static inline void PUTC_LL(char ch)
> > +{
> > + while (!(readl(DEBUG_LL_UART_ST) & UART_ST_TX_RDY))
> > + ; /* noop */
> > +
> > + writel(0x06, DEBUG_LL_UART_ST);
>
> could you use a macro here for 0x06?
It is not so easy. See comments in serial driver: there is no public documentation
on Canon Digic chips. This driver is result of reverse engineering.
I have no complete information about the bitfield meaning. In this situation
it's very difficult to invent adequate names for the bitfield macros.
E.g. I have given the DEBUG_LL_UART_ST name to the register (I mean STATUS register).
But now I see that It is not just a status register but a control register too.
Just now I prefere to keep the names and constants as-is.
Moreover the situation with digic timers is worse as I don't know meaning of most bitfields :)
> > + writel(ch, DEBUG_LL_UART_TX);
> > +}
> > +
> > +#endif
>
> thx,
>
> Jason.
--
--
Best regards,
Antony Pavlov
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
