[PATCH] hxd8-resume.patch

matt_hsu matt_hsu at openmoko.org
Tue May 22 17:55:36 CEST 2007


於 二,2007-05-22 於 13:50 +0200,Harald Welte 提到:
> Hi Matt!
> 
> The patch seems fine to me (style-wise), however:
> 
> On Sun, May 20, 2007 at 04:13:20PM +0800, matt_hsu wrote:
> > Log:
> > 	HXD8: GPG[15:13] must be selected as input for resuming from u-boot.
> 
> the most important bit is missing: WHY?
> 
I checked the source, it's result from s3c2410_ts_connect() in
s3c2410_ts.c. Even the GPIO configuration (GPGCON) is correct in uboot.
TS module would modify these important bits. 

But doesn't these bits affect resume logic for GTA01?
This caution is not mentioned in S3C2410 datasheet. 
That is, GPG[13:15] are not functional pins in NAND boot mode in 2410.

So I think this patch should be applied on GTA02 if TS module is
included. 

> This information should be present in both the Log, as well as a comment
> in the actual source code.  Remember, lots of people will only have
> access to the source code, and not the schematics.  Still, they want to
> understand what is actually going on.
> 
Absolutely

> Please resubmit.  Thanks!





More information about the openmoko-kernel mailing list