Re: experimental patch for toshiba_acpi

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

 



On Thu, 2009-02-26 at 14:03 +0000, Richard Hughes wrote:
> On Thu, 2009-02-26 at 08:12 -0500, John Belmonte wrote:
> > The proposed patch makes /dev/toshiba, and hence toshset, work on more
> > laptops-- it's a no-brainer to anyone with a practical mindset.
> 
> Making toshset work is not a practical goal.

I suggest you get a dictionary out and look up the work practical.
Because making it works is eminently practical.

>  Making hardware "just work"
> without any vendor specific commands is a better goal. Exposing low
> level, device specific, hardware details up through a device node isn't
> a good plan at all.
> 

Well Toshiba who in part wrote the ACPI specification take *EXACTLY* the
same route on Windows. There is a device driver that creates a TVALD
device (the name has changed slightly on newwer models and I cannot
remember it off hand). To change an HCI setting you open up the TVALD
and do a ioctl in *exactly* the same manner as you do on /dev/toshiba.
The similarity is so striking I could almost say Toshiba copied me.

Now if Toshiba who in part wrote the ACPI specification think this is
the way to go, and the people who know the most about the HCI in the
free software world also think this is the way to go, what are your
qualifications to argue with them, given you have virtually no knowledge
of the Toshiba HCI and would not appear to even own a Toshiba designed
and manufactured laptop anyway.


JAB.

-- 
Jonathan A. Buzzard                 Email: jonathan (at) buzzard.me.uk
Fife, United Kingdom.

--
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