[PATCH] iwlwifi: remove incorrect comment in pnvm

[PATCH] iwlwifi: remove incorrect comment in pnvm

[Luca Coelho]

From: Luca Coelho <luciano.coelho@intel.com>

We use this driver as a backport that also runs on older kernels (as
part of the backports project).  So we use some checks to backport or
prevent code from compiling in incompatible kernel version.

When I took one of the PNVM patches from the backport, I accidentally
left the comment that a certain part of the code doesn't work in older
kernels.  This obviously should never be valid for the mainline.
Remove this comment.

Reported-by: Kalle Valo <kvalo@codeaurora.org>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
---
 drivers/net/wireless/intel/iwlwifi/fw/pnvm.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/net/wireless/intel/iwlwifi/fw/pnvm.c b/drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
index 2cd07d2690e0..fd070ca5e517 100644
--- a/drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
+++ b/drivers/net/wireless/intel/iwlwifi/fw/pnvm.c
@@ -220,11 +220,6 @@ static int iwl_pnvm_parse(struct iwl_trans *trans, const u8 *data,
 	return -ENOENT;
 }
 
-/*
- * This is known to be broken on v4.19 and to work on v5.4.  Until we
- * figure out why this is the case and how to make it work, simply
- * disable the feature in old kernels.
- */
 #if defined(CONFIG_EFI)
 
 #define IWL_EFI_VAR_GUID EFI_GUID(0x92daaf2f, 0xc02b, 0x455b,	\
-- 
2.30.0

Re: [PATCH] iwlwifi: remove incorrect comment in pnvm

[Kalle Valo]

Luca Coelho <luca@coelho.fi> writes:

> From: Luca Coelho <luciano.coelho@intel.com>
>
> We use this driver as a backport that also runs on older kernels (as
> part of the backports project).  So we use some checks to backport or
> prevent code from compiling in incompatible kernel version.
>
> When I took one of the PNVM patches from the backport, I accidentally
> left the comment that a certain part of the code doesn't work in older
> kernels.  This obviously should never be valid for the mainline.
> Remove this comment.
>
> Reported-by: Kalle Valo <kvalo@codeaurora.org>
> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

Thanks, I assigned this to myself on patchwork and apply directly to
wireless-drivers-next.

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

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

Re: [PATCH] iwlwifi: remove incorrect comment in pnvm

[Kalle Valo]

Luca Coelho <luca@coelho.fi> wrote:

> From: Luca Coelho <luciano.coelho@intel.com>
> 
> We use this driver as a backport that also runs on older kernels (as
> part of the backports project).  So we use some checks to backport or
> prevent code from compiling in incompatible kernel version.
> 
> When I took one of the PNVM patches from the backport, I accidentally
> left the comment that a certain part of the code doesn't work in older
> kernels.  This obviously should never be valid for the mainline.
> Remove this comment.
> 
> Reported-by: Kalle Valo <kvalo@codeaurora.org>
> Signed-off-by: Luca Coelho <luciano.coelho@intel.com>

Patch applied to wireless-drivers-next.git, thanks.

3304b6f937a3 iwlwifi: remove incorrect comment in pnvm

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/iwlwifi.20210211223049.40d545a0fa89.I04793aaa5312b926335c8db32131f000432df511@changeid/

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