Hi Andreas,

Thank you for your review.


> -----Original Message-----
> From: Andreas Bießmann [mailto:andr...@biessmann.org]
> Sent: 2016年5月3日 19:18
> To: Yang, Wenyou <wenyou.y...@atmel.com>
> Cc: U-Boot Mailing List <u-boot@lists.denx.de>
> Subject: Re: [U-Boot] [PATCH 1/2] pinctrl: at91-pio4: add pinctrl driver
> 
> Dear Wenyou,
> 
> On 2016-04-07 04:15, Wenyou Yang wrote:
> > AT91 PIO4 controller is a combined gpio-controller, pin-mux and
> > pin-config module. The peripherals are assigned pins through per-pin
> > based muxing logic. And the pin configuration are performed on
> > specific registers which are shared along with the gpio controller.
> >
> > Signed-off-by: Wenyou Yang <wenyou.y...@atmel.com>
> > ---
> >
> >  arch/arm/mach-at91/include/mach/atmel_pio4.h |  35 ++++++
> >  drivers/pinctrl/Kconfig                      |   7 ++
> >  drivers/pinctrl/Makefile                     |   1 +
> >  drivers/pinctrl/pinctrl-at91-pio4.c          | 164
> > +++++++++++++++++++++++++++
> >  4 files changed, 207 insertions(+)
> >  create mode 100644 drivers/pinctrl/pinctrl-at91-pio4.c
> >
> > diff --git a/arch/arm/mach-at91/include/mach/atmel_pio4.h
> > b/arch/arm/mach-at91/include/mach/atmel_pio4.h
> > index 8bb4b12..6760bec 100644
> > --- a/arch/arm/mach-at91/include/mach/atmel_pio4.h
> > +++ b/arch/arm/mach-at91/include/mach/atmel_pio4.h
> > @@ -29,6 +29,41 @@ struct atmel_pio4_port {
> >
> >  #endif
> >
> > +/*
> > + * PIO Configuration Register Fields
> > + */
> > +#define ATMEL_PIO_CFGR_FUNC_MASK   GENMASK(2, 0)
> > +#define ATMEL_PIO_CFGR_FUNC_GPIO   (0x0 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_A       (0x1 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_B       (0x2 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_C       (0x3 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_D       (0x4 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_E       (0x5 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_F       (0x6 << 0)
> > +#define ATMEL_PIO_CFGR_FUNC_PERIPH_G       (0x7 << 0)
> > +#define ATMEL_PIO_DIR_MASK         BIT(8)
> > +#define ATMEL_PIO_PUEN_MASK                BIT(9)
> > +#define ATMEL_PIO_PDEN_MASK                BIT(10)
> > +#define ATMEL_PIO_IFEN_MASK                BIT(12)
> > +#define ATMEL_PIO_IFSCEN_MASK              BIT(13)
> > +#define ATMEL_PIO_OPD_MASK         BIT(14)
> > +#define ATMEL_PIO_SCHMITT_MASK             BIT(15)
> > +#define ATMEL_PIO_CFGR_EVTSEL_MASK GENMASK(26, 24)
> > +#define ATMEL_PIO_CFGR_EVTSEL_FALLING      (0 << 24)
> > +#define ATMEL_PIO_CFGR_EVTSEL_RISING       (1 << 24)
> > +#define ATMEL_PIO_CFGR_EVTSEL_BOTH (2 << 24)
> > +#define ATMEL_PIO_CFGR_EVTSEL_LOW  (3 << 24)
> > +#define ATMEL_PIO_CFGR_EVTSEL_HIGH (4 << 24)
> 
> just realized that most of these definitions are already in the existing GPIO 
> driver.
> Could you please put another patch before moving these definitions from the
> driver code to the header. It is ok for me to rephrase them when doing so.

Add a new patch to handle it.

Thank you.


Best Regards,
Wenyou Yang
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to