| Message ID | 1537183857-29173-1-git-send-email-jacopo+renesas@jmondi.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | media: adv748x: Allow probe with a single output endpoint | expand |
Hi Jacopo, Thank you for the patches. On Monday, 17 September 2018 14:30:53 EEST Jacopo Mondi wrote: > Hello Laurent, Kieran, Niklas, > to address the Ebisu board use case, this series allows the adv748x > driver to probe with a single output connection defined. > > Compared to v2, I have dropped the last patch, as without any dynamic > routing support it is not that helpful, and I've fixed most of commit > messages as suggested by Kieran. > > I have tested in 3 conditions on Salvator-X M3-W: > - AFE input not registered > - TXB not registered (Ebisu use case) > - AFE and TXB not registered > > Let me know if I can help testing this on Ebisu. For the whole series, Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> on Ebisu with your "[PATCH/RFT v2 0/8] arm64: dts: renesas: Ebisu: Add HDMI and CVBS input" patches. The driver now probes properly and I can capture frames, but they're all black :-S I don't think that's related to this series though. > v2 -> v3: > - Drop v2 patch [5/5] > - Add Kieran's tags and modify commit messages as he suggested > > Jacopo Mondi (4): > media: i2c: adv748x: Support probing a single output > media: i2c: adv748x: Handle TX[A|B] power management > media: i2c: adv748x: Conditionally enable only CSI-2 outputs > media: i2c: adv748x: Register only enabled inputs > > drivers/media/i2c/adv748x/adv748x-afe.c | 2 +- > drivers/media/i2c/adv748x/adv748x-core.c | 83 +++++++++++++++------------- > drivers/media/i2c/adv748x/adv748x-csi2.c | 29 ++++------- > drivers/media/i2c/adv748x/adv748x-hdmi.c | 2 +- > drivers/media/i2c/adv748x/adv748x.h | 19 ++++++-- > 5 files changed, 68 insertions(+), 67 deletions(-)
Hi Jacopo, On 17/09/18 12:30, Jacopo Mondi wrote: > Hello Laurent, Kieran, Niklas, > to address the Ebisu board use case, this series allows the adv748x driver > to probe with a single output connection defined. > > Compared to v2, I have dropped the last patch, as without any dynamic routing > support it is not that helpful, and I've fixed most of commit messages as > suggested by Kieran. Thank you, I've applied all of these patches to my tree and submitted as a pull request to Hans. -- Regards Kieran > > I have tested in 3 conditions on Salvator-X M3-W: > - AFE input not registered > - TXB not registered (Ebisu use case) > - AFE and TXB not registered > > Let me know if I can help testing this on Ebisu. > > Thanks > j > > v2 -> v3: > - Drop v2 patch [5/5] > - Add Kieran's tags and modify commit messages as he suggested > > Jacopo Mondi (4): > media: i2c: adv748x: Support probing a single output > media: i2c: adv748x: Handle TX[A|B] power management > media: i2c: adv748x: Conditionally enable only CSI-2 outputs > media: i2c: adv748x: Register only enabled inputs > > drivers/media/i2c/adv748x/adv748x-afe.c | 2 +- > drivers/media/i2c/adv748x/adv748x-core.c | 83 +++++++++++++++++--------------- > drivers/media/i2c/adv748x/adv748x-csi2.c | 29 ++++------- > drivers/media/i2c/adv748x/adv748x-hdmi.c | 2 +- > drivers/media/i2c/adv748x/adv748x.h | 19 ++++++-- > 5 files changed, 68 insertions(+), 67 deletions(-) > > -- > 2.7.4 >