On Fri, Feb 03, 2012 at 10:08:12AM +, Dave Airlie wrote:
> On Wed, Feb 1, 2012 at 1:05 PM, Sascha Hauer
> wrote:
> > On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
> >>
> >>
> >> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> >> > > index 8d593ad..cdbbb40 10
On Fri, Feb 03, 2012 at 10:08:12AM +, Dave Airlie wrote:
> On Wed, Feb 1, 2012 at 1:05 PM, Sascha Hauer wrote:
> > On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
> >>
> >>
> >> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> >> > > index 8d593ad..cdbbb40 10064
On Wed, Feb 1, 2012 at 1:05 PM, Sascha Hauer wrote:
> On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
>>
>>
>> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
>> > > index 8d593ad..cdbbb40 100644
>> > > --- a/include/drm/drm_crtc.h
>> > > +++ b/include/drm/drm_crtc.h
On Wed, Feb 1, 2012 at 1:05 PM, Sascha Hauer wrote:
> On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
>>
>>
>> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
>> > > index 8d593ad..cdbbb40 100644
>> > > --- a/include/drm/drm_crtc.h
>> > > +++ b/include/drm/drm_crtc.h
On Wed, Feb 01, 2012 at 02:05:38PM +0100, Sascha Hauer wrote:
> On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
> >
> >
> > > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > > > index 8d593ad..cdbbb40 100644
> > > > --- a/include/drm/drm_crtc.h
> > > > +++ b/inclu
On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
>
>
> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > > index 8d593ad..cdbbb40 100644
> > > --- a/include/drm/drm_crtc.h
> > > +++ b/include/drm/drm_crtc.h
> > > @@ -394,7 +394,7 @@ struct drm_crtc {
> > > s64 fr
On Wed, Feb 01, 2012 at 11:48:53AM +, Chris Wilson wrote:
> On Wed, 1 Feb 2012 11:38:32 +0100, Sascha Hauer
> wrote:
> > Creating an enum property is a common pattern, so create
> > a convenience function for this and use it where appropriate.
>
> Similar naming comments apply as for drm_pr
On Wed, 1 Feb 2012 11:38:32 +0100, Sascha Hauer
wrote:
> Creating an enum property is a common pattern, so create
> a convenience function for this and use it where appropriate.
Similar naming comments apply as for drm_property_create_range. However,
I did spot something anomalous...
> diff --
Creating an enum property is a common pattern, so create
a convenience function for this and use it where appropriate.
Signed-off-by: Sascha Hauer
---
drivers/gpu/drm/drm_crtc.c| 100 +---
drivers/gpu/drm/i915/intel_modes.c| 28 +++-
drivers
- Original Message -
> From: "Chris Wilson"
> To: "Sascha Hauer" , dri-devel at
> lists.freedesktop.org
> Cc: kernel at pengutronix.de
> Sent: Wednesday, 1 February, 2012 11:48:53 AM
> Subject: Re: [PATCH 14/20] drm: add convenience function to cre
On Wed, Feb 01, 2012 at 02:05:38PM +0100, Sascha Hauer wrote:
> On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
> >
> >
> > > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > > > index 8d593ad..cdbbb40 100644
> > > > --- a/include/drm/drm_crtc.h
> > > > +++ b/inclu
On Wed, Feb 01, 2012 at 07:55:40AM -0500, David Airlie wrote:
>
>
> > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > > index 8d593ad..cdbbb40 100644
> > > --- a/include/drm/drm_crtc.h
> > > +++ b/include/drm/drm_crtc.h
> > > @@ -394,7 +394,7 @@ struct drm_crtc {
> > > s64 fr
- Original Message -
> From: "Chris Wilson"
> To: "Sascha Hauer" , dri-devel@lists.freedesktop.org
> Cc: ker...@pengutronix.de
> Sent: Wednesday, 1 February, 2012 11:48:53 AM
> Subject: Re: [PATCH 14/20] drm: add convenience function to create an enum
On Wed, Feb 01, 2012 at 11:48:53AM +, Chris Wilson wrote:
> On Wed, 1 Feb 2012 11:38:32 +0100, Sascha Hauer
> wrote:
> > Creating an enum property is a common pattern, so create
> > a convenience function for this and use it where appropriate.
>
> Similar naming comments apply as for drm_pr
On Wed, 1 Feb 2012 11:38:32 +0100, Sascha Hauer wrote:
> Creating an enum property is a common pattern, so create
> a convenience function for this and use it where appropriate.
Similar naming comments apply as for drm_property_create_range. However,
I did spot something anomalous...
> diff --g
Creating an enum property is a common pattern, so create
a convenience function for this and use it where appropriate.
Signed-off-by: Sascha Hauer
---
drivers/gpu/drm/drm_crtc.c| 100 +---
drivers/gpu/drm/i915/intel_modes.c| 28 +++-
drivers
16 matches
Mail list logo
