Re: [PATCH] accel/qaic: Add xbl_config image entry for AIC200

[Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>]

On 10/14/25 1:47 AM, Carl Vanderlip wrote:
> On 10/8/2025 12:49 AM, Konrad Dybcio wrote:
>> On 10/8/25 12:15 AM, Youssef Samir wrote:
>>> From: Aswin Venkatesan <aswivenk@qti.qualcomm.com>
>>>
>>> Update the Sahara image table for the AIC200 to add an entry for xbl_config image at id 38.
>>>
>>> Signed-off-by: Aswin Venkatesan <aswivenk@qti.qualcomm.com>
>>> Signed-off-by: Youssef Samir <youssef.abdulrahman@oss.qualcomm.com>
>>> ---
>>>  drivers/accel/qaic/sahara.c | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/drivers/accel/qaic/sahara.c b/drivers/accel/qaic/sahara.c
>>> index 3ebcc1f7ff58..04e8acb94c04 100644
>>> --- a/drivers/accel/qaic/sahara.c
>>> +++ b/drivers/accel/qaic/sahara.c
>>> @@ -194,6 +194,7 @@ static const char * const aic200_image_table[] = {
>>>  	[23] = "qcom/aic200/aop.mbn",
>>>  	[32] = "qcom/aic200/tz.mbn",
>>>  	[33] = "qcom/aic200/hypvm.mbn",
>>> +	[38] = "qcom/aic200/xbl_config.elf",
>>
>> This is the third oneliner extending this array you sent within 24h,
>> please get the list of required files and update this list once and
>> for good
>>
>> Konrad
> 
> I wish it was that easy during bring-up.
> 
> These patches are being upstreamed from our internal branch and thus
> reflect the different latencies across the sub-teams to that request.

That's upstream-first-first ;)

I would assume there's a single list of files that get loaded, perhaps
within the device programmer image.. Unless you're altering that on the
fly as well?

Konrad