On 10 July 2014 10:16, Dave Airlie <airlied@xxxxxxxxx> wrote:
> On 10 July 2014 17:09, Daniel Vetter <daniel@xxxxxxxx> wrote:
>> On Thu, Jul 10, 2014 at 12:02:15PM +1000, Dave Airlie wrote:
>>> On 9 July 2014 21:42, Sachin Kamat <sachin.kamat@xxxxxxxxxxx> wrote:
>>> > Commit 34ea3d386347 ("drm: add register and unregister functions
>>> > for connectors") probably missed out converting the
>>> > drm_sysfs_connector_remove instances in the following files.
>>> > Without this patch we get the following compilation error:
>>> > ERROR: "drm_sysfs_connector_remove" [drivers/gpu/drm/tilcdc/tilcdc.ko] undefined!
>>> >
>>> > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxxx>
>>> > CC: Thomas Wood <thomas.wood@xxxxxxxxx>
>>> > CC: David Herrmann <dh.herrmann@xxxxxxxxx>
>>> > CC: Daniel Vetter <daniel.vetter@xxxxxxxx>
>>> > ---
>>> > Only compile tested.
>>>
>>> Oops, applied thanks,
>>>
>>> I did an arm build just on the drm dir, forgot to let it run to
>>> completion over the whole tree.
>>
>> Yeah, my apologies for breaking things. Nowadays I rely on the 0-day
>> builder for arm build testing, dunno why it didn't report this one here.
>> Might be good to check with Wu Fengguang that your driver is included - he
>> tends to catch other drm breakage on arm drivers well.
It actually looks like the calls to drm_sysfs_connector_remove were
added after the patch to remove it was applied:
16dcbde drm/tilcdc: tfp410: fix dangling sysfs connector node
daa15b4 drm/tilcdc: slave: fix dangling sysfs connector node
e396900 drm/tilcdc: panel: fix dangling sysfs connector node
This probably explains why it wasn't reported from the builder on the
original patch.
>
> I just got the report now :-) after I fixed it!
>
> Dave.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
