Re: [PATCH v4 1/3] xilinx: dfu: Fill directly update_info.dfu_string

[Michal Simek <michal.simek@amd.com>]

On 2/28/25 19:15, Jon Humphreys wrote:
> Heinrich Schuchardt <xypron.glpk@gmx.de> writes:
> 
>> On 28.02.25 16:41, Michal Simek wrote:
>>>
>>>
>>> On 2/28/25 15:33, Heinrich Schuchardt wrote:
>>>> On 26.02.25 23:35, Jonathan Humphreys wrote:
>>>>> From: Michal Simek <michal.simek@amd.com>
>>>>>
>>>>> Directly fill update_info.dfu_string to prepare platforms to switch
>>>>> from using dfu_alt_info variable to dfu_string which contains
>>>>> description
>>>>> for capsule update when switch is done.
>>>>
>>>> Other platforms only set the environment variable.
>>>
>>> Not really.
>>>
>>> git grep \.dfu_string board/
>>>
>>> beagle, imx, rpi, ti and others are also doing it.
>>
>> But why do we have platforms that don't?
>>
> 
> I'm not aware of the history of the Xilinx platforms and why they didn't,
> but Michal has fixed this.  I'm not aware of any other platforms not
> setting .dfu_string.

It has never been wired like this from the beginning.
In April 2022 the patch
741ef867288b ("capsule: board: Add information needed for capsule updates")
added description for capsules without defining dfu_string because dfu_alt_info 
has been properly generated.

M