linux-kernel - RE: [PATCH] Document: dt: binding: imx: Fix PAD_CTL_DSE

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]

Message-ID: <VI1PR04MB42227AA158BA1FCBF314AA27807B0@VI1PR04MB4222.eurprd04.prod.outlook.com>
Date:   Tue, 26 Feb 2019 12:21:49 +0000
From:   Aisheng Dong <aisheng.dong@....com>
To:     Christina Quast <cquast@...overdisplays.com>
CC:     "festevam@...il.com" <festevam@...il.com>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "stefan@...er.ch" <stefan@...er.ch>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "linus.walleij@...aro.org" <linus.walleij@...aro.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] Document: dt: binding: imx: Fix PAD_CTL_DSE_X*

> From: Christina Quast [mailto:cquast@...overdisplays.com]
> Sent: Saturday, February 23, 2019 1:01 AM
> 
> In the iMX7d datasheet, the PAD_CTL_DSE_X* values are different from the
> documentation.
> 

It's a doc problem.
Latest RM seems got updated.

As here it's a reference definition in binding doc and device tree actually does not
use it (IMX Pinctrl use raw data to set pad configuration). So it won't cause any
compatibility issue to me.

Please update the patch title to:
dt-bindings: pinctrl: imx7d: xxxxx

Otherwise:
Ack-by: Dong Aisheng <aisheng.dong@....com>

Regards
Dong Aisheng

> Signed-off-by: Christina Quast <cquast@...overdisplays.com>
> ---
>  .../devicetree/bindings/pinctrl/fsl,imx7d-pinctrl.txt       | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/fsl,imx7d-pinctrl.txt
> b/Documentation/devicetree/bindings/pinctrl/fsl,imx7d-pinctrl.txt
> index 6666277c3acb..8ac1d0851a0f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/fsl,imx7d-pinctrl.txt
> +++ b/Documentation/devicetree/bindings/pinctrl/fsl,imx7d-pinctrl.txt
> @@ -48,9 +48,9 @@ PAD_CTL_HYS                     (1 << 3)
>  PAD_CTL_SRE_SLOW                (1 << 2)
>  PAD_CTL_SRE_FAST                (0 << 2)
>  PAD_CTL_DSE_X1                  (0 << 0)
> -PAD_CTL_DSE_X2                  (1 << 0)
> -PAD_CTL_DSE_X3                  (2 << 0)
> -PAD_CTL_DSE_X4                  (3 << 0)
> +PAD_CTL_DSE_X4                  (1 << 0)
> +PAD_CTL_DSE_X2                  (2 << 0)
> +PAD_CTL_DSE_X6                  (3 << 0)
> 


>  Examples:
>  While iomuxc-lpsr is intended to be used by dedicated peripherals to take
> --
> 2.20.1
> 
> 
> 
> 
> 
> Please consider the environment before printing this email
> 
> 
> 
> 
> 
> The information transmitted is intended only for the person or entity to which
> it is addressed and may contain confidential and/or privileged material. Any
> review, retransmission, dissemination or other use of, or taking of any action in
> reliance upon, this information by persons or entities other than the intended
> recipient is prohibited.
> If you received this in error, please contact the sender or postmaster
> (postmaster@...overdisplays.com) and delete the material from any
> computer.
> Although we routinely screen for viruses, addressees should check this e-mail
> and any attachment for viruses. We make no warranty as to absence of viruses
> in this e-mail or any attachments.
> Our Company's email policy is to permit incidental personal use. If this email is
> of a personal nature, it must not be relied upon as expressing the views or
> opinions of the company.