Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp3587830ybk; Tue, 19 May 2020 08:13:28 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxX27fnMprmafb06P3NgZKW+eN6TTlbVp8M7raU1LJgFzVgZPV5Ujs3g29r06TA1EtnqDz1 X-Received: by 2002:a05:6402:21cc:: with SMTP id bi12mr18231460edb.294.1589901208531; Tue, 19 May 2020 08:13:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589901208; cv=none; d=google.com; s=arc-20160816; b=JVo48lN/Xe+0q7xFQercJFpcJyQKVfFAl5K3tmQIZxq5MmiHUyMhLvIHhoZH6Arhdw KJYl4aG0bOjo5BIUqi3QIUwFvAPCPD8gT0fe+9g6FeupThf4hcgnyY6dh8R/I6HkeEuU Y84WZ2PMI8n6euco1XpAzuAJzn5AZE+7ypVq9aT3KOX9LrWaC4MaUaXAa5jj3ep5nA7M a+F/4FjUwAvYkn7ia1QaaftdppOOJeJpkZvMvB7VUVqIlEKu1vpQ4UgsyW/v3vQmbmu3 kegcdpsIkhHUQ6QZ1MLFE2qPaGVS++fXwbtzyRyb1F9qGKiDr+nxN+0bksMI49T866gH 7OIg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-transfer-encoding:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=raFbFZ0QTf3Z1s7+3bsi6rYdTh+H/Frejet3Giqguqw=; b=oTRAE3tq+NQEEDF53PXZsaNNur7hjgV1sXKUlhMqdExI5FVUX+efWag+vKfwhp3bNS wRb05rYD754xt8fobcVwBQTlSvPZu1X1zuNYn8T9QcFfM+vpRrRgUHdBKWdSWoWA+gKQ yawqCx8Y10tedZHMgMGBAWtNj3SeEH7uh2axAaiR09o5M4UCE4fuebIpb7iyscor8IEr NA5lTaF3YwfVCqAzGzdikQu1LWC5Epn2J2Obw+KOV/ANwEM/FSQG6b6CfXiEBAZ9kUMm xxtEWkdKrpF2kY5fVqq08QHtmkIG2qIiuI2wHz4Cm8L7YayAQqkzha6SCBtaWbJ4saOG wuEQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=jiZiBdZ9; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ny24si43631ejb.445.2020.05.19.08.12.49; Tue, 19 May 2020 08:13:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=jiZiBdZ9; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729218AbgESPMM (ORCPT + 99 others); Tue, 19 May 2020 11:12:12 -0400 Received: from mail.kernel.org ([198.145.29.99]:60220 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728775AbgESPML (ORCPT ); Tue, 19 May 2020 11:12:11 -0400 Received: from pali.im (pali.im [31.31.79.79]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id BB2FF207FB; Tue, 19 May 2020 15:12:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1589901131; bh=em25szXdtZK2RR5pD26erJ6INoBREuJbq5qd9DHazRg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=jiZiBdZ91SCPzYVRTVYwYgcAQzgd+eMvx4PGxt0aUOtzdH35lFvG+INCWrLeJ1k4k uxJBsSduXbkp77wAoHUWv9MGilOmh9MMzsDNld1lIHBiBsyrf25lLljOQpes9YDBGp t22aj/i7ZBdgDUKwmAHuau7GZDbkBLzH327zavCs= Received: by pali.im (Postfix) id 81FDB6F0; Tue, 19 May 2020 17:12:08 +0200 (CEST) Date: Tue, 19 May 2020 17:12:08 +0200 From: Pali =?utf-8?B?Um9ow6Fy?= To: David Lin , =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= , Steve deRosier , Kalle Valo , Johannes Berg , Chor Teck Law , James Lin , Pete Hsieh Cc: linux-wireless@vger.kernel.org Subject: Re: [PATCH v9] Add new mac80211 driver mwlwifi. Message-ID: <20200519151208.g35qpxdtroedjma7@pali> References: <3e57f30c29254db4a906e3e71ac36da5@SC-EXCH02.marvell.com> <8492870c897543a1b3c635a96f1066cb@SC-EXCH02.marvell.com> <4b33da1fb23a476ca5bc21eb74f1ffd2@SC-EXCH02.marvell.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20180716 Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org On Wednesday 21 November 2018 09:24:55 Rafał Miłecki wrote: > On Wed, 8 Feb 2017 at 08:55, David Lin wrote: > > > From: Rafał Miłecki [mailto:zajec5@gmail.com] worte: > > > On 8 February 2017 at 08:28, David Lin wrote: > > > >> From: Rafał Miłecki [mailto:zajec5@gmail.com] worte: > > > >> Sent: Wednesday, February 08, 2017 3:24 PM On 8 February 2017 at > > > >> 07:30, David Lin wrote: > > > >> > steve.derosier@gmail.com [mailto:steve.derosier@gmail.com] wrote: > > > >> >> On Tue, Feb 7, 2017 at 10:15 PM, David Lin wrote: > > > >> >> >> Rafał Miłecki [mailto:zajec5@gmail.com] wrote: > > > >> >> >> Please use ieee80211-freq-limit: > > > >> >> >> https://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git > > > >> >> >> /co > > > >> >> >> mmi > > > >> >> >> t/?id=b3 > > > >> >> >> 30b25eaabda00d74e47566d9200907da381896 > > > >> >> >> > > > >> >> >> Most likely with wiphy_read_of_freq_limits helper: > > > >> >> >> https://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git > > > >> >> >> /co > > > >> >> >> mmi > > > >> >> >> t/?id=e6 > > > >> >> >> 91ac2f75b69bee743f0370d79454ba4429b175 > > > >> >> > > > > >> >> > I already replied meaning of these parameters: > > > >> >> > is used to disable 2g band. > > > >> >> > is used to disable 5g band. > > > >> >> > is used to specify antenna number (if > > > >> >> > default number > > > >> >> is suitable for you, there is no need to use this parameter). > > > >> >> > should not be used for chip with device > > > >> >> > power > > > >> table. > > > >> >> In fact, this parameter should not be used any more. > > > >> >> > > > > >> >> > > > >> >> David, I think you're not understanding the comment, or at least > > > >> >> that's what it looks like to me. Yes, you did reply as to the meaning. > > > >> >> And, your reply, while informative, didn't tell us you understood > > > >> >> and were willing to fix the problem. I doubt you meant it this > > > >> >> way, but it feels defensive and like a brush-off. > > > >> >> > > > >> >> First off, you will still have to document any DT bindings you're > > > >> >> adding. Just because you answer the question in the review doesn't > > > >> >> mean you're not responsible for doing so. > > > >> >> > > > >> >> And second off, I think that Rafal (and sorry about my spelling, > > > >> >> looks like there's some sort of accent on the l that I don't know > > > >> >> how to make my keyboard do) is saying: there's already some > > > >> >> generic bindings that can be used to disable the 2g or 5g bands. > > > >> >> Granted they're even newer than your patch, but I do think if said > > > >> >> bindings exist and > > > >> are appropriate, you should use them. > > > >> >> > > > >> >> - Steve > > > >> > > > > >> > These parameters are marvell proprietary parameters, I don't think > > > >> > it should > > > >> be added to DT bindings. > > > >> > > > >> Steve is correct. > > > >> > > > >> You have to document new properties, just because they are Marvell > > > >> specific doesn't change anything. You just document them in a proper > > > place. > > > >> > > > > > > > > All right. I will do that. > > > > > > > >> > > > >> > BTW, and are only used for mwlwifi to > > > >> > report > > > >> supported bands, it is not related to limitation of frequency. > > > >> > > > >> How reporting a single band doesn't limit reported frequencies? You > > > >> can achieve exactly the same using generic property, so there is no > > > >> place for Marvell specific ones. > > > >> > > > >> In fact there were drivers of 3 vendors requiring band/freq-related > > > >> description in DT: Broadcom, Marvell & Mediatek. This property was > > > >> discussed & designed to support all limitation cases we found > > > >> possible to make it usable with > > > >> (hopefully) all drivers. > > > >> > > > > > > > > I only need simple way to disable 2g or 5g band. I will follow your suggestion > > > to document these marvell proprietary parameters. > > > > > > Seriously? Refusing to use generic binding because you think marvell,5ghz; is > > > simpler than ieee80211-freq-limit = <2402000 2482000>; (not to mention your > > > property seems reversed!)? > > > > > > I don't know how else to explain this to you. We don't want duplicated > > > properties where one can work. Just use existing one. Don't add new one even > > > if documented. > > > > > > > All right. I will check and let patch v10 to use it. For previous parameters, they will only be used by previous OpenWrt projects. > > Hi David, how are things going? Could we get v10? Hello! I would like to ask, is there any plan for future updates for this mwlwifi softmac driver?