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=-7.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 2483EC48BE6 for ; Wed, 16 Jun 2021 21:16:26 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7F5B9613DF for ; Wed, 16 Jun 2021 21:16:25 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7F5B9613DF Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id E7ED4804E7; Wed, 16 Jun 2021 23:16:22 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=konsulko.com header.i=@konsulko.com header.b="POPxBtyj"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 6096E8060C; Wed, 16 Jun 2021 23:16:21 +0200 (CEST) Received: from mail-qv1-xf2b.google.com (mail-qv1-xf2b.google.com [IPv6:2607:f8b0:4864:20::f2b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 2534A801A0 for ; Wed, 16 Jun 2021 23:16:18 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-qv1-xf2b.google.com with SMTP id if15so517298qvb.2 for ; Wed, 16 Jun 2021 14:16:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=6e+zTvi6hjpCgqriEzPPf7WaZlaEIP3TEcBecb11skI=; b=POPxBtyjR3lqO5UqMaph24rocam+EQib4ZMUYqdUROrmDZbjo3TjF1I4obmjoxU0jG X85ZStq8c3o/FtKuEMcJ1yn7rjTVYv8RL8S6SzRMXTmOZOv0Yv2vrwdJKAB0KU9qfhH6 MHS1sdtsZiA6YSrLzWlR+By3Fv+AhL7rCSZyQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=6e+zTvi6hjpCgqriEzPPf7WaZlaEIP3TEcBecb11skI=; b=eqTIneT22MNXhvERoELP8TZJs7vBXn1DsAg6S79g64jbpZiVJrnGHGHhlULDUxVWyA IXMeHhMVfQNEakz8lnllnKB57vA1oW0bwhi8c/P/0yOUHmiHw2cVQZdQSVmkX7bdBhRp yBOgmO8KovlisE3XbKhOHju+kiCc7kNnY5Yhgpr8WXtjNP9dNcTI+ekXugsi7XwuwmSo Bt8Y9n1dmmKoXkpVgVYJQvHzQQEpJJoK5AtF043Dk/rMr4Vm1/MKPFw7jLE8JVPD+oGR DUVWJTHEB3cZItG5pQsg9OcdT+2ey6h8BovdGFFuclN6DAJ1kNdOM4LsfZ51DSvcGNC9 2RRQ== X-Gm-Message-State: AOAM532WbLcLm/TpxpNoMabK7xJT08oL4wUINDsA5xIUg9zIsEYFpv2m NbaenYKwyNWSOKwXYK00VXkGjA== X-Google-Smtp-Source: ABdhPJy0yjY1oQVVFQweYsPP2UuL7OKfrP1/WE85a9eKvQWcUh0gsCAagIPrjbQLY91KZ29baYwQXA== X-Received: by 2002:a0c:ef0c:: with SMTP id t12mr2111662qvr.5.1623878176823; Wed, 16 Jun 2021 14:16:16 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b01-cbda-8009-22e2-67e2-281e.res6.spectrum.com. [2603:6081:7b01:cbda:8009:22e2:67e2:281e]) by smtp.gmail.com with ESMTPSA id h9sm356975qkj.66.2021.06.16.14.16.15 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Wed, 16 Jun 2021 14:16:15 -0700 (PDT) Date: Wed, 16 Jun 2021 17:16:14 -0400 From: Tom Rini To: Merlijn Wajer Cc: Ivaylo Dimitrov , Pali =?iso-8859-1?Q?Roh=E1r?= , maemo-leste@lists.dyne.org, u-boot@lists.denx.de Subject: Re: [maemo-leste] [PATCH] arm: Remove nokia_rx51 board Message-ID: <20210616211614.GA9516@bill-the-cat> References: <20210521144418.GI17669@bill-the-cat> <20210521213645.2y5dawkvr6wfaymy@pali> <20210615123436.GH9516@bill-the-cat> <5848e9c9-4dab-00cc-07dc-ffa57b9417cd@gmail.com> <20210616121008.GR9516@bill-the-cat> <20210616121313.GS9516@bill-the-cat> <20210616173702.GW9516@bill-the-cat> <42d863df-7b71-ef27-8032-bb463e941768@wizzup.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="BClrmmvvLKCkynDZ" Content-Disposition: inline In-Reply-To: <42d863df-7b71-ef27-8032-bb463e941768@wizzup.org> X-Clacks-Overhead: GNU Terry Pratchett User-Agent: Mutt/1.9.4 (2018-02-28) X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean --BClrmmvvLKCkynDZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jun 16, 2021 at 10:44:52PM +0200, Merlijn Wajer wrote: > Hi Tom, >=20 > On 16/06/2021 19:37, Tom Rini wrote: >=20 > >>>>> Fixing those requires enabling of OF_CONTROL and this in turn means= the > >>>>> board must be migrated to DT, unless I am missing something. That's= why my > >>>>> "please advice..." stance. > >>>> > >>>> Please post the patches that bring you to the above link errors, yes, > >>>> thanks. > >>> > >>> To be clearer, finish up a patch that completes the migration but is = too > >>> large to install on the hardware so that others can take a look. > >> > >> I am not sure I understand that - a patch that completes the migration= to > >> DM_USB cannot be done ATM as the binary does not link without OF_CONTR= OL. > >> And I am not going to enable OF_CONTROL as this means I will have to m= igrate > >> everything to DT. That's why I enable DM_USB only - see reply to the o= ther > >> mail. > >=20 > > My advise is to provide a linkable but not runnable (or, only runnable > > in QEMU, the problem is the fixed layout of the actual device flash, > > right?) patch so that we can figure out how and what needs to be tuned > > where so that we can see what to do about this platform. > >=20 >=20 > It looks to me that there are at least two problems: >=20 > * Size of the produced u-boot with additional features could be too large > * Porting RX51 to DM_USB seemingly also requires porting RX51 to > device-tree (since OF_CONTROL is required), adding additional complicatio= ns. >=20 > When you ask for patches, are you asking for patches that just enable > DM_USB and stub functions to make the linking succeed (although it > clearly won't result in a functional u-boot), or are you asking for > patches that enable DM_USB and OF_CONTROL, and also port the device to > device tree? To me it seems like you meant the former, but I'd like to > confirm. >=20 > Also, I believe Ivaylo and Pali are wondering if porting to DTS is a > hard requirement for porting to DM_USB (since OF_CONTROL would be require= d)? So, part of the high level answer here is that long term, yes, enabling DM for everything is required. That may mean additional infrastructure work, if we don't already have all of the knobs in place today to let space constrained platforms like this continue to work. Does that mean we _have_to_ have OF_CONTROL? Well, today we don't have platdata for non-SPL/TPL. Saying we can't support this platform without adding as an option and then enablig OF_PLATDATA for U-Boot itself is something to bring up on the U-Boot mailing list and cc the relevant custodians. --=20 Tom --BClrmmvvLKCkynDZ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmDKah0ACgkQFHw5/5Y0 tyw9VQv/Z6xd4FZolDl22SIegA5kwgdTLoQgu3Fb/RoVQE51HHr09IVa/loZWeG8 xmxKXRdCZoHroXraxj7pYRUWhHSegJNXOp4t7H2rVlyGzBa3D6/D2qfpNWvkkVKO R+7DuvTQileL4KNxMnd0iq6a7Uu4ysOZc3aDkFj8kTTZ3uQBJAVdIEABwvSwDZYE 8Lf8LJaCqaPCNmgP896nG1p0UM7EBmfR1lFs1F+aVswIbvw15ohCIVlFd8VZ9j81 U3/p3YBnZFrBioeUFYBTnv6mowx4E6t2G71hPyGdP5x0BQEnQ5txpNMPa4MQZuMz jxUglq2e/mmF+jNpd+5mMgUzFQ7rrN5sqp91ISoGHgHwgEbQ0+H03ka6tyIAM0HJ KG/vOD8K7j5Ss1nfxrgZS6HzFbOPoePpy+/S59jmuvIHqsqHsX5jSIsM2U2otza6 TNg2peyouswsvA/SoGD534nI/Depvaoy8d1cU/+6RD0RS3VMVMs/wNjxOje2SRpk qTCbiOZR =k49q -----END PGP SIGNATURE----- --BClrmmvvLKCkynDZ--