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 95EA0C433F5 for ; Tue, 18 Jan 2022 12:50:49 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 979ED83817; Tue, 18 Jan 2022 13:50:42 +0100 (CET) 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="MQaae7uR"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id AA4FA8326F; Tue, 18 Jan 2022 13:50:38 +0100 (CET) Received: from mail-qk1-x730.google.com (mail-qk1-x730.google.com [IPv6:2607:f8b0:4864:20::730]) (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 5163E83817 for ; Tue, 18 Jan 2022 13:50:33 +0100 (CET) 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-qk1-x730.google.com with SMTP id 193so21758208qkh.13 for ; Tue, 18 Jan 2022 04:50:33 -0800 (PST) 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=N6bVRIifat9uwwRRBK71vZcNJuyJIhoO82IRERrSwLA=; b=MQaae7uRNcYuwt30KFnH1O3mBsQURkYyFGV/KYn90AfZCfayKVExOmsQNzhlhsmdZ+ 25ab5r1vDEHSWwELRZ0dQzsclnu+Il4KJmJyDuC3YS1x4QxCn7xzpa6KzjyM8KHdXLFw mEcD87EQYBCMZAC1CV6Wc84iMQNAA9j7RVeyk= 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=N6bVRIifat9uwwRRBK71vZcNJuyJIhoO82IRERrSwLA=; b=GNwkJIW4zWMxuvIKSEme731TebtZ6bShmOXm+hWMKRlTdI0mqC9OUDjqhe8daal+rd Q21IktDhdkrxAuRE1iMce+LJ6eSrIwywB2rHF0foXvKsYpxlEaWXyE+0CnW5l3Bm0DUt 2wGTZV3h/M/tYPEKd5T3/Vsybb9rP/HGsQi5vnlhff1oCs+m8SwDNvURRJW2htyZ9DpW u/2oY7MFHKIluo9YdPQXu1AqCgAzKZk/ibMx5TJGdResmD3VX4Tew2mhg3aNOcJoRczt 9O4BYc9qNIKHn+KEZ5ajMhj06rJAh3hANBoCVX67GVBb9OT0u1BiT7z/NttktJHROPWo 2ppA== X-Gm-Message-State: AOAM533DTA0zR0cbGWqksd1CQzr8kdzELGoWaFrl4zvGe77lCqrEQR/t XAVIV3BH6vvrsjn177I75LpCyg== X-Google-Smtp-Source: ABdhPJyDH5H8yKzjFtkUU7PSVn+9yfo9jt818ux1zes2YckOtFf7a5KfqI2lzq6JKl3nKNvkDGtDzQ== X-Received: by 2002:a05:620a:284a:: with SMTP id h10mr17390629qkp.4.1642510231593; Tue, 18 Jan 2022 04:50:31 -0800 (PST) Received: from bill-the-cat (2603-6081-7b01-cbda-8dc3-db57-8e4a-695a.res6.spectrum.com. [2603:6081:7b01:cbda:8dc3:db57:8e4a:695a]) by smtp.gmail.com with ESMTPSA id h6sm3531988qtb.15.2022.01.18.04.50.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 18 Jan 2022 04:50:30 -0800 (PST) Date: Tue, 18 Jan 2022 07:50:28 -0500 From: Tom Rini To: Aswath Govindraju Cc: u-boot@lists.denx.de, Wolfgang Denk , Hari Nagalla , David Huang , Keerthy , Dave Gerlach , Suman Anna , Nishanth Menon , Tero Kristo , Kishon Vijay Abraham I , Vignesh Raghavendra , Christian Hewitt , Tim Harvey , Peter Robinson , Kever Yang , Neil Armstrong , Fabio Estevam , Andre Przywara , Jaehoon Chung , Sean Anderson , Lukasz Majewski Subject: Re: [PATCH v2 01/20] remoteproc: k3_system_controller: Support optional boot_notification channel Message-ID: <20220118125028.GM2631111@bill-the-cat> References: <20220111075545.1880-1-a-govindraju@ti.com> <20220111075545.1880-2-a-govindraju@ti.com> <20220113141228.GO9207@bill-the-cat> <8e646a92-daf9-6457-d8dc-94b226647442@ti.com> <39a92323-8c8f-32aa-60b2-b4c43739a9f3@ti.com> <20220117135413.GM9207@bill-the-cat> <87211b68-1cf5-c105-ba4c-2b61ddb56764@ti.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="5PPnVwj6ulmYFu5t" Content-Disposition: inline In-Reply-To: <87211b68-1cf5-c105-ba4c-2b61ddb56764@ti.com> 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.2 at phobos.denx.de X-Virus-Status: Clean --5PPnVwj6ulmYFu5t Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 18, 2022 at 11:26:50AM +0530, Aswath Govindraju wrote: > Hi Tomi, >=20 > On 17/01/22 7:24 pm, Tom Rini wrote: > > On Mon, Jan 17, 2022 at 12:22:52PM +0530, Aswath Govindraju wrote: > >> Hi Tom, > >> > >> On 17/01/22 11:01 am, Aswath Govindraju wrote: > >>> Hi Tom, > >>> > >>> On 13/01/22 7:42 pm, Tom Rini wrote: > >>>> On Tue, Jan 11, 2022 at 01:25:26PM +0530, Aswath Govindraju wrote: > >>>> > >>>>> From: Nishanth Menon > >>>>> > >>>>> If there is an optional boot notification channel that an SoC uses > >>>>> separate from the rx path, use the same. > >>>>> > >>>>> Signed-off-by: Nishanth Menon > >>>>> --- > >>>>> .../remoteproc/k3-system-controller.txt | 3 +++ > >>>>> drivers/remoteproc/k3_system_controller.c | 20 +++++++++++++++= +++- > >>>>> 2 files changed, 22 insertions(+), 1 deletion(-) > >>>> > >>>> Binding docs are rst these days, so we should sync with upstream and > >>>> then this property is already there, right? > >>>> > >>> > >>> I will create a followup patch to convert documentation to rst. Also, > >>> about the property, mbox-names property is already present but > >>> "boot_notify" is a newly added channel and not are required property. > >>> So, this was additionally added. > >>> > >> > >> One more question regarding documentation, should it be changed to rst > >> or yaml, as this is a device tree binding? > >=20 > > I mis-spoke, yeah. It should be yaml and pushed upstream first, then > > brought back here. > >=20 >=20 > I am sorry, I have one more question. This above documentation file is > not present in kernel documentation, so I did not understand how can > this be pushed there first. >=20 > Also, as converting to yaml would be a different work. Wouldn't it be > better to separate that work from this series? Sigh, it should have been upstreamed first. So yeah, make the changes you need here now and then please start pushing it upstream, thanks. --=20 Tom --5PPnVwj6ulmYFu5t Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmHmt5EACgkQFHw5/5Y0 tywN9wv/VRbSds+XOK90y4dNuuCkC3q2vi4UvyLIud/IjcT3khp53pTv8IuiXCwS HdfGzimcjKZSa3An+BxM6/QTl3/n6Gg3te4SuiTKl+liklOttrhtEmaiaPLQ0zuS qzt4sU2nycgAU3j8HxvO/udHCOdo6AD9m5QL38CNtue+VvHvGpv/30OkXq2pGAz/ iqHTugBh92EQjTmoyynYuiqAcOIrxivK3PaU5AxMLiMo6aYvl+9+O9wMFOOQvBbE CXSq9xi2jT4foiywucdWXKDqtzumRM8BEp+1LI24vXNt947IZwtIKwhVEZ1scsFO slGHKgGMbVoBgGwgvlUJQt+t4zQkLuvO3WdVz90lQtNFzBT8DBk35gVWwUMk7K6i d9jwm7xYyK6MUPWLYdwlhFqK3q985K6/RkYHHiyW0apL+I6O4Sw7jsN3eaQ61wiv Bo2CxA4iSNYYL5OqJdtpAUfafnnOqu8xNP+n47FJ3KQUPj0BumlUmDpD/Zos5aJd fuwPwMJ1 =rOoh -----END PGP SIGNATURE----- --5PPnVwj6ulmYFu5t--