Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-24 Thread Chee, Tien Fong
On Fri, 2019-02-22 at 10:16 +0100, Michal Simek wrote: > On 22. 02. 19 4:49, Chee, Tien Fong wrote: > > > > On Thu, 2019-02-21 at 22:22 -0500, Tom Rini wrote: > > > > > > On Thu, Feb 21, 2019 at 08:45:37AM +0100, Michal Simek wrote: > > > > > > > > > > > > Hi Tom, > > > > > > > > On 20. 02.

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-22 Thread Michal Simek
On 22. 02. 19 4:49, Chee, Tien Fong wrote: > On Thu, 2019-02-21 at 22:22 -0500, Tom Rini wrote: >> On Thu, Feb 21, 2019 at 08:45:37AM +0100, Michal Simek wrote: >>> >>> Hi Tom, >>> >>> On 20. 02. 19 2:58, Tom Rini wrote: On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.co

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Chee, Tien Fong
On Thu, 2019-02-21 at 22:22 -0500, Tom Rini wrote: > On Thu, Feb 21, 2019 at 08:45:37AM +0100, Michal Simek wrote: > > > > Hi Tom, > > > > On 20. 02. 19 2:58, Tom Rini wrote: > > > > > > On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.co > > > m wrote: > > > > > > > > > > >

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Tom Rini
On Thu, Feb 21, 2019 at 08:45:37AM +0100, Michal Simek wrote: > Hi Tom, > > On 20. 02. 19 2:58, Tom Rini wrote: > > On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.com wrote: > > > >> From: Tien Fong Chee > >> > >> Drop the statically allocated get_contents_vfatname_block and >

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Adam Ford
On Thu, Feb 21, 2019 at 6:51 AM Marek Vasut wrote: > > On 2/21/19 1:13 PM, Michal Simek wrote: > > On 21. 02. 19 10:04, Marek Vasut wrote: > >> On 2/21/19 9:55 AM, Alexander Graf wrote: > >>> > >>> > >>> On 21.02.19 09:49, Marek Vasut wrote: > On 2/21/19 9:44 AM, Alexander Graf wrote: >

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Marek Vasut
On 2/21/19 1:13 PM, Michal Simek wrote: > On 21. 02. 19 10:04, Marek Vasut wrote: >> On 2/21/19 9:55 AM, Alexander Graf wrote: >>> >>> >>> On 21.02.19 09:49, Marek Vasut wrote: On 2/21/19 9:44 AM, Alexander Graf wrote: > > > On 21.02.19 09:41, Marek Vasut wrote: >> On 2/21/19

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Michal Simek
On 21. 02. 19 10:04, Marek Vasut wrote: > On 2/21/19 9:55 AM, Alexander Graf wrote: >> >> >> On 21.02.19 09:49, Marek Vasut wrote: >>> On 2/21/19 9:44 AM, Alexander Graf wrote: On 21.02.19 09:41, Marek Vasut wrote: > On 2/21/19 9:40 AM, Chee, Tien Fong wrote: >> On Thu,

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Marek Vasut
On 2/21/19 9:55 AM, Alexander Graf wrote: > > > On 21.02.19 09:49, Marek Vasut wrote: >> On 2/21/19 9:44 AM, Alexander Graf wrote: >>> >>> >>> On 21.02.19 09:41, Marek Vasut wrote: On 2/21/19 9:40 AM, Chee, Tien Fong wrote: > On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote:

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Alexander Graf
On 21.02.19 09:49, Marek Vasut wrote: > On 2/21/19 9:44 AM, Alexander Graf wrote: >> >> >> On 21.02.19 09:41, Marek Vasut wrote: >>> On 2/21/19 9:40 AM, Chee, Tien Fong wrote: On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote: > > On 21.02.19 09:23, Chee, Tien Fong wrote:

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Marek Vasut
On 2/21/19 9:44 AM, Alexander Graf wrote: > > > On 21.02.19 09:41, Marek Vasut wrote: >> On 2/21/19 9:40 AM, Chee, Tien Fong wrote: >>> On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote: On 21.02.19 09:23, Chee, Tien Fong wrote: > > On Thu, 2019-02-21 at 08:45 +0100,

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Alexander Graf
On 21.02.19 09:41, Marek Vasut wrote: > On 2/21/19 9:40 AM, Chee, Tien Fong wrote: >> On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote: >>> >>> On 21.02.19 09:23, Chee, Tien Fong wrote: On Thu, 2019-02-21 at 08:45 +0100, Michal Simek wrote: > > Hi Tom, > > On 20.

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Marek Vasut
On 2/21/19 9:40 AM, Chee, Tien Fong wrote: > On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote: >> >> On 21.02.19 09:23, Chee, Tien Fong wrote: >>> >>> On Thu, 2019-02-21 at 08:45 +0100, Michal Simek wrote: Hi Tom, On 20. 02. 19 2:58, Tom Rini wrote: > > > On

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Chee, Tien Fong
On Thu, 2019-02-21 at 09:29 +0100, Alexander Graf wrote: > > On 21.02.19 09:23, Chee, Tien Fong wrote: > > > > On Thu, 2019-02-21 at 08:45 +0100, Michal Simek wrote: > > > > > > Hi Tom, > > > > > > On 20. 02. 19 2:58, Tom Rini wrote: > > > > > > > > > > > > On Mon, Feb 11, 2019 at 02:56:19PM

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Alexander Graf
On 21.02.19 09:23, Chee, Tien Fong wrote: > On Thu, 2019-02-21 at 08:45 +0100, Michal Simek wrote: >> Hi Tom, >> >> On 20. 02. 19 2:58, Tom Rini wrote: >>> >>> On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.com >>> wrote: >>> From: Tien Fong Chee Drop the

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-21 Thread Chee, Tien Fong
On Thu, 2019-02-21 at 08:45 +0100, Michal Simek wrote: > Hi Tom, > > On 20. 02. 19 2:58, Tom Rini wrote: > > > > On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.com > > wrote: > > > > > > > > From: Tien Fong Chee > > > > > > Drop the statically allocated

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-20 Thread Michal Simek
Hi Tom, On 20. 02. 19 2:58, Tom Rini wrote: > On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.com wrote: > >> From: Tien Fong Chee >> >> Drop the statically allocated get_contents_vfatname_block and >> dynamically allocate a buffer only if required. This saves >> 64KiB of memory.

Re: [U-Boot] [U-Boot, v3, 1/2] fs: fat: dynamically allocate memory for temporary buffer

2019-02-19 Thread Tom Rini
On Mon, Feb 11, 2019 at 02:56:19PM +0800, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > Drop the statically allocated get_contents_vfatname_block and > dynamically allocate a buffer only if required. This saves > 64KiB of memory. > > Signed-off-by: Stefan Agner > Signed-off-by: