diff mbox

gma500: remove duplicate FB_REG09 define

Message ID 20140609152023.GO9600@mwanda (mailing list archive)
State New, archived
Headers show

Commit Message

Dan Carpenter June 9, 2014, 3:20 p.m. UTC
The FB_REG09 define is cut and pasted twice so we can remove the second
instance.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

Comments

Jani Nikula June 9, 2014, 3:41 p.m. UTC | #1
Patrik, Alan -

Should you add a MAINTAINERS entry for gma500 with one or both of you as
maintainers? What's the status?

BR,
Jani.


On Mon, 09 Jun 2014, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> The FB_REG09 define is cut and pasted twice so we can remove the second
> instance.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>
> diff --git a/drivers/gpu/drm/gma500/mid_bios.c b/drivers/gpu/drm/gma500/mid_bios.c
> index a97e38e..d75ecb3 100644
> --- a/drivers/gpu/drm/gma500/mid_bios.c
> +++ b/drivers/gpu/drm/gma500/mid_bios.c
> @@ -39,7 +39,6 @@ static void mid_get_fuse_settings(struct drm_device *dev)
>  #define FB_REG06 0xD0810600
>  #define FB_MIPI_DISABLE  (1 << 11)
>  #define FB_REG09 0xD0810900
> -#define FB_REG09 0xD0810900
>  #define FB_SKU_MASK  0x7000
>  #define FB_SKU_SHIFT 12
>  #define FB_SKU_100 0
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
Patrik Jakobsson June 10, 2014, 9:18 p.m. UTC | #2
On Mon, Jun 9, 2014 at 5:41 PM, Jani Nikula <jani.nikula@linux.intel.com> wrote:
>
> Patrik, Alan -
>
> Should you add a MAINTAINERS entry for gma500 with one or both of you as
> maintainers? What's the status?
>
> BR,
> Jani.

Hi Jani

Adding an entry would make people think that I have time to spend on gma500
development or is in some way responsible for it. At the moment, that is sadly
not the case. I have a few things on my todo-list which I intend to fix but
after that I would much rather work on something more productive.

My gma500 itch has been scratched for quite a while and perhaps it's time to
move on.

Regards
Patrik

>
>
> On Mon, 09 Jun 2014, Dan Carpenter <dan.carpenter@oracle.com> wrote:
>> The FB_REG09 define is cut and pasted twice so we can remove the second
>> instance.
>>
>> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>>
>> diff --git a/drivers/gpu/drm/gma500/mid_bios.c b/drivers/gpu/drm/gma500/mid_bios.c
>> index a97e38e..d75ecb3 100644
>> --- a/drivers/gpu/drm/gma500/mid_bios.c
>> +++ b/drivers/gpu/drm/gma500/mid_bios.c
>> @@ -39,7 +39,6 @@ static void mid_get_fuse_settings(struct drm_device *dev)
>>  #define FB_REG06 0xD0810600
>>  #define FB_MIPI_DISABLE  (1 << 11)
>>  #define FB_REG09 0xD0810900
>> -#define FB_REG09 0xD0810900
>>  #define FB_SKU_MASK  0x7000
>>  #define FB_SKU_SHIFT 12
>>  #define FB_SKU_100 0
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
> --
> Jani Nikula, Intel Open Source Technology Center
Alan Cox June 10, 2014, 9:42 p.m. UTC | #3
> Adding an entry would make people think that I have time to spend on gma500
> development or is in some way responsible for it. At the moment, that is sadly
> not the case. I have a few things on my todo-list which I intend to fix but
> after that I would much rather work on something more productive.

A wise man indeed.

I'll try and keep an eye on it for minor fixes but hopefully its "done".
Apart from one person trying to port Linux to the Lenovo Tablet 2 I
don't think there is anything that is likely to happen here new ?

Alan
diff mbox

Patch

diff --git a/drivers/gpu/drm/gma500/mid_bios.c b/drivers/gpu/drm/gma500/mid_bios.c
index a97e38e..d75ecb3 100644
--- a/drivers/gpu/drm/gma500/mid_bios.c
+++ b/drivers/gpu/drm/gma500/mid_bios.c
@@ -39,7 +39,6 @@  static void mid_get_fuse_settings(struct drm_device *dev)
 #define FB_REG06 0xD0810600
 #define FB_MIPI_DISABLE  (1 << 11)
 #define FB_REG09 0xD0810900
-#define FB_REG09 0xD0810900
 #define FB_SKU_MASK  0x7000
 #define FB_SKU_SHIFT 12
 #define FB_SKU_100 0