Karthikeyan Periyasamy <quic_periyasa@xxxxxxxxxxx> writes:
> Currently, the mac80211 hw registration procedure is tightly coupled with
> the handling of link/radio (ar). Define a new helper function to separate
> the link/radio handling from the mac80211 hw registration procedure for
> improved code readability. Also, it can be easy to scale these
> functionality to support single/multi link operation in the future.
>
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1
>
> Signed-off-by: Karthikeyan Periyasamy <quic_periyasa@xxxxxxxxxxx>
> ---
> drivers/net/wireless/ath/ath12k/core.c | 61 +++++++++++-
> drivers/net/wireless/ath/ath12k/mac.c | 132 ++++++++++---------------
> drivers/net/wireless/ath/ath12k/mac.h | 4 +-
> 3 files changed, 109 insertions(+), 88 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath12k/core.c b/drivers/net/wireless/ath/ath12k/core.c
> index e10c5f2cd8eb..d1ac00c59b8c 100644
> --- a/drivers/net/wireless/ath/ath12k/core.c
> +++ b/drivers/net/wireless/ath/ath12k/core.c
> @@ -498,11 +498,62 @@ static void ath12k_core_soc_destroy(struct ath12k_base *ab)
> ath12k_qmi_deinit_service(ab);
> }
>
> +static int ath12k_core_mac_register(struct ath12k_base *ab)
> +{
> + struct ath12k *ar;
> + struct ath12k_pdev *pdev;
> + int i;
> + int ret;
> +
> + if (test_bit(ATH12K_FLAG_REGISTERED, &ab->dev_flags))
> + return 0;
> +
> + /* Initialize channel counters frequency value in hertz */
> + ab->cc_freq_hz = 320000;
> + ab->free_vdev_map = (1LL << (ab->num_radios * TARGET_NUM_VDEVS)) - 1;
> +
> + for (i = 0; i < ab->num_radios; i++) {
> + pdev = &ab->pdevs[i];
> + ar = pdev->ar;
> +
> + ret = ath12k_mac_hw_register(ar);
> + if (ret)
> + goto err_cleanup;
> + }
> +
> + return 0;
> +
> +err_cleanup:
> + for (i = i - 1; i >= 0; i--) {
> + pdev = &ab->pdevs[i];
> + ar = pdev->ar;
> + ath12k_mac_hw_unregister(ar);
> + }
> +
> + return ret;
> +}
Is there a reason why you moved these two functions from mac.c to
core.c? This is mac level code anyway, right?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
