From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B0E17C43334 for ; Thu, 21 Jul 2022 13:49:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230406AbiGUNtn (ORCPT ); Thu, 21 Jul 2022 09:49:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58166 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230473AbiGUNtk (ORCPT ); Thu, 21 Jul 2022 09:49:40 -0400 Received: from mail-lj1-x22a.google.com (mail-lj1-x22a.google.com [IPv6:2a00:1450:4864:20::22a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E19036313 for ; Thu, 21 Jul 2022 06:49:38 -0700 (PDT) Received: by mail-lj1-x22a.google.com with SMTP id j26so1911025lji.1 for ; Thu, 21 Jul 2022 06:49:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=Df5PaXBQvWi3ElHoTsEdWiLtMQ6HsSl/JnMKyaUxUZw=; b=u/jvunMnyd49xDidrkd2IjPWAeumQ8e8y/ZLGpFdZyEHxSJEMo2CR8SUicSbdC74NE JMb7LSsarT5VL72kvSyrJo9Nv5G9/KfHABcXTLUO7U6qIMBfWsJa4I91DCLXglTvcafB fNp40vmMAN5kmvqf1PzUVuQHws+ep/jtnM/oswkgfBbOXM8pihXM81a9zOatRFYovNkp nxkqdts7DyqcEMvNwIk6EtaHnV40krQtd8xHHxqt+HKytqXQsTa1soVCwJXesGyXdmKW /p/4P4PvZ/Qc4YH2R0uSC6ECgSR1yStB8saq+ayFfYAKxkBsFySs77mV6/ouRJjO/aSs NCyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=Df5PaXBQvWi3ElHoTsEdWiLtMQ6HsSl/JnMKyaUxUZw=; b=b0QUBDfpD8bkkt6Fmu4BlO0lZaA445YKyW86sSmKhuYPqnhDTxMEO/1X5kQikQjWnC eJzQOe2nUrnGjMK8UXRePUhCtK5G7YvJkjGH24q8d3iIRaTPKMBf3cXgZk/ybKx9JZdn U4UuA9cfsllVwmnIDQxr/f1r2de6H6hAjKb8L8o9koIpRciRdHOHMFZDTOBe9YLekPOy mLVvJp5AG1AP3KTHRMBDqi1254/FesaVO7bioQi849OrgMv571G7S2eeQYmIF7dBcL0o CE9q4FsI5Tcahi3yX9xTHHnBPPNqZ9uUBndC37tDjf4bbSYIo5dQX2qHWPcUy3V9LTHZ Hfyw== X-Gm-Message-State: AJIora94CD6w5BlWq1OVp7gbAxj7u0MUZGKalptNEk+nlFmHo+zX//Tp Wec+ZkL/6Drhxe/tdzYQKFB3Xw== X-Google-Smtp-Source: AGRyM1thiRH/aj+PE2hEZO3seebses8tiG7Wdmeu6aB+i44xAKChmd5OawfeBgmZpsvVxmmTUcXReg== X-Received: by 2002:a2e:a591:0:b0:25d:7366:3344 with SMTP id m17-20020a2ea591000000b0025d73663344mr19932544ljp.242.1658411376877; Thu, 21 Jul 2022 06:49:36 -0700 (PDT) Received: from [192.168.115.193] (89-162-31-138.fiber.signal.no. [89.162.31.138]) by smtp.gmail.com with ESMTPSA id a8-20020a2eb548000000b0025a6e47056csm518302ljn.124.2022.07.21.06.49.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 21 Jul 2022 06:49:36 -0700 (PDT) Message-ID: Date: Thu, 21 Jul 2022 15:49:34 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH V3 3/7] dt-bindings: soc: imx: add i.MX8MP vpu blk ctrl Content-Language: en-US To: Peng Fan , "Peng Fan (OSS)" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "l.stach@pengutronix.de" Cc: "kernel@pengutronix.de" , "festevam@gmail.com" , dl-linux-imx , "laurent.pinchart@ideasonboard.com" , "marex@denx.de" , "paul.elder@ideasonboard.com" , "aford173@gmail.com" , "Markus.Niebel@ew.tq-group.com" , "alexander.stein@ew.tq-group.com" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Aisheng Dong References: <20220719055054.3855979-1-peng.fan@oss.nxp.com> <20220719055054.3855979-4-peng.fan@oss.nxp.com> <27ab24ec-61c7-cafb-6665-a16c58808649@linaro.org> <9c1735bf-3f29-4bf1-f7cf-f9e211c505ac@linaro.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 21/07/2022 12:38, Peng Fan wrote: >> Subject: Re: [PATCH V3 3/7] dt-bindings: soc: imx: add i.MX8MP vpu blk ctrl >> >> On 19/07/2022 11:58, Peng Fan wrote: >>>>> + interconnects: >>>>> + items: >>>>> + - description: G1 decoder interconnect >>>>> + - description: G2 decoder interconnect >>>>> + - description: VC8000E encoder interconnect >>>>> + >>>>> + interconnect-names: >>>>> + items: >>>>> + - const: g1 >>>>> + - const: g2 >>>>> + - const: vc8000e >>>> >>>> Include interconnects+names in list of all properties and disallow >>>> them for other variants. >>> >>> I not understand well about >>> " Include interconnects+names in list of all properties ", could you >>> please explain a bit more? >>> >>> And there is already an "if" to check whether the compatible contains >>> " fsl,imx8mp-vpu-blk-ctrl" to make sure the interconnect valid for >>> i.MX8MP, so it is not valid to other variants. >> >> Defining properties in some if: clause is not really readable and >> maintainable. The properties should be defined in top-level properties:. > > Maybe I should also include i.MX8MM interconnect and make it > a separate patch, then i.MX8MP/i.MX8MM just use their own > names just like other properities. In final, as below: > > Tow patch: > 1. Add interconnect property for i.MX8MM VPU BLK CTRL > 2. Add i.MX8MP VPU BLK CTRL dt bindings > > In top-level: > + interconnects: > + maxItems: 3 > > + interconnect-names: > + maxItems: 3 > > For i.MX8MM > + interconnects: > + items: > + - description: G1 decoder interconnect > + - description: G2 decoder interconnect > + - description: H1 encoder interconnect > + > + interconnect-names: > + items: > + - const: g1 > + - const: g2 > + - const: h1 > > For i.MX8MP: > + interconnects: > + items: > + - description: G1 decoder interconnect > + - description: G2 decoder interconnect > + - description: VC8000E encoder interconnect > + > + interconnect-names: > + items: > + - const: g1 > + - const: g2 > + - const: vc8000e > > How do you think? I don't understand what is the question here to me. Shall I tell you what to do or not to do with iMX 8MM? I don't know. I am not a maintainer of that SoC. Best regards, Krzysztof From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 31391CCA479 for ; Thu, 21 Jul 2022 13:57:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=gz71cmqkYE+Y6Ktlkmtm8cGLrlxy2+npL0CVj4L23OM=; b=zP8BoJ/+OZtcFn IenQXsRUXRXkEYSzcLBsTTtIcvRCqM9QFxLY07qSTtibIqGR/ZaRvCKGkBo5ng04sZva3O+Kzem+I qFyLvsw88E+8ACxow/y4ZiQ19T/C8mE7+2KdL+kjIRD/4nC9Dn41WsDzSDUtgMyUCddMUY+/VgM7t kw7ibiEfOS4L546Kt9iYZ8LG4ZXv+9zRzEf7UnDL+byZE4VeRuFpBEgw39xRneP+DeuR87+zaUjzC hXnU/YnYN1qbljv+07gP/KbPAceRELHkMnmfZx42mfjKAJzlLwMxZg3E5s+X0QynRnGqZ9z74mMyc 34DqHhuNuYg5v3sXO47A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oEWey-007M9P-Sg; Thu, 21 Jul 2022 13:56:13 +0000 Received: from mail-lj1-x231.google.com ([2a00:1450:4864:20::231]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oEWYe-007GYx-02 for linux-arm-kernel@lists.infradead.org; Thu, 21 Jul 2022 13:49:41 +0000 Received: by mail-lj1-x231.google.com with SMTP id u14so1927473lju.0 for ; Thu, 21 Jul 2022 06:49:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=Df5PaXBQvWi3ElHoTsEdWiLtMQ6HsSl/JnMKyaUxUZw=; b=u/jvunMnyd49xDidrkd2IjPWAeumQ8e8y/ZLGpFdZyEHxSJEMo2CR8SUicSbdC74NE JMb7LSsarT5VL72kvSyrJo9Nv5G9/KfHABcXTLUO7U6qIMBfWsJa4I91DCLXglTvcafB fNp40vmMAN5kmvqf1PzUVuQHws+ep/jtnM/oswkgfBbOXM8pihXM81a9zOatRFYovNkp nxkqdts7DyqcEMvNwIk6EtaHnV40krQtd8xHHxqt+HKytqXQsTa1soVCwJXesGyXdmKW /p/4P4PvZ/Qc4YH2R0uSC6ECgSR1yStB8saq+ayFfYAKxkBsFySs77mV6/ouRJjO/aSs NCyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=Df5PaXBQvWi3ElHoTsEdWiLtMQ6HsSl/JnMKyaUxUZw=; b=74brkZprM18LBj1VCYjP74PRS1ibcDiuoE5MbOdiyDnNO1NYPYnt22mBd2MRXSyzR1 Uz6aTZgwvPT4lNVRRwwdC4rflTzSkFX09p1gSoXBmly3CWswYapW22ePQDeA+YRUZ/CU MqrApUJPFWoKaT/MBsNDz4Sbbd9oO6wERxKynT0Fn0S6J3/JTngKhM46+CUegfoaPIan 2qc/8yE3i0AQZV4043M1DRDbQ+MQFymq3n9TQjZd4oIeWYdEJJdt6B3Al2V1US9+Z90f otktyAjS1mfh4tAJ04HZaelTbLoQkpwSgPgNJAeT7c8qhzxIYLZuf78InOAYyQ3Wnj57 prNg== X-Gm-Message-State: AJIora9SOvLnwjKfci/5nwRbII2C0L4tiKiz4v+gDf7OeHG8XopcZFE5 co4sID9Z6bywm+2Sxn4P0IKckg== X-Google-Smtp-Source: AGRyM1thiRH/aj+PE2hEZO3seebses8tiG7Wdmeu6aB+i44xAKChmd5OawfeBgmZpsvVxmmTUcXReg== X-Received: by 2002:a2e:a591:0:b0:25d:7366:3344 with SMTP id m17-20020a2ea591000000b0025d73663344mr19932544ljp.242.1658411376877; Thu, 21 Jul 2022 06:49:36 -0700 (PDT) Received: from [192.168.115.193] (89-162-31-138.fiber.signal.no. [89.162.31.138]) by smtp.gmail.com with ESMTPSA id a8-20020a2eb548000000b0025a6e47056csm518302ljn.124.2022.07.21.06.49.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 21 Jul 2022 06:49:36 -0700 (PDT) Message-ID: Date: Thu, 21 Jul 2022 15:49:34 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH V3 3/7] dt-bindings: soc: imx: add i.MX8MP vpu blk ctrl Content-Language: en-US To: Peng Fan , "Peng Fan (OSS)" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "l.stach@pengutronix.de" Cc: "kernel@pengutronix.de" , "festevam@gmail.com" , dl-linux-imx , "laurent.pinchart@ideasonboard.com" , "marex@denx.de" , "paul.elder@ideasonboard.com" , "aford173@gmail.com" , "Markus.Niebel@ew.tq-group.com" , "alexander.stein@ew.tq-group.com" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Aisheng Dong References: <20220719055054.3855979-1-peng.fan@oss.nxp.com> <20220719055054.3855979-4-peng.fan@oss.nxp.com> <27ab24ec-61c7-cafb-6665-a16c58808649@linaro.org> <9c1735bf-3f29-4bf1-f7cf-f9e211c505ac@linaro.org> From: Krzysztof Kozlowski In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220721_064940_101531_2DE7C1F9 X-CRM114-Status: GOOD ( 15.68 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 21/07/2022 12:38, Peng Fan wrote: >> Subject: Re: [PATCH V3 3/7] dt-bindings: soc: imx: add i.MX8MP vpu blk ctrl >> >> On 19/07/2022 11:58, Peng Fan wrote: >>>>> + interconnects: >>>>> + items: >>>>> + - description: G1 decoder interconnect >>>>> + - description: G2 decoder interconnect >>>>> + - description: VC8000E encoder interconnect >>>>> + >>>>> + interconnect-names: >>>>> + items: >>>>> + - const: g1 >>>>> + - const: g2 >>>>> + - const: vc8000e >>>> >>>> Include interconnects+names in list of all properties and disallow >>>> them for other variants. >>> >>> I not understand well about >>> " Include interconnects+names in list of all properties ", could you >>> please explain a bit more? >>> >>> And there is already an "if" to check whether the compatible contains >>> " fsl,imx8mp-vpu-blk-ctrl" to make sure the interconnect valid for >>> i.MX8MP, so it is not valid to other variants. >> >> Defining properties in some if: clause is not really readable and >> maintainable. The properties should be defined in top-level properties:. > > Maybe I should also include i.MX8MM interconnect and make it > a separate patch, then i.MX8MP/i.MX8MM just use their own > names just like other properities. In final, as below: > > Tow patch: > 1. Add interconnect property for i.MX8MM VPU BLK CTRL > 2. Add i.MX8MP VPU BLK CTRL dt bindings > > In top-level: > + interconnects: > + maxItems: 3 > > + interconnect-names: > + maxItems: 3 > > For i.MX8MM > + interconnects: > + items: > + - description: G1 decoder interconnect > + - description: G2 decoder interconnect > + - description: H1 encoder interconnect > + > + interconnect-names: > + items: > + - const: g1 > + - const: g2 > + - const: h1 > > For i.MX8MP: > + interconnects: > + items: > + - description: G1 decoder interconnect > + - description: G2 decoder interconnect > + - description: VC8000E encoder interconnect > + > + interconnect-names: > + items: > + - const: g1 > + - const: g2 > + - const: vc8000e > > How do you think? I don't understand what is the question here to me. Shall I tell you what to do or not to do with iMX 8MM? I don't know. I am not a maintainer of that SoC. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel