| Message ID | 20200610124623.51085-1-kieran@bingham.xyz (mailing list archive) |
|---|---|
| Headers | show |
| Series | GMSL fixups ready for v10. | expand |
Hi Kieran,
You forgot to mention that the previous 3 patches posted as v9.1 have
already been merged into your GMSL branch and these are based upon that...:
- Use the same default mbus_fmt everywhere
- Don't provide GPIO names
- Fix dev->of_node refcnting
--
Kieran
On 10/06/2020 13:46, Kieran Bingham wrote:
> From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
>
> Unfortunately we missed the 5.8 window with reviews that came in really
> late in the merge window, but hey ... more time to do more fixups to
> GMSL....
>
> Jacopo has done various updates to fix the DT Yaml validation, though
> there is probably some scope there that means we might need a
> meta-schema for I2C-Muxes ... but that gets more complicated and
> probably a bit out of scope for now.
>
> I've taken more review comments into consideration and handled more
> fixes for the drivers, which we expect to get to a point that these can
> now be merged for the next release.
>
> Sakari has at least provisionalyl given us his blessing - so lets hope
> v10 is the last - and we can finally see max9286/rdacm20 get upstream.
>
> --
> Kieran
>
>
> Jacopo Mondi (1):
> fixes! [max9286-dt]: Fix dt-validation
>
> Kieran Bingham (8):
> fixes! [max9286]: Fix whitespace indent
> fixes! [max9286]: Validate link formats
> fixes! [max9286]: Use single sample per pixel
> fixes! [max9286]: Remove redundant DPHY check
> fixes! [max9286]: Remove redundant call
> fixes! [max9286-dt]: Add GPIO controller support
> fixes! [max9286-dt]: Correctly match the hex camera node reg
> fixes! [rdacm20]: Use usleep_range over mdelay(10)
>
> .../bindings/media/i2c/maxim,max9286.yaml | 91 +++++++++++++++++--
> drivers/media/i2c/max9286.c | 45 ++++-----
> drivers/media/i2c/rdacm20.c | 4 +-
> 3 files changed, 107 insertions(+), 33 deletions(-)
>
Hi Kieran, On Wed, Jun 10, 2020 at 01:46:14PM +0100, Kieran Bingham wrote: > From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > Unfortunately we missed the 5.8 window with reviews that came in really > late in the merge window, but hey ... more time to do more fixups to > GMSL.... > > Jacopo has done various updates to fix the DT Yaml validation, though > there is probably some scope there that means we might need a > meta-schema for I2C-Muxes ... but that gets more complicated and > probably a bit out of scope for now. I've replied with a patch that fixes the dt-schema validation warning we had. The example in the schema was actually wrong. If you agree with the patch, please squash it in forthcoming v10. Thanks j > > I've taken more review comments into consideration and handled more > fixes for the drivers, which we expect to get to a point that these can > now be merged for the next release. > > Sakari has at least provisionalyl given us his blessing - so lets hope > v10 is the last - and we can finally see max9286/rdacm20 get upstream. > > -- > Kieran > > > Jacopo Mondi (1): > fixes! [max9286-dt]: Fix dt-validation > > Kieran Bingham (8): > fixes! [max9286]: Fix whitespace indent > fixes! [max9286]: Validate link formats > fixes! [max9286]: Use single sample per pixel > fixes! [max9286]: Remove redundant DPHY check > fixes! [max9286]: Remove redundant call > fixes! [max9286-dt]: Add GPIO controller support > fixes! [max9286-dt]: Correctly match the hex camera node reg > fixes! [rdacm20]: Use usleep_range over mdelay(10) > > .../bindings/media/i2c/maxim,max9286.yaml | 91 +++++++++++++++++-- > drivers/media/i2c/max9286.c | 45 ++++----- > drivers/media/i2c/rdacm20.c | 4 +- > 3 files changed, 107 insertions(+), 33 deletions(-) > > -- > 2.25.1 >
From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> Unfortunately we missed the 5.8 window with reviews that came in really late in the merge window, but hey ... more time to do more fixups to GMSL.... Jacopo has done various updates to fix the DT Yaml validation, though there is probably some scope there that means we might need a meta-schema for I2C-Muxes ... but that gets more complicated and probably a bit out of scope for now. I've taken more review comments into consideration and handled more fixes for the drivers, which we expect to get to a point that these can now be merged for the next release. Sakari has at least provisionalyl given us his blessing - so lets hope v10 is the last - and we can finally see max9286/rdacm20 get upstream. -- Kieran Jacopo Mondi (1): fixes! [max9286-dt]: Fix dt-validation Kieran Bingham (8): fixes! [max9286]: Fix whitespace indent fixes! [max9286]: Validate link formats fixes! [max9286]: Use single sample per pixel fixes! [max9286]: Remove redundant DPHY check fixes! [max9286]: Remove redundant call fixes! [max9286-dt]: Add GPIO controller support fixes! [max9286-dt]: Correctly match the hex camera node reg fixes! [rdacm20]: Use usleep_range over mdelay(10) .../bindings/media/i2c/maxim,max9286.yaml | 91 +++++++++++++++++-- drivers/media/i2c/max9286.c | 45 ++++----- drivers/media/i2c/rdacm20.c | 4 +- 3 files changed, 107 insertions(+), 33 deletions(-)