Re: [PATCH v5 1/1] applesmc: Re-work SMC comms

2020-11-11 Thread Henrik Rydberg
overlong udelay()") Reported-by: Andreas Kemnade Tested-by: Andreas Kemnade # MacBookAir6,2 Acked-by: Arnd Bergmann Signed-off-by: Brad Campbell Signed-off-by: Henrik Rydberg --- Changelog : v1 : Initial attempt v2 : Address logic and coding style v3 : Removed some debug hangover. Added tested-b

Re: [PATCH v3] applesmc: Re-work SMC comms

2020-11-09 Thread Henrik Rydberg
Hi Brad, Out of morbid curiosity I grabbed an older MacOS AppleSMC.kext (10.7) and ran it through the disassembler. Every read/write to the SMC starts the same way with a check to make sure the SMC is in a sane state. If it's not, a read command is sent to try and kick it back into line : Wa

Re: [PATCH v3] applesmc: Re-work SMC comms

2020-11-08 Thread Henrik Rydberg
On 2020-11-08 12:57, Brad Campbell wrote: On 8/11/20 9:14 pm, Henrik Rydberg wrote: On Sun, Nov 08, 2020 at 09:35:28AM +0100, Henrik Rydberg wrote: Hi Brad, On 2020-11-08 02:00, Brad Campbell wrote: G'day Henrik, I noticed you'd also loosened up the requirement for SMC_STAT

Re: [PATCH v3] applesmc: Re-work SMC comms

2020-11-08 Thread Henrik Rydberg
On Sun, Nov 08, 2020 at 09:35:28AM +0100, Henrik Rydberg wrote: > Hi Brad, > > On 2020-11-08 02:00, Brad Campbell wrote: > > G'day Henrik, > > > > I noticed you'd also loosened up the requirement for SMC_STATUS_BUSY in > > read_smc(). I assume > >

Re: [PATCH v3] applesmc: Re-work SMC comms

2020-11-08 Thread Henrik Rydberg
restore function with the changes previously committed. Tested on : MacbookAir6,2 MacBookPro11,1 iMac12,2 Fixes: fff2d0f701e6 ("hwmon: (applesmc) avoid overlong udelay()") Reported-by: Andreas Kemnade Tested-by: Andreas Kemnade # MacBookAir6,2 Acked-by: Arnd Bergmann Signed-off-by: B

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-08 Thread Henrik Rydberg
Hi Brad, G'day Henrik, Which kernel was this based on? It won't apply to my 5.9 tree. I was being lazy and applied the diff to linus/master on top of my current stable branch. More importantly, I sent the mail out from an email client that may not format the patch properly; I'll fix that.

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-07 Thread Henrik Rydberg
On 2020-11-06 21:02, Henrik Rydberg wrote: So as it stands, it does not work at all. I will continue to check another machine, and see if I can get something working. On the MacBookAir3,1 the situation is somewhat better. The first three tree positions result in zero failures and 10 reads per

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-06 Thread Henrik Rydberg
So as it stands, it does not work at all. I will continue to check another machine, and see if I can get something working. On the MacBookAir3,1 the situation is somewhat better. The first three tree positions result in zero failures and 10 reads per second. The fourth yields zero failues and

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-06 Thread Henrik Rydberg
I can't guarantee it won't break older machines which is why I've asked for help testing it. I only have a MacbookPro 11,1 and an iMac 12,2. It fixes both of those. Help testing would be much appreciated. I see, this makes much more sense. I may be able to run some tests tonight. Meanwhile,

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-05 Thread Henrik Rydberg
On 2020-11-05 09:30, Brad Campbell wrote: On 5/11/20 6:56 pm, Henrik Rydberg wrote: Hi Brad, Great to see this effort, it is certainly an area which could be improved. After having seen several generations of Macbooks while modifying much of that code, it became clear that the SMC

Re: [PATCH] applesmc: Re-work SMC comms v2

2020-11-05 Thread Henrik Rydberg
Hi Brad, Great to see this effort, it is certainly an area which could be improved. After having seen several generations of Macbooks while modifying much of that code, it became clear that the SMC communication got refreshed a few times over the years. Every tiny change had to be tested on a

Re: [PATCH] hwmon: applesmc: check status earlier.

2020-08-20 Thread Henrik Rydberg
; - else - return snprintf(sysfsbuf, PAGE_SIZE, "%d\n", count); + + count = ((u32)buffer[0]<<24) + ((u32)buffer[1]<<16) + + ((u32)buffer[2]<<8) + buffer[3]; + return snprintf(sysfsbuf, PAGE_SIZE, "%d\n", count); } static ssize_t applesmc_key_at_index_read_show(struct device *dev, Looks good, thank you. Reviewed-by: Henrik Rydberg Henrik

Re: [PATCH v3 02/49] Input: introduce input_mt_report_slot_inactive

2019-09-17 Thread Henrik Rydberg
Hi Jiada, input_mt_report_slot_state() ignores the tool when the slot is closed. which has caused a bit of confusion. This patch introduces input_mt_report_slot_inactive() to report slot inactive state. replaces all input_mt_report_slot_state() with input_mt_report_slot_inactive() in case of clo

Re: [PATCH 0/2] Add Apple SPI keyboard and trackpad driver

2019-02-04 Thread Henrik Rydberg
Hi Ronald, This changeset adds a driver for the SPI keyboard and trackpad on recent MacBook's and MacBook Pro's. The driver has seen a fair amount of use over the last 2 years (basically anybody running linux on these machines), with only relatively small changes in the last year or so. For thos

Re: [PATCH] [v3] HID: add support for Apple Magic Trackpad 2

2018-09-30 Thread Henrik Rydberg
anged, 134 insertions(+), 16 deletions(-) This version looks good. Reviewed-by: Henrik Rydberg Thank you! Henrik

Re: [PATCH] [v2] HID: add support for Apple Magic Trackpad 2

2018-08-28 Thread Henrik Rydberg
Hi Sean, Thanks for the driver. Looking good, but I do have some comments below. > diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h > index 79bdf0c7e351..d6d0b20cc015 100644 > --- a/drivers/hid/hid-ids.h > +++ b/drivers/hid/hid-ids.h > @@ -88,9 +88,11 @@ > #define USB_DEVICE_ID_ANTON_T

Re: [RFC/PATCH] Input: make input_report_slot_state() return boolean

2018-06-05 Thread Henrik Rydberg
void input_mt_report_finger_count(struct input_dev *dev, int count);   Reviewed-by: Henrik Rydberg Thanks, Dmitry. Henrik

Re: [PATCH 1/2] HID: multitouch: report MT_TOOL_PALM for non-confident touches

2018-06-04 Thread Henrik Rydberg
Hi Dmitry, Logically, the confidence state is a property of a contact, not a new type of contact. Trying to use it in any other way is bound to lead to confusion. Problem is that MT_TOOL_PALM has been introduced in the kernel since v4.0 (late 2015 by a736775db683 "Input: add MT_TOOL_PALM"). It'

Re: [PATCH 1/2] HID: multitouch: report MT_TOOL_PALM for non-confident touches

2018-06-01 Thread Henrik Rydberg
However, I interpret a firmware that send (confidence 1, tip switch 1) and then (confidence 0, tip switch 0) a simple release, and the confidence bit should not be relayed. This unfortunately leads to false clicks: you start with finger, so confidence is 1, then you transition the same touch t

Re: [PATCH v3] input: bcm5974 - Add driver for Apple Magic Trackpad 2

2018-03-09 Thread Henrik Rydberg
Hi Stephan, I would like to have Touchpad 2 properly supported. You will find proper prior support for Magic Trackpads in drivers/hid/hid-magicmouse.c. Henrik

Re: [PATCH v5] HID: hid-multitouch: support fine-grain orientation reporting

2017-11-29 Thread Henrik Rydberg
| 52 +--- include/linux/hid.h | 1 + 3 files changed, 54 insertions(+), 8 deletions(-)    Reviewed-by: Henrik Rydberg Thank you, Wei-Ning, and sorry for the delay. Dmitry, did you plan to add this to your pull request already? Henrik

Re: [PATCH v4] HID: hid-multitouch: support fine-grain orientation reporting

2017-10-11 Thread Henrik Rydberg
> rather have [-90,90], with 0 being "north". > > > > ... I'd like the documentation fix to go in together in one go with this > > patch if possible. > > > > Sounds like a plan. How about this patch? Henrik --- >From b14f92066dfab3f8a255ec7b5a30cb1a864dc62

Re: [PATCH v4] HID: hid-multitouch: support fine-grain orientation reporting

2017-10-10 Thread Henrik Rydberg
value -= field->logical_maximum; > + td->curdata.a = -value; > + td->curdata.has_azimuth = true; > + break; > case HID_DG_TOUCH: > /* do nothing */ >

Re: [PATCH 1/2] HID: multitouch: report MT_TOOL_PALM for non-confident touches

2017-08-11 Thread Henrik Rydberg
Hi Dmitry, The meaning of confidence is literally "contact is too large to be a finger", so it is not touch state, but really tool identity. We do allow tool identity to change over time. What I am arguing is rather that since "palm" is a property, just like contact size, there should be no nee

Re: [PATCH 1/2] HID: multitouch: report MT_TOOL_PALM for non-confident touches

2017-08-10 Thread Henrik Rydberg
Hi Dmitry, On 08/11/2017 02:44 AM, Dmitry Torokhov wrote: According to Microsoft specification [1] for Precision Touchpads (and Touchscreens) the devices use "confidence" reports to signal accidental touches, or contacts that are "too large to be a finger". Instead of simply marking contact ina

Re: [PATCH] Input: Do not add SYN_REPORT in between a single packet data

2016-04-06 Thread Henrik Rydberg
Hi Aniroop, >> I am not sure what the urgency is. It is more of a theoretical problem >> ans so far the proposed solutions were actually introducing more >> problems than they were solving. >> >> I am sorry, bit this particular topic is not a priority for me. >> > > There is no hurry at all. :-)

Re: [PATCH] Input: Do not add SYN_REPORT in between a single packet data

2016-03-10 Thread Henrik Rydberg
Hi Dmitry, >> diff --git a/drivers/input/input.c b/drivers/input/input.c >> index 8806059..262ef77 100644 >> --- a/drivers/input/input.c >> +++ b/drivers/input/input.c >> @@ -401,8 +401,7 @@ static void input_handle_event(struct input_dev *dev, >> if (dev->num_vals >= 2) >>

[PATCH 2/3] HID: apple: Add support for the 2015 Macbook Pro

2015-07-24 Thread Henrik Rydberg
: Janez Urevc Signed-off-by: Henrik Rydberg --- drivers/hid/hid-apple.c | 6 ++ drivers/hid/hid-core.c | 6 ++ drivers/hid/hid-ids.h | 3 +++ 3 files changed, 15 insertions(+) diff --git a/drivers/hid/hid-apple.c b/drivers/hid/hid-apple.c index f822fd2..884d82f 100644 --- a/drivers/hid

[PATCH 1/3] Input: bcm5974 - Prepare for a new trackpad generation

2015-07-24 Thread Henrik Rydberg
Radmacher Tested-by: Yang Hongyang Tested-by: Yen-Chin, Lee Tested-by: George Hilios Tested-by: Janez Urevc Signed-off-by: Henrik Rydberg --- drivers/input/mouse/bcm5974.c | 132 ++ 1 file changed, 81 insertions(+), 51 deletions(-) diff --git a/drivers/input

[PATCH 3/3] Input: bcm597 - Add support for the 2015 Macbook Pro

2015-07-24 Thread Henrik Rydberg
Signed-off-by: Henrik Rydberg --- drivers/input/mouse/bcm5974.c | 33 +++-- 1 file changed, 31 insertions(+), 2 deletions(-) diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c index a596b9b..30e3442 100644 --- a/drivers/input/mouse/bcm5974.c

Re: [PATCH 2/2] Input - synaptics: pin 3 touches when the firmware reports 3 fingers

2015-04-25 Thread Henrik Rydberg
Benjamin, For old kernels this is not a problem because max_slots was 2 and libinput/ xorg-synaptics knew how to deal with that. Now that max_slot is 3, the clients ignore BTN_TOOL_TRIPLETAP and count the actual used slots (so 2). It then gets confused when receiving the BTN_TO

Re: [PATCH v2] Input - mt: Fix input_mt_get_slot_by_key

2015-04-23 Thread Henrik Rydberg
Peter, It may be a long time ago now, but we had very vocal discussions regarding the MT protocol back then, and I am quite sure all the subtleties are well understood. In order to fully appreciate the simplicity of the protocol, one only needs to stop misintepreting it. In order to do that, pleas

Re: [PATCH v2] Input - mt: Fix input_mt_get_slot_by_key

2015-04-23 Thread Henrik Rydberg
> My guess is that none of these drivers are > able to handle a silent closing of a slot and the easiest solution might > be to simply change the documentation if in fact nobody uses the > compressed notation (which removes just one ABS_SLOT event within the > frame, so not sure we can call it comp

Re: [PATCH v2] Input - mt: Fix input_mt_get_slot_by_key

2015-04-23 Thread Henrik Rydberg
> "Creation, replacement and destruction of contacts is achieved by > modifying the ABS_MT_TRACKING_ID of the associated slot. A > non-negative tracking id is interpreted as a contact, and the value -1 > denotes an unused slot. A tracking id not previously present is > considered new, and a track

Re: [PATCH v2 0/3] Balanced slots, attempt #2

2015-03-31 Thread Henrik Rydberg
On 03/31/2015 05:07 PM, Benjamin Tissoires wrote: > Hi, > > so this is the v2 with Hans' ack and Henrik's respin of the first patch. The whole series looks good, thank you Benjamin. Acked-by: Henrik Rydberg Henrik -- To unsubscribe from this list: send the line "unsu

Re: [PATCH 1/3] Input: mt - prevent balanced slot assignment to assign twice the slot

2015-03-31 Thread Henrik Rydberg
er with a different commit message. I did not have any opportunity to check this in hardware. I you like it and find it working, then please take over the authorship and just add a signed-off from me. Thanks, Henrik --- >From e00d63f6cadf20d871bed763b3531428b34d785c Mon Sep 17 00:00:00 2001

Re: [PATCH] Input - mt: Fix input_mt_get_slot_by_key

2015-02-03 Thread Henrik Rydberg
Hi Benjamin, > Slot 0 is released correclty, but when we look for Contact ID 2, the slot > 0 is then picked up again because it is marked as inactive (trackingID < 0). > > This is wrong, and we should not reuse a slot in the same frame. > The test should also check for input_mt_is_used(). Good c

Re: [PATCH] Input: MT - Add support for balanced slot assignment

2015-02-01 Thread Henrik Rydberg
tested-by. Done, and tested against Linus' master. I take it you have a patch on this coming up as well? Thanks, Henrik >From 2a4986420e634df2f0ba09198cb1e55714f61577 Mon Sep 17 00:00:00 2001 From: Henrik Rydberg Date: Sun, 1 Feb 2015 09:16:10 +0100 Subject: [PATCH v2] Input: MT - Add support

Re: [PATCH] Input: MT - Add support for balanced slot assignment

2015-01-22 Thread Henrik Rydberg
Hi Dmitry, On 01/22/2015 09:02 PM, Dmitry Torokhov wrote: > On Thu, Jan 22, 2015 at 08:52:25PM +0100, Henrik Rydberg wrote: >> int input_mt_assign_slots(struct input_dev *dev, int *slots, >> - const struct input_mt_pos *pos, int num_pos) >> +

[PATCH] Input: MT - Add support for balanced slot assignment

2015-01-22 Thread Henrik Rydberg
problem formulation for dmax > 0 cannot benefit from the speedup for positive definite matrices, but since the convergence is faster, the result is about the same. For a handful of contacts, the latency difference is truly negligible. Suggested-by: Benjamin Tissoires Not-yet-signed-off-by: Hen

Re: Fixing touch point jumps in the kernel (was Re: [PATCH] MAINTAINERS: Update rydberg's addresses)

2015-01-22 Thread Henrik Rydberg
Hi Peter, > from the testing I've done, you cannot trigger this except by lifting a > finger and raising a finger. There is no per-use-case requirement, it's a > hardware deficiency that the hardware simply cannot detect this specific > case of finger change. This is a good selling point, but...

Re: [PATCH] MAINTAINERS: Update rydberg's addresses

2015-01-21 Thread Henrik Rydberg
Hi Benjamin, > - there is a fragmentation problem: we would have to fix the bug in > xorg-synaptics (which is slowly waiting for its death), libinput, > ChromeOS, Qt Embedded, Kivy (I think), etc... Indeed, this is the problem I wanted to highlight. As the fragmentation problem grows (graphics, i

Re: [PATCH] MAINTAINERS: Update rydberg's addresses

2015-01-21 Thread Henrik Rydberg
Hi Peter, > it wasn't the first reaction. it was the third, after we hacked up > synaptics [1], then libinput [2], both rather unsatisfactory. Not sure what > chromeos does but they'll probably would need similar code. And any other > users of the evdev API of course. So if this approach did not

Re: [PATCH] MAINTAINERS: Update rydberg's addresses

2015-01-20 Thread Henrik Rydberg
Hi Benjamin, > Henrik, I just used the get_maintainer to add you on CC to an input-mt > patch series, and it ended up using the @euromail.se instead of your > still valid one. I can resend to you the patch series if you want. Thanks, that won't be necessary. I looked through the the patchset, an

[PATCH] MAINTAINERS: Update rydberg's addresses

2014-12-19 Thread Henrik Rydberg
My ISP finally gave up on the old mail address, so I am moving things over to bitmath.org instead. Also change the status fields to better reflect reality. Signed-off-by: Henrik Rydberg --- MAINTAINERS | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/MAINTAINERS

Re: [PATCH] x86, ia64: Do not lose track of the EFI default VGA device

2014-11-14 Thread Henrik Rydberg
is enforced. Here is a tested patch that does just that, thanks for the suggestion. Henrik >From 43c16bbc7adbcb17aac73d09f046bf2779771c4c Mon Sep 17 00:00:00 2001 From: Henrik Rydberg Date: Fri, 14 Nov 2014 20:01:21 +0100 Subject: [PATCH v2] x86, ia64: Do not lose track of the EFI default VGA

Re: [PATCH] x86, ia64: Do not lose track of the EFI default VGA device

2014-11-14 Thread Henrik Rydberg
On 11/14/2014 03:42 PM, Bruno Prémont wrote: > On Fri, 14 Nov 2014 13:53:30 +0100 Henrik Rydberg wrote: >> Since commit 20cde694027e ("x86, ia64: Move EFI_FB >> vga_default_device() initialization to pci_vga_fixup()") in the 3.17 >> merge window, the EFI framebuffer

[PATCH] x86, ia64: Do not lose track of the EFI default VGA device

2014-11-14 Thread Henrik Rydberg
FB_EFI is configured without VGA_ARB. Add a select clause to remedy this. Cc: Bruno Prémont Signed-off-by: Henrik Rydberg --- Hi Peter, I stumbled upon this bug from the 3.17 merge window when updating to Linus's 3.18 git head yesterday. The patch has been tested on two different EFI machines; one

Re: [PATCH] Input: synaptics - properly initialize slots for semi-MT

2014-07-26 Thread Henrik Rydberg
Hi Dmitry, On 07/26/2014 02:27 AM, Dmitry Torokhov wrote: > Semi-MT devices are pointers too, so let's tell that to > input_mt_init_slots(), as well as let it set up the devices as semi-MT, > instead of us doing it manually. > > Reviewed-by: Daniel Kurtz > Reviewed-by: Benson Leung > Signed-off

Re: [PATCH] Input: fix defuzzing logic

2014-07-20 Thread Henrik Rydberg
ndrew de los Reyes > Reviewed-by: Benson Leung > Signed-off-by: Dmitry Torokhov > --- > drivers/input/input.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) Oh my, that was a bad one. Reviewed-by: Henrik Rydberg Thanks, Henrik -- To unsubscribe from this list:

Re: [PATCH v5 4/7] Input: pixcir_i2c_ts: Use Type-B Multi-Touch protocol

2014-05-19 Thread Henrik Rydberg
ed for a better model of interrupted contacts in the core. > + } > break; > + } > > msleep(20); > } > @@ -333,6 +389,13 @@ static int pixcir_i2c_ts_probe(struct i2c_client *client, > input_set_abs_

Re: [PATCH v2 4/8] Input: pixcir_i2c_ts: Use Type-B Multi-Touch protocol

2014-03-08 Thread Henrik Rydberg
Hi Roger, the MT implementation seems mostly fine, just one curiosity: > static irqreturn_t pixcir_ts_isr(int irq, void *dev_id) > { > struct pixcir_i2c_ts_data *tsdata = dev_id; > const struct pixcir_ts_platform_data *pdata = tsdata->chip; > + struct pixcir_report_data report;

Re: [PATCH 0/2] input: mt: Add helper function to send end events

2014-01-01 Thread Henrik Rydberg
Hi Dmitry, >> What problematic scenario is this supposed to solve? >> >> The 'release on suspend' is only an approximation to the 'close >> laptop' scenario, it is certainly not correct in the coffee table >> case, > > Why would it not be correct for coffee table? Do we expect the touches > to re

Re: [PATCH 0/2] input: mt: Add helper function to send end events

2013-12-31 Thread Henrik Rydberg
Hi Felipe, > Adds a helper function to send end touch events for active tracking ids. > And also implements it in egalax driver. What problematic scenario is this supposed to solve? The 'release on suspend' is only an approximation to the 'close laptop' scenario, it is certainly not correct in

Re: [patch 3/3] HID: multitouch: add support of other generic collections in hid-mt

2013-12-21 Thread Henrik Rydberg
Hi Benjamin, > The ANTEC Touch Pad is a device which can switch from a multitouch > touchpad to a mouse. It thus presents several generic collections which > are currently ignored by hid-multitouch. Enable them by using the generic > protocol. Adding also a suffix for them depending on their appli

Re: [PATCH 7/9] Input: pixcir_i2c_ts: Implement Type B Multi Touch reporting

2013-12-21 Thread Henrik Rydberg
Hi Roger, > It seems the controller the original driver was written for does not report > touch ID and just reports 2 touch co-ordinates. I'm not sure how this fits > into > Type B reporting model. Look in drivers/input/mouse/cypress_ps2.c for an example of how to deal with anonymous touch coord

Re: small regression: hwmon: (applesmc) Check key count before proceeding - 5f4513864304672e6ea9eac60583eeac32e679f2

2013-11-24 Thread Henrik Rydberg
Hi Chris, > Well, it seems to be a another one off event. It's the same hardware as before, and it was booting from a USB stick containing Fedora 20 final test candidate 2 which uses kernel 3.11.8. An immediate reboot did not reproduce the problem, nor multiple subsequent reboots. I think I previo

Re: small regression: hwmon: (applesmc) Check key count before proceeding - 5f4513864304672e6ea9eac60583eeac32e679f2

2013-11-24 Thread Henrik Rydberg
; [8.603053] applesmc: key count changed from 261 to 1392508929 I was under the impression that this machine was tested before, and that commit 25f2bd7f5add608c1d1405938f39c96927b275ca Author: Henrik Rydberg Date: Wed Oct 2 19:15:03 2013 +0200 hwmon: (applesmc) Always read until end of data

Re: applesmc oops in 3.10/3.11

2013-10-09 Thread Henrik Rydberg
Hi Guenter, > > I want to make one more test tomorrow, then I'll send a proper patch. > > Thanks! > > > Ok, great. And here it is, with compiler warning fixed. Thanks everybody, Henrik >From 18fa0f55b764ad0fe5fc49f81bae281e5110ed56 Mon Sep 17 00:00:00 2001 From:

Re: applesmc oops in 3.10/3.11

2013-10-08 Thread Henrik Rydberg
Hi Guenter, > > > So, what should we do with this patch ? Apply it ? > > > > So far I'm getting nothing on the original machine. As of today it's > > applied as the last patch on 3.12.0-0.rc4.git0.1.fc20.x86_64. Unfortunately > > at the moment I'm a bit too dense to figure out how to get a new

Re: applesmc oops in 3.10/3.11

2013-10-02 Thread Henrik Rydberg
On Wed, Oct 02, 2013 at 09:47:18AM -0700, Guenter Roeck wrote: > On Wed, Oct 02, 2013 at 06:34:18PM +0200, Henrik Rydberg wrote: > > > >>One thing I have seen in all logs is the earlier "send_byte fail" > > > >>message, so > > > >>I thi

Re: applesmc oops in 3.10/3.11

2013-10-02 Thread Henrik Rydberg
> >>One thing I have seen in all logs is the earlier "send_byte fail" message, > >>so > >>I think that is a pre-requisite. > > > >Not necessarily - it could be that the patch actually fixes the root > >cause. One possible scenario is that on recent SMCs, some of the > >commands produce more data t

Re: applesmc oops in 3.10/3.11

2013-10-02 Thread Henrik Rydberg
> > Patch added on top of 3.12.0-0.rc3.git0.1.fc20.x86_64 and built. But after > > ~dozen reboots, I'm not triggering the problem. The only items in dmesg > > with smc in it: > > > > [ 13.799819] applesmc: key=261 fan=2 temp=14 index=14 acc=1 lux=2 kbd=1 > > [ 13.833402] input: applesmc as

Re: applesmc oops in 3.10/3.11

2013-10-01 Thread Henrik Rydberg
machines seem fine, I have never encountered the problem myself. Here is a patch to test that theory. It has been tested to be pretty harmless on two different generations. I really really do not want to add an 'if (value is insane)' check ;-) Thanks, Henrik >From d48a9e4e6e45dcd9c7e7

Re: applesmc oops in 3.10/3.11

2013-09-26 Thread Henrik Rydberg
> >Yes - I agree that the error state is far-fetched, but it is hard to > >see any other logical explanation. There is of course always the > >possibility that the problem is somewhere else completely. > > > There are also ACPI conflicts in each of the bug reports I looked at. > Can this play a rol

Re: applesmc oops in 3.10/3.11

2013-09-26 Thread Henrik Rydberg
> Applied. I'll run my sanity tests before I send it upstream. > I'll also Cc: stable. Great. > Interesting is that this started to happen with 3.10, even though > I did not find any relevant changes in the driver. Is it possible that > changed boot timing (ie reduced boot time) exposes this prob

Re: applesmc oops in 3.10/3.11

2013-09-25 Thread Henrik Rydberg
es - I agree that the error state is far-fetched, but it is hard to see any other logical explanation. There is of course always the possibility that the problem is somewhere else completely. Proper patch attached. Thanks, Henrik --- >From dedefba9167913c46e1896ce0624e68ffe95d532 Mon Sep 17 00:

Re: applesmc oops in 3.10/3.11

2013-09-25 Thread Henrik Rydberg
On Wed, Sep 25, 2013 at 12:56:28PM -0700, Guenter Roeck wrote: > On Wed, Sep 25, 2013 at 03:06:25PM -0400, Josh Boyer wrote: > > Hi All, > > > > Chris has reported[1] an issue with the applesmc driver in the 3.10 > > and 3.11 kernels. This seems to be a bit transient, but the oops is > > consiste

[PATCH] HID: Correct the USB IDs for the new Macbook Air 6

2013-09-01 Thread Henrik Rydberg
: Dmitry Torokhov Reported-and-tested-by: Ian Munsie Tested-by: Linus G Thiel Signed-off-by: Henrik Rydberg --- Hi Jiri, Dmitry, it looks like this change has been sufficiently tested now, in addition to making perfect sense. Jiri, would you mind taking it, if Dmitry approves? The patch should

Re: [PATCH 1/2] HID: apple: Add another device ID for the mid-2013 Macbook Air

2013-09-01 Thread Henrik Rydberg
On Mon, Aug 26, 2013 at 02:51:32PM +0200, Linus G Thiel wrote: > I'm on an ISO layout, this is what makes sense to me and it also works fine > on my laptop (6,2 with Swedish layout). > > > On Mon, Aug 26, 2013 at 3:50 AM, Ian Munsie wrote: > > > > Brad, Linus, does the above patch work for you

Re: [PATCH] Input: add driver for Neonode zForce based touchscreens

2013-09-01 Thread Henrik Rydberg
Hi Heiko, > This adds a driver for touchscreens using the zforce infrared > technology from Neonode connected via i2c to the host system. > > It supports multitouch with up to two fingers and tracking of the > contacts in hardware. > > Signed-off-by: Heiko Stuebner Thanks for the driver. Pleas

Re: [PATCH 1/2] HID: apple: Add another device ID for the mid-2013 Macbook Air

2013-08-25 Thread Henrik Rydberg
Hi Ian, > This patch adds a device ID found for mid-2013 Macbook Air 6,1 from > lsusb: > > Bus 001 Device 003: ID 05ac:0290 Apple, Inc. > > Since IDs already exist for this generation Macbook air as WELLSPRING8, > name this one WELLSPRING8A. This only adds an ANSI version since it's > device ID

Re: [PATCH 1/2] HID: apple: Add another device ID for the mid-2013 Macbook Air

2013-08-24 Thread Henrik Rydberg
On Sat, Aug 24, 2013 at 04:26:30PM -0700, Dmitry Torokhov wrote: > Hi Ian, > > On Sun, Aug 25, 2013 at 12:17:52AM +1000, Ian Munsie wrote: > > This patch adds a device ID found for mid-2013 Macbook Air 6,1 from > > lsusb: > > > > Bus 001 Device 003: ID 05ac:0290 Apple, Inc. > > > > Since IDs alr

Re: [PATCH v3 0/3] HID: Win 8 multitouch panels detection in core

2013-08-22 Thread Henrik Rydberg
ll not impact this. Maybe just wait for Henrik to put his "Reviewed-by" > before > giving one last test. Here goes: Reviewed-by: Henrik Rydberg Cheers, Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majo

Re: [PATCH v2 2/3] HID: detect Win 8 multitouch devices in core

2013-08-21 Thread Henrik Rydberg
Hi Benjamin, > Detecting Win 8 multitouch devices in core allows us to set quirks > before the device is parsed through hid_hw_start(). > It also simplifies the detection of those devices in hid-multitouch and > makes the handling of those devices cleaner. > > As Win 8 multitouch panels are in th

Re: [PATCH v2 1/3] HID: Use hid_parser for pre-scanning the report descriptors

2013-08-21 Thread Henrik Rydberg
Hi Benjamin, this looks pretty good to me, just a few nitpicks below. > hid_scan_report() implements its own HID report descriptor parsing. It is > going to be really bad with the detection of Win 8 certified touchscreen, > as this detection relies on a special feature and on the report_size and

[REGRESSION] nouveau: Resume hung after protecting against client races (MBA3,1)

2013-06-04 Thread Henrik Rydberg
Hi Ben, The new mutexes in nvc0/nv50 (fadb17190/b509656) break resume on my MBA3,1. A dead-lock somewhere, perhaps? Reverting fixes the problem. Thanks, Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More maj

Re: Linux 3.9-rc8

2013-04-25 Thread Henrik Rydberg
> > Yes, I was really hoping (and originally planning) to release 3.9 > > final this weekend, but we had enough issues that I just didn't feel > > comfy about it. It was borderline, and none of the issues were huge, > > and maybe I could have called this just 3.9 and opened the merge > > window, bu

Re: Linux 3.9-rc8

2013-04-25 Thread Henrik Rydberg
Hi Matthew, Peter, Linus, > Yes, I was really hoping (and originally planning) to release 3.9 > final this weekend, but we had enough issues that I just didn't feel > comfy about it. It was borderline, and none of the issues were huge, > and maybe I could have called this just 3.9 and opened the m

[PATCH] Input: MT - handle semi-mt devices in core

2013-04-07 Thread Henrik Rydberg
Most semi-mt drivers use the slots in a manual way, but really only need to treat the finger count manually. With this patch, a semi-mt driver may use the input-mt core for everything else. Signed-off-by: Henrik Rydberg --- Hi Dmitry, an earlier version of this patch came about as a test to

Re: Atmel updates to atmel_mxt_ts touch controller driver

2013-03-29 Thread Henrik Rydberg
Hi Nick, > The following patches are a large series of updates in functionality to the > atmel_mxt_ts touch driver. They apply cleanly to input/next. I guess this is no longer true. > These changes address some of the same issues that appear in the patchsets > submitted by Daniel Kurtz and Peter

Re: [PATCH 1/1] Feature: Support ALPS v6.

2013-03-29 Thread Henrik Rydberg
Hi Ivan, > Tested on Dell 7720. Works fine. > > Signed-off-by: Valyushkov Ivan > --- > drivers/input/mouse/alps.c | 464 > +--- > drivers/input/mouse/alps.h |2 + > 2 files changed, 443 insertions(+), 23 deletions(-) This patch does not seem to app

Re: [PATCH 2/4] Input: cyapa - Firmware update via request firmware

2013-03-19 Thread Henrik Rydberg
Hi Benson, > On Sat, Mar 16, 2013 at 1:00 PM, Henrik Rydberg wrote: > >> Userspace can initiate the firmware update procedure by copying cyapa.bin > >> to /lib/firmware, and then writing anything to the device sysfs attribute: > >> /sys/bus/i2c/devices/7-0067/upd

Re: [PATCH 0/7] HID: multitouch: support of hybrid finger/pen devices

2013-03-19 Thread Henrik Rydberg
Hi Jiri, Benjamin, > > So the solution consists in relying inconditionaly to the quirk MULTI_INPUT > > for > > hid-multitouch. Before registering the input device in hid-input, we can > > test if > > it has been populated, and if not, then we simply don't register it. In > > order to > > preven

Re: [PATCH 6/7] HID: multitouch: append " Pen" to the name of the stylus input

2013-03-19 Thread Henrik Rydberg
Hi Benjamin, > This is not just cosmetics, it can help to write udev and X.org > rules. > > Signed-off-by: Benjamin Tissoires > --- > drivers/hid/hid-multitouch.c | 24 +++- > 1 file changed, 23 insertions(+), 1 deletion(-) > > diff --git a/drivers/hid/hid-multitouch.c b/dr

Re: [PATCH 4/7] HID: multitouch: add handling for pen in dual-sensors device

2013-03-19 Thread Henrik Rydberg
Hi Benjamin, > Dual sensors devices reports pen and touch on two different reports. > Using the quirk HID_QUIRK_MULTI_INPUT allows us to create a new input > device to forward pen events. > > The quirk HID_QUIRK_NO_EMPTY_INPUT avoids the creation of input devices > for the not used mouse emulatio

Re: [PATCH 1/7] HID: input: don't register unmapped input devices

2013-03-19 Thread Henrik Rydberg
Hi Benjamin, > There is no need to register an input device containing no events. > This allows drivers using the quirk MULTI_INPUT to register one input > per report effectively used. > > For backward compatibility, we need to add a quirk to request > this behavior. > > Signed-off-by: Benjamin

Re: [PATCH 2/4] Input: cyapa - Firmware update via request firmware

2013-03-16 Thread Henrik Rydberg
Hi Benson, > From: Daniel Kurtz > > Use the kernel request_firmware API to allow a hotplug script to load new > firmware into CYAPA device. When request_firmware is called by a driver, > the kernel creates 'loading' and 'data' sysfs entries, and generates a > firmware udev event containing the

Re: [PATCH 1/4] Input: cyapa - Move common initialization to cyapa_detect

2013-03-16 Thread Henrik Rydberg
Hi Benson, > cyapa_check_is_operational and cyapa_create_input_dev are common to > the probe and firmware update paths. Pull those out into > cyapa_detect. > > Signed-off-by: Benson Leung > --- > drivers/input/mouse/cyapa.c | 57 > +++-- > 1 file changed

Re: [PATCH v3 1/2] Input: atmel_mxt_ts - Support for touchpad variant

2013-03-10 Thread Henrik Rydberg
Hi Benson, > From: Daniel Kurtz > > This same driver can be used by atmel based touchscreens and touchpads > (buttonpads). Platform data may specify a device is a touchpad > using the is_tp flag. > > This will cause the driver to perform some touchpad specific > initializations, such as: > *

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
On Wed, Mar 06, 2013 at 01:49:15PM -0800, H. Peter Anvin wrote: > On 03/06/2013 01:33 PM, Yinghai Lu wrote: > > On Wed, Mar 6, 2013 at 12:58 PM, H. Peter Anvin > > wrote: > > > >> Excellent. Yinghai, can you write up the patch with a proper > >> description and I'll put it into x86/urgent. > >

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
On Wed, Mar 06, 2013 at 08:08:34AM -0800, Linus Torvalds wrote: > On Wed, Mar 6, 2013 at 12:06 AM, Henrik Rydberg wrote: > > > > Or not. ;-) This commit breaks boot on my MacBookAir3,1: > > > > commit 8d57470d8f859635deffe3919d7d4867b488b85a > > Author: Yinghai Lu

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
d calculating, it should be PMD_SIZE align. > Otherwise in extreme case we could have less then 2M for the first step. > In your case is about only (1M - 32k) for first step. > > Please try attached patch. Bingo. Excellent, thank you Yinghai. I verified that it also boots on top of Linus&#

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
> >> Can you check bootloader like grub.efi ? > > > > I checked, same story. I tried without EFI_STUB, no joy. I ran with > > and without nouveau, just in case. Without the patch, everything > > works. With the patch, nothing works, and no output at all. > > > > With a bit of luck, I could maybe

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
Hi Yinghai, > >> Can you get a boot log with "debug memblock=debug" from the last > >> successful commit point? Are you booting EFI or BootCamp? > > > > Attached the dmesg log, booting from f763ad1 which is on top of > > 3.7-rc6. I am booting with EFI_STUB, straight into the kernel. > > The comma

Re: Linux 3.9-rc1

2013-03-06 Thread Henrik Rydberg
Hi Linus, Peter, > I don't know if it's just me, but this merge window had more "Uhhuh" > moments than I'm used to. I stopped merging a couple of times, because > we had bugs that looked really scary, but thankfully each time people > were on them like paparazzi on Justin Bieber. Special thanks to

Re: [PATCH v2 0/8] hid driver transport cleanup

2013-02-25 Thread Henrik Rydberg
On Mon, Feb 25, 2013 at 01:29:40PM +0100, Jiri Kosina wrote: > On Mon, 25 Feb 2013, Benjamin Tissoires wrote: > > > Hi guys, > > > > this is the v2 of the hid transport cleanup. > > > > Changes since v1: > > - gathered reviewed/acked/etc.. > > - changed commit messages of patches 4-6 > > - add n

Re: [PATCH v4] Input: synaptics - use firmware data for Cr-48

2013-02-20 Thread Henrik Rydberg
eds to be. The input core can handle slot allocation these days, so I wonder if the the two patches below work for you, as an alternative? Thanks, Henrik --- >From 47629b43c260cb9c1ccbd5c474d89da81a029d27 Mon Sep 17 00:00:00 2001 From: Henrik Rydberg Date: Wed, 20 Feb 2013 22:36:52 +0100

Re: [PATCH v2] Input: atmel_mxt_ts - refactor i2c error handling

2013-02-20 Thread Henrik Rydberg
gt; @@ -438,16 +454,7 @@ static int __mxt_write_reg(struct i2c_client *client, > u16 reg, u16 len, > buf[1] = (reg >> 8) & 0xff; > memcpy(&buf[2], val, len); > > - ret = i2c_master_send(client, buf, count); > - if (ret == count) { > - ret = 0; > - } else { > - if (ret >= 0) > - ret = -EIO; > - dev_err(&client->dev, "%s: i2c send failed (%d)\n", > - __func__, ret); > - } > - > + ret = mxt_i2c_send(client, buf, count); > kfree(buf); > return ret; > } > -- > 1.8.1 > Reviewed-by: Henrik Rydberg Thanks, Henrik -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] HID: ntrig: use input_configured() callback to set the name

2013-02-20 Thread Henrik Rydberg
N-Trig Touchscreen" : > - "N-Trig MultiTouch"; > - break; > - } > - } > - > /* This is needed for devices with more recent firmware versions */ > report = hdev->report_enum[HID_FEATURE_REPORT].repo

  1   2   3   4   >