From: Tom Rini <trini@konsulko.com>
To: Aswath Govindraju <a-govindraju@ti.com>
Cc: Lokesh Vutla <lokeshvutla@ti.com>,
Vignesh Raghavendra <vigneshr@ti.com>,
Kishon Vijay Abraham I <kishon@ti.com>,
Pratyush Yadav <p.yadav@ti.com>, Tom Warren <twarren@nvidia.com>,
Stephen Warren <swarren@nvidia.com>,
Peter.Chubb@data61.csiro.au,
Oleksandr Suvorov <oleksandr.suvorov@toradex.com>,
Heiko Schocher <hs@denx.de>,
Andreas Geisreiter <ageisreiter@dh-electronics.de>,
Ludwig Zenz <lzenz@dh-electronics.de>,
Samuel Egli <samuel.egli@siemens.com>,
Allen Martin <amartin@nvidia.com>,
Jaehoon Chung <jh80.chung@samsung.com>,
Lukasz Majewski <lukma@denx.de>,
Chander Kashyap <k.chander@samsung.com>,
Richard Hu <richard.hu@technexion.com>,
Fabio Estevam <fabio.estevam@nxp.com>,
Matthias Brugger <mbrugger@suse.com>,
Robert Baldyga <r.baldyga@samsung.com>,
Dinh Nguyen <dinguyen@kernel.org>,
Chin-Liang See <clsee@altera.com>, Marek Vasut <marex@denx.de>,
Dalon Westergreen <dwesterg@gmail.com>,
Wolfgang Grandegger <wg@aries-embedded.de>,
Stefan Roese <sr@denx.de>,
Mike Looijmans <mike.looijmans@topic.nl>,
Otavio Salvador <otavio@ossystems.com.br>,
Michal Simek <michal.simek@xilinx.com>,
Adam Ford <aford173@gmail.com>,
Jagan Teki <jagan@amarulasolutions.com>,
Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>,
Ley Foon Tan <ley.foon.tan@intel.com>,
Simon Glass <sjg@chromium.org>,
Patrick Delaunay <patrick.delaunay@foss.st.com>,
Ovidiu Panait <ovidiu.panait@windriver.com>,
u-boot@lists.denx.de
Subject: Re: [PATCH v3] usb: dfu: Migrate CONFIG_SYS_DFU_DATA_BUF_SIZE and CONFIG_SYS_DFU_MAX_FILE_SIZE to Kconfig
Date: Tue, 8 Jun 2021 08:07:41 -0400 [thread overview]
Message-ID: <20210608120741.GS9516@bill-the-cat> (raw)
In-Reply-To: <696174a2-8f09-d15a-5aaa-2b0e8bbab4c5@ti.com>
[-- Attachment #1: Type: text/plain, Size: 1050 bytes --]
On Tue, Jun 08, 2021 at 01:34:33PM +0530, Aswath Govindraju wrote:
> + Tom Rini
>
> I saw that this patch is delegated to Tom Rini in patchwork page but I
> did not add him in the cc-list. So, I added him now.
>
> Hi Tom,
>
> On 01/06/21 4:51 pm, Aswath Govindraju wrote:
> > Currently the config options CONFIG_SYS_DFU_DATA_BUF_SIZE and
> > CONFIG_SYS_DFU_MAX_FILE_SIZE are being set in include/configs/<board>.h
> > files and also in <board_name>_defconfig files without a Kconfig option. It
> > is easier for users to set these configs in defconfig files than in config
> > header files as they are a part of the source code.
> >
> > Add Kconfig symbols, and update the defconfigs by using tools/moveconfig.py
> > script.
> >
>
> May I know if this patch is good to be picked?
Yes but some of the recent "this cleanup caused a hard to spot
regression" have left me of the opinion that Kconfig migrations need to
go in to -next or early -rc rather than late -rc, so, soon now. Thanks
for your patience.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
next prev parent reply other threads:[~2021-06-09 3:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-01 11:21 [PATCH v3] usb: dfu: Migrate CONFIG_SYS_DFU_DATA_BUF_SIZE and CONFIG_SYS_DFU_MAX_FILE_SIZE to Kconfig Aswath Govindraju
2021-06-01 15:58 ` Matthias Brugger
2021-06-02 8:37 ` Lukasz Majewski
2021-06-08 8:04 ` Aswath Govindraju
2021-06-08 12:07 ` Tom Rini [this message]
2021-06-09 12:19 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210608120741.GS9516@bill-the-cat \
--to=trini@konsulko.com \
--cc=Peter.Chubb@data61.csiro.au \
--cc=a-govindraju@ti.com \
--cc=aford173@gmail.com \
--cc=ageisreiter@dh-electronics.de \
--cc=amartin@nvidia.com \
--cc=ashok.reddy.soma@xilinx.com \
--cc=clsee@altera.com \
--cc=dinguyen@kernel.org \
--cc=dwesterg@gmail.com \
--cc=fabio.estevam@nxp.com \
--cc=hs@denx.de \
--cc=jagan@amarulasolutions.com \
--cc=jh80.chung@samsung.com \
--cc=k.chander@samsung.com \
--cc=kishon@ti.com \
--cc=ley.foon.tan@intel.com \
--cc=lokeshvutla@ti.com \
--cc=lukma@denx.de \
--cc=lzenz@dh-electronics.de \
--cc=marex@denx.de \
--cc=mbrugger@suse.com \
--cc=michal.simek@xilinx.com \
--cc=mike.looijmans@topic.nl \
--cc=oleksandr.suvorov@toradex.com \
--cc=otavio@ossystems.com.br \
--cc=ovidiu.panait@windriver.com \
--cc=p.yadav@ti.com \
--cc=patrick.delaunay@foss.st.com \
--cc=r.baldyga@samsung.com \
--cc=richard.hu@technexion.com \
--cc=samuel.egli@siemens.com \
--cc=sjg@chromium.org \
--cc=sr@denx.de \
--cc=swarren@nvidia.com \
--cc=twarren@nvidia.com \
--cc=u-boot@lists.denx.de \
--cc=vigneshr@ti.com \
--cc=wg@aries-embedded.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox