linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* LPC32XX architecture files (updated v4)
@ 2010-02-19 23:25 wellsk40 at gmail.com
  2010-02-19 23:25 ` [PATCH 01/13] ARM: LPC32XX: Initial architecture header files wellsk40 at gmail.com
                   ` (12 more replies)
  0 siblings, 13 replies; 22+ messages in thread
From: wellsk40 at gmail.com @ 2010-02-19 23:25 UTC (permalink / raw)
  To: linux-arm-kernel


Thanks again for the review comments. This patchset is the 4rth
release with many review comments incorporated.

Changes from previous release to v4 (02/19/2010)

Review comment changes...
Removed the include/mach/board.h file until it is actually needed
Changed IRQ macros for SIC1 and SIC2 and handling in SIC1/SIC2
	chained handlers
Size macros in platform.h changed to numeric values
Changed uncompress macros to use __raw_readl/writel instead of
	direct register access using volatile unsigned long *
Removed unused LPC32XX_CLKPWR_SET_PLL_USBPDIV macro
printascii debug macro assembly optimized
Fixed an issue with macros using _REG being doubled casted. After
	looking at this, _REG isn't really needed at all. The
	macros that rely on it can use io_p2v() directly.
_REG macro removed in platform.h. Other macros that use it now
	use io_p2v directly
Grammar and typo fixes
DEFINE_MUTEX used instead of init_mutex()
PLL397 and main osc clock rates hardcoded (0 rate not needed for a
	disabled clock)
clock structure usecount type changes to u32
Added WARN_ON for clock disable when usecount <= 0
Clock functions pointers no longer have the address of(&) prefix
	for function names, not needed
IRAM size computation algorithm changed
arch specific name changes on global platform structs specific to
	the lpc32xx
LPC32xx gpiolib macro change for GPIO3 to hide pin gap details
Made pll_postdivs static in clock.c. Moved support function
	(clk_get_pllrate_from_reg) from common.c that used that
	function into clock.c. Prototype for support function
	left in common.h (used in timer.c).
Used ffs() instead of a case block for a support function in the
	clock driver
Changed clock enable/disable logic to fix an issue with clock
	usecount accumulators and disable logic on failure. Clocks
	are now enabled from the bottom up. This is ok on the 32xx
	and makes the logic much simpler.
generic_handle_irq() used instead of handle_level_irq() for
	chained handlers
SUBIRQs initially left masked, chained handler init enables them
	now
suspend_valid_only_mem used instead of local variant
Small tweaks to suspend.S
icache invalidation added after SRAM copy and cache flush placed
	in the correct place
Removed the ARCH specific register swap in include/amba/clcd.h
	(compiles fine, but requires RK PL111 patch to work)
Montavista copyright and original GPLv2 licensing terms to pm.c
	and suspend.S. INAL either, but if they get involved,
	it would be easier to completely rewrite them than try
	to sort out the aftermath.

Additional changes...
Added missing LPC32XX_CLKPWR_USBPDIV_PLL_MASK mask macros to USB
	PCLK div computation (clock.c)

Unchanged from review comments...
IO_ADDRESS macro name unchanged. Macro is used in an existing
	driver (WDT) and provides support for assembly code
	that can't use the typecasts for sparse
I liked the idea of using functions for adding platform drivers
	specific to different device variants, but didn't want to
	do this...yet. (It makes good sense though.)

^ permalink raw reply	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2010-02-23 21:28 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-19 23:25 LPC32XX architecture files (updated v4) wellsk40 at gmail.com
2010-02-19 23:25 ` [PATCH 01/13] ARM: LPC32XX: Initial architecture header files wellsk40 at gmail.com
2010-02-20 16:12   ` Uwe Kleine-König
     [not found]     ` <a08234131002201112x66a595c4j235bf1a31aef4f92@mail.gmail.com>
2010-02-20 19:14       ` Kevin Wells
2010-02-20 19:17       ` Kevin Wells
2010-02-22 21:41         ` Kevin Wells
2010-02-23  8:02           ` Uwe Kleine-König
2010-02-19 23:25 ` [PATCH 02/13] ARM: LPC32XX: Debug and IRQ macros wellsk40 at gmail.com
2010-02-19 23:25 ` [PATCH 03/13] ARM: LPC32XX: Clock driver wellsk40 at gmail.com
2010-02-20 16:33   ` Russell King - ARM Linux
2010-02-23 21:28     ` Kevin Wells
2010-02-19 23:26 ` [PATCH 04/13] ARM: LPC32XX: GPIO, timer, and IRQ drivers wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 05/13] ARM: LPC32XX: System suspend support wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 06/13] ARM: LPC32XX: Serial support code wellsk40 at gmail.com
2010-02-20 16:34   ` Russell King - ARM Linux
2010-02-19 23:26 ` [PATCH 07/13] ARM: LPC32XX: Misc support functions wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 08/13] ARM: LPC32XX: Phytec 3250 platform support wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 09/13] ARM: LPC32XX: Arch config menu supoport and makefiles wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 10/13] ARM: LPC32XX: Default PHY3250 kernel config (ramdisk) wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 11/13] ARM: Add support for the LPC32XX arch wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 12/13] i2c: " wellsk40 at gmail.com
2010-02-19 23:26 ` [PATCH 13/13] WATCHDOG: " wellsk40 at gmail.com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).