Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759062AbbFBREs (ORCPT ); Tue, 2 Jun 2015 13:04:48 -0400 Received: from down.free-electrons.com ([37.187.137.238]:35479 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753822AbbFBREk (ORCPT ); Tue, 2 Jun 2015 13:04:40 -0400 Date: Tue, 2 Jun 2015 19:04:35 +0200 From: Boris Brezillon To: Cyrille Pitchen Cc: , , , , , , , , , , Subject: Re: [PATCH v2 2/2] clk: at91: add Flexcom clock Message-ID: <20150602190435.4698a4db@bbrezillon> In-Reply-To: <62cb26952d0688c6f37ad84d253d5302818a20ae.1433264017.git.cyrille.pitchen@atmel.com> References: <62cb26952d0688c6f37ad84d253d5302818a20ae.1433264017.git.cyrille.pitchen@atmel.com> X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.23; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1392 Lines: 38 Hi Cyrille, On Tue, 2 Jun 2015 18:57:19 +0200 Cyrille Pitchen 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/