RE: [PATCH 2/9] omap2/3/4: ioremap omap_globals module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx]
> Sent: Friday, February 19, 2010 3:55 AM
> To: Shilimkar, Santosh
> Cc: tony@xxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 2/9] omap2/3/4: ioremap omap_globals module
> 
> Santosh Shilimkar <santosh.shilimkar@xxxxxx> writes:
> 
> > This is a clean-up patch towards dynamic allocation of IO space
> > instead of using harcoded macros to calculate virtual addresses.
> >
> > Also update the sdrc, prcm, tap and control module to
> > allocate iospace dynamically
> 
> You should drop the referenece to the TAP here since it's dropped.
> 
> > As per Tony's suggestion V2 version drops tap changes
> > becasue ioremap uses cpu_is_omap2420() and cpu_is_omap2430(),
> > so we can't use that for setting tap_base. Hence ioremap()
> > won't work for tap until omap2_check_revision() is done
> 
> And this type of comment should come after the '---' since
> it's relevant to the reviewers, but not to the final git history.
> 
> 
> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
> > CC: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
> > CC: Tony Lindgren <tony@xxxxxxxxxxx>
> 
> After that, you can change the CC for me to a Reviewed-by.

Refreshed version attached with style fix and adding the Paul, Kevin's
Reviewed-by.

Regards,
Santosh
 

Attachment: 0002-omap2-3-4-ioremap-omap_globals-module.patch
Description: 0002-omap2-3-4-ioremap-omap_globals-module.patch


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux