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;
Message-ID: <7c41d634-e602-d5fa-05a2-8661fcd7ad55@quicinc.com>
Date:   Wed, 9 Nov 2022 08:49:34 -0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH 33/50] wifi: ath12k: add mhi.c
Content-Language: en-US
To:     Kalle Valo <kvalo@kernel.org>
CC:     <linux-wireless@vger.kernel.org>, <ath12k@lists.infradead.org>
References: <20220812161003.27279-1-kvalo@kernel.org>
 <20220812161003.27279-34-kvalo@kernel.org>
 <05841a52-3d2f-8704-b885-4b02e7a439e0@quicinc.com>
 <87h6z9eeb2.fsf@kernel.org>
From:   Jeff Johnson <quic_jjohnson@quicinc.com>
In-Reply-To: <87h6z9eeb2.fsf@kernel.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/8/2022 6:00 AM, Kalle Valo wrote:
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>>> +static struct mhi_event_config ath12k_mhi_events_qcn9274[] = {
>>
>> seems this should be const
>> but for some reason struct mhi_controller_config has:
>> 	struct mhi_event_config *event_cfg;
>>
>> (not const) so this can't be const :(
>>
>> perhaps someone can propose a MHI interface change?
>> especially since internally to MHI in parse_ev_cfg() we have:
>> 	const struct mhi_event_config *event_cfg;
>> 	[...]
>> 	for (i = 0; i < num; i++) {
>> 		event_cfg = &config->event_cfg[i];
>>
>> so it is treated as const
> 
> You submitted a patch for this, thanks for that:
> 
> https://lore.kernel.org/all/20220830171147.24338-1-quic_jjohnson@quicinc.com/
> 
> But oddly I cannot find anywhere in git, I sent a question about that to
> the mhi list.

This was already discussed on the MHI list, but just to follow up here, 
that MHI patch could not be accepted since there is one use case where 
MHI needs to write back into the event_cfg.