All of lore.kernel.org
 help / color / mirror / Atom feed
* S5PC100 squashing
@ 2010-01-26  8:32 ` Ben Dooks
  0 siblings, 0 replies; 4+ messages in thread
From: Ben Dooks @ 2010-01-26  8:32 UTC (permalink / raw)
  To: linux-arm-kernel, Linux Samsung SoC

Does anyone have any opinion on whether plat-s5pc1xx should stay? At the
moment it is only servicing mach-s5pc100 and thenew support for v210 and
s5pc110 seem to be using plat-s5p and not plat-s5pc1xx.

>From the looks of it, s5pc100-init.c could probably be pushed into
the arch/arm/mach-s5pc100/cpu.c file. The s5pc100-clock.c should be
moved into arch/arm/mach-s5pc100/ as well.

Not sure yet what to do with the rest.

-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.

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

* S5PC100 squashing
@ 2010-01-26  8:32 ` Ben Dooks
  0 siblings, 0 replies; 4+ messages in thread
From: Ben Dooks @ 2010-01-26  8:32 UTC (permalink / raw)
  To: linux-arm-kernel

Does anyone have any opinion on whether plat-s5pc1xx should stay? At the
moment it is only servicing mach-s5pc100 and thenew support for v210 and
s5pc110 seem to be using plat-s5p and not plat-s5pc1xx.

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

* RE: S5PC100 squashing
  2010-01-26  8:32 ` Ben Dooks
@ 2010-01-26  9:25   ` Marek Szyprowski
  -1 siblings, 0 replies; 4+ messages in thread
From: Marek Szyprowski @ 2010-01-26  9:25 UTC (permalink / raw)
  To: 'Ben Dooks', linux-arm-kernel, 'Linux Samsung SoC'
  Cc: 'Kyungmin Park', Marek Szyprowski

Hello,

On Tuesday, January 26, 2010 9:32 AM Ben Dooks wrote:

> Does anyone have any opinion on whether plat-s5pc1xx should stay? At the
> moment it is only servicing mach-s5pc100 and thenew support for v210 and
> s5pc110 seem to be using plat-s5p and not plat-s5pc1xx.

I vote for moving s5pc100 support to plat-s5p directory and removing
plat-s5pc1xx dir completely. I want to prepare patches for this, but
waited until S5PV210 support gets merged.

> From the looks of it, s5pc100-init.c could probably be pushed into
> the arch/arm/mach-s5pc100/cpu.c file. The s5pc100-clock.c should be
> moved into arch/arm/mach-s5pc100/ as well.

Right.

> Not sure yet what to do with the rest.

Most of c100-specific code should be moved to mach-s5pc100 directory.
The rest can be probably shared with all S5P platforms. 

Best regards
--
Marek Szyprowski
Samsung Poland R&D Center

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

* S5PC100 squashing
@ 2010-01-26  9:25   ` Marek Szyprowski
  0 siblings, 0 replies; 4+ messages in thread
From: Marek Szyprowski @ 2010-01-26  9:25 UTC (permalink / raw)
  To: linux-arm-kernel

Hello,

On Tuesday, January 26, 2010 9:32 AM Ben Dooks wrote:

> Does anyone have any opinion on whether plat-s5pc1xx should stay? At the
> moment it is only servicing mach-s5pc100 and thenew support for v210 and
> s5pc110 seem to be using plat-s5p and not plat-s5pc1xx.

I vote for moving s5pc100 support to plat-s5p directory and removing
plat-s5pc1xx dir completely. I want to prepare patches for this, but
waited until S5PV210 support gets merged.

> From the looks of it, s5pc100-init.c could probably be pushed into
> the arch/arm/mach-s5pc100/cpu.c file. The s5pc100-clock.c should be
> moved into arch/arm/mach-s5pc100/ as well.

Right.

> Not sure yet what to do with the rest.

Most of c100-specific code should be moved to mach-s5pc100 directory.
The rest can be probably shared with all S5P platforms. 

Best regards
--
Marek Szyprowski
Samsung Poland R&D Center

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

end of thread, other threads:[~2010-01-26  9:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-01-26  8:32 S5PC100 squashing Ben Dooks
2010-01-26  8:32 ` Ben Dooks
2010-01-26  9:25 ` Marek Szyprowski
2010-01-26  9:25   ` Marek Szyprowski

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.