On 30/05/2023 14:25, Krzysztof Kozlowski wrote: > On Tue, 30 May 2023 03:51:37 -0600, Pranavi Somisetty wrote: >> watermark value is the minimum amount of packet data >> required to activate the forwarding process. The watermark >> implementation and maximum size is dependent on the device >> where Cadence MACB/GEM is used. >> >> Signed-off-by: Pranavi Somisetty <pranavi.somisetty@xxxxxxx> >> --- >> Changes v2: >> None (patch added in v2) >> >> Changes v3: >> 1. Fixed DT schema error: "scalar properties shouldn't have array keywords". >> 2. Modified description of rx-watermark to include units of the watermark value. >> 3. Modified the DT property name corresponding to rx_watermark in >> pbuf_rxcutthru to "cdns,rx-watermark". >> 4. Modified commit description to remove references to Xilinx platforms, >> since the changes aren't platform specific. >> --- >> Documentation/devicetree/bindings/net/cdns,macb.yaml | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for dtbs_check. > This will change in the future. > > Full log is available here: https://patchwork.ozlabs.org/patch/1787378 > > > ethernet@e000b000: ethernet-phy@0: Unevaluated properties are not allowed ('device_type', 'marvell,reg-init' were unexpected) > arch/arm/boot/dts/zynq-parallella.dtb > Unrelated warnings, can be ignored. Best regards, Krzysztof