<div class="gmail_quote">On Fri, Apr 13, 2012 at 17:18, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Fri, 13 Apr 2012 17:08:41 -0300, Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com">eugeni.dodonov@intel.com</a>> wrote:<br>
> Prevent a NULL pointer exception when we are trying to retrieve EDID data<br>
> from non-existent adapter.<br>
<br>
</div>This just means that a HDMI with a garbage ddc_bus is never detected.<br>
Since we control ddc_bus entirely, it means that the initialisation is<br>
completely bogus. Please fix the root cause and not paper over<br>
programming bugs.<br></blockquote><div><br></div><div><div>Sorry, I think I wasn't clear in my cover letter about this one.. I am not hitting any issue with this current branch I sent; but the nature of the code we have after the gmbus refactoring made me a bit paranoid about accessing properties of something that could potential be NULL.</div>

<div><br></div><div>As far as I can see, we should not hit this with any combination of outputs on any hardware that is already covered by the wikipedia :). So I am fine with dropping this patch if you also think that it does not adds anything relevant to us. Or perhaps I should refactor it into adding some WARN's instead?</div>

</div><div><br></div></div>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br></a><br>