Re: [PATCH 2/3] m68k: compile fix - ADBREQ_RAW missing declaration

On Thu, Dec 15, 2005 at 06:58:29PM +0000, Al Viro wrote:
> On Thu, Dec 15, 2005 at 06:14:05PM +0000, Al Viro wrote:
> > >From my reading of the code it's a way for mac/misc.c to send a packet that
> > starts with CUDA_PACKET or PMU_PACKET instead of ADB_PACKET, but otherwise
> > is the same as normal adb_request() ones...
> >
> > Used for access to timer, nvram, etc. - looks like that puppy used to
> > use the same protocol for more than just ADB and the first byte of packet
> > really selects the destination...
>
> After reading some more... Is there any reason why mac/misc.c can't
> simply use cuda_request() and pmu_request() instead? At least for
> read/write for time and nvram we end up with identical sequence of
> operations anyway - if you expand the calls in
> adb_request((struct adb_request *) &req, NULL,
> ADBREQ_RAW|ADBREQ_SYNC,
> 2, CUDA_PACKET, CUDA_GET_TIME);
> [m68k]
> and
> if (cuda_request(&req, NULL, 2, CUDA_PACKET, CUDA_GET_TIME) < 0)
> /* bail out */
> while (!req.complete)
> cuda_poll();
> [ppc]
> until you get to call of cuda_write(), you'll see the same code. Come
> to think of that... Shouldn't the ifdefs for CONFIG_ADB_PMU in there be for
> CONFIG_ADB_PMU68? The former depends on PMAC_PPC, so it's not particulary
> useful thing to check on m68k...

Honestly, the RTC/PRAM code is completely broken in 2.6 for mac68k anyway.
We definitely shouldn't be using CONFIG_ADB_PMU in the m68k code. I suspect
that wasn't found due to the fact that the pmu68k driver has never been
reliable enough to use, so everyone blames that for stuff breaking. Changing
that use of CONFIG_ADB_PMU to CONFIG_ADB_PMU68K seems like the correct
thing to do in this case.

I would like to stop using adb_request in mac/misc.c as well, but it's not
as simple as just changing it to use cuda_request and pmu_request. That
should do it for the cuda and pmu based models, but the egret (Mac IIsi
and friends) based models get left out by that fix. If noone else looks
at it before me, I'll check this out after I fix some other stuff related
to m68k mac support.

Brad Boyer
flar@xxxxxxxxxxxxx

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

Relevant Pages

  • Re: Cleaning rust from transformer laminations?
    ... The NVRAM is a 28-pin DIP package, ... pretty sure I now have two functional chips! ... I tried a spare power ... replacement NVRAM chip to fix the clock. ...
    (rec.crafts.metalworking)
  • Re: [9fans] Vmware playground but in qemu.
    ... should write the lines suggested into the nvram partition. ... On the first boot, nvram will not be found, so, i created an nvram ... And fix the erroneous bits. ...
    (comp.os.plan9)
  • Re: [PATCH 2/3] m68k: compile fix - ADBREQ_RAW missing declaration
    ... > Used for access to timer, nvram, etc. - looks like that puppy used to ... useful thing to check on m68k... ...
    (Linux-Kernel)
  • problem after an nvram replacement.
    ... i replaced the nvram in an old sparcstation 2. ... and this reduces the hex number to 7. ... is there any easy way to fix this with eeprom? ... the boot process keeps telling to replace the ...
    (comp.sys.sun.hardware)
  • Re: Sparc10 will not boot without KB/M
    ... > in NVRAM to fix this? ... Can some one tell me how to fix this?? ... Make sure ttya-ignore-cd is true, output-device is screen, ... no keyboard is attached and revert to TTY A.) ...
    (comp.unix.solaris)