On 22/12/2022 09:42, Shubhrajyoti Datta wrote: > The LogiCORE IP AXI Performance Monitor core enables AXI system > performance measurement for multiple slots (AXI4/AXI3/ > AXI4-Stream/AXI4-Lite) activity. Add the devicetree binding for > xilinx APM. You did not implement what I asked for in the subject. > > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxx> > > > --- > > Changes in v2: > - Use boolean for the values xlnx,enable-profile , xlnx,enable-trace > and xlnx,enable-event-count > - Update the file name > - use generic node name pmu > > .../bindings/perf/xlnx,axi-perf-monitor.yaml | 133 ++++++++++++++++++ > 1 file changed, 133 insertions(+) > create mode 100644 Documentation/devicetree/bindings/perf/xlnx,axi-perf-monitor.yaml > > diff --git a/Documentation/devicetree/bindings/perf/xlnx,axi-perf-monitor.yaml b/Documentation/devicetree/bindings/perf/xlnx,axi-perf-monitor.yaml > new file mode 100644 > index 000000000000..72843504c946 > --- /dev/null > +++ b/Documentation/devicetree/bindings/perf/xlnx,axi-perf-monitor.yaml > @@ -0,0 +1,133 @@ > +# SPDX-License-Identifier: GPL-2.0-or-later OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/perf/xlnx,axi-perf-monitor.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Xilinx Axi Performance Monitor > + > +maintainers: > + - Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxx> > + > +properties: > + compatible: > + const: xlnx,axi-perf-monitor > + > + reg: > + maxItems: 1 > + > + clocks: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + xlnx,profile: > + description: > + Has profile mode support. Event counting in profile mode consists of a > + fixed number of accumulators for each AXI4/AXI3/AXI4-Lite slot. All the > + events that can be counted are detected and given to the accumulator > + which calculates the aggregate value. There is no selection of events, > + and in this mode, event counting is done only on AXI4/AXI3/AXI4-Lite > + monitor slots. > + type: boolean > + > + xlnx,trace: > + description: > + Has trace mode support. In trace mode, the APM provides event logging in a > + reduced dynamic configuration. It captures the specified AXI events, > + external events and the time stamp difference between two successive > + events into the streaming FIFO. The selection of events to be captured > + is set through parameter configuration. Streaming agents are not > + supported in trace mode. > + type: boolean > + > + xlnx,num-monitor-slots: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: > + Number of monitor slots. > + minimum: 1 > + maximum: 8 > + > + xlnx,event-count: > + description: > + Has event counting capability.. No need for double full stop. > + type: boolean > + > + xlnx,event-log: > + type: boolean > + description: > + Has event logging support. > + > + xlnx,have-sampled-metric-cnt: > + type: boolean > + description: > + Sampled metric counters enabled in APM. > + > + xlnx,metric-count-width: > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - enum: [32, 64] Second time, the same problem. I wrote long time ago: "All of such comments apply everywhere." I'll stop review. Implement all the comments, not subset of them. Best regards, Krzysztof
