[PATCH 0/8] drm: atmel-hlcdc: various fixes/improvements

2016-03-17 Thread Dave Airlie
On 1 March 2016 at 20:55, Boris Brezillon
 wrote:
> On Wed,  6 Jan 2016 11:19:21 +0100
> Boris Brezillon  wrote:
>
>> Hello,
>>
>> This series is a collection of fixes and improvements for the
>> atmel-hlcdc driver.
>>
>> The main feature added here is the support for external RGB -> XXX
>> bridges (patch 6 and 7).
>>
>> The first patch is a fix preventing a potential memory leak.
>> Patch 2 is adding support for asynchronous mode setting, which was
>> supported before the migration to atomic mode setting.
>>
>> Patch 3 is just a minor fix to expose the real encoder and connector
>> types (we are currently exposing an LVDS encoder/connector, which is
>> wrong since the display controller output the pixel stream in raw
>> RGB).
>>
>> Patch 4 is removing useless fields and functions which were left
>> when moving to atomic modesetting.
>>
>> And patch 8 is just a cosmetic patch moving the mode checking code
>> from ->atomic_check() to ->mode_fixup().
>
> Dave, Daniel, any comment on this series (AKA ping)?

On internal driver stuff, I'd usually only look on a git pull :-)

Dave.


[PATCH 0/8] drm: atmel-hlcdc: various fixes/improvements

2016-03-17 Thread Boris Brezillon
On Thu, 17 Mar 2016 12:43:11 +1000
Dave Airlie  wrote:

> On 1 March 2016 at 20:55, Boris Brezillon
>  wrote:
> > On Wed,  6 Jan 2016 11:19:21 +0100
> > Boris Brezillon  wrote:
> >
> >> Hello,
> >>
> >> This series is a collection of fixes and improvements for the
> >> atmel-hlcdc driver.
> >>
> >> The main feature added here is the support for external RGB -> XXX
> >> bridges (patch 6 and 7).
> >>
> >> The first patch is a fix preventing a potential memory leak.
> >> Patch 2 is adding support for asynchronous mode setting, which was
> >> supported before the migration to atomic mode setting.
> >>
> >> Patch 3 is just a minor fix to expose the real encoder and connector
> >> types (we are currently exposing an LVDS encoder/connector, which is
> >> wrong since the display controller output the pixel stream in raw
> >> RGB).
> >>
> >> Patch 4 is removing useless fields and functions which were left
> >> when moving to atomic modesetting.
> >>
> >> And patch 8 is just a cosmetic patch moving the mode checking code
> >> from ->atomic_check() to ->mode_fixup().
> >
> > Dave, Daniel, any comment on this series (AKA ping)?
> 
> On internal driver stuff, I'd usually only look on a git pull :-)

Yep, Daniel told me that both of you usually don't review internal
driver changes. I'll send a PR soon (after the merge window of course).

Thanks,

Boris

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


[PATCH 0/8] drm: atmel-hlcdc: various fixes/improvements

2016-03-16 Thread Nicolas Ferre
Le 06/01/2016 11:19, Boris Brezillon a écrit :
> Hello,
> 
> This series is a collection of fixes and improvements for the
> atmel-hlcdc driver.
> 
> The main feature added here is the support for external RGB -> XXX
> bridges (patch 6 and 7).
> 
> The first patch is a fix preventing a potential memory leak.
> Patch 2 is adding support for asynchronous mode setting, which was
> supported before the migration to atomic mode setting.
> 
> Patch 3 is just a minor fix to expose the real encoder and connector
> types (we are currently exposing an LVDS encoder/connector, which is
> wrong since the display controller output the pixel stream in raw
> RGB).
> 
> Patch 4 is removing useless fields and functions which were left
> when moving to atomic modesetting.
> 
> And patch 8 is just a cosmetic patch moving the mode checking code
> from ->atomic_check() to ->mode_fixup().

To give more weight to this series, you can also add my:

Tested-by: Nicolas Ferre 

Bye,


> Boris Brezillon (8):
>   drm: atmel-hlcdc: add a ->cleanup_fb() operation
>   drm: atmel-hlcdc: support asynchronous atomic commit operations
>   drm: atmel-hlcdc: fix connector and encoder types
>   drm: atmel-hlcdc: remove leftovers from atomic mode setting migration
>   drm: atmel-hlcdc: support extended timing ranges on sama5d4 and
> sama5d2
>   drm: atmel-hlcdc: move output mode selection in CRTC implementation
>   drm: atmel-hlcdc: rework the output code to support drm bridges
>   drm: atmel-hlcdc: check display mode validity in crtc->mode_fixup()
> 
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c   | 148 +-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 123 ++-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h |  16 ++
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 249 
> ++-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c  |  22 +-
>  5 files changed, 400 insertions(+), 158 deletions(-)
> 


-- 
Nicolas Ferre


[PATCH 0/8] drm: atmel-hlcdc: various fixes/improvements

2016-03-01 Thread Boris Brezillon
On Wed,  6 Jan 2016 11:19:21 +0100
Boris Brezillon  wrote:

> Hello,
> 
> This series is a collection of fixes and improvements for the
> atmel-hlcdc driver.
> 
> The main feature added here is the support for external RGB -> XXX
> bridges (patch 6 and 7).
> 
> The first patch is a fix preventing a potential memory leak.
> Patch 2 is adding support for asynchronous mode setting, which was
> supported before the migration to atomic mode setting.
> 
> Patch 3 is just a minor fix to expose the real encoder and connector
> types (we are currently exposing an LVDS encoder/connector, which is
> wrong since the display controller output the pixel stream in raw
> RGB).
> 
> Patch 4 is removing useless fields and functions which were left
> when moving to atomic modesetting.
> 
> And patch 8 is just a cosmetic patch moving the mode checking code
> from ->atomic_check() to ->mode_fixup().

Dave, Daniel, any comment on this series (AKA ping)?

> 
> Best Regards,
> 
> Boris
> 
> Boris Brezillon (8):
>   drm: atmel-hlcdc: add a ->cleanup_fb() operation
>   drm: atmel-hlcdc: support asynchronous atomic commit operations
>   drm: atmel-hlcdc: fix connector and encoder types
>   drm: atmel-hlcdc: remove leftovers from atomic mode setting migration
>   drm: atmel-hlcdc: support extended timing ranges on sama5d4 and
> sama5d2
>   drm: atmel-hlcdc: move output mode selection in CRTC implementation
>   drm: atmel-hlcdc: rework the output code to support drm bridges
>   drm: atmel-hlcdc: check display mode validity in crtc->mode_fixup()
> 
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c   | 148 +-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 123 ++-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h |  16 ++
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 249 
> ++-
>  drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c  |  22 +-
>  5 files changed, 400 insertions(+), 158 deletions(-)
> 



-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com


[PATCH 0/8] drm: atmel-hlcdc: various fixes/improvements

2016-01-06 Thread Boris Brezillon
Hello,

This series is a collection of fixes and improvements for the
atmel-hlcdc driver.

The main feature added here is the support for external RGB -> XXX
bridges (patch 6 and 7).

The first patch is a fix preventing a potential memory leak.
Patch 2 is adding support for asynchronous mode setting, which was
supported before the migration to atomic mode setting.

Patch 3 is just a minor fix to expose the real encoder and connector
types (we are currently exposing an LVDS encoder/connector, which is
wrong since the display controller output the pixel stream in raw
RGB).

Patch 4 is removing useless fields and functions which were left
when moving to atomic modesetting.

And patch 8 is just a cosmetic patch moving the mode checking code
from ->atomic_check() to ->mode_fixup().

Best Regards,

Boris

Boris Brezillon (8):
  drm: atmel-hlcdc: add a ->cleanup_fb() operation
  drm: atmel-hlcdc: support asynchronous atomic commit operations
  drm: atmel-hlcdc: fix connector and encoder types
  drm: atmel-hlcdc: remove leftovers from atomic mode setting migration
  drm: atmel-hlcdc: support extended timing ranges on sama5d4 and
sama5d2
  drm: atmel-hlcdc: move output mode selection in CRTC implementation
  drm: atmel-hlcdc: rework the output code to support drm bridges
  drm: atmel-hlcdc: check display mode validity in crtc->mode_fixup()

 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c   | 148 +-
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 123 ++-
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h |  16 ++
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 249 ++-
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c  |  22 +-
 5 files changed, 400 insertions(+), 158 deletions(-)

-- 
2.1.4