Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Kalle Valo <kvalo@kernel.org>
To:     Luca Coelho <luca@coelho.fi>
Cc:     johannes@sipsolutions.net, socketcan@hartkopp.net, kuba@kernel.org,
        linux-wireless@vger.kernel.org
Subject: Re: [PATCH] iwlwifi: mvm: Don't fail if PPAG isn't supported
References: <iwlwifi.20220322173828.fa47f369b717.I6a9c65149c2c3c11337f3a802dff22f514a3a436@changeid>
        <3166a024cd5bef43bf192418e95d3c75409a861f.camel@coelho.fi>
Date:   Tue, 22 Mar 2022 21:36:16 +0200
In-Reply-To: <3166a024cd5bef43bf192418e95d3c75409a861f.camel@coelho.fi> (Luca
        Coelho's message of "Tue, 22 Mar 2022 17:43:31 +0200")
Message-ID: <87a6dhwzin.fsf@tynnyri.adurom.net>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Luca Coelho <luca@coelho.fi> writes:

> On Tue, 2022-03-22 at 17:39 +0200, Luca Coelho wrote:
>> From: Miri Korenblit <miriam.rachel.korenblit@intel.com>
>> 
>> When we're copying the PPAG table into the cmd structure we're failing
>> if the table doesn't exist in ACPI or is invalid, or if the FW doesn't
>> support PPAG setting etc.
>> 
>> This is wrong because those are valid scenarios.  Fix this by not
>> failing in those cases.
>> 
>> Fixes: e8e10a37c51c ("iwlwifi: acpi: move ppag code from mvm to fw/acpi")
>> Tested-by: Oliver Hartkopp <socketcan@hartkopp.net>
>> Signed-off-by: Miri Korenblit <miriam.rachel.korenblit@intel.com>
>> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
>> ---
>>  drivers/net/wireless/intel/iwlwifi/mvm/fw.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
>> index 4632d3ad1a2b..e842816134f1 100644
>> --- a/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
>> +++ b/drivers/net/wireless/intel/iwlwifi/mvm/fw.c
>> @@ -1015,8 +1015,9 @@ int iwl_mvm_ppag_send_cmd(struct iwl_mvm *mvm)
>>  	int ret, cmd_size;
>>  
>>  	ret = iwl_read_ppag_table(&mvm->fwrt, &cmd, &cmd_size);
>> +	/* Not supporting PPAG table is a valid scenario */
>>  	if(ret < 0)
>> -		return ret;
>> +		return 0;
>>  
>>  	IWL_DEBUG_RADIO(mvm, "Sending PER_PLATFORM_ANT_GAIN_CMD\n");
>>  	ret = iwl_mvm_send_cmd_pdu(mvm, WIDE_ID(PHY_OPS_GROUP,
>
> Hi,
>
> Jakub, this is the fix for the PPAG regression that we talked about
> earlier.  If it's fine with Kalle, you can apply it directly to net-
> next to expedite it.

You didn't CC netdev though so it's not visible on their patchwork.

> Kalle can you ack?

Acked-by: Kalle Valo <kvalo@kernel.org>

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches