This patch fixes the following checkpatch.pl warning in p80211types.h:
WARNING: do not add new typedefs
Signed-off-by: Sergio Paracuellos
---
drivers/staging/wlan-ng/cfg80211.c | 12 +-
drivers/staging/wlan-ng/p80211metastruct.h | 248 ++---
drivers/staging/wlan-
The dgnc_tty_uninit() doesn't match with dgnc_tty_init() at all.
And also the dgnc_cleanup_tty() is only called for exiting the module.
Signed-off-by: Daeseok Youn
---
V2: the subject line was cut off, I put it completely.
drivers/staging/dgnc/dgnc_driver.c | 2 +-
drivers/staging/dgnc/dgnc_tty
The dgnc_finalize_board_init() function has only job for
requesting the IRQ. It should be renamed to dgnc_request_irq()
Signed-off-by: Daeseok Youn
---
V2: the subject line was cut off, I put it completely.
drivers/staging/dgnc/dgnc_driver.c | 6 +++---
1 file changed, 3 insertions(+), 3 deleti
The functions related with tty device initialization are needed
to be moved from dgnc_found_board() to dgnc_init_one().
Signed-off-by: Daeseok Youn
---
V2: the subject line was cut off, I put it completely.
drivers/staging/dgnc/dgnc_driver.c | 81 --
1 file c
The board structure should be freed when any function was failed
in dgnc_found_board(). And the board strucure will be stored
into dgnc_board array when the dgnc_found_board() function has no error.
Signed-off-by: Daeseok Youn
---
V2: the subject line was cut off, I put it completely.
drivers/s
The ioremap() function can be failed, so it need to have error
handling in dgnc_do_remap(). And also the return type of
dgnc_do_remap() should be changed from "void" to "int"
Signed-off-by: Daeseok Youn
---
V2: the subject line was cut off, I put it completely.
drivers/staging/dgnc/dgnc_driver.
The channel array in board_t was initialized in dgnc_found_board()
with NULL. But the channel is going to initialize in dgnc_tty_init()
again. So the channel array doesn't need to set NULL
for initailization in dgnc_found_board().
Signed-off-by: Daeseok Youn
---
V2: The subject line was cut off,
2016-09-22 16:21 GMT+09:00 Greg KH :
> On Thu, Sep 22, 2016 at 02:22:03PM +0900, Daeseok Youn wrote:
>> The board structure should be freed when any function was failed
>> in dgnc_found_board(). And the board strucure will be stored
>> into dgnc_board array when the dgnc_found_board() function has
Fixed a coding style issue by removing unnecessary spaces before casts.
Signed-off-by: Jannik Becher
---
drivers/staging/wlan-ng/hfa384x.h | 4 ++--
drivers/staging/wlan-ng/p80211netdev.c | 12 ++--
drivers/staging/wlan-ng/p80211req.c| 16
drivers/staging/wlan-
On Fri, 2016-09-23 at 00:39 +0200, becher.jan...@gmail.com wrote:
> On Thu, Sep 22, 2016 at 03:05:02PM -0700, Joe Perches wrote:
> > On Thu, 2016-09-22 at 23:56 +0200, Jannik Becher wrote:
> > >r emoved a space after a cast to obtain the coding style.
> > Better would be to change the subject to so
On Thu, Sep 22, 2016 at 03:05:02PM -0700, Joe Perches wrote:
> On Thu, 2016-09-22 at 23:56 +0200, Jannik Becher wrote:
> > removed a space after a cast to obtain the coding style.
>
> Better would be to change the subject to something like:
>
> [PATCH] staging: wlan-ng: Remove unnecessary spaces
On Thu, 2016-09-22 at 23:56 +0200, Jannik Becher wrote:
> removed a space after a cast to obtain the coding style.
Better would be to change the subject to something like:
[PATCH] staging: wlan-ng: Remove unnecessary spaces before casts
___
devel maili
removed a space after a cast to obtain the coding style.
Signed-off-by: Jannik Becher
---
drivers/staging/wlan-ng/hfa384x.h | 4 ++--
drivers/staging/wlan-ng/p80211netdev.c | 12 ++--
drivers/staging/wlan-ng/p80211req.c| 16
drivers/staging/wlan-ng/prism2fw.c
On Thu, Sep 22, 2016 at 10:52:18PM +0200, becher.jan...@gmail.com wrote:
> I always wondered why I shouldn't make more than one change in a patch,
> but in all talks I watched they said that it's easier for them to merge
> small patches.
> So you think I should make one "big" patch and resend it?
>
On Thu, 2016-09-22 at 22:52 +0200, becher.jan...@gmail.com wrote:
> I always wondered why I shouldn't make more than one change in a patch,
> but in all talks I watched they said that it's easier for them to merge
> small patches.
>
> So you think I should make one "big" patch and resend it?
Make
The preferred indentation for cases and switches has the cases at the same
level as the switch.
Signed-off-by: Christopher H. Pezley
---
drivers/staging/android/ion/ion_of.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/android/ion/ion_of
On Thu, Sep 22, 2016 at 11:32:02PM +0300, Andrey Utkin wrote:
> On Thu, Sep 22, 2016 at 09:11:46PM +0200, Jannik Becher wrote:
> > removed a space after a cast to obtain the coding style.
> >
> > Signed-off-by: Jannik Becher
> > ---
> > drivers/staging/wlan-ng/p80211req.c | 2 +-
> > 1 file chan
On Thu, Sep 22, 2016 at 09:11:46PM +0200, Jannik Becher wrote:
> removed a space after a cast to obtain the coding style.
>
> Signed-off-by: Jannik Becher
> ---
> drivers/staging/wlan-ng/p80211req.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/wlan-ng
On 09/22/2016 08:59 AM, Antti Keränen wrote:
Correctly align an unaligned block comment
Signed-off-by: Antti Keränen
---
drivers/staging/android/uapi/ion.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/android/uapi/ion.h
b/drivers/staging/android/uapi/ion
On 09/22/2016 08:59 AM, Antti Keränen wrote:
Remove unnecessary braces surrounding a single statement block
In the future, when sending different versions of a patch please
give a short summary of what's changed. I think this is just
a resend from v2 with more reviewers added but I'm not 100%
On Thu, Sep 22, 2016 at 02:49:22PM -0500, Tom Lendacky wrote:
> > I thought that reduction is the reservation of bits for the SME mask.
> >
> > What other reduction is there?
>
> There is a reduction in physical address space for the SME mask and the
> bits used to aid in identifying the ASID ass
On 09/22/2016 02:11 PM, Borislav Petkov wrote:
> On Thu, Sep 22, 2016 at 02:04:27PM -0500, Tom Lendacky wrote:
>> That's not what I mean here. If the BIOS sets the SMEE bit in the
>> SYS_CFG msr then, even if the encryption bit is never used, there is
>> still a reduction in physical address space
On 09/22/2016 09:45 AM, Paolo Bonzini wrote:
>
>
> On 22/09/2016 16:35, Borislav Petkov wrote:
@@ -230,6 +230,10 @@ int __init efi_setup_page_tables(unsigned long
pa_memmap, unsigned num_pages)
efi_scratch.efi_pgt = (pgd_t *)__sme_pa(efi_pgd);
pgd = efi_pgd;
On 09/22/2016 09:59 AM, Borislav Petkov wrote:
> On Thu, Sep 22, 2016 at 04:45:51PM +0200, Paolo Bonzini wrote:
>> The main difference between the SME and SEV encryption, from the point
>> of view of the kernel, is that real-mode always writes unencrypted in
>> SME and always writes encrypted in SE
On Thu, Sep 22, 2016 at 02:04:27PM -0500, Tom Lendacky wrote:
> That's not what I mean here. If the BIOS sets the SMEE bit in the
> SYS_CFG msr then, even if the encryption bit is never used, there is
> still a reduction in physical address space.
I thought that reduction is the reservation of bi
removed a space after a cast to obtain the coding style.
Signed-off-by: Jannik Becher
---
drivers/staging/wlan-ng/p80211req.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/p80211req.c
b/drivers/staging/wlan-ng/p80211req.c
index 40627d5..ecd2fff 1006
On 09/22/2016 09:17 AM, Greg KH wrote:
> On Wed, Sep 21, 2016 at 11:12:05PM +0200, Christopher Pezley wrote:
>> There is a line with space indentation instead of tabs. I've changed it to
>> use tabs for indentation.
>>
>> Signed-off-by: Christopher H. Pezley
>> ---
>> drivers/staging/rtl8712/rtl8
On 09/22/2016 12:07 PM, Borislav Petkov wrote:
> On Thu, Sep 22, 2016 at 05:05:54PM +0200, Paolo Bonzini wrote:
>> Which paragraph?
>
> "Linux relies on BIOS to set this bit if BIOS has determined that the
> reduction in the physical address space as a result of enabling memory
> encryption..."
>
On 22/09/2016 20:47, Tom Lendacky wrote:
> > Because the firmware volume is written to high memory in encrypted form,
> > and because the PEI phase runs in 32-bit mode, the firmware code will be
> > encrypted; on the other hand, data that is placed in low memory for the
> > kernel can be unencryp
On 22/09/2016 20:37, Borislav Petkov wrote:
>> > Unless this is part of some spec, it's easier if things are the same in
>> > SME and SEV.
> Yeah, I was pondering over how sprinkling sev_active checks might not be
> so clean.
>
> I'm wondering if we could make the EFI regions presented to the gu
On Thu, Sep 22, 2016 at 08:23:36PM +0200, Paolo Bonzini wrote:
> Unless this is part of some spec, it's easier if things are the same in
> SME and SEV.
Yeah, I was pondering over how sprinkling sev_active checks might not be
so clean.
I'm wondering if we could make the EFI regions presented to th
On 22/09/2016 19:46, Tom Lendacky wrote:
>> > Do you mean, it is encrypted here because we're in the guest kernel?
> Yes, the idea is that the SEV guest will be running encrypted from the
> start, including the BIOS/UEFI, and so all of the EFI related data will
> be encrypted.
Unless this is par
Hello Yannis!
There is a typo in your one-line description/subject ("ion").
On 09/22/2016 07:55 PM, Yannis Damigos wrote:
> This patch fixes 1 error, 1 warning and 14 checks found by
> checkpatch.
Please be more specific in your commit message and mention what exactly
you are doing to the code a
On 09/22/2016 09:35 AM, Borislav Petkov wrote:
> On Mon, Aug 22, 2016 at 07:25:25PM -0400, Brijesh Singh wrote:
>> From: Tom Lendacky
>>
>> EFI data is encrypted when the kernel is run under SEV. Update the
>> page table references to be sure the EFI memory areas are accessed
>> encrypted.
>>
>> S
This patch fixes the following checkpatch.pl warning in spi.c:
WARNING: line over 80 characters
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/spi.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/rts5208/spi.c b/drivers/staging/rts5208/spi.
This patch fixes the following checkpatch.pl warning in rtsx.h:
WARNING: do not add new typedefs
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx.h | 4
drivers/staging/rts5208/rtsx_chip.h | 2 +-
2 files changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/s
This patch fixes the following checkpatch.pl warning in xd.c:
WARNING: else is not generally useful after a break or return
It also makes code more uniform with the new changes
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/xd.c | 38 ++
1 file
This patch fixes the following checkpatch.pl warning in sd.c:
WARNING: else is not generally useful after a break or return
It also makes code more uniform with the new changes
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/sd.c | 173 +++
This patchset fix several style warnings reported by checkpatch.pl script.
Sergio Paracuellos (8):
staging: rts5208: fix style warnings in rtsx.h
staging: rts5208: fix style warnings in rtsx.c
staging: rts5208: fix style warnings in rtsx_chip.h
staging: rts5208: fix style warnings in rtsx.
This patch fixes the following checkpatch.pl warning in rtsx_sys.h:
WARNING: do not add new typedefs
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx_sys.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx_sys.h
b/drivers/staging/rts5208/rtsx_sy
This patch fixes the following checkpatch.pl warning in rtsx.h:
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx.h | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx
This patch fixes the following checkpatch.pl warning in rtsx_chip.h:
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx_chip.h | 52 +++--
1 file changed, 27 insertions(+), 25 deletions(-
This patch fixes the following checkpatch.pl warning in rtsx.c:
WARNING: Symbolic permissions are not preferred.
Signed-off-by: Sergio Paracuellos
---
drivers/staging/rts5208/rtsx.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx.c b/
Rename CamelCased function getChipType to get_chip_type.
This issue was found by checkpatch.pl
Signed-off-by: Moshe Green
---
drivers/staging/sm750fb/ddk750_chip.c | 16
drivers/staging/sm750fb/ddk750_chip.h | 2 +-
drivers/staging/sm750fb/ddk750_mode.c | 4 ++--
drivers/st
Fixes warnings found by checkpatch
Missing a blank line after declarations
Signed-off-by: Johan Svensson
---
drivers/staging/ks7010/ks_hostif.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/staging/ks7010/ks_hostif.c
b/drivers/staging/ks7010/ks_hostif.c
index aad8048..c57ca58 1
changed comparison "wlandev == NULL" to "!wlandev" to obtain the coding
style.
Signed-off-by: Jannik Becher
---
drivers/staging/wlan-ng/cfg80211.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlan-ng/cfg80211.c
b/drivers/staging/wlan-ng/cfg80211.c
index f4
This patch fixes 1 error, 1 warning and 14 checks found by
checkpatch.
Signed-off-by: Yannis Damigos
---
drivers/staging/android/ion/ion_of.c | 49 ++--
1 file changed, 24 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/android/ion/ion_of.c
b/drivers
On 22/09/16 10:43, Arnd Bergmann wrote:
> The newly added __sca3000_get_base_freq function handles all valid
> modes of the SCA3000_REG_ADDR_MODE register, but gcc notices
> that any other value (i.e. 0x00) causes the base_freq variable to
> not get initialized:
>
> drivers/staging/iio/accel/sca30
On Thu, Sep 22, 2016 at 07:08:50PM +0200, Paolo Bonzini wrote:
> That's not how I read it. I just figured that the BIOS has some magic
> things high in the physical address space and if you reduce the physical
> address space the BIOS (which is called from e.g. EFI runtime services)
> would have p
On 22/09/2016 19:07, Borislav Petkov wrote:
>> Which paragraph?
> "Linux relies on BIOS to set this bit if BIOS has determined that the
> reduction in the physical address space as a result of enabling memory
> encryption..."
>
> Basically, you can enable SME in the BIOS and you're all set.
Tha
On Thu, Sep 22, 2016 at 05:05:54PM +0200, Paolo Bonzini wrote:
> Which paragraph?
"Linux relies on BIOS to set this bit if BIOS has determined that the
reduction in the physical address space as a result of enabling memory
encryption..."
Basically, you can enable SME in the BIOS and you're all se
Correctly align an unaligned block comment
Signed-off-by: Antti Keränen
---
drivers/staging/android/uapi/ion.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/android/uapi/ion.h
b/drivers/staging/android/uapi/ion.h
index 647f130..e629394 100644
--- a/drivers/
Remove unnecessary braces surrounding a single statement block
Signed-off-by: Antti Keränen
---
drivers/staging/android/ion/hisilicon/hi6220_ion.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/android/ion/hisilicon/hi6220_ion.c
b/drivers/staging/android
On Thu, Sep 22, 2016 at 05:13:07PM +0300, Antti Keränen wrote:
> This set of patches fix two coding style warnings found with checkpatch.pl
> I hope I got this correctly this time around :)
>
> Antti Keränen (2):
> staging: ion: Fix a coding style issue
> staging: ion: Fix a coding style issue
On 22/09/2016 16:59, Borislav Petkov wrote:
> On Thu, Sep 22, 2016 at 04:45:51PM +0200, Paolo Bonzini wrote:
>> The main difference between the SME and SEV encryption, from the point
>> of view of the kernel, is that real-mode always writes unencrypted in
>> SME and always writes encrypted in SEV
On Mon, Aug 22, 2016 at 07:24:19PM -0400, Brijesh Singh wrote:
> From: Tom Lendacky
Subject: [RFC PATCH v1 04/28] x86: Secure Encrypted Virtualization (SEV) support
Please start patch commit heading with a verb, i.e.:
"x86: Add AMD Secure Encrypted Virtualization (SEV) support"
--
Regards/Gru
On Thu, Sep 22, 2016 at 04:45:51PM +0200, Paolo Bonzini wrote:
> The main difference between the SME and SEV encryption, from the point
> of view of the kernel, is that real-mode always writes unencrypted in
> SME and always writes encrypted in SEV. But UEFI can run in 64-bit mode
> and learn abou
rf_type is always equal to ODM_1T1R.
So, only RF PATH A exists for r8188eu device...
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/odm.c | 35 -
drivers/staging/rtl8188eu/hal/phy.c | 16 +++--
drivers/staging/rtl8188eu/hal/rtl81
rf_type is always equal to RF_1T1R.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/bb_cfg.c| 5 +
drivers/staging/rtl8188eu/hal/phy.c | 5 +
drivers/staging/rtl8188eu/hal/rf_cfg.c| 5 +
drivers/staging/rtl8188eu/hal/rtl8188e_dm.c
On 22/09/2016 16:35, Borislav Petkov wrote:
>> > @@ -230,6 +230,10 @@ int __init efi_setup_page_tables(unsigned long
>> > pa_memmap, unsigned num_pages)
>> >efi_scratch.efi_pgt = (pgd_t *)__sme_pa(efi_pgd);
>> >pgd = efi_pgd;
>> >
>> > + flags = _PAGE_NX | _PAGE_RW;
>> > + if (sev_ac
On Mon, Aug 22, 2016 at 07:25:25PM -0400, Brijesh Singh wrote:
> From: Tom Lendacky
>
> EFI data is encrypted when the kernel is run under SEV. Update the
> page table references to be sure the EFI memory areas are accessed
> encrypted.
>
> Signed-off-by: Tom Lendacky
> ---
> arch/x86/platform
Hi Michael,
El 22/09/16 a las 06:43, Michael Ellerman escribió:
Emilio López writes:
+CFLAGS += -I../../../../include/uapi/
Please don't include the unprocessed uapi headers, they are not meant to
be directly included in userspace programs.
They even say so:
include/uapi/linux/types.h:#war
This set of patches fix two coding style warnings found with checkpatch.pl
I hope I got this correctly this time around :)
Antti Keränen (2):
staging: ion: Fix a coding style issue
staging: ion: Fix a coding style issue
drivers/staging/android/ion/hisilicon/hi6220_ion.c | 4 ++--
drivers/sta
On Mon, Aug 22, 2016 at 07:24:32PM -0400, Brijesh Singh wrote:
> From: Tom Lendacky
>
> Currently the nested_ctl variable in the vmcb_control_area structure is
> used to indicate nested paging support. The nested paging support field
> is actually defined as bit 0 of the this field. In order to s
Remove unnecessary braces surrounding a single statement block
Signed-off-by: Antti Keränen
---
drivers/staging/android/ion/hisilicon/hi6220_ion.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/android/ion/hisilicon/hi6220_ion.c
b/drivers/staging/android
Correctly align an unaligned block comment
Signed-off-by: Antti Keränen
---
drivers/staging/android/uapi/ion.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/android/uapi/ion.h
b/drivers/staging/android/uapi/ion.h
index 647f130..e629394 100644
--- a/drivers/
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
staging-testing
head: 2a38ef60ef4ffe135566d1b8acc197b803c48ec1
commit: bce83de2578b37f1be039977278e8691a597ea7d [1000/1004] staging: i4l:
act2000: Replace printk(KERN_DEBUG..) with netdev_dbg(dev, ..)
config: i386-allmod
On Thu, Sep 22, 2016 at 11:53:57AM +0200, Greg Kroah-Hartman wrote:
> On Thu, Sep 22, 2016 at 05:38:27PM +0800, kbuild test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
> > staging-testing
> > head: 2a38ef60ef4ffe135566d1b8acc197b803c48ec1
> > commi
Emilio López writes:
> These tests are based on the libsync test suite from Android.
> This commit lays the ground for future tests, as well as includes
> tests for a variety of basic allocation commands.
Hi Emilio,
Just a few comments on the Makefile.
> diff --git a/tools/testing/selftests/sy
On Thu, Sep 22, 2016 at 05:38:27PM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
> staging-testing
> head: 2a38ef60ef4ffe135566d1b8acc197b803c48ec1
> commit: 94b9eba50e3ad6fafe88d71da4a7345ebe87cdfb [998/1004] staging: i4l:
> act200
The newly added __sca3000_get_base_freq function handles all valid
modes of the SCA3000_REG_ADDR_MODE register, but gcc notices
that any other value (i.e. 0x00) causes the base_freq variable to
not get initialized:
drivers/staging/iio/accel/sca3000_core.c: In function 'sca3000_write_raw':
drivers/
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
staging-testing
head: 2a38ef60ef4ffe135566d1b8acc197b803c48ec1
commit: 94b9eba50e3ad6fafe88d71da4a7345ebe87cdfb [998/1004] staging: i4l:
act2000: Replace printk(KERN_WARNING..) with netdev_warn(dev, ..)
config: i386-allm
This patch allows to call the write() function for synchronous and
isochronous channels with buffers of any size. The AIM simply waits for
data to fill up the MOST buffer object according to the network interface
controller specification for streaming channels, before it submits the
buffer to the H
On Thu, Sep 22, 2016 at 02:22:03PM +0900, Daeseok Youn wrote:
> The board structure should be freed when any function was failed
> in dgnc_found_board(). And the board strucure will be stored
> into dgnc_board array when the dgnc_found_board() function has no error.
>
> Signed-off-by: Daeseok Youn
On Thu, Sep 22, 2016 at 02:21:38PM +0900, Daeseok Youn wrote:
> The ioremap() function can be failed, so it need to have error
> handling in dgnc_do_remap(). And also the return type of
> dgnc_do_remap() should be changed from "void" to "int"
>
> Signed-off-by: Daeseok Youn
> ---
> drivers/stagi
On Thu, Sep 22, 2016 at 02:20:51PM +0900, Daeseok Youn wrote:
> The channel array in board_t was initialized in dgnc_found_board()
> with NULL. But the channel is going to initialize in dgnc_tty_init().
> So the channel array doesn't need to set NULL for initailization.
>
> Signed-off-by: Daeseok
On Wed, Sep 21, 2016 at 11:12:05PM +0200, Christopher Pezley wrote:
> There is a line with space indentation instead of tabs. I've changed it to
> use tabs for indentation.
>
> Signed-off-by: Christopher H. Pezley
> ---
> drivers/staging/rtl8712/rtl871x_ioctl_linux.c | 2 +-
> 1 file changed, 1
On Thu, Sep 22, 2016 at 09:57:16AM +0300, Antti Keränen wrote:
> This patch fixes two coding style issues found with checkpatch.pl. Both
> of them are warnings.
Ah, then this should be two separate patches :)
Also, always use scripts/get_maintainer.pl to determine who to cc: on a
patch.
>
> uap
On Wed, Sep 21, 2016 at 02:49:07PM +0200, Christian Gromm wrote:
> From: Andrey Shvetsov
>
> This patch allows to call the write() function for synchronous and
> isochronous channels with buffers of any size. The AIM simply waits for
> data to fill up the MOST buffer object according to the netwo
On Wed, Sep 21, 2016 at 02:49:06PM +0200, Christian Gromm wrote:
> From: Andrey Shvetsov
>
> This patch removes the labels 'put_mbo' and 'unlock' and relocates the
> code accordingly making the code look simpler.
But this is the opposite of what you should normally do. Now you have
to keep trac
80 matches
Mail list logo
