[Intel-gfx] [PATCH] drm/i915: support address only i2c-over-aux transactions

Daniel Vetter daniel at ffwll.ch
Mon Apr 7 13:00:48 PDT 2014


On Mon, Apr 7, 2014 at 4:35 PM, Alex Deucher <alexdeucher at gmail.com> wrote:
> On Mon, Apr 7, 2014 at 5:37 AM, Jani Nikula <jani.nikula at intel.com> wrote:
>> To support bare address requests used by the drm dp helpers.
>>
>> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
>>
>> ---
>>
>> Hi Alex, similar to Thierry's patch for i915.
>>
>
> Looks good to me.
>
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>
> Do you want me to add this to the patch set?

Afaict we've done an unconditional msg_bytes = send_bytes + 4; in the
i915 driver before the conversion to the dp aux helper, which is why
I've slapped an r-b onto your patch without asking for a i915
adjustement. Otoh we have a pile of bugs for dp dongles still. Imo my
preferred approach would be to get the helper + radeon stuff in and
then hawk the i915 patch to a bunch of bug reporter and see whether it
sticks. Ofc if we already know that we need it it would be best to
merge it in one pull together with all your other patches.

Jani?
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the dri-devel mailing list