Thread (23 messages) 23 messages, 5 authors, 2022-09-12

RE: [PATCH 3/4] bindings: firmware: Update binding doc for the zynqmp afi config node

From: Manne, Nava kishore <hidden>
Date: 2022-08-30 05:41:21
Also in: linux-devicetree, linux-fpga, lkml

Hi Krzysztof,

	Please find my response inline. 
-----Original Message-----
From: Krzysztof Kozlowski <redacted>
Sent: Wednesday, August 24, 2022 6:28 PM
To: Manne, Nava kishore <redacted>; git (AMD-Xilinx)
[off-list ref]; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org;
michal.simek@xilinx.com; mdf@kernel.org; hao.wu@intel.com;
yilun.xu@intel.com; trix@redhat.com; p.zabel@pengutronix.de;
gregkh@linuxfoundation.org; ronak.jain@xilinx.com; rajan.vaja@xilinx.com;
abhyuday.godhasara@xilinx.com; piyush.mehta@xilinx.com;
lakshmi.sai.krishna.potthuri@xilinx.com; harsha.harsha@xilinx.com;
linus.walleij@linaro.org; nava.manne@xilinx.com;
devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
kernel@vger.kernel.org; linux-fpga@vger.kernel.org
Subject: Re: [PATCH 3/4] bindings: firmware: Update binding doc for the
zynqmp afi config node

Use subject prefixes matching the subsystem (git log --oneline -- ...).

"bindings" is no correct.
Will fix in v2.
On 24/08/2022 06:55, Nava kishore Manne wrote:
quoted
Updates binding document for the zynqmp afi config node to handle the
PS_PL Bus-width and resets.
Use imperative language:
https://elixir.bootlin.com/linux/v5.17.1/source/Documentation/process/sub
mitting-patches.rst#L95
Will fix in v2.
quoted
Signed-off-by: Nava kishore Manne <redacted>
---
 .../bindings/firmware/xilinx/xlnx,zynqmp-firmware.yaml     | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git
a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-
firmwa
quoted
re.yaml
b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-
firmwa
quoted
re.yaml index f14f7b454f07..9504665cad95 100644
---
a/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-
firmwa
quoted
re.yaml
+++ b/Documentation/devicetree/bindings/firmware/xilinx/xlnx,zynqmp-fi
+++ rmware.yaml
@@ -59,6 +59,13 @@ properties:
       controller.
     type: object

+  zynqmp-fpga:
Just: fpga
 
Will fix in v2.

Regards,
Navakishore.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help