Re: [PATCH V7 2/4] MIPS: Add board support for Loongson1B

From: Ralf Baechle
Date: Thu Jun 21 2012 - 07:32:39 EST


On Wed, Jun 20, 2012 at 10:10:26PM +0200, Florian Fainelli wrote:

> > > This redefines a function that already is declared in <linux/clk.h> and
> > > defined in drivers/clk/clkdev.c. Why?
> >
> > Because he doesn't support clkdev? clkdev support is optional.
>
> I don't think it is a good idea not to support clkdev for new targets. Ralf
> what do you think about it?

My gut feeling is that if there's a suitable generic infrastructure we
should use it, so use clkdev for new targets. I was just wondering if
there's a good reason to doing things the way he did.

Ralf
--
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/