Hi Neil,
On 03-03-2017 09:07, Neil Armstrong wrote:
>
> The problem is that the HPD/RxSense is tied to this phy_mask and glued into
> the
> dw-hdmi driver.
>
> The *real* solution would be to completely separate the HPD/RxSense irq
> handling to
> a separate driver as a shared irq...
>
> If Jose is willing to give me some documentation and Freescale some boards,
> I'll be
> happy to do it !
>
>
Hmm, why don't get rid of phy_mask totally and just return the
new mask in update_hpd() function? Or add a get_hpd_status()
callback. (I also think there are too many callbacks. For example
we could have: setup, set_status, clear and then just use
parameters when needed:
void setup(bool force, bool disabled, bool rxsense)
void set_status(bool enable, bool enable_ints)
void clear()
What do you think? I only checked quickly the code, don't know if
this is enough.
Best regards,
Jose Miguel Abreu
