On 25 September 2014 03:59, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: > On Wed, Sep 24, 2014 at 6:24 PM, Ilia Mirkin <imirkin@xxxxxxxxxxxx> wrote: >> On Wed, Sep 24, 2014 at 6:24 AM, Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: >>> Hi Dave, >>> >>> Just noticed that you've picked up the header rework stuff already, so >>> I've rebased that out again. Otherwise just two stragglers from the vblank >>> rework and the universal cursor planes locking fix. Plus sprinkling >>> container_of all over fbdev emulation from Fabian. >> >> I thought it was questionable whether these were desired... the >> pattern of casting between structs that contain others as bases is a >> pretty common one, and container_of just makes it harder to read. Did >> the discussion end in a way that concluded the opposite? I might have >> missed it. > > I like them since you can employ gcc to check mundane stuff for you. > And container_of is so massively established in the kernel (with > list_head alone) that I don't think any serious kernel hacker has > problem's reading it. Alan Cox complained a bit a bit on imo elitist > grounds so until someone goes on record with an official Nack I think > they're worth it. I like them, but I don't want to encourage them, they are over time going to just be conflict creators, which makes it sounds like work for me, So I don't mind this batch for the fbdev, but I don't want to see anymore unless they provide more value. also I'm on holidays until Tuesday, I'll have some access to stuff, but won't be on top of urgent things, so hopefully nothing urgent happens. Dave. _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
