From: Vinod Koul <vkoul@kernel.org>
To: "周琰杰 (Zhou Yanjie)" <zhouyanjie@wanyeetech.com>
Cc: balbi@kernel.org, gregkh@linuxfoundation.org,
mturquette@baylibre.com, sboyd@kernel.org, robh+dt@kernel.org,
kishon@ti.com, linux-clk@vger.kernel.org,
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, dongsheng.qiu@ingenic.com,
aric.pzqi@ingenic.com, rick.tyliu@ingenic.com,
yanfei.li@ingenic.com, sernia.zhou@foxmail.com,
zhenwenjin@gmail.com, paul@crapouillou.net
Subject: Re: [PATCH v9 0/3] Use the generic PHY framework for Ingenic USB PHY.
Date: Sat, 5 Dec 2020 13:39:41 +0530 [thread overview]
Message-ID: <20201205080941.GX8403@vkoul-mobl> (raw)
In-Reply-To: <20201116141906.11758-1-zhouyanjie@wanyeetech.com>
On 16-11-20, 22:19, 周琰杰 (Zhou Yanjie) wrote:
> v3->v4:
> Only add new generic-PHY driver, without removing the old one. Because the
> jz4740-musb driver is not ready to use the generic PHY framework. When the
> jz4740-musb driver is modified to use the generic PHY framework, the old
> jz4770-phy driver can be "retired".
>
> v4->v5:
> 1.Add an extra blank line between "devm_of_phy_provider_register" and "return".
> 2.Remove unnecessary "phy_set_drvdata".
> 3.Add Paul Cercueil's Reviewed-by.
>
> v5->v6:
> 1.Revert the removal of "phy_set_drvdata" in v5, removing "phy_set_drvdata" will
> cause a kernel panic on CI20.
> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
> 2.Rewrite the macro definitions, replace the original code with "FIELD_PREP()"
> and "u32p_replace_bits()" according to Vinod Koul's suggestion.
>
> v6->v7:
> 1.Remove the stray tab character.
> 2.Remove unnecessary "platform_set_drvdata".
> 3.Remove the "dev" field in priv structure, and use &phy->dev instead.
>
> v7->v8:
> Add support for Ingenic JZ4775 SoC and X2000 SoC.
>
> v8->v9:
> Correct the path errors in "ingenic,phy-usb.yaml" and "ingenic,cgu.yaml".
Applied, thanks
--
~Vinod
prev parent reply other threads:[~2020-12-05 8:23 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-16 14:19 [PATCH v9 0/3] Use the generic PHY framework for Ingenic USB PHY 周琰杰 (Zhou Yanjie)
2020-11-16 14:19 ` [PATCH v9 1/3] USB: PHY: JZ4770: Remove unnecessary function calls 周琰杰 (Zhou Yanjie)
2020-11-16 14:19 ` [PATCH v9 2/3] dt-bindings: USB: Add bindings for Ingenic JZ4775 and X2000 周琰杰 (Zhou Yanjie)
2020-11-25 2:20 ` Stephen Boyd
2020-11-30 21:15 ` Rob Herring
2020-11-16 14:19 ` [PATCH v9 3/3] PHY: Ingenic: Add USB PHY driver using generic PHY framework 周琰杰 (Zhou Yanjie)
2020-12-05 8:09 ` Vinod Koul [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201205080941.GX8403@vkoul-mobl \
--to=vkoul@kernel.org \
--cc=aric.pzqi@ingenic.com \
--cc=balbi@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dongsheng.qiu@ingenic.com \
--cc=gregkh@linuxfoundation.org \
--cc=kishon@ti.com \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=paul@crapouillou.net \
--cc=rick.tyliu@ingenic.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=sernia.zhou@foxmail.com \
--cc=yanfei.li@ingenic.com \
--cc=zhenwenjin@gmail.com \
--cc=zhouyanjie@wanyeetech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox