On Mon, Sep 07, 2020 at 06:11:18PM +0200, Krzysztof Kozlowski wrote:
> The ADC in S3C/S5P/Exynos SoCs can be used also for handling touch
> screen. In such case the second interrupt is required. This second
> interrupt can be anyway provided, even without touch screens. This
> fixes dtbs_check warnings like:
>
> arch/arm/boot/dts/s5pv210-aquila.dt.yaml: adc@e1700000: interrupts: [[23], [24]] is too long
>
> Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> ---
> .../bindings/iio/adc/samsung,exynos-adc.yaml | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> index cc3c8ea6a894..89b4f9c252a6 100644
> --- a/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/samsung,exynos-adc.yaml
> @@ -41,7 +41,10 @@ properties:
> maxItems: 2
>
> interrupts:
> - maxItems: 1
> + description:
> + ADC interrupt followed by optional touchscreen interrupt.
> + minItems: 1
> + maxItems: 2
>
> "#io-channel-cells":
> const: 1
> @@ -107,6 +110,16 @@ allOf:
> items:
> - const: adc
>
> + - if:
> + properties:
> + has-touchscreen:
> + true
This evaluates as true if 'has-touchscreen' is not present too. You
should use 'required' here.
> + then:
> + properties:
> + interrupts:
> + minItems: 2
> + maxItems: 2
> +
> examples:
> - |
> adc: adc@12d10000 {
> --
> 2.17.1
>
