AW: (EXT) Re: [PATCH 1/2] dt-bindings: mtd: spi-nor: Add output-driver-strength property
From: Alexander Stein <hidden>
Date: 2021-10-08 07:10:52
Also in:
lkml
On Mon, 05 Oct 2021 14:26:28 +0200, Rob Herring wrote:
From: Rob Herring <robh@kernel.org> On Mon, 04 Oct 2021 13:15:28 +0200, Alexander Stein wrote:quoted
From: Alexander Stein <redacted> This property is for optimizing output voltage impedance and is specific to each board. Signed-off-by: Alexander Stein <redacted> --- I checked Micron and Macronix datasheets. Both have similar but not identical supported values. Also the register locations are different. For those reasons I decided to specify the Ohms value directly and let the device specfic driver figure out if it is supported where to write it to. BTW: Are the Ohm values and the corresponding register bits standardized somewhere? Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml | 6 ++++++ 1 file changed, 6 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/1536062
Thanks for pointing that out. I checked the amount of error messages related to jedec,spi-nor.yaml before and after applying these patches using
ARCH=arm64 make -j12 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml 2>&1 > /dev/null | wc -l
with all arm64 platforms enabled. There wasn't a change in the line count, so I assume this new binding didn't cause them. Best regards, Alexander