Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754594AbbEMLpq (ORCPT ); Wed, 13 May 2015 07:45:46 -0400 Received: from mail-wg0-f43.google.com ([74.125.82.43]:34597 "EHLO mail-wg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753446AbbEMLpi (ORCPT ); Wed, 13 May 2015 07:45:38 -0400 MIME-Version: 1.0 In-Reply-To: <2789447.8FUTAOJOHR@wuerfel> References: <1431158038-3813-1-git-send-email-mcoquelin.stm32@gmail.com> <1431158038-3813-15-git-send-email-mcoquelin.stm32@gmail.com> <2789447.8FUTAOJOHR@wuerfel> Date: Wed, 13 May 2015 13:45:36 +0200 Message-ID: Subject: Re: [PATCH v8 14/16] ARM: dts: Introduce STM32F429 MCU From: Maxime Coquelin To: Arnd Bergmann , Daniel Thompson Cc: =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , =?UTF-8?Q?Andreas_F=C3=A4rber?= , Geert Uytterhoeven , Rob Herring , Philipp Zabel , Linus Walleij , Stefan Agner , Peter Meerwald , Paul Bolle , Peter Hurley , Andy Shevchenko , Chanwoo Choi , Russell King , Daniel Lezcano , Joe Perches , Vladimir Zapolskiy , Lee Jones , Jonathan Corbet , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Thomas Gleixner , Greg Kroah-Hartman , Jiri Slaby , Andrew Morton , "David S. Miller" , Mauro Carvalho Chehab , Antti Palosaari , Tejun Heo , Will Deacon , Nikolay Borisov , Rusty Russell , Kees Cook , Michal Marek , "linux-doc@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-gpio@vger.kernel.org" , "linux-serial@vger.kernel.org" , Linux-Arch , "linux-api@vger.kernel.org" , Nicolae Rosia , Kamil Lulko Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1848 Lines: 50 2015-05-12 23:21 GMT+02:00 Arnd Bergmann : > On Saturday 09 May 2015 09:53:56 Maxime Coquelin wrote: >> +#include >> + >> > > Can you find a way to avoid this dependency? > > Maybe you can change the bindings so that the numbers you pass as > arguments to the reset and clock specifiers reflect the numbers that > the hardware use? If I understand correctly, you prefer the way I did in v7 [0]? I don't have a strong opinion on this. Either way is fine to me. I changed the bindings in the v8 after discussions with Daniel Thompson, who is implementing the clock driver part of the RCC IP. He proposed we used common defines, because each peripheral has a reset line and a clock gate. Both reset and clock are represented as a single bit, with only the base offset differing between clock and reset. You can have a look at chapter 6 of the reference manual [1] if you find some time. Having common defines between clocks and reset make sense to me, but I also understand your point of avoiding dependencies. Maybe I can revert back to v7 bindings for now, and then we can reconsider using common defines when Daniel will send the clock patches. Note that doing that won't break the DT binary compatibility, as the raw reset values, or the ones from defines are the same. Daniel, could you share an example of the bindings you would use for the clocks? Kind regards, Maxime > > Arnd [0]: http://lkml.iu.edu/hypermail/linux/kernel/1504.3/04523.html [1]: http://www.st.com/web/en/resource/technical/document/reference_manual/DM00031020.pdf -- 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/