Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

2021-11-19 Thread Alejandro Colomar (man-pages)
On 11/19/21 17:18, Arnd Bergmann wrote: > On Fri, Nov 19, 2021 at 5:10 PM Andy Shevchenko > wrote: >> On Fri, Nov 19, 2021 at 04:57:46PM +0100, Arnd Bergmann wrote: > >>> The main problem with this approach is that as soon as you start >>> actually reducing the unneeded indirect includes, you

Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

2021-11-19 Thread Alejandro Colomar (man-pages)
Hi Arnd, On 11/19/21 16:57, Arnd Bergmann wrote: > > From what I can tell, linux/stddef.h is tiny, I don't think it's really > worth optimizing this part. I have spent some time last year > trying to untangle some of the more interesting headers, but ended > up not completing this as there are

Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

2021-11-19 Thread Alejandro Colomar (man-pages)
Hi Andy, On 11/19/21 16:34, Andy Shevchenko wrote: > On Fri, Nov 19, 2021 at 04:06:27PM +0100, Alejandro Colomar (man-pages) wrote: >> Yes, I would like to untangle the dependencies. >> >> The main reason I started doing this splitting >> is because I wouldn't be

Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

2021-11-19 Thread Alejandro Colomar (man-pages)
Hi Jani, On 11/19/21 13:47, Jani Nikula wrote: On Fri, 19 Nov 2021, Alejandro Colomar wrote: In the first and second commits I changed a lot of stuff in many parts, and that's why I CCd so many people (also in this cover letter). However, to avoid spamming, and since it would be a nightmare

Re: [Intel-gfx] [PATCH 00/17] Add memberof(), split some headers, and slightly simplify code

2021-11-19 Thread Alejandro Colomar (man-pages)
Hi Arnd, On 11/19/21 15:47, Arnd Bergmann wrote: > On Fri, Nov 19, 2021 at 12:36 PM Alejandro Colomar > wrote: >> >> Alejandro Colomar (17): >> linux/container_of.h: Add memberof(T, m) >> Use memberof(T, m) instead of explicit NULL dereference >> Replace some uses of memberof() by its