[PATCH v3] GPIO: Add support for GPIO on CLPS711X-target platform

Russell King - ARM Linux linux at arm.linux.org.uk
Sun Oct 7 11:20:04 EDT 2012


On Sun, Oct 07, 2012 at 07:12:27PM +0400, Alexander Shiyan wrote:
> On Sun, 7 Oct 2012 11:28:03 +0100
> Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
> 
> > > ...
> > > > > +++ b/arch/arm/mach-clps711x/include/mach/gpio.h
> > > > > @@ -0,0 +1,27 @@
> > > > > +/*
> > > > > + *  This file contains the CLPS711X GPIO definitions.
> > > > > + *
> > > > > + *  Copyright (C) 2012 Alexander Shiyan <shc_work at mail.ru>
> > > > > + *
> > > > > + * This program is free software; you can redistribute it and/or modify
> > > > > + * it under the terms of the GNU General Public License as published by
> > > > > + * the Free Software Foundation; either version 2 of the License, or
> > > > > + * (at your option) any later version.
> > > > > + */
> > > > > +
> > > > > +/* Simple helper for convert port & pin to GPIO number */
> > > > > +#define CLPS711X_GPIO(port, bit)	((port) * 8 + (bit))
> > > > > +
> > > > > +/* Temporaty definitions for GPIO-ports */
> > > > > +/* Will be removed after remove clps_read(write) macros */
> > > > > +#include <mach/hardware.h>
> > > > > +#define _PADR	(CLPS711X_VIRT_BASE + PADR)
> > > > > +#define _PBDR	(CLPS711X_VIRT_BASE + PBDR)
> > > > > +#define _PCDR	(CLPS711X_VIRT_BASE + PCDR)
> > > > > +#define _PDDR	(CLPS711X_VIRT_BASE + PDDR)
> > > > > +#define _PADDR	(CLPS711X_VIRT_BASE + PADDR)
> > > > > +#define _PBDDR	(CLPS711X_VIRT_BASE + PBDDR)
> > > > > +#define _PCDDR	(CLPS711X_VIRT_BASE + PCDDR)
> > > > > +#define _PDDDR	(CLPS711X_VIRT_BASE + PDDDR)
> > > > > +#define _PEDR	(CLPS711X_VIRT_BASE + PEDR)
> > > > > +#define _PEDDR	(CLPS711X_VIRT_BASE + PEDDR)
> > > > 
> > > > Why can't this file (or the bulk of it) live in drivers/gpio ?
> > > We should have access for macros in gpio.h from board support files
> > > and other drivers. From drivers/gpio it not possible.
> > > If you ask about port definitions, as I say before, it will be
> > > removed later after rework on platform hardware definitions.
> > 
> > No other drivers should be directly accessing the GPIO registers -
> > doing so is likely a bug because of the inherent lack of locking,
> > which will cause race conditions.
> > 
> > All accesses to GPIOs should be done via gpiolib, even from board
> > support files.
> 
> Yes, and I am say only about macro that calculate GPIO-number from
> port number and bit position. Access to this is a helpful for understand
> source code.

Sorry, I don't think you're understanding.

Why can't you put:

#define _PADR        (CLPS711X_VIRT_BASE + PADR)

and the other macros inside drivers/gpio/gpio-clps711x.c ?  This should
be the _only_ file which uses them.



More information about the linux-arm-kernel mailing list