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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id ED278C43334 for ; Sat, 9 Jul 2022 15:02:49 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 6B0A0845E3; Sat, 9 Jul 2022 17:02:47 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (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="Xm3A5GV7"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 2E271845D9; Sat, 9 Jul 2022 17:02:46 +0200 (CEST) Received: from mail-qk1-x72c.google.com (mail-qk1-x72c.google.com [IPv6:2607:f8b0:4864:20::72c]) (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 61464845CA for ; Sat, 9 Jul 2022 17:02:43 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-qk1-x72c.google.com with SMTP id f14so985431qkm.0 for ; Sat, 09 Jul 2022 08:02:43 -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; bh=sr+oMKCxe9uqL+zD4mO0dpUK/i+xQYLiR8GbdlX1fOc=; b=Xm3A5GV7+I1mS3Ka7sOPEPYA1U25n4nvsF0p5sEuU/Dtn6MdVdfFiZo6iZkIo2c3qd JIabE/29FDoXYAsfE+ARJzN/4ZCexHQQmfErqmI3T4mm+PgmVZPoz/d1GLHZbj31uy72 M+VUPt6aVV0en1xDhOZdgAgMd/0E4Vk7O/rco= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=sr+oMKCxe9uqL+zD4mO0dpUK/i+xQYLiR8GbdlX1fOc=; b=OPAo62uPeEJ/fuvYrmwZb3mCB9lsOBuNWNlENanq1JI+rGlZts+LgLi2HTIMECxenp S2gxPpzFXUVJAn3CP85wQ8J6ylMzuVPtWwZ8J4bHbqJ++XcYpdpflYXKsIt4dwLbmTD2 FkPV35pCYL0iHqS8HLw3LUjhtULk5REPYYOkdCptHc2pAhe9L9JzTdnxv3O9cE8idw98 67++XiyqS/taNFDamSXtJyHq5JDfk4mxyJ0ISe1I4GM5ivatvplkR1UlLMpkeEYhXFww NZQMECcI1Kir/Aae516JWnI9zpyJSpMyKt5IFf/CYuAwxEM5rD7wDFbrEaiaD6I2tLGW NCQg== X-Gm-Message-State: AJIora+zJLPXo98aB4Rc2n/XHR6jO14gD8NWC4/r5nAbH8OtVC1Kul6A 7ioxnQR4QJFnjw6APNVdkIC7Zg== X-Google-Smtp-Source: AGRyM1uyV1WztxJVxqIuzpLWS9OGFduoaINenJrMLzS3CdpYBfex9VMCCCx5Cvkv+j3ae+KSX0LnkA== X-Received: by 2002:a05:620a:3186:b0:6af:3f3b:3837 with SMTP id bi6-20020a05620a318600b006af3f3b3837mr6116993qkb.482.1657378961510; Sat, 09 Jul 2022 08:02:41 -0700 (PDT) Received: from bill-the-cat (cpe-65-184-195-139.ec.res.rr.com. [65.184.195.139]) by smtp.gmail.com with ESMTPSA id x18-20020a05620a449200b006a981a2c483sm1288634qkp.39.2022.07.09.08.02.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 09 Jul 2022 08:02:40 -0700 (PDT) Date: Sat, 9 Jul 2022 11:02:38 -0400 From: Tom Rini To: Heinrich Schuchardt Cc: u-boot@lists.denx.de, Simon Glass Subject: Re: [PATCH 4/4] doc: Migrate Process wiki page to sphinx Message-ID: <20220709150238.GI1146598@bill-the-cat> References: <20220627171722.1153337-1-trini@konsulko.com> <20220627171722.1153337-5-trini@konsulko.com> <2b600d2f-1534-062e-f8b0-ba65676fdfcc@gmx.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="ikO5TyXgUjwX5WPf" Content-Disposition: inline In-Reply-To: <2b600d2f-1534-062e-f8b0-ba65676fdfcc@gmx.de> X-Clacks-Overhead: GNU Terry Pratchett X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.6 at phobos.denx.de X-Virus-Status: Clean --ikO5TyXgUjwX5WPf Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Jul 09, 2022 at 08:55:48AM +0200, Heinrich Schuchardt wrote: > On 6/27/22 19:17, Tom Rini wrote: [snip] > > +Phases of the Development Process > > +--------------------------------- > > + > > +U-Boot development takes place in `Release Cycles > > +`_. A Release Cycle las= ts > > +normally for three months. > > + > > +The first two weeks of each Release Cycle are called *Merge Window*. > > + > > +It is followed by a *Stabilization Period*. > > + > > +The end of a Release Cycle is marked by the release of a new U-Boot ve= rsion. >=20 > Don't repeat yourself. I'm not seeing repetition really here. [snip] > > +Work flow of a Custodian > > +------------------------ > > + > > +The normal flow of work in the U-Boot development process will look > > +like this: > > + > > +#. A developer submits a patch via e-mail to the u-boot-users mailing = list. > > + U-Boot has adopted the `Linux kernel signoff policy `_, so the submitter = must > > + include a ``Signed-off-by:`` line. >=20 > Keep lines at 80 characters. >=20 > > +#. Everybody who can is invited to review and test the changes. Revie= ws should > > + reply on the mailing list with ``Acked-by`` lines. >=20 > %s/Acked-by/Reviewed-by/ ? >=20 > Please, refer to > https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html > for the usage of Reviewed-by and Acked-by. Yes, it would be good to reword this to reference the current kernel.org documentation, I'll try something. > > +#. The responsible custodian > > + > > + #. inspects this patch, especially for: >=20 > This should not be a bullet but go into the line above. That would read better, yes. [snip] > > +#. Once tests are passed, some agreed time limit expires, the custodian >=20 > No clue what agreed time limit you are referring to. >=20 > The custodian will create a merge request when the merge window matching > the type of the patch (feature or bug) is open. This has long been more informal rather than following a defined process, yes. I'll try and reword things a bit. >=20 > > + requests that the changes in his public git repository be merged in= to the > > + main tree. If necessary, the custodian may have to adapt his change= s to > > + allow for a clean merge. > > + Todo: define a reasonable time limit. 3 weeks? >=20 > This todo-line is not helpful. If the patch contains a new feature, the > custodian has to wait for the next merge window before issuing a pull > request for it. --=20 Tom --ikO5TyXgUjwX5WPf Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmLJmIUACgkQFHw5/5Y0 tyw3Zgv6AvoQYHBrzaIxXLPlsGfmxYrloOQVT7KfROcb8Gs9aeIme8efod+6xgww 8TXE2lRgc1WJ227o6t+5HbgCriuWxI9FhYetnGAyBoUtL4XQufPyUEDPR+OhehmQ uBsgctpuQD/d1gI7i1diWpYxWWU2I/4goyPeaXrIyvL6VlzfYRpQ5Hy+4/7W8WiM Gdh5E2j6K1gYtsHmf91S5weKNW85PetHtaSvDCpRv96WG3P2zXvoqbUVfFOtiX2z yJzXKYmmcf0SvbVj5aKYXnNjUVCz/BAhS/S9Oc5SEAyvqDIAq59NizQkRbwp51Gp hUIo6+5MrDvdPbmyftGm28W60J/KxWLMDKsXkXUE8zhV7tNqZEmf6WoBweR8kN+I +5B3YppBDOfo+ASvJti6bnLsvaH/kuYgriBSCnsI8UMMSIO7f+XzlxVRRiOyzPKG JWWKPkEb+SqUi/Cum35YZuazXjv49Pgc4iKKgXx5VRlLzhOO93gDLhR2r6qGbmXJ SiuaP1ZP =p5SK -----END PGP SIGNATURE----- --ikO5TyXgUjwX5WPf--