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 9296BCA0FEF for ; Fri, 1 Sep 2023 18:39:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239306AbjIASjb (ORCPT ); Fri, 1 Sep 2023 14:39:31 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53324 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237600AbjIASja (ORCPT ); Fri, 1 Sep 2023 14:39:30 -0400 Received: from mail-wm1-x332.google.com (mail-wm1-x332.google.com [IPv6:2a00:1450:4864:20::332]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B5156107 for ; Fri, 1 Sep 2023 11:39:27 -0700 (PDT) Received: by mail-wm1-x332.google.com with SMTP id 5b1f17b1804b1-3fee8af9cb9so22974695e9.1 for ; Fri, 01 Sep 2023 11:39:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=jrtc27.com; s=gmail.jrtc27.user; t=1693593566; x=1694198366; darn=vger.kernel.org; h=to:references:message-id:content-transfer-encoding:cc:date :in-reply-to:from:subject:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=bA5Lm6a1t+KFAs7vnfL5/17M6lm4KSTTxqyCI/pc5BY=; b=ZiqUuk9p4uZ/yOZAr5a+RcMr0ew02/QrP2SaKXTOtNRKAhMTGpo/XtFQGJu0Omhu1K qhpkoPCaQFESaNA6OeK3whxRVLW/NkzbKlzwIo+kCnF8K+4ckSIjxaA5NeQyi9+e5nPo ShDY1K5wYQudm+SXQKDcI2MUete8t0pGLXXpxfy70IeHaXjTbeidkHbDBM5wCnpDrnRV 240RgKdpMmDvmkJsWumT31Sm/IK7KK6lbeuPndS69jE4WaLoQibwU97TQdGjgHA//Cq1 frJIX1mfv3AlWny/85CNz67tz4O94lqM7DUTW+kg/ZDXIPdAxk0UTChPNey1MvEU8bv/ kBeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1693593566; x=1694198366; h=to:references:message-id:content-transfer-encoding:cc:date :in-reply-to:from:subject:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=bA5Lm6a1t+KFAs7vnfL5/17M6lm4KSTTxqyCI/pc5BY=; b=kdLSQ5gO3ZB/JSkmH+LjBE3uzfqAewp7mlwUdWzLiYeQq4AV5LWsLYvCGAaqlC/oNW oaHoUjkA/j3/sOfP19OlyBEFMfonqS8HEDF9oU4gFycvJt5aMbJ7MNGodvSTcd+PE/Ck cj2ga84a1CuLJblsCsisGtIwj0xfwmSPk5zpw9oi9H4KxnwljXo2h9OWNXGB0ATl1Rtd y3Vb0kzOgl12QPOMtsgkQ8yR36ohw6qI2oqL+Rln2JsfPt+OjKhTadXjElZ3OetUvs5j 90U600rHHuDB9ES7OJ1V7b6Qkif2NiSvBWNvO5Ih+4o5Ej2qtVHmCQd1iSvuWxlm6WoF wZnw== X-Gm-Message-State: AOJu0Yzhaq/JEde+KBT3J6hrN4RMsxdT0TI9sekkJsUOEkXEBmrb1Ctw fR3MHvRyYDft5l/WYIk18O+AJQ== X-Google-Smtp-Source: AGHT+IHwt+wcmiHzIzpBrobUBa991WScHeOvwtCAq4Ysh3WkCDFvhnNCS5xE0Y9TRcLz8axynzkOjA== X-Received: by 2002:a7b:c8ca:0:b0:401:b204:3b95 with SMTP id f10-20020a7bc8ca000000b00401b2043b95mr2441626wml.15.1693593566116; Fri, 01 Sep 2023 11:39:26 -0700 (PDT) Received: from smtpclient.apple ([131.111.5.246]) by smtp.gmail.com with ESMTPSA id l8-20020a05600c1d0800b003fef6881350sm5840720wms.25.2023.09.01.11.39.25 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 01 Sep 2023 11:39:25 -0700 (PDT) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3731.600.7\)) Subject: Re: [PATCH v1 1/3] dt-bindings: mmc: Drop unused properties From: Jessica Clarke In-Reply-To: <20230901-affected-wanting-ab517791a870@spud> Date: Fri, 1 Sep 2023 19:39:14 +0100 Cc: William Qiu , Conor Dooley , Krzysztof Kozlowski , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , LKML , linux-riscv , linux-mmc@vger.kernel.org, Emil Renner Berthing , Rob Herring , Jaehoon Chung , Ulf Hansson , Krzysztof Kozlowski , Conor Dooley , Paul Walmsley , Palmer Dabbelt , Albert Ou Content-Transfer-Encoding: quoted-printable Message-Id: References: <20230830031846.127957-1-william.qiu@starfivetech.com> <20230830031846.127957-2-william.qiu@starfivetech.com> <20230830-commence-trickery-40eaa193cb15@wendy> <20230830-procedure-frostbite-56c751f7c276@wendy> <20230901-remold-sublease-a1ddb1fc6348@spud> <9EF26965-10E5-4BCA-AC5E-93C5AA55A0DF@jrtc27.com> <20230901-affected-wanting-ab517791a870@spud> To: Conor Dooley X-Mailer: Apple Mail (2.3731.600.7) Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 1 Sep 2023, at 18:43, Conor Dooley wrote: >=20 > On Fri, Sep 01, 2023 at 06:20:38PM +0100, Jessica Clarke wrote: >> On 1 Sep 2023, at 16:42, Conor Dooley wrote: >>>=20 >>> On Fri, Sep 01, 2023 at 10:33:13AM +0800, William Qiu wrote: >>>>=20 >>>>=20 >>>> On 2023/8/30 16:34, Conor Dooley wrote: >>>>> On Wed, Aug 30, 2023 at 09:29:20AM +0200, Krzysztof Kozlowski = wrote: >>>>>> On 30/08/2023 08:50, Conor Dooley wrote: >>>>>>> On Wed, Aug 30, 2023 at 11:18:44AM +0800, William Qiu wrote: >>>>>>>> Due to the change of tuning implementation, it's no longer = necessary to >>>>>>>> use the "starfive,sysreg" property in dts, so drop the relevant >>>>>>>> description in dt-bindings here. >>>>>>>=20 >>>>>>> How does changing your software implantation invalidate a = description of >>>>>>> the hardware? >>>>>>>=20 >>>>>>=20 >>>>>> Which is kind of proof that this syscon was just to substitute >>>>>> incomplete hardware description (e.g. missing clocks and phys). = We >>>>>> should have rejected it. Just like we should reject them in the = future. >>>>>=20 >>>>> :s I dunno what to do with this... I'm inclined to say not to = remove it >>>>> from the binding or dts at all & only change the software. >>>>>=20 >>>>>> There are just few cases where syscon is reasonable. All others = is just >>>>>> laziness. It's not only starfivetech, of course. Several other >>>>>> contributors do the same. >>>>>=20 >>>>> I'm not sure if laziness is fair, lack of understanding is usually = more >>>>> likely. >>>>=20 >>>> For this, I tend to keep it in binding, but remove it from = required. Because >>>> we only modify the tuning implementation, it doesn't mean that this = property >>>> need to be removed, it's just no longer be the required one. >>>=20 >>> Please only remove it from required if the current driver doesn't = break >>> if the regmap is removed. >>=20 >> Either way please make sure the documentation clearly states =E2=80=9Cn= ever use >> this, if you=E2=80=99re using it you=E2=80=99re doing it wrong, this = only exists >> because it was wrongly used in the past=E2=80=9D. Otherwise people = writing >> drivers for other OSes will probably use it too thinking they need = to. >=20 > Maybe we should just delete it if the impact is going to be = negligible, > sounds like you're not using it in FreeBSD, which was part of what I = was > worried about. Guess it depends on what Emil & the distro heads think. FreeBSD doesn=E2=80=99t have StarFive drivers yet; I don=E2=80=99t have = time to write them, and a community member has taken it upon themselves as a hobby but is rather inexperienced and has been struggling for months. OpenBSD has drivers, including a modified dwmmc, but doesn=E2=80=99t use this = property (in fact its driver doesn=E2=80=99t use the compatible other than to = probe the generic driver). I don=E2=80=99t think anyone else has a serious port; = Haiku=E2=80=99s the closest but also has no StarFive support. Jess