Re: [PATCH v6 2/2] lib/string: shrink lib/string.i via IWYU

2023-12-26 Thread Tanzir Hasan
On Tue, Dec 26, 2023 at 10:35 AM Andy Shevchenko wrote: > > On Tue, Dec 26, 2023 at 8:00 PM Tanzir Hasan wrote: > > > > This diff uses an open source tool include-what-you-use (IWYU) to modify > > the include list, changing indirect includes to direct includes. IWYU is > > implemented using the I

Re: [PATCH v6 2/2] lib/string: shrink lib/string.i via IWYU

2023-12-26 Thread Andy Shevchenko
On Tue, Dec 26, 2023 at 8:00 PM Tanzir Hasan wrote: > > This diff uses an open source tool include-what-you-use (IWYU) to modify > the include list, changing indirect includes to direct includes. IWYU is > implemented using the IWYUScripts github repository which is a tool that > is currently unde

[PATCH v6 2/2] lib/string: shrink lib/string.i via IWYU

2023-12-26 Thread Tanzir Hasan
This diff uses an open source tool include-what-you-use (IWYU) to modify the include list, changing indirect includes to direct includes. IWYU is implemented using the IWYUScripts github repository which is a tool that is currently undergoing development. These changes seek to improve build times.

[PATCH v6 1/2] kernel.h: removed REPEAT_BYTE from kernel.h

2023-12-26 Thread Tanzir Hasan
This patch creates wordpart.h and includes it in asm/word-at-a-time.h for all architectures. WORD_AT_A_TIME_CONSTANTS depends on kernel.h because of REPEAT_BYTE. Moving this to another header and including it where necessary allows us to not include the bloated kernel.h. Making this implicit depend

[PATCH v6 0/2] shrink lib/string.i via IWYU

2023-12-26 Thread Tanzir Hasan
This patch series changes the include list of string.c to minimize the preprocessing size. The patch series intends to remove REPEAT_BYE from kernel.h and move it into its own header file because word-at-a-time.h has an implicit dependancy on it but it is declared in kernel.h which is bloated. ---

Re: [PATCH v4] iio: sx9324: avoid copying property strings

2023-12-26 Thread Jonathan Cameron
On Tue, 19 Dec 2023 21:34:15 + Justin Stitt wrote: > We're doing some needless string copies when trying to assign the proper > `prop` string. We can make `prop` a const char* and simply assign to > string literals. > > For the case where a format string is used, let's extract the parsing >