Messages in this thread | | | Date | Mon, 26 Jun 2006 10:43:19 -0700 | From | Brad Boyer <> | Subject | Re: [PATCH 08/21] gcc 4 fix |
| |
On Sat, Jun 24, 2006 at 02:45:06PM +1000, Finn Thain wrote: > On Fri, 23 Jun 2006, Roman Zippel wrote: > > On Fri, 23 Jun 2006, Al Viro wrote: > > > Broken. Proper fix is to rename the function so that it wouldn't > > > clash. > > > > Well, I wouldn't call it broken, as both versions can never be compiled > > into the same kernel, but I don't care much how it's fixed. > > > > Does anyone know the relationship between via-pmu.c and via-pmu68k.c? If > > it's intended to keep the differences small, a rename would be the wrong > > fix. > > The relationship is (and was) just that they share the pmu.h header file > declarations. In the patch in question I used the powerpc definition as > well.
I believe that pmu68k started out as a copy of pmu. The versions of PMU on the 68k PowerBooks are just older than the ones on ppc. They are fundamentally the same, it's just that the ppc systems have various fixes and new features. Ideally the files would be merged back together and split in a more logical fashion to share the code that is the same. Each one already supports multiple versions of the chip.
> The powerpc version exports pmu_queue_request (apparently for the use of > low_i2c.c). The m68k version doesn't, but if it needed to export it, I > don't see why it shouldn't implement the same "API"?
Several other parts of the kernel submit pmu requests, but most of them use pmu_request(). It's probably better to unify this API set. The cuda and IIsi drivers have equivalent calls, for that matter. I wanted to make a generic call that would switch to the active one of those, but the idea didn't go over too well. It was probably due to the bad taste everyone had from when adb_request() was used for that purpose.
Brad Boyer flar@allandria.com
- 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/
| |