On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: > I added a simple clock source for the p1010rdb so the flexcan driver > could determine a clock frequency. The p1010 flexcan device only has > an oscillator of system bus frequency divided by 2. > > Signed-off-by: Robin Holt <h...@sgi.com> > Acked-by: Marc Kleine-Budde <m...@pengutronix.de>, > Acked-by: Wolfgang Grandegger <w...@grandegger.com>, > Cc: U Bhaskar-B22300 <b22...@freescale.com> > Cc: socketcan-c...@lists.berlios.de, > Cc: net...@vger.kernel.org, > Cc: PPC list <linuxppc-dev@lists.ozlabs.org> > Cc: Kumar Gala <ga...@kernel.crashing.org> > --- > arch/powerpc/platforms/85xx/Kconfig | 2 + > arch/powerpc/platforms/85xx/Makefile | 2 + > arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ > arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ > 4 files changed, 64 insertions(+), 0 deletions(-) > create mode 100644 arch/powerpc/platforms/85xx/clock.c
I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. This either seems to global or I'm missing something. I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. - k _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev