Re: [PATCH v2 07/14] log: Avoid including function names by default

2024-09-20 Thread Simon Glass
Hi Quentin, On Mon, 12 Aug 2024 at 10:56, Quentin Schulz wrote: > > Hi Simon, > > On 8/11/24 4:50 PM, Simon Glass wrote: > > Hi Quentin, > > > > On Tue, 6 Aug 2024 at 08:16, Quentin Schulz > > wrote: > >> > >> Hi Simon, > >> > >> On 7/21/24 5:25 PM, Simon Glass wrote: > >>> Unless function name

Re: [PATCH v2 07/14] log: Avoid including function names by default

2024-08-12 Thread Quentin Schulz
Hi Simon, On 8/11/24 4:50 PM, Simon Glass wrote: Hi Quentin, On Tue, 6 Aug 2024 at 08:16, Quentin Schulz wrote: Hi Simon, On 7/21/24 5:25 PM, Simon Glass wrote: Unless function names are requested, the logging system should not compile these into the code. Adjust the macros to handle this.

Re: [PATCH v2 07/14] log: Avoid including function names by default

2024-08-11 Thread Simon Glass
Hi Quentin, On Tue, 6 Aug 2024 at 08:16, Quentin Schulz wrote: > > Hi Simon, > > On 7/21/24 5:25 PM, Simon Glass wrote: > > Unless function names are requested, the logging system should not > > compile these into the code. Adjust the macros to handle this. > > > > This means that turning on func

Re: [PATCH v2 07/14] log: Avoid including function names by default

2024-08-06 Thread Quentin Schulz
Hi Simon, On 7/21/24 5:25 PM, Simon Glass wrote: Unless function names are requested, the logging system should not compile these into the code. Adjust the macros to handle this. This means that turning on function names at runtime won't work unless CONFIG_LOGF_FUNC is enabled. We could perhaps

[PATCH v2 07/14] log: Avoid including function names by default

2024-07-21 Thread Simon Glass
Unless function names are requested, the logging system should not compile these into the code. Adjust the macros to handle this. This means that turning on function names at runtime won't work unless CONFIG_LOGF_FUNC is enabled. We could perhaps split this into a separate option if that is a prob