Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp901365pxu; Wed, 6 Jan 2021 07:22:03 -0800 (PST) X-Google-Smtp-Source: ABdhPJxlkPJeRd5CeW7g/W2OxyAAW17TTKriNg0vjSC0LeTNNL62KgJEHYJ0JajN+5IZzB+at4PL X-Received: by 2002:a50:d2d2:: with SMTP id q18mr4214227edg.346.1609946523620; Wed, 06 Jan 2021 07:22:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1609946523; cv=none; d=google.com; s=arc-20160816; b=ugVFqDodfSpukWHiosRc9JSZZtK83GIpz8vCSazFx/hDeqSdzL/yyWe7DbpI+mBD/T /T3n34QWtWD++oiHrMIMlDc2Q+35a+wjdWYxuMKTx4p77/9sD/Gze5dMRIIJijNABpwK HXZGuJ0M1On0mG8EtHTEypa6lzoGrUsx2hhzZOZ1PgXfufDBt2AVZMRJ7PX1IA/LrD6Z 6Stqdp6S+snNy7FPE/vO0cKXjwjddHcykAWRg8Yl7EZpRYLAPDdHocZAg+zBl/q/jeBx YAl5p8kzkmed1eL+B9txq5MiU7oRIlMK+DGHT2qFaKViP+LOHUD02BJOpsvyPeJK99MV YKYA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=SUyx/diqi2F1STfXm7QYrRLhMzAeEdwwZ/ZMvDjelN4=; b=KFtpFfoZIGZEy8rQ5tL4rGwSjVKMHCiJFkjnVPlpuVfhceEkqhvxgE3o6Z4HGraeNg VjJILk7YGjS4z+iGF8XXfNVTSx1H1DyzwfYCo82+jmET3eH7MmAp3qpF1jef7JT9NV2s 1SM9D7ItTdtideukSmKExaHOG2VRDEoWxpV+H3FTcFR18v3qfjF1dpZb/kqx9bfsk2um pTwgKxoLEyPchg964F8/ZMvnj7kFTN+hpbuzr0VIZYHJlOiSZ13BlKteTHybJ7yE5o3c pnRWUuWqIHkfmjTCLy+FM8/rZ8MQIoIz4axzj6xGqFdgFersYgTEMxAGf59c7CKWz1yd +LoQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=WotmtCzj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ah12si1019543ejc.483.2021.01.06.07.21.38; Wed, 06 Jan 2021 07:22:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=WotmtCzj; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726826AbhAFPUY (ORCPT + 99 others); Wed, 6 Jan 2021 10:20:24 -0500 Received: from mail.kernel.org ([198.145.29.99]:40040 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726663AbhAFPUY (ORCPT ); Wed, 6 Jan 2021 10:20:24 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 068C32311D; Wed, 6 Jan 2021 15:19:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1609946383; bh=u/iJ2yg1n0QdcVgVvjTf284lvZQ11lam/cryS8nLa+Y=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=WotmtCzjWxb2CAu2KAtsKTJqh9iCxeOMgicnKsFyUrueNxBBcVCCmpWMJbg8PY8SO VFt7v5v95F3h4lF7UoXFeabAeqZHgU866dLUQQrZGu0jdI9RZPELcSdjK+OoEvW37U q0mdbZDCAzvwLFBCun8X7E9ZgDEt8NFeSCvFas0qa3f/4K3XPVwl1lsjP+t6l5pJhA SROujlVC3B1fXZGLceKzxvi7TQ727HRqhu9pKNB7rcomm3nMWJVGR1utKmSrS7+NR5 lH1EYBshQUWRi4+1RJmbENh1kBUWaVTGKiMLhpqvzthK/KzcB3xdc0qFVMdk2eVMrp CRIZb2wQnRAIg== Received: by mail-ej1-f42.google.com with SMTP id x16so5495813ejj.7; Wed, 06 Jan 2021 07:19:42 -0800 (PST) X-Gm-Message-State: AOAM533zkV39EsVMBHq76NgypnulOxwz225Y3H8GGq3XcKbeJu/Vphjk og8es/ZLkyWIdnCQBrhldGBFDw82mxvEU9BlLg== X-Received: by 2002:a17:906:d87:: with SMTP id m7mr3151012eji.108.1609946381645; Wed, 06 Jan 2021 07:19:41 -0800 (PST) MIME-Version: 1.0 References: <20210104131755.2979203-1-bert@biot.com> <20210104131755.2979203-2-bert@biot.com> <1609866288.019144.372417.nullmailer@robh.at.kernel.org> In-Reply-To: From: Rob Herring Date: Wed, 6 Jan 2021 08:19:30 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2 1/2] dt-bindings: interrupt-controller: Add Realtek RTL838x/RTL839x support To: Bert Vermeulen Cc: devicetree@vger.kernel.org, John Crispin , Marc Zyngier , "linux-kernel@vger.kernel.org" , Birger Koblitz , Thomas Gleixner Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 6, 2021 at 2:55 AM Bert Vermeulen wrote: > > On 1/5/21 6:04 PM, Rob Herring wrote: > > On Mon, 04 Jan 2021 14:17:54 +0100, Bert Vermeulen wrote: > >> Signed-off-by: Bert Vermeulen > >> --- > >> .../realtek,rtl-intc.yaml | 57 +++++++++++++++++++ > >> 1 file changed, 57 insertions(+) > >> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/realtek,rtl-intc.yaml > >> > > > > My bot found errors running 'make dt_binding_check' on your patch: > > I fixed the syntax error that caused this to fail. But running the test now > produces some errors that are not so obvious: > > interrupt-controller@3000: interrupt-map: [[31, 4294967295, 2], [30, > 4294967295, 1], [29, 4294967295, 5]] is too short > > Not sure where this comes from, but running dt-validate -v gives this: > > Failed validating 'minItems' in schema['properties']['interrupt-map']: > {'maxItems': 24, 'minItems': 24} > > Where did that 24 come from? I don't even have 24 initerrupts to map :-) If you don't define minItems, then the default is the same as maxItems. So you need to set 'minItems'. > > Another error is this: > > interrupt-controller@3000: 'interrupt-map-mask' is a dependency of > 'interrupt-map' > > I can see this in your dtschema package's schemas/interrupt-controller.yaml > where it's defined as a dependency, but why? I don't need that property. This should be the other way around. I'll fix it. Rob