Received-SPF: pass (google.com: domain of linux-wireless+bounces-5583-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <ef6b6a7a4a9d3b01c0eb6abf0991e7e27425e102.camel@sipsolutions.net>
Subject: Re: [PATCH 01/13] wifi: cfg80211: Add provision to advertise
 multiple radio in one wiphy
From: Johannes Berg <johannes@sipsolutions.net>
To: Vasanthakumar Thiagarajan <quic_vthiagar@quicinc.com>, Karthikeyan
	Periyasamy <quic_periyasa@quicinc.com>, ath12k@lists.infradead.org
Cc: linux-wireless@vger.kernel.org
Date: Fri, 29 Mar 2024 15:30:42 +0100
In-Reply-To: <033185b0-f878-a50b-d0d9-57fa79439bdf@quicinc.com>
References: <20240328072916.1164195-1-quic_periyasa@quicinc.com>
	 <20240328072916.1164195-2-quic_periyasa@quicinc.com>
	 <e36599d3269496de59505b36be46f873976f8706.camel@sipsolutions.net>
	 <033185b0-f878-a50b-d0d9-57fa79439bdf@quicinc.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.50.4 (3.50.4-1.fc39) 
Precedence: bulk
MIME-Version: 1.0

On Fri, 2024-03-29 at 19:41 +0530, Vasanthakumar Thiagarajan wrote:
> >=20
> > > + * @hw_chans: list of the channels supported by every constituent un=
derlying
> > > + *	hardware. Drivers abstracting multiple discrete hardware (radio) =
under
> > > + *	one wiphy can advertise the list of channels supported by each ph=
ysical
> > > + *	hardware in this list. Underlying hardware specific channel list =
can be
> > > + *	used while describing interface combination for each of them.
> >=20
> > I'd expect there to be a limit on channels being within a single band o=
n
> > a single "hardware"?
> >=20
>=20
> There are ath12k hardware supporting multiple band which need to be=20
> registered under one mac80211_hw/wiphy. This design is to support such=
=20
> hardware.

Oh OK, that was something that I didn't have in mind any more, or never
knew or paid attention to.

> I agree, it is adding complexities. I was thinking if this can=20
> be done in steps , first limiting to single band for a single hardware=
=20
> then extending it with multiple bands later but that may bring in=20
> different set of challenges...

Probably not point, yeah.

johannes