lkml.org 
[lkml]   [2005]   [Nov]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Sharp power management: split into sharpsl-dependend and generic parts
From
Date
On Mon, 2005-11-21 at 23:47 +0100, Pavel Machek wrote:
> Hi!
>
> This splits sharpsl_pm.c into sharpsl_pm.c and
> sharp_pm.c. sharpsl_pm.c contains stuff that is shared between spitz
> and corgi, sharp_pm.c contains more widely usable code. I'd like
> something like this to be eventually merged... [Of course, I'll
> cleanup #ifdef COLLIE's, I did not realize some were still pending.]
>
> Comments?

The patch makes it very difficult to see what you've changed. The other
ifdef COLLIEs look fairly straight forward to clean up and once that's
done I'm ok with the principle of it.

I don't think the filenames are quite right as collie is still a Sharp
SL series device. How about leaving the common code in sharpsl_pm.c and
adding sharpsl_pm_pxa.c for the corgi/spitz common code? That should
make the diffs easier to read. Eventually, sharpsl_pm.c should probably
end up in arm/common but for now, I'd like to see easy to read diffs.

Thinking further about the structure, I have another idea which might be
worth thinking about. If the common sharpsl_pm code was moved to
arm/common, the files mach-sa1100/sharpsl_pm.c and mach-pxa/sharpsl_pm.c
could provide the mach dependent functions Only one of the two would
ever get compiled into a kernel so the files can provide the same
function namespace and avoid any ugly ifdefs or tables of function
pointers.

Out of interest, does the code work on collie yet?

Richard

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-11-22 01:40    [W:0.045 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site