linux-kernel - Re: [PATCH 1/2] dt-bindings: ahci-fsl-qoriq: added explanation for reg-names

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 18 Nov 2016 08:21:38 -0600
From:   Rob Herring <robh@...nel.org>
To:     yuantian.tang@....com
Cc:     tj@...nel.org, mark.rutland@....com, devicetree@...r.kernel.org,
        catalin.marinas@....com, will.deacon@....com,
        linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
        shawnguo@...nel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 1/2] dt-bindings: ahci-fsl-qoriq: added explanation for
 reg-names

On Thu, Nov 17, 2016 at 03:59:19PM +0800, yuantian.tang@....com wrote:
> From: Tang Yuantian <Yuantian.Tang@....com>
> 
> Added explanation for reg-names to make it more clear.
> 
> Signed-off-by: Tang Yuantian <yuantian.tang@....com>
> ---
>  Documentation/devicetree/bindings/ata/ahci-fsl-qoriq.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/ata/ahci-fsl-qoriq.txt b/Documentation/devicetree/bindings/ata/ahci-fsl-qoriq.txt
> index fc33ca0..80cf10c 100644
> --- a/Documentation/devicetree/bindings/ata/ahci-fsl-qoriq.txt
> +++ b/Documentation/devicetree/bindings/ata/ahci-fsl-qoriq.txt
> @@ -10,6 +10,8 @@ Required properties:
>  Optional properties:
>    - dma-coherent: Enable AHCI coherent DMA operation.
>    - reg-names: register area names when there are more than 1 register area.
> +		example: 'ahci' is for sata controller register.
> +			 'sata-ecc' is for sata ecc register.

This is not an example, but should be exactly what names are used.

When is there more than one area? That should be defined by the 
compatible string and spelled out here.

Rob