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 X-Spam-Level: X-Spam-Status: No, score=-6.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B3E20C433E2 for ; Tue, 1 Sep 2020 06:44:26 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6F7D42098B for ; Tue, 1 Sep 2020 06:44:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="a5vBf/wM"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=bidouilliste.com header.i=@bidouilliste.com header.b="YrKuyRgS" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6F7D42098B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=bidouilliste.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:Mime-Version:References:In-Reply-To:Message-Id: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=30J1PRIwU6n1muFxpDI8WxQF6t1smqkPUr+WZVktK68=; b=a5vBf/wMNHM470s6CGB1MHmZQ rZ2bKyXTqkKCcw2zSNStR4kLpaWjITm9NdtG7uQQp2YsTTdcMGMY+AZjw4yvezRR/rzzru5yxyVF0 hI2uJFdxp9SKHVzmTluFFc11lTdrRV2WhLnEJpSZaplK5qKxqYuNgJNjNmaDEsb6RvZfJZnkjxm5H qxW0sQZ6rd+d4FO3w79Uf+faMHc9J+lLUsUAz2gNHyrkl38mTGbu817pyzraGALPPKJ9BamB/tEiB EXo5dywarg/fR+3z/vrLXKF25ZevSyc7Vjam3vr7hINqVrind+F18GCQwvqZLbNzbWXulsXC2AGLj FvXHUcpEg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kD00P-0006Df-Fu; Tue, 01 Sep 2020 06:42:57 +0000 Received: from mx.blih.net ([212.83.155.74]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kD00L-0006Cs-NZ for linux-arm-kernel@lists.infradead.org; Tue, 01 Sep 2020 06:42:54 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bidouilliste.com; s=mx; t=1598942569; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=nZvkdgfEsx8IUABfhx71Dpoxee70fmEUTqE8AA7jJGE=; b=YrKuyRgS7QNfdF7qc4hPTROgfFajA7Fk9Ph+iwvL4aPMHfAkWgKP6nOnmArJ1pn+u1TWq2 Cd91ByVr7E2rDIilyqYa/3Zyd/93ANKbFJxeHCP+m8YDm64qxE65dqaI+MukDy6JM7NyRu rCmQsqbvJSUHjiMTKOj+S6geJCxe3A0= Received: from amy.home (lfbn-idf2-1-1138-237.w90-92.abo.wanadoo.fr [90.92.20.237]) by mx.blih.net (OpenSMTPD) with ESMTPSA id 8cb32919 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Tue, 1 Sep 2020 06:42:49 +0000 (UTC) Date: Tue, 1 Sep 2020 08:42:49 +0200 From: Emmanuel Vadot To: =?ISO-8859-1?Q?Andr=E9?= Przywara Subject: Re: [PATCH v2 0/6] dt-bindings: Convert SP804 to Json-schema (and fix users) Message-Id: <20200901084249.b72bb69d617cc32733432da1@bidouilliste.com> In-Reply-To: <28acf821-fa6f-7259-ec1b-4810ca1ff48f@arm.com> References: <20200828142018.43298-1-andre.przywara@arm.com> <28acf821-fa6f-7259-ec1b-4810ca1ff48f@arm.com> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.32; amd64-portbld-freebsd13.0) Mime-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200901_024254_048724_46FB305C X-CRM114-Status: GOOD ( 32.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Lorenzo Pieralisi , Scott Branden , Ray Jui , Linus Walleij , Daniel Lezcano , Liviu Dudau , "linux-kernel@vger.kernel.org" , Wei Xu , Rob Herring , Haojian Zhuang , Sudeep Holla , Florian Fainelli , Thomas Gleixner , Chanho Min , Linux ARM Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 28 Aug 2020 16:44:28 +0100 Andr=E9 Przywara wrote: > On 28/08/2020 15:54, Linus Walleij wrote: > = > Hi, > = > > On Fri, Aug 28, 2020 at 4:20 PM Andre Przywara = wrote: > > = > >> This is the second attempt at converting the SP804 timer binding to ya= ml. > >> Compared to v1, I forbid additional properties, and included the prime= cell > >> binding. Also the clock-names property is now listed, although without > >> further requirements on the names. Changelog below. > > = > > The series: > > Acked-by: Linus Walleij > > = > >> I couldn't test any of those DT files on actual machines, but tried > >> to make the changes in a way that would be transparent to at least the > >> Linux driver. The only other SP804 DT user I could find is FreeBSD, > >> but they seem to use a different binding (no clocks, but a > >> clock-frequency property). > > = > > That's annoying. I suppose FreeBSD just made that up and doesn't > > even have a binding document for it? > = > I couldn't find bindings at all in their git tree. That's because I don't merge the bindings in the main branch. > I don't think they > treat this very formally, it seems to be more use-case driven. > Their SP804 driver does not know how to handle clock properties, so most > of the DTs (in sys/gnu/dts, so apparently copied from Linux) would not > work really well, because the driver assumes a hardcoded frequency of > 1MHz by default. In addition to sys/gnu/dts we also have sys/dts/ which are our own DTs before we used the Linux ones (a long time ago but some platform weren't converted, they will just die sometime in the futur if nobody takes care of them I guess). > There is only one DT (Annapurna Alpine with Cortex-A15) that provides > this clock-frequency property. The Linux DT does not mention the SP804 > in there at all, interestingly. I'm not familiar with this platform at all, it was done under contract by Semihalf and I'm sure that if something fails and their client starts to complain they will fix it. > > In an ideal world I suppose we should go and fix FreeBSD but I have > > no idea how easy or hard that is. > = > It seems to be messy, at least in this case, and I guess unifying DTs > means some work on drivers as well. I wouldn't worry about us on this case, this binding requirements seems to have be done a long time ago before we had any clock framework and if our drivers needs to be updated we will do it when we imports DTS from whatever Linux version this will be merged in. > But AFAIK most of the more modern platforms copy the DTs (and thus > implicitly the bindings) from Linux, so there is probably much less > deviation for many more relevant boards. Yes, I (and others) insist on using the DTs from Linux and not doing any patches to it without sending them to the Linux ML. > Cheers, > Andre Cheers, > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- = Emmanuel Vadot _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel