Re: [PATCH 09/11] staging: cxt1e1: musycc.c: Return negative error codes
On Fri, Sep 27, 2013 at 09:36:36AM +0530, Sachin Kamat wrote: Return negative error codes as is followed in the rest of the kernel. This doesn't apply to linux-next (Sep 26) for some reason that I can't figure out. The first chunk fails. Btw, the caller eventually flips any stray positive returns to negative using the mkret() function. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 00/51] DMA mask changes
On Thu, Sep 26, 2013 at 10:23:08PM +0200, Rafał Miłecki wrote: 2013/9/19 Russell King - ARM Linux li...@arm.linux.org.uk: This email is only being sent to the mailing lists in question, not to anyone personally. The list of individuals is far to great to do that. I'm hoping no mailing lists reject the patches based on the number of recipients. Huh, I think it was enough to send only 3 patches to the b43-dev@. Like: [PATCH 01/51] DMA-API: provide a helper to set both DMA and coherent DMA masks [PATCH 14/51] DMA-API: net: b43: (...) [PATCH 15/51] DMA-API: net: b43legacy: (...) ;) I believe Joe has some nice script for doing it that way. When fixing some coding style / formatting, he sends only related patches to the given ML. If I did that, then the mailing lists would not get the first patch, because almost none of the lists would be referred to by patch 1. Moreover, people complain when they don't have access to the full patch series - they assume patches are missing for some reason, and they ask for the rest of the series. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 1/1] staging: dgnc: Remove KERNEL_VERSION check
This check is not required. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org Cc: Lidza Louina lidza.lou...@gmail.com --- drivers/staging/dgnc/dgnc_kcompat.h |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_kcompat.h b/drivers/staging/dgnc/dgnc_kcompat.h index ed85152..eaec7e6 100644 --- a/drivers/staging/dgnc/dgnc_kcompat.h +++ b/drivers/staging/dgnc/dgnc_kcompat.h @@ -28,11 +28,6 @@ #ifndef __DGNC_KCOMPAT_H #define __DGNC_KCOMPAT_H -# ifndef KERNEL_VERSION -# define KERNEL_VERSION(a,b,c) (((a) 16) + ((b) 8) + (c)) -# endif - - #if !defined(TTY_FLIPBUF_SIZE) # define TTY_FLIPBUF_SIZE 512 #endif -- 1.7.9.5 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [v3.11][Regression] HID: hyperv: convert alloc+memcpy to memdup
On Wed, 25 Sep 2013, Joseph Salisbury wrote: After further testing reverting the following commit does in fact resolve the bug: commit b1a1442a23776756b254b69786848a94d92445ba Author: Jiri Kosina jkos...@suse.cz Date: Mon Jun 3 11:27:48 2013 +0200 HID: core: fix reporting of raw events Reverting this commit in v3.12-rc2 prevents the system from locking up, which happens when connecting a bluetooth trackpad. Jiri, do you think we should revert this patch, or is there some further debugging/data collecting you would like to do? Hi Joseph, in this mail: Message-ID: 5241992e.3090...@canonical.com Date: Tue, 24 Sep 2013 09:52:46 -0400 you said that reverting this commit doesn't prevent the lockups, so I am rather confused ... ? Thanks, -- Jiri Kosina SUSE Labs ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [v3.11][Regression] HID: hyperv: convert alloc+memcpy to memdup
On 09/27/2013 06:50 AM, Jiri Kosina wrote: On Wed, 25 Sep 2013, Joseph Salisbury wrote: After further testing reverting the following commit does in fact resolve the bug: commit b1a1442a23776756b254b69786848a94d92445ba Author: Jiri Kosina jkos...@suse.cz Date: Mon Jun 3 11:27:48 2013 +0200 HID: core: fix reporting of raw events Reverting this commit in v3.12-rc2 prevents the system from locking up, which happens when connecting a bluetooth trackpad. Jiri, do you think we should revert this patch, or is there some further debugging/data collecting you would like to do? Hi Joseph, in this mail: Message-ID: 5241992e.3090...@canonical.com Date: Tue, 24 Sep 2013 09:52:46 -0400 you said that reverting this commit doesn't prevent the lockups, so I am rather confused ... ? Thanks, The testing was invalid. Reverting commit b1a1442 does resolve the bug and stop the lockups. Thanks, Joe ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor
On Tue, Sep 03, K. Y. Srinivasan wrote: Hyper-V supports a mechanism for retrieving the local APIC frequency.Use this and bypass the calibration code in the kernel. This would allow us to boot the Linux kernel as a modern VM on Hyper-V where many of the legacy devices (such as PIT) are not emulated. + if (ms_hyperv.features HV_X64_MSR_APIC_FREQUENCY_AVAILABLE) { + /* + * Get the APIC frequency. + */ + rdmsrl(HV_X64_MSR_APIC_FREQUENCY, hv_lapic_frequency); + hv_lapic_frequency /= HZ; This leads to build errors on 32bit: linux-3.12-rc2/arch/x86/kernel/cpu/mshyperv.c:88: undefined reference to `__udivdi3' Up to now I have only tested on 64bit, sorry for that. Olaf ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [v3.11][Regression] HID: hyperv: convert alloc+memcpy to memdup
On Fri, Sep 27, 2013 at 10:42:10AM -0400, Joseph Salisbury wrote: On 09/27/2013 06:50 AM, Jiri Kosina wrote: On Wed, 25 Sep 2013, Joseph Salisbury wrote: After further testing reverting the following commit does in fact resolve the bug: commit b1a1442a23776756b254b69786848a94d92445ba Author: Jiri Kosina jkos...@suse.cz Date: Mon Jun 3 11:27:48 2013 +0200 HID: core: fix reporting of raw events Reverting this commit in v3.12-rc2 prevents the system from locking up, which happens when connecting a bluetooth trackpad. Jiri, do you think we should revert this patch, or is there some further debugging/data collecting you would like to do? Hi Joseph, in this mail: Message-ID: 5241992e.3090...@canonical.com Date: Tue, 24 Sep 2013 09:52:46 -0400 you said that reverting this commit doesn't prevent the lockups, so I am rather confused ... ? Thanks, The testing was invalid. Reverting commit b1a1442 does resolve the bug and stop the lockups. It looks like magicmouse_raw_event() returns 1 on success and 0 on failure. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor
On Fri, Sep 27, KY Srinivasan wrote: This leads to build errors on 32bit: linux-3.12-rc2/arch/x86/kernel/cpu/mshyperv.c:88: undefined reference to `__udivdi3' Up to now I have only tested on 64bit, sorry for that. Thanks Olaf. I am travelling currently; I will fix it when I get back (early next week). This change seems to fix it: -+ hv_lapic_frequency /= HZ; ++ div_u64(hv_lapic_frequency, HZ); Olaf ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
RE: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor
-Original Message- From: Olaf Hering [mailto:o...@aepfle.de] Sent: Friday, September 27, 2013 8:33 AM To: KY Srinivasan Cc: x...@kernel.org; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; a...@canonical.com; jasow...@redhat.com; t...@linutronix.de; h...@zytor.com; jbeul...@suse.com; b...@alien8.de Subject: Re: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor On Fri, Sep 27, KY Srinivasan wrote: This leads to build errors on 32bit: linux-3.12-rc2/arch/x86/kernel/cpu/mshyperv.c:88: undefined reference to `__udivdi3' Up to now I have only tested on 64bit, sorry for that. Thanks Olaf. I am travelling currently; I will fix it when I get back (early next week). This change seems to fix it: -+ hv_lapic_frequency /= HZ; ++ div_u64(hv_lapic_frequency, HZ); Thanks Olaf. Could you submit the patch. Regards, K. Y ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] Staging: sm7xxfb: fixed line break coding style issues
On Fri, Sep 27, 2013 at 05:39:11PM +0200, Martin Berglund wrote: On Fri, Sep 27, 2013 at 03:27:29PM +0300, Dan Carpenter wrote: On Fri, Sep 27, 2013 at 02:08:07PM +0200, Martin Berglund wrote: @@ -508,9 +506,8 @@ static void sm7xx_set_timing(struct smtcfb_info *sfb) /* init SEQ register SR30 - SR75 */ for (i = 0; i SIZE_SR30_SR75; i++) - if (((i + 0x30) != 0x62) \ - ((i + 0x30) != 0x6a) \ - ((i + 0x30) != 0x6b)) + if (((i + 0x30) != 0x62) ((i + 0x30) != 0x6a) + ((i + 0x30) != 0x6b)) The prefered way would be to align this like this: if ((i + 0x30) != 0x62 (i + 0x30) != 0x6a (i + 0x30) != 0x6b) regards, dan carpenter New patch. Looks nice, but could you resend it in a way that can be a applied? Take your original patch and change the subject to: [PATCH v2] Staging: sm7xxfb: fixed line break coding style issues Add v2: minor style change under the --- line. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] Staging: sm7xxfb: fixed line break coding style issues
On Fri, Sep 27, 2013 at 03:27:29PM +0300, Dan Carpenter wrote: On Fri, Sep 27, 2013 at 02:08:07PM +0200, Martin Berglund wrote: @@ -508,9 +506,8 @@ static void sm7xx_set_timing(struct smtcfb_info *sfb) /* init SEQ register SR30 - SR75 */ for (i = 0; i SIZE_SR30_SR75; i++) - if (((i + 0x30) != 0x62) \ -((i + 0x30) != 0x6a) \ -((i + 0x30) != 0x6b)) + if (((i + 0x30) != 0x62) ((i + 0x30) != 0x6a) +((i + 0x30) != 0x6b)) The prefered way would be to align this like this: if ((i + 0x30) != 0x62 (i + 0x30) != 0x6a (i + 0x30) != 0x6b) regards, dan carpenter New patch. Signed-off-by: Martin Berglund mar...@rogsta.net --- drivers/staging/sm7xxfb/sm7xxfb.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging/sm7xxfb/sm7xxfb.c index 8add64b..8ba0097 100644 --- a/drivers/staging/sm7xxfb/sm7xxfb.c +++ b/drivers/staging/sm7xxfb/sm7xxfb.c @@ -259,8 +259,7 @@ static int smtc_setcolreg(unsigned regno, unsigned red, unsigned green, if (sfb-fb.var.bits_per_pixel == 16) { u32 *pal = sfb-fb.pseudo_palette; val = chan_to_field(red, sfb-fb.var.red); - val |= chan_to_field(green, \ - sfb-fb.var.green); + val |= chan_to_field(green, sfb-fb.var.green); val |= chan_to_field(blue, sfb-fb.var.blue); #ifdef __BIG_ENDIAN pal[regno] = @@ -274,8 +273,7 @@ static int smtc_setcolreg(unsigned regno, unsigned red, unsigned green, } else { u32 *pal = sfb-fb.pseudo_palette; val = chan_to_field(red, sfb-fb.var.red); - val |= chan_to_field(green, \ - sfb-fb.var.green); + val |= chan_to_field(green, sfb-fb.var.green); val |= chan_to_field(blue, sfb-fb.var.blue); #ifdef __BIG_ENDIAN val = @@ -508,9 +506,9 @@ static void sm7xx_set_timing(struct smtcfb_info *sfb) /* init SEQ register SR30 - SR75 */ for (i = 0; i SIZE_SR30_SR75; i++) - if (((i + 0x30) != 0x62) \ -((i + 0x30) != 0x6a) \ -((i + 0x30) != 0x6b)) + if ((i + 0x30) != 0x62 + (i + 0x30) != 0x6a + (i + 0x30) != 0x6b) smtc_seqw(i + 0x30, VGAMode[j].Init_SR30_SR75[i]); -- 1.8.4 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] Staging: sm7xxfb: fixed line break coding style issues
On Fri, Sep 27, 2013 at 05:39:11PM +0200, Martin Berglund wrote: On Fri, Sep 27, 2013 at 03:27:29PM +0300, Dan Carpenter wrote: On Fri, Sep 27, 2013 at 02:08:07PM +0200, Martin Berglund wrote: @@ -508,9 +506,8 @@ static void sm7xx_set_timing(struct smtcfb_info *sfb) /* init SEQ register SR30 - SR75 */ for (i = 0; i SIZE_SR30_SR75; i++) - if (((i + 0x30) != 0x62) \ - ((i + 0x30) != 0x6a) \ - ((i + 0x30) != 0x6b)) + if (((i + 0x30) != 0x62) ((i + 0x30) != 0x6a) + ((i + 0x30) != 0x6b)) The prefered way would be to align this like this: if ((i + 0x30) != 0x62 (i + 0x30) != 0x6a (i + 0x30) != 0x6b) regards, dan carpenter New patch. Signed-off-by: Martin Berglund mar...@rogsta.net Can you please resend this in a format I can apply it in, without needing to edit the patch body by hand? I deal with thousands of patches every month, and if I have to edit them all, that cuts down my ability to apply them greatly. thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
RE: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor
-Original Message- From: Olaf Hering [mailto:o...@aepfle.de] Sent: Friday, September 27, 2013 8:10 AM To: KY Srinivasan Cc: x...@kernel.org; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; a...@canonical.com; jasow...@redhat.com; t...@linutronix.de; h...@zytor.com; jbeul...@suse.com; b...@alien8.de Subject: Re: [PATCH V2 1/1] X86: Hyper-V: Get the local APIC timer frequency from the hypervisor On Tue, Sep 03, K. Y. Srinivasan wrote: Hyper-V supports a mechanism for retrieving the local APIC frequency.Use this and bypass the calibration code in the kernel. This would allow us to boot the Linux kernel as a modern VM on Hyper-V where many of the legacy devices (such as PIT) are not emulated. + if (ms_hyperv.features HV_X64_MSR_APIC_FREQUENCY_AVAILABLE) { + /* +* Get the APIC frequency. +*/ + rdmsrl(HV_X64_MSR_APIC_FREQUENCY, hv_lapic_frequency); + hv_lapic_frequency /= HZ; This leads to build errors on 32bit: linux-3.12-rc2/arch/x86/kernel/cpu/mshyperv.c:88: undefined reference to `__udivdi3' Up to now I have only tested on 64bit, sorry for that. Thanks Olaf. I am travelling currently; I will fix it when I get back (early next week). K. Y Olaf ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel