On 5/5/2025 10:46 AM, Jeff Johnson wrote: > On 5/2/2025 4:02 PM, Muna Sinada wrote: >> From: Karthik M <quic_karm@xxxxxxxxxxx> >> >> In MLO configuration, ath12k_mac_radio_start() iterates through all >> the radios and makes the ar state 'ON'. Even though some bands are >> not supported in certain countries, ath12k_reg_update_chan_list() >> tries to update the channel list for all the active pdevs and ends >> up in the warn_on for non-supported band. >> >> To prevent this, disable the pdev when the num of channels in a band > > s/num/number/ > >> for a particular country is zero. > > Can you explain what is meant by disabling the pdev? There are plenty of > countries where not all bands are supported, but I'd expect the pdev to be > active on the bands that are supported. > > Or is it more accurate to say: > disable the pdev when the number of channels across all bands supported by the > pdev is zero for a particular country? > > I just want to make sure the description is accurate. > > /jeff You are correct that it is more accurate to specify that we are disabling the pdev when the number of channels across all bands supported by the pdev is zero for a particular country. I will update commit message in new patchset
