[elbe-devel] [PATCH v4 03/16] schema: copy_to_partition, set_partition_type: Add attributes

[Bastian Germann]

Am 18.10.22 um 16:50 schrieb Viraj Shah:
> * Add documentation for attributes and rewrite documentation
> of finetuning actions copy_to_partition, set_partition_type.
> * Replace tabs with spaces.
> 
> Signed-off-by: Viraj Shah <viraj.shah at linutronix.de>

Reviewed-by: Bastian Germann <bage at linutronix.de>

> ---
>   schema/dbsfed.xsd | 50 ++++++++++++++++++++++++++++++++++++-----------
>   1 file changed, 39 insertions(+), 11 deletions(-)
> 
> diff --git a/schema/dbsfed.xsd b/schema/dbsfed.xsd
> index a995983f1..4e252c66c 100644
> --- a/schema/dbsfed.xsd
> +++ b/schema/dbsfed.xsd
> @@ -2422,29 +2422,57 @@
>     <complexType name="copy_to_partition">
>       <annotation>
>         <documentation>
> -        Attribute 'part' describes the partition to be mounted (nr).
> -	The value of the tag describes the filename where the artifact shall
> -	be copied to, onto the filesystem, after it has been mounted.
> -	Attribute 'artifact' is the artifact name.
> +        It copies the artifact to a given partition that will be
> +        mounted.
> +        Please note that this element's value is destination's path in "part"
> +        partition.
>         </documentation>
>       </annotation>
>       <simpleContent>
>         <extension base="rfs:string">
> -        <attribute name="part" type="integer" use="required" />
> -        <attribute name="artifact" type="string" use="required" />
> +        <attribute name="part" type="integer" use="required">
> +          <annotation>
> +            <documentation>
> +              Describes the partition number that the required file is copied to.
> +            </documentation>
> +          </annotation>
> +        </attribute>
> +        <attribute name="artifact" type="string" use="required">
> +          <annotation>
> +            <documentation>
> +              The name of the file that is copied to the destination is termed
> +              as "artifact"
> +            </documentation>
> +          </annotation>
> +        </attribute>
>         </extension>
>       </simpleContent>
>     </complexType>
>     <complexType name="set_partition_type">
>       <annotation>
>         <documentation>
> -        Attribute 'part' describes the partition to be modified (nr).
> -	Attribute 'type' is the numerical partition id, as understood by
> -	fdisk.
> +        Sets the partition type of a particular partition to the specified type
> +        for disk labels.
> +        When changing partition type, it asks user the partition number in case
> +        of many partitions, and then the type.
>         </documentation>
>       </annotation>
> -    <attribute name="part" type="integer" use="required" />
> -    <attribute name="type" type="string" use="required" />
> +      <attribute name="part" type="integer" use="required">
> +        <annotation>
> +          <documentation>
> +            Describes the partition to be modified in integer.
> +          </documentation>
> +        </annotation>
> +      </attribute>
> +      <attribute name="type" type="string" use="required">
> +        <annotation>
> +          <documentation>
> +            Describes the numerical partition id, as understood by fdsik.
> +            It is the "partition type" and can be a numerical id or a textual alias
> +            for it.
> +          </documentation>
> +        </annotation>
> +      </attribute>
>     </complexType>
>     <complexType name="addgroup">
>       <annotation>