linux-kernel - Re: [PATCH] [PATCH v3] phy: add phy hisi inno usb2

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:	Mon, 25 Jul 2016 10:26:29 -0500
From:	Rob Herring <robh+dt@...nel.org>
To:	Pengcheng Li <lpc.li@...ilicon.com>
Cc:	Kishon Vijay Abraham I <kishon@...com>,
	Mark Rutland <mark.rutland@....com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	xuejiancheng@...ilicon.com, Dongpo Li <lidongpo@...ilicon.com>,
	zhangzhenxing@...ilicon.com
Subject: Re: [PATCH] [PATCH v3] phy: add phy hisi inno usb2

On Mon, Jul 25, 2016 at 8:50 AM, Pengcheng Li <lpc.li@...ilicon.com> wrote:
> Add support for inno usb2 phy integrated on some hisilicon SOCs.

Bindings need to be sent to DT list.

>
> Signed-off-by: Pengcheng Li <lpc.li@...ilicon.com>
> ---
> change log:
> v3:
>  1. inno phy do not use phy framework, remove it.
> ---
>  .../devicetree/bindings/phy/phy-hisi-inno-usb2.txt |  48 ++++
>  drivers/phy/Kconfig                                |  10 +
>  drivers/phy/Makefile                               |   1 +
>  drivers/phy/phy-hisi-inno-usb2.c                   | 290 +++++++++++++++++++++
>  4 files changed, 349 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/phy/phy-hisi-inno-usb2.txt
>  create mode 100644 drivers/phy/phy-hisi-inno-usb2.c
>
> diff --git a/Documentation/devicetree/bindings/phy/phy-hisi-inno-usb2.txt b/Documentation/devicetree/bindings/phy/phy-hisi-inno-usb2.txt
> new file mode 100644
> index 0000000..59eaf73
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/phy-hisi-inno-usb2.txt
> @@ -0,0 +1,48 @@
> +HiSilicon INNO USB2 PHY
> +-----------------------
> +Required properties:
> +- compatible: Should be "hisilicon,inno_usb2_phy"

Use '-', not '_'.

> +- #phy-cells: Must be 0
> +- hisilicon,peripheral-syscon: Phandle of syscon used to control phy.
> +- hisilicon,reg-num: Number of phy registers which should be configured
> +at phy intialization stage
> +- hisilicon,reg-seq: Sequence of triplets of (address, value, delay-us).
> +The number of triplets is equal to "hisilicon,reg-num". Each triplet is
> +used to write one phy register. The delay-us cell represents the delay
> +time in microseconds to be applied after each write.

Don't create a scripting language in DT. The programming needed should
be defined by the compatible string and possibly some specific
properties of things that vary board to board.

Rob