From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Subject: Re: [PATCH 4/4] wl1251: spi: add device tree support Date: Mon, 28 Oct 2013 14:30:45 -0500 Message-ID: <3381D0FB-2AAD-446D-8650-E4BE5F22A2E3@codeaurora.org> References: <1382890469-25286-1-git-send-email-sre@debian.org> <1382890469-25286-5-git-send-email-sre@debian.org> Mime-Version: 1.0 (Apple Message framework v1283) Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Sebastian Reichel , Mark Rutland , linux-doc@vger.kernel.org, Tony Lindgren , Russell King , Sachin Kamat , Ian Campbell , Sebastian Reichel , Luciano Coelho , devicetree@vger.kernel.org, Pawel Moll , Stephen Warren , "John W. Linville" , Rob Herring , Bill Pemberton , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Greg Kroah-Hartman , "linux-wireless@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Felipe Balb To: Grazvydas Ignotas Return-path: In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Oct 28, 2013, at 12:15 PM, Grazvydas Ignotas wrote: > On Mon, Oct 28, 2013 at 8:37 AM, Kumar Gala wr= ote: >> On Oct 27, 2013, at 11:14 AM, Sebastian Reichel wrote: >>> +++ b/Documentation/devicetree/bindings/net/wireless/ti,wl1251.txt >>> @@ -0,0 +1,36 @@ >>> +* Texas Instruments wl1251 controller >>> + >>> +The wl1251 chip can be connected via SPI or via SDIO. The linux >>> +kernel currently only supports device tree for the SPI variant. >>> + >>=20 >> From the binding I have no idea what this chip actually does, also w= e don't normally reference linux kernel support in bindings specs (so p= lease remove it). >>=20 >> However, what would expect the SDIO binding to look like? Or more s= pecifically, how would you distinguish the SPI vs SDIO binding/connecti= on? I'm wondering if the compatible should be something like "ti,wl125= 1-spi" and than the sdio can be "ti,wl1251-sdio" >=20 > When connected to SDIO, it doesn't act as standard SDIO device and > can't be probed (standard SDIO registers missing), so information has > to come some other way. That used to partially come through > platform_data and partially through a callback from mmc subsystem (se= e > pandora_wl1251_init_card() in > arch/arm/mach-omap2/board-omap3pandora.c). I don't know much about DT= , > but maybe the information that comes from SDIO registers on "normal" > SDIO devices should come through DT in this case too? I don't really > know how that should be integrated with mmc subsystem though.. Ok, my point is still valid that we can use a different compatible for = the SDIO case even if its no standard SDIO vs the SPI case. >=20 >>> +Required properties: >>> +- compatible : Should be "ti,wl1251" >>=20 >> reg is not listed as a required prop. >>=20 >>> +- interrupts : Should contain interrupt line >>> +- interrupt-parent : Should be the phandle for the interrupt >>> + controller that services interrupts for this device >>> +- vio-supply : phandle to regulator providing VIO >>> +- power-gpio : GPIO connected to chip's PMEN pin >>=20 >> should be vendor prefixed: ti,power-gpio >>=20 >>> +- For additional required properties on SPI, please consult >>> + Documentation/devicetree/bindings/spi/spi-bus.txt >>> + >>> +Optional properties: >>> +- ti,use-eeprom : If found, configuration will be loaded from eepr= om. >>=20 >> can you be a bit more specific on what cfg will be loaded. Also, is= this property a boolean, if so how do I know which eeprom the cfg is l= oaded from (is it one that is directly connected to the wl1251? >=20 > wl1251 is a wifi chip that can have an optional eeprom connected to i= t > to store things like calibration stuff and MAC address, and that > eeprom is usually inside a single module with some additional radio > related chips. If the eeprom is connected (like the module on pandora > board has), the driver can issue command to the firmware running on > chip to load that data on it's startup, alternatively the driver can > load calibration from other storage (like it happens on N900). So sounds like a boolean. I think just adding that its a boolean and m= aybe something like "configuration (calibration data, MAC addr, etc..)" - k >=20 >=20 > Gra=9Evydas > -- > To unsubscribe from this list: send the line "unsubscribe devicetree"= in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html --=20 Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, host= ed by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html