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 779BAC4725D for ; Fri, 19 Jan 2024 08:41:51 +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:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=CnjqttjPZboxzqiaNCvAUDVEBS2BsJKIWPTkRu1ZbDA=; b=i5mdH2ovlP5iRY LGPCo657KbWwFbs8yr/DibTPL8Ic2ewqQ+SGg/CSrG3cQakrNuisNgAO193Sey5xMFmfRDC0haIHK I4IGxMZUWUn6jPuCxgX8eB/Yf8+jKsa3zsn8EOVc2Yn58tBt1fIynAqgzJrzPvmaWzI3yfabxBFS3 3f1etZzCqMUKGt/4Pk0bqztVpe7ghIQ4wPob63Fni98W8dzyLRzSkrA4zbHJaefBHbfZhqS15FPTy bTyUdteEkQgLOqGi+dUwSozo7nKd8G1BNswS3kQPtdH8HaKlulqQlsJeNyl52vrI5ryxm7HO/hrOz 1Vk7QKtcuAHwi+v6jzYQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rQkRF-004ros-1V; Fri, 19 Jan 2024 08:41:21 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rQkRB-004rn2-1w for linux-arm-kernel@lists.infradead.org; Fri, 19 Jan 2024 08:41:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1705653677; x=1737189677; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=IcPAsYyb8agVNnqTKryYS5Rg90OgTBUYYVzfBKNm/5E=; b=OoPFgN0AGDZMQ/ePLY113Z0tmCFH40ZexECXE6EXZdBMX48HIDY4eiby xoLlyFhaEHBEWeCdtbXil5P++Uiia4h/3UIpqAaAnPHJxhklIvT3qGher VV9QMROlVdWatPJeotThRECbAQrO5lYQe65Di6Q2DE/N0rl/nnjcNVG7M FhaDROj9kvEM7yiwBVRwsfaUOmCPSTtHJtzr30DYTJwUg78dFVD1CWO7d 7QIGF6VN6haoAsiT+w80nm61kukcJJKEMOLAVb1OuNqtBmHyLujTR5OWQ 6CrtdIy3hbvUz37LQUbTbfkXgCtLmWhtTOLQKt6VQf5iFCOXf2Nt/G1hB A==; X-CSE-ConnectionGUID: RtxeGdiTQi2EA5IUfTpDHA== X-CSE-MsgGUID: YefU/4fSQo+lq3HZihisNA== X-IronPort-AV: E=Sophos;i="6.05,204,1701154800"; d="scan'208";a="182227076" X-Amp-Result: SKIPPED(no attachment in message) Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa6.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 19 Jan 2024 01:41:11 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Fri, 19 Jan 2024 01:41:10 -0700 Received: from NAM04-BN8-obe.outbound.protection.outlook.com (10.10.215.250) by email.microchip.com (10.10.87.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Fri, 19 Jan 2024 01:41:10 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KUtZOLHTalZHl2+REqN8+v8STYXvNxYrApc8VFVVXvuJdSkjRtQ6x9DE/O/NwYwBEpu6fqAlx9UHpYo9RQfsDM824TANRv3hu57o3P3st/D9rhyWWJmH8VCJEDIrq6Xi5NtCmcYu7Y9PAwFkfECIX72ijuHIenS7Ql+6EPB5hEk/k5CnMRN8n6N8NJGG9V1X2BO/L4ohcUFFUPQSmXm5CmqZDNf9a7R4HDNXpsBFt+P1E2CxEzPhew6pJwegoUYVWDQckoQCzZ9w1axuuQcKOUrvQeLz48IL4q1Jus4TyJZSG0HnjAwqR6jUjxaUvGjWxU7Hg8Z/TBIOLJqnrwyf8w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=IcPAsYyb8agVNnqTKryYS5Rg90OgTBUYYVzfBKNm/5E=; b=K8o9vEbM9L+enTUUxtXGbzs32eBcigf/oF4iHq9QL96SF+XOEHA2SB4DG0xVa1OVJEYkqmaC8VKG3giEHcGPJtdK+5w3LNJnUyFHdvfPKJlgwiTCaeumdg31dSporNalUHONsm5kR1vpQZziLmY0d43CpaDWxIKXNZN/BKWVnyjwxyg6If3OeMGxU32TC152QgSkRiP5LsiVxM2sFqqHzYEsk/ZbqB8WvwqGFqhTol5sLM0iGPjBvqyZMAKBRzQ2xnEjqQeaqebMenNYh+ACDgJLRmmf9Lz05OGCUP1g3C+drUK/U5LkXoO3QUL1YAveB2Sez6VHeSl8ZAN/TkREXQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchip.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=IcPAsYyb8agVNnqTKryYS5Rg90OgTBUYYVzfBKNm/5E=; b=RiFIXUTA3T2Bjg1S83g6dtEznbdQ+W635mGN/2RDzmDdABOKb2r54O77aA4hlgj0l9WU+TC6hWeqI20w9cEKVybI9c7hVRyTfi8ftquK3QVuazKO0GV7fi6V3Zo867Cjp9sTJQKkjOqNPClq//6NfrxOwiNseMybCPOokLKUvRdGasUS3LgTbOgQotpLup34ocQKK46FK1FPNMedURtlP59gzTMr57ZJFfFGYRJ4hYWifcnq30OJL5SIPPlF4K3mzqCyFRNTGpfUALmF7rDuB9SwvulDJJzvsXFWEM5UoWys8LiQxzjYHkT0jLMsBKEosYNofMqA8kEoSm1ark5Xiw== Received: from PH7PR11MB6451.namprd11.prod.outlook.com (2603:10b6:510:1f4::16) by PH7PR11MB7027.namprd11.prod.outlook.com (2603:10b6:510:20a::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7202.23; Fri, 19 Jan 2024 08:41:04 +0000 Received: from PH7PR11MB6451.namprd11.prod.outlook.com ([fe80::80b9:80a3:e88a:57ee]) by PH7PR11MB6451.namprd11.prod.outlook.com ([fe80::80b9:80a3:e88a:57ee%3]) with mapi id 15.20.7202.020; Fri, 19 Jan 2024 08:41:04 +0000 From: To: Subject: Re: [PATCH v3 0/3] Convert Microchip's HLCDC Text based DT bindings to JSON schema Thread-Topic: [PATCH v3 0/3] Convert Microchip's HLCDC Text based DT bindings to JSON schema Thread-Index: AQHaSfBtFpABnI0MwEGuolZ8EVFW4bDf9cIAgADc1AA= Date: Fri, 19 Jan 2024 08:41:04 +0000 Message-ID: References: <20240118092612.117491-1-dharma.b@microchip.com> <20240118193040.GA223383@ravnborg.org> In-Reply-To: <20240118193040.GA223383@ravnborg.org> Accept-Language: en-GB, en-US Content-Language: en-GB X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PH7PR11MB6451:EE_|PH7PR11MB7027:EE_ x-ms-office365-filtering-correlation-id: 2aba715f-7188-4e98-a2e8-08dc18ca5f7a x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: ZrDWEFsJyDlBoFw0A/iADArXWpGeAQ/qwsDcFF34W7zrlK9lSf3MpcxEUbGAKOJBJMLaf0Nhb2XoRlJJ+PniZUzv/aFSyQNVC/64ei2e1y/yTib58H+/7ywuaniRWbcP6LgVuxCYWYRcJ3hyzx8b+7Sko0uY8gC8kx/lS/4ZKNKmKQVqpBn9rQy5GOv8iiprJZ8NIZWMIaYkKbsWrYIfj6DDuF7lfPHUHVrbMuPLmIcms84JL1ksmIFkJ1oZn3oFEzj8vLuUDkAcvhvgHN+8T1fgiO6Owo//VSeFCdTIAl/14pUJlY/9H6tz1CXBsFzRecnts80iJtLO5cLG+I/khfX5qkx2ZM3+6VMHgyIvCbR/apmzZU91t61gVU5pCx/eL6Oa/M3AtqyVZtttk0pmyXtbhEtl+dIOEmig7fEMiT2Hf3oK8eZ35driMj6iE2HyEb8nXuhM4DtxJ1TpHO0IX8A6fi126YiZ0THE0Cn8zfMjxbtdd/JC/Gt4O74WZMQsHPde4FtkRzKNkboX8wVMQJ4/mJROJKIZKtyPv/4xf3fBLQDGKN6xp8RNsu01lvenQGJMj+rjwQ9cqFzoUarzN9bmqr8t4+OWv94iAxjp2wIR/qraHeWKLpJSvELQ22MAxfyNgyLleom1QlY+8yv2MD7Y5RcW0F0vG0Rb6jloJd0uA7nzPcWJ8A/iQUtt+qZwtTzEU+hEIB0xZXof0E+QUTcnwNIfRpSpXicEEY6Twa0= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH7PR11MB6451.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(396003)(136003)(366004)(376002)(346002)(39860400002)(230273577357003)(230473577357003)(230373577357003)(230173577357003)(230922051799003)(1800799012)(64100799003)(186009)(451199024)(54906003)(5660300002)(2906002)(7416002)(31686004)(4326008)(8676002)(8936002)(316002)(66946007)(91956017)(6916009)(66446008)(64756008)(66556008)(66476007)(76116006)(478600001)(2616005)(6506007)(6486002)(71200400001)(966005)(6512007)(107886003)(53546011)(83380400001)(122000001)(26005)(41300700001)(38070700009)(38100700002)(36756003)(31696002)(86362001)(45980500001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?eUF0d0VIK2ZXakNHd1BUcU1oRjFBY2dKRmFYWmF4enJxL0hyWTcvZXcwRGcz?= =?utf-8?B?UlNoL0FtUnFmL3VTNTVURlJzYjg0NlFpVExvcDZKR25MMHU4QUEwdjVydVVS?= =?utf-8?B?V2FaaHdrNWZXaW45ZllUWTdjTy9xdnpQWlR6bHNjWGdXK211Z3Y5L1RZY09h?= =?utf-8?B?cG55MjRmQnZqRTZtTWluK0p4blpKT0YvKzhPQUZjaGJHWVMxMXlTR0JTdDhO?= =?utf-8?B?cWtLQnJPWjdOYi9RRHl0UjIzSG94UzN5VGU1OTg0N1F6WmhYT01MOUdQTmNS?= =?utf-8?B?MG9LUWlqZ0lHZDhYOEdwZHgwV2s0eEx3eEI0MGNMNnoxUzV4OU52TmYwV0Ju?= =?utf-8?B?d3NhRnpDa2RMTDlSRWx1L244QWI5cTlqN3VVWE8wS1YwMUNuNzI5emVuVGI2?= =?utf-8?B?ckViblZvYWhaV2Fia0hmVXRtM1NrVVpFODFieUNWZWNWeUpqY1JIditkeHMr?= =?utf-8?B?L1FvTWluVjBGd0QrbXRjTEJqUHJkNW9EQU9pZFJVWGIxU21WdW5pamFWcE1J?= =?utf-8?B?RC9EK0lVVTltWW9vSUZhQklVUitoVzkreHdMU0lCOXhKb1Nwek9QNk1iNzJK?= =?utf-8?B?dFN3djlYNHN5eWZNNXJsZmFRLzBTQktJQUFOWkNNVGUxMTByajhRS1pCdTRI?= =?utf-8?B?SG5GV01tWlRWVmJhYU1hWjhzV2pOM2NpdDI4WSsyOW44eG5pL1AxeWpqUzZv?= =?utf-8?B?N2lVVjlkSStJU2t1d29IVHVMTkV0c1Y3N0NYWW1vSVJMcFZhakdtenJnUXQ4?= =?utf-8?B?RHZlR1dRNWViZjdMcno2YnBiWWt2WWszbXlJUnlTaGR5c1FWeEpmZ21jRTRL?= =?utf-8?B?NXVYbFRENWhiWS81MFZVUStoZGJEc28rMGVqd2J1QTFieFlvekl1RnFSU3Vp?= =?utf-8?B?a1dJODlVR3BBUEptUzVuZUxQQXVZbjJsdUFyOUtzS2VVYW1mZUJRcWxOeTdr?= =?utf-8?B?UGw4aEJIS0g1bXovcnd2aHkxZUlpMUZQWXRaTm9jdlJESTFNL0Fya0hDMUpr?= =?utf-8?B?b1ZCMWNRSDVkdzA0QXpyK0o5MFp1RmxUdHkzaFRJcThqY29Sd2FrU3RmN0JX?= =?utf-8?B?S2NLaFNjRVh6aGdiRlVOeXJyRDFMYWowUTlaYUVEaU9SdTRsL3NCNGJES3px?= =?utf-8?B?d2tjVGpnWjFCdjRQWU1OWEhUK2R5dGZZTWJUaG1jZ1RWbk8vMmY1R1NUSzJu?= =?utf-8?B?ZlU1cHpkTURoTTdiZjRrSXYwU2NadElZV0J0L0NVU0N5Sk00VXZyRmdIUWJ1?= =?utf-8?B?TGdXd0tNdkFxNEE4bnprVUhKUUZHVFRQdGxiTTJ0cFY0L2UyQnB4NEpnL0Rp?= =?utf-8?B?bkRBOU0zL2Q2bHpjRnVaWUdOOTNLSE5taVFKb2wzUHZYaFBDTUZma0JzbVlD?= =?utf-8?B?T1kwaXFIOWNmUGRFbWdyU25ISHc0M1pOeWNsOXp4cFAxYkg4YjVwN2VEd0p0?= =?utf-8?B?S0xtRGxtM0NmcVJZMk42ODhNZHROejkyTUZVcHdsVU9xblZDYnc4Q1RJdU8r?= =?utf-8?B?ODY5bHRIQW9ZRFFjU04xMkpMNG1jay9xbExkd1dhcGxrQkRicGtnYkp0MU5w?= =?utf-8?B?cDRmbnBPOFg3NGx4Q3RrSDVzQm4rVlh2RGx3ZlZZem5RT0dTMTlFVXB2aGdR?= =?utf-8?B?R3Bra1VTN0NvUDk2cnU3emNLUFVrVzRMVk1QcXk5a2t1bis0Sm8zUkZLdVMy?= =?utf-8?B?TGJlazFES0VLS0lSS05Tc2xVM0R3SzBHZXd4bHVoeWRpQ3p1TFBkR0lKU1hK?= =?utf-8?B?Sm5LWndnYzJMNExQMVJqNGFnOUttS3JWQk9NcTFpR1hqQ3VJaXJmalJDN3gv?= =?utf-8?B?bnl0RU5Kcmo2MzhVdkp2eDB1RXhNY01TdWYzN1VLN1Q3K0xoR24ydjhmaWoy?= =?utf-8?B?eXQvTGNqQ2w4bEJFSGN6MjhiV3YvUHFFNlo3MGczbWVGMm1HdnBNblZBaGJN?= =?utf-8?B?blE1WE5jNnN3cjg1M2dnWTd2eVUwWU1QZHRScWdDUmtVNGlsNnlkNkR4L0x5?= =?utf-8?B?YTM2Zm1ZZDNCUS8xa0F3cVlhMzBNRVYrdTZQUEVBREFNc21Wd3grb0hNckdh?= =?utf-8?B?MVVmTGY4bVMvbEMzOUpmaFhXUlduSGp4Umh3NnNjNldtR0FGTHFmSFJxbW53?= =?utf-8?Q?l1HxuJllF3sAf0QMTOrocfXAC?= Content-ID: <1E92E73D5A3BF04B9FE8933EE12CE74E@namprd11.prod.outlook.com> MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH7PR11MB6451.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2aba715f-7188-4e98-a2e8-08dc18ca5f7a X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Jan 2024 08:41:04.4023 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: vl56lj2kESTWdB/vF/fJ8qKqYHJkpp2AtP4qoEvzCzPN80fw6xQKuASIwIFg2iiRL5jF1s0sOZogB+L90EWCWg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7027 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240119_004117_740621_6D0C0F0B X-CRM114-Status: GOOD ( 28.30 ) 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: , Cc: Linux4Microchip@microchip.com, linux-pwm@vger.kernel.org, alexandre.belloni@bootlin.com, dri-devel@lists.freedesktop.org, Conor.Dooley@microchip.com, thierry.reding@gmail.com, krzysztof.kozlowski+dt@linaro.org, claudiu.beznea@tuxon.dev, airlied@gmail.com, lee@kernel.org, u.kleine-koenig@pengutronix.de, devicetree@vger.kernel.org, conor+dt@kernel.org, tzimmermann@suse.de, maarten.lankhorst@linux.intel.com, mripard@kernel.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org, bbrezillon@kernel.org, linux-kernel@vger.kernel.org, daniel@ffwll.ch 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 Hi Sam, On 19/01/24 1:00 am, Sam Ravnborg wrote: > [You don't often get email from sam@ravnborg.org. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > Hi Dharma et al. > > On Thu, Jan 18, 2024 at 02:56:09PM +0530, Dharma Balasubiramani wrote: >> Converted the text bindings to YAML and validated them individually using following commands >> >> $ make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/ >> $ make dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/ >> >> changelogs are available in respective patches. >> >> Dharma Balasubiramani (3): >> dt-bindings: display: convert Atmel's HLCDC to DT schema >> dt-bindings: atmel,hlcdc: convert pwm bindings to json-schema >> dt-bindings: mfd: atmel,hlcdc: Convert to DT schema format > > I know this is a bit late to ask - sorry in advance. > > The binding describes the single IP block as a multi functional device, > but it is a single IP block that includes the display controller and a > simple pwm that can be used for contrast or backlight. yes. > > If we ignore the fact that the current drivers for hlcdc uses an mfd > abstraction, is this then the optimal way to describe the HW? > > > In one of my stale git tree I converted atmel lcdc to DT, and here Are you referring the "bindings/display/atmel,lcdc.txt"? > I used: > > + "#pwm-cells": > + description: > + This PWM chip use the default 3 cells bindings > + defined in ../../pwm/pwm.yaml. > + const: 3 > + > + clocks: > + maxItems: 2 > + > + clock-names: > + maxItems: 2 > + items: > + - const: lcdc_clk > + - const: hclk > > This proved to be a simple way to describe the HW. > > To make the DT binding backward compatible you likely need to add a few > compatible that otherwise would have been left out - but that should do > the trick. again you mean the compatibles from atmel,lcdc binding? > > The current atmel hlcdc driver that is split in three is IMO an > over-engineering, and the driver could benefit merging it all in one. > And the binding should not prevent this. could you please confirm if my understanding is correct: you want a unified display binding that encompasses the properties of the two subdevices (display controller and pwm), eliminating the need to reference them in additional bindings? > > Sam -- With Best Regards, Dharma B. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel