Re: [PATCH 1/4] usb: dwc3: of-simple: add compatible for rockchip

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Felipe and Brian,

On 05/10/2016 03:15 PM, Felipe Balbi wrote:
Hi,

Brian Norris <briannorris@xxxxxxxxxxxx> writes:
Hi William,

Did you leave off linux-rockchip@xxxxxxxxxxxxxxxxxxx intentionally? IMO,
it's nice to have that list in CC, so interested parties can follow your
work, even if they aren't as fortunate as me to have been CC'd on your
patch directly.
Actually, I don't know the linux-rockchip@xxxxxxxxxxxxxxxxxxx before.
I'll add the list in CC next patch version.
Thanks~

On Mon, May 09, 2016 at 07:46:14PM +0800, William Wu wrote:
Signed-off-by: William Wu <william.wu@xxxxxxxxxxxxxx>
---
  drivers/usb/dwc3/dwc3-of-simple.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of-simple.c
index 9743353..1f3665b 100644
--- a/drivers/usb/dwc3/dwc3-of-simple.c
+++ b/drivers/usb/dwc3/dwc3-of-simple.c
@@ -162,6 +162,7 @@ static const struct dev_pm_ops dwc3_of_simple_dev_pm_ops = {
  static const struct of_device_id of_dwc3_simple_match[] = {
  	{ .compatible = "qcom,dwc3" },
  	{ .compatible = "xlnx,zynqmp-dwc3" },
+	{ .compatible = "rockchip,dwc3" },
Add to Documentation/devicetree/bindings/. Do we need a new
Documentation/devicetree/bindings/usb/rockchip,dwc3.txt, to match the
pattern of qcom and xlnx? Or can we just add to dwc3.txt, since so far,
all bindings are documented in the common file?
dwc3.txt is for dwc3.ko. We need separate files for rockchip, xilinx and
qualcomn :-)
I have already prepared a new Documentation/devicetree/bindings/usb/rockchip,dwc3.txt, But considering that rockchip,dwc3.txt should contains phys node description which are
not ready yet, so I don't add the dt-bindings patch here.
Is it better to add the dt-bindings patch(rockchip,dwc3.txt) without phys node description here?
If it is, I'll add  rockchip,dwc3.txt next patch verison.

And I have some doubts about the name format of the usb dwc3 documents.
I notice that there are at least two types of name format:
1. dwc3-xx.txt  (e.g. dwc3-st.txt )
2. xx, dwc3.txt (e.g. qcom,dwc3.txt)

Which one do you prefer?



--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux