From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752030AbcF3LvB (ORCPT ); Thu, 30 Jun 2016 07:51:01 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:56482 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751858AbcF3Lu7 (ORCPT ); Thu, 30 Jun 2016 07:50:59 -0400 From: Arnd Bergmann To: Christopher Covington Cc: linux-arm-kernel@lists.infradead.org, Austin Christ , wsa@the-dreams.de, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, timur@codeaurora.org, rruigrok@codeaurora.org, nkaje@codeaurora.org Subject: Re: [PATCH v4 1/2] i2c: qup: add ACPI support Date: Thu, 30 Jun 2016 13:52:56 +0200 Message-ID: <4043322.Uc6lMkrL43@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-22-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <577503F2.90204@codeaurora.org> References: <1465409985-17113-1-git-send-email-austinwc@codeaurora.org> <4749471.m1cxc4jj96@wuerfel> <577503F2.90204@codeaurora.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:ZuZYe0YO/YTMI+oxxO0OpLpf6FlJ/o+B+bPoCBA1Gtx0v/r6UDb GpN+n1RCTTNsBVt5jfrtlJ4P5U3yC+a7dt4qxooa72EzlgBKaZ2PzpvN5hwcoLZqrm+8FN6 ZLx8SNJx268iBQLtnEWlYMl2HN66TYUxvnwEHpJ94o6Bz+6N8hHN5oJXDmsdvm6FBHZjdJ9 apBSBwF0bJH4M2+kMYp/w== X-UI-Out-Filterresults: notjunk:1;V01:K0:9pCrjIsdwLw=:PlFH6EK+LDfOOePfqZ9oam CJxSXlrBAlclOY/DDrqi2jl1GW69P64Oh7nGZk6hqit0l6An8FBeY5AYmT5GaG/Z9fZlLc4Q2 RZaZIPfTR/MOb8j1Dfy0+F724BtmchnrIoMAzbCG8oVmQf2RPIA6+Cye9LXvQmeLwgix5qFr0 DdCBx+vC+70+gdgAsCDNk4DUvZ0OokdBceh+UndrD6mKceQzWuRrjbs/T45oCHyOPn1dhLpIy Pbx1VoWwlUsOOhTul0bqbzUX+17lcuccaYjeK+AsRofJTyIehuDD0ja6/boxgNIIjfnXmTR6R kUCAs+cSMNOBktnaMJY04fjGLBGdjFYe7stt/LXvc8J3SDlgq8eK+r9CZobncgUFhBkyJ2rtu iNfP3aDXe2I+bA8TYiyJ7M4QLuLteLzCGeK5Bya016W+ze3n8P0ET8cTWMj1fji8/yedHFP9H 3rfTLTk3nkw3sFRnAw7LLxyz9fFetCs937QM5Q2Rs0Rh2Tt8SiBqOe1Ja0wDQbRJTyZ6vYvwc y/tpMrfg+OxcfktaoHaib8dt8tmkZ86ubPMau1v2tzcIMmq04V03sBji9pXDFGUoL1OCFlO1H uwwpv5w9Uo79qVuA15UnNiIPbZIndaPg55p4KECYsAI9mGnfmuWrqUs3WIMkxScojp7uitOiC DrahN3JoiQk9r2L1kYc313Bmc8py2Ux9UB2HkyurHxNk7Y3GDZj0C4QdETGF6LQb6bAIppGDZ 9z0hkTmNS3MCrCKk Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday, June 30, 2016 7:35:14 AM CEST Christopher Covington wrote: > Hi Arnd, > > On 06/08/2016 05:02 PM, Arnd Bergmann wrote: > > On Wednesday, June 8, 2016 12:19:44 PM CEST Austin Christ wrote: > >> + ret = device_property_read_u32(qup->dev, > >> + "src-clock-hz", &src_clk_freq); > >> + if (ret) { > >> + dev_warn(qup->dev, "using default src-clock-hz %d", > >> + DEFAULT_SRC_CLK); > >> + src_clk_freq = DEFAULT_SRC_CLK; > >> + } > >> > > > > Where is this property documented? > > We plan on submitting documentation via dsd@acpica.org to > https://github.com/ahs3/dsd once it is operational. As I understand it > the project is brand new. It may take several months to begin accepting > submissions. In the mean time, we could potentially include > documentation in a reply to this thread, the cover of the next series, a > wiki page on codeaurora.org, a file in Documentation (perhaps to be > replaced by ACPICA style imports of the OS-neutral DSD project or a git > submodule) or potentially other means. Please let us know what you think > is sufficient. As you are reusing part of the DT binding, it seems appropriate to put the documentation for this into the binding documentation in the kernel. Not sure what the devicetree maintainers think about that though. Arnd