Re: [PATCH v2 2/2] clk: at91: add Flexcom clock

[Boris Brezillon]

Hi Cyrille,

On Tue, 2 Jun 2015 18:57:19 +0200
Cyrille Pitchen <cyrille.pitchen@xxxxxxxxx> wrote:

> This driver supports the new Atmel Flexcom. The Flexcom is a wrapper which
> integrates one SPI controller, one I2C controller and one USART. Only one
> function can be enabled at a time. This driver selects the function once for
> all, when the Flexcom is probed, according to the value of the new
> "atmel,flexcom-mode" device tree property.
> 
> This driver has chosen to present the Flexcom to the system as a clock so the
> implementation is seamless for the existing Atmel SPI, I2C and USART drivers.

Could detail a bit more why you chose to represent this flexcom IP as
a clock device ?

I don't like to use this 'DT should represent real hardware' argument,
but to me, it looks like you're trying to use a false hardware
representation to avoid changing the peripheral drivers code, which is
wrong since the DT is supposed to represent the hardware blocks.

How about representing the flexcom as an MFD instead ?

Best Regards,

Boris


-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
--
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/