[U-Boot] [PATCH 06/22] ARM sunxi: Basic GPIO driver

Wolfgang Denk wd at denx.de
Sun Nov 25 20:50:43 CET 2012


Dear Henrik Nordström,

In message <1353843623.17518.17.camel at home.hno.se> you wrote:
> GPIO driver for Allwinner sun4i/sun5i family of SoCs
> 
> GPIO Pins are named by their symbolic pin names P<g><#>
> such as PH19 or H19.
> 
> Note: This do not perform any validation if the pin is in use
> for some other I/O function. Use with care.

SoB missing.


Um.... are these patches actually bisectable???  It appears not; for
example, you reference gpio_direction_input() in ealier patches, but
define it only now.

This patch series need a full rework.

Review stops here.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Knowledge, sir, should be free to all!
	-- Harry Mudd, "I, Mudd", stardate 4513.3


More information about the U-Boot mailing list