From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751449AbdJDKCy (ORCPT ); Wed, 4 Oct 2017 06:02:54 -0400 Received: from mail-wm0-f48.google.com ([74.125.82.48]:49574 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751290AbdJDKCw (ORCPT ); Wed, 4 Oct 2017 06:02:52 -0400 X-Google-Smtp-Source: AOwi7QAkRao2qy5omB3QpUn5dGf4qEiJmwsALAU9w16oFsR82+5xmOaYBvRviXI7SRwnLnEF8J+VwA== Subject: Re: [PATCH v3 1/2] dt-bindings: add device tree binding for Allwinner XR819 SDIO Wi-Fi To: Icenowy Zheng , Kalle Valo References: <20171003165944.13056-1-icenowy@aosc.io> <20171003165944.13056-2-icenowy@aosc.io> <871smjxp46.fsf@kamboji.qca.qualcomm.com> Cc: Rob Herring , Maxime Ripard , Chen-Yu Tsai , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@googlegroups.com From: Arend van Spriel Message-ID: <59D4B1C8.8020105@broadcom.com> Date: Wed, 4 Oct 2017 12:02:48 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/4/2017 11:03 AM, Icenowy Zheng wrote: > > > 于 2017年10月4日 GMT+08:00 下午5:02:17, Kalle Valo 写到: >> Icenowy Zheng writes: >> >>> Allwinner XR819 is a SDIO Wi-Fi chip, which has the functionality to >> use >>> an out-of-band interrupt pin instead of SDIO in-band interrupt. >>> >>> Add the device tree binding of this chip, in order to make it >> possible >>> to add this interrupt pin to device trees. >>> >>> Signed-off-by: Icenowy Zheng >>> Acked-by: Rob Herring >>> --- >>> Changes in v3: >>> - Renames the node name. >>> - Adds ACK from Rob. >>> Changes in v2: >>> - Removed status property in example. >>> - Added required property reg. >>> >>> .../bindings/net/wireless/allwinner,xr819.txt | 38 >> ++++++++++++++++++++++ >>> 1 file changed, 38 insertions(+) >>> create mode 100644 >> Documentation/devicetree/bindings/net/wireless/allwinner,xr819.txt >> >> Like I asked already last time, AFAICS there is no upstream xr819 >> wireless driver in drivers/net/wireless directory. Do we still accept >> bindings like this for out-of-tree drivers? > > See esp8089. > > There's also no in-tree driver for it. The question is whether we should. The above might be a precedent, but it may not necessarily be the way to go. The commit message for esp8089 seems to hint that there is intent to have an in-tree driver: """ Note that at this point there only is an out of tree driver for this hardware, there is no clear timeline / path for merging this. Still I believe it would be good to specify the binding for this in tree now, so that any future migration to an in tree driver will not cause compatiblity issues. Cc: Icenowy Zheng Signed-off-by: Hans de Goede Signed-off-by: Rob Herring """ Regardless the bindings are in principle independent of the kernel and just describing hardware. I think there have been discussions to move the bindings to their own repository, but apparently it was decided otherwise. Regards, Arend