From mboxrd@z Thu Jan 1 00:00:00 1970 From: kvalo@codeaurora.org (Kalle Valo) Date: Sat, 02 Jul 2016 09:59:55 +0300 Subject: [linux-sunxi] Re: [PATCH 1/4] brcmfmac: Add brcm, nvram_file_name dt property In-Reply-To: (Jonas Gorski's message of "Fri, 1 Jul 2016 10:58:43 +0200") References: <1467209074-15634-1-git-send-email-hdegoede@redhat.com> <8d8bd933-717b-48e4-f002-7b834abed6fd@redhat.com> <3960223.GqB9zXL8s8@wuerfel> <1f44df41-0111-441b-4671-718eec0c4346@broadcom.com> Message-ID: <87k2h44j9w.fsf@kamboji.qca.qualcomm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Jonas Gorski writes: > Hi, > > On 30 June 2016 at 21:23, Arend Van Spriel wrote: >> >> >> On 30-6-2016 13:31, Arnd Bergmann wrote: >>> On Thursday, June 30, 2016 12:25:15 PM CEST Hans de Goede wrote: >>>>> So then how about making use of a more specific compatible string? >>>>> >>>>> e.g. >>>>> >>>>> brcmf { >>>>> compatible = "foo,ap6210", "brcm,bcm4329-fmac"; >>>>> ... >>>>> }; >>>>> >>>>> and if the compatible has more than one element you request >>>>> FW_NAME_.txt as the nvram file. Or try each comptible (and >>>>> lastly no suffix) until you get a match. (AFAICT, this is what the >>>>> "model" property was originally intended for anyway, but almost nobody >>>>> did it right, and everyone put a user readable string into "model" for >>>>> boards instead of the ePAPR defined compatible string). >>>> >>>> Hmm, interesting idea. Not sure how easy / hard it will be to implement >>>> this, but from a dt binding point of view it seems elegant. >>>> >>>> Kalle, Arend, what do you think of this ? >> >> At first glance I like the suggestion, but this would mean updating the >> bindings document for each new wifi module that we want to add. Not a >> big problem, but it makes that I have a slight preference to using a >> property for it, eg. brcm,module = "ap6210"; > > If you want a separate property, then I repeat my very first > suggestion, the well defined model property. > e.g. > > brcmf at 0 { > model = "ampak,ap6210"; > compatible = "brcm,bcm4329-fmac"; > ... > }; > > All device nodes may have a model property, not just the top "machine" one. I like this model property but I'm no DT expert. What others think about it, would it work? -- Kalle Valo