Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dtschema: Add basis points property unity #73

Merged
merged 1 commit into from May 13, 2022

Conversation

krzk
Copy link

@krzk krzk commented May 13, 2022

Document the basis points property unit: one hundredth of 1 percentage
point. The unit is not very popular but recently new bindings using it
were submitted.

See also: https://lore.kernel.org/all/1652282793-5580-2-git-send-email-quic_kriskura@quicinc.com/

Signed-off-by: Krzysztof Kozlowski krzk@kernel.org

Document the basis points property unit: one hundredth of 1 percentage
point.  The unit is not very popular but recently new bindings using it
were submitted.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
@robherring robherring merged commit ea4ca82 into devicetree-org:main May 13, 2022
@krzk krzk deleted the bp-property-unit branch May 13, 2022 12:20
@z3ntu
Copy link

z3ntu commented Oct 5, 2023

Sorry to necrobump, but this suffix seems to conflict with the bindings phy/qcom,usb-snps-femto-v2.yaml in mainline. Those expect a signed integer (the minimum value is negative) but the base unit here sets it as unsigned integer.

So not sure where to fix this really..

foo.dtb: phy@88e3000: qcom,squelch-detector-bp:0:0: 4294965206 is greater than the maximum of 1590
        from schema $id: http://devicetree.org/schemas/phy/qcom,usb-snps-femto-v2.yaml#

Any ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants