From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
To: Corentin Chary <corentin.chary@gmail.com>
Cc: Andy Ross <andy.ross@windriver.com>,
linux-input@vger.kernel.org,
acpi4asus-user@lists.sourceforge.net,
platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH 0/4] Pegatron Lucid tablet acceleromter/ALS
Date: Tue, 25 Jan 2011 09:12:38 -0800 [thread overview]
Message-ID: <20110125171238.GE19701@core.coreip.homeip.net> (raw)
In-Reply-To: <AANLkTi=tQpQ0wjfx-Cp13FuCSQNFitN7XAg-V2AcSHz2@mail.gmail.com>
On Tue, Jan 25, 2011 at 05:05:05PM +0000, Corentin Chary wrote:
> On Tue, Jan 25, 2011 at 4:43 PM, Andy Ross <andy.ross@windriver.com> wrote:
> > On 01/24/2011 11:30 PM, Corentin Chary wrote:
> >> Dmitry Torokhov wrote:
> >> > I am happy with the input part but I wonder if it really needs to be
> >> > separate? If you roll it up into asus-laptop you would not need a new
> >> > platform device and then it would be what - 20 lines of code?
> >> >
> >> > Corentin, what do you think?
> >>
> >> I think that it should be consistent, if there is already some code
> >> for pegatron tablet in asus-laptop, then why a new platform device
> >> only for the accelerometer ?
> >
> > No strong feelings here. I made it a separate file because
> > asus-laptop.c is already very large and this was something that could
> > be fairly easily split out, but obviously it's a really trivial
> > device.
> >
> >> Andy, what is shared with asus-laptop exactly ? What sysfs files are
> >> working and enabled ? Are backlight and led classes working ?
> >
> > Just the key device, which exports the single touch area in the corner
> > of the tablet as KEY_MEDIA. Some of the other sysfs devices
> > (bluetooth, display, leds) appear because the methods exist, but
> > they're stubbed out in the ASL and thus noops.
> >
> > Andy
>
> If everybody is ok with that, I think you should move the code back
> into asus-laptop.c. But if you do so, don't forget to modify the help
> text of ASUS_LAPTOP section in KConfig.
>
> Should we also change the description to
> "Asus/Medion/JVC/Lenovo/Pegatron Extra" ? That seems really ugly
> but....
"Kitchen sink" so we do not need to adjust the description going
forward? ;P
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-01-25 17:12 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-24 22:48 [PATCH 0/4] Pegatron Lucid tablet acceleromter/ALS Andy Ross
[not found] ` <1295909288-32650-1-git-send-email-andy.ross-CWA4WttNNZF54TAoqtyWWQ@public.gmane.org>
2011-01-24 22:48 ` [PATCH 1/4] asus-laptop: Device detection for Pegatron Lucid tablets Andy Ross
2011-01-24 23:05 ` [PATCH 0/4] Pegatron Lucid tablet acceleromter/ALS Dmitry Torokhov
2011-01-25 7:30 ` Corentin Chary
2011-01-25 16:43 ` Andy Ross
2011-01-25 17:05 ` Corentin Chary
[not found] ` <AANLkTi=tQpQ0wjfx-Cp13FuCSQNFitN7XAg-V2AcSHz2-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-01-25 17:09 ` Andy Ross
[not found] ` <4D3F03B0.7030607-CWA4WttNNZF54TAoqtyWWQ@public.gmane.org>
2011-01-25 17:10 ` Corentin Chary
2011-01-25 17:12 ` Dmitry Torokhov [this message]
2011-01-24 22:48 ` [PATCH 2/4] asus-laptop: Pegatron Lucid ALS support Andy Ross
2011-01-24 22:48 ` [PATCH 3/4] asus-laptop: Support pega_accel accelerometer driver Andy Ross
2011-01-24 22:48 ` [PATCH 4/4] input: Pegatron Lucid accelerometer Andy Ross
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110125171238.GE19701@core.coreip.homeip.net \
--to=dmitry.torokhov@gmail.com \
--cc=acpi4asus-user@lists.sourceforge.net \
--cc=andy.ross@windriver.com \
--cc=corentin.chary@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).