Re: [patch 19/19] asus_acpi: Add support for Asus Z81SP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 22 Dec 2006 21:13:23 -0500
Len Brown <lenb@xxxxxxxxxx> wrote:

> NAK
> 
> The proper way to update the asus_acpi driver
> is through its (new) maintainer and mailing list on acpi4asus-user@xxxxxxxxxxxxxxxxxxxxx
> which will forward the patches through the acpi tree when tested.
> 

So why the NAK?  The patch is good, isn't it?



> 
> On Tuesday 19 December 2006 15:56, akpm@xxxxxxxx wrote:
> > From: Matthew C Campbell <calvinmc@xxxxxxxxx>
> > 
> > Adds support in asus_acpi for the Asus Z81SP laptop.  This preserves all
> > old functionality when improperly detected as well as enabling Bluetooth
> > support.
> > 
> > Signed-off-by: Matthew C Campbell <calvinmc@xxxxxxxxx>
> > Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
> > ---
> > 
> >  drivers/acpi/asus_acpi.c |   14 +++++++++++++-
> >  1 files changed, 13 insertions(+), 1 deletion(-)
> > 
> > diff -puN drivers/acpi/asus_acpi.c~asus_acpi-add-support-for-asus-z81sp drivers/acpi/asus_acpi.c
> > --- a/drivers/acpi/asus_acpi.c~asus_acpi-add-support-for-asus-z81sp
> > +++ a/drivers/acpi/asus_acpi.c
> > @@ -145,6 +145,7 @@ struct asus_hotk {
> >  		W5F,            //W5F
> >  		V6V,            //V6V
> >  		xxN,		//M2400N, M3700N, M5200N, M6800N, S1300N, S5200N
> > +		A4S,            //Z81sp
> >  		//(Centrino)
> >  		END_MODEL
> >  	} model;		//Models currently supported
> > @@ -450,7 +451,16 @@ static struct model_data model_conf[END_
> >  	 .brightness_set = "SPLV",
> >  	 .brightness_get = "GPLV",
> >  	 .display_set = "SDSP",
> > -	 .display_get = "\\ADVG"}
> > +	.display_get = "\\ADVG"},
> > +
> > +	{
> > +		.name              = "A4S",
> > +		.brightness_set    = "SPLV",
> > +		.brightness_get    = "GPLV",
> > +		.mt_bt_switch      = "BLED",
> > +		.mt_wled           = "WLED"
> > +	}
> > +
> >  };
> >  
> >  /* procdir we use */
> > @@ -1188,6 +1198,8 @@ static int asus_model_match(char *model)
> >  		return W5F;
> >  	else if (strncmp(model, "V6V", 3) == 0)
> >  		return V6V;
> > +	else if (strncmp(model, "A4S", 3) == 0)
> > +		return A4S;
> >  	else
> >  		return END_MODEL;
> >  }
> > _
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > 
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux