[PATCH 1/15] Update for ucc_geth driver
Paul Mackerras
paulus at samba.org
Mon Oct 2 17:24:58 EST 2006
Kim Phillips writes:
> if (bcsr_regs == NULL)
> - bcsr_regs = (u8 *) ioremap(BCSR_PHYS_ADDR, BCSR_SIZE);
> + bcsr_regs = get_bcsr();
> bcsr_regs[14] |= 0x40;
Umm, if the BSCR is really so unique to each board, how come this
driver is frobbing a specific bit? How does the ucc_geth driver know
about the layout of a given board's BCSR?
Also, this patch means that the kernel won't build (assuming you have
the ucc_geth driver in your config) until we get to patch 9/15, where
get_bcsr gets defined.
Lastly, shouldn't this patch be going through Jeff Garzik?
Paul.
More information about the Linuxppc-dev
mailing list