On Fri, Jun 20, Haiyang Zhang wrote:
> This command ("/etc/init.d/network restart") exists on our supported distros
> currently. We will also look into some better ways to refresh the network for
> the distros without this command. I have tried setting IF_OPER_DORMANT then
> IF_OPER_UP, but not w
Hi Uffe,
do you accepted this patch?
if accepted, I will submit again with jones ack.
Best Regards.
micky.
On 06/19/2014 09:57 AM, micky wrote:
On 06/18/2014 07:03 PM, Ulf Hansson wrote:
On 18 June 2014 12:08, micky wrote:
On 06/18/2014 03:39 PM, Ulf Hansson wrote:
On 18 June 2014 03:17, m
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/DDRInit.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index f1d7cb8..b4f0ea3 100644
--- a/drivers/staging/bcm/DDRInit.c
+++ b/drivers/staging/bcm/
The inner if-statement was aligned just like the outer one. Why?
This indention was introduced by
f34c488c3894968e8cdbdc3b1ed617d78315cace
which is a indention-fix patch itself. That's why I'm curious about it.
I did not merge these nested if-statements, as I don't know if I'm
destroying lo
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/DDRInit.c | 31 +--
1 file changed, 21 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 423bfd9..4564f40 100644
--- a/drivers/staging/bcm/DDRInit.c
+++
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/DDRInit.c | 176 --
1 file changed, 134 insertions(+), 42 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DDRInit.c
index 4c7f518..cfaa2c1 100644
--- a/drivers/staging/bcm/
This patch fixes some indentation errors, where multi-line statements
where not indented.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/DDRInit.c | 56 +--
1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/drivers/staging/bcm/DDRInit.
This patch moves the doubled if block inside the retval-checking one
level up.
As the if-else-block it was in is now a one-statement if-else block,
I removed the braces from it.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/DDRInit.c | 16 +---
1 file changed, 5 insertions(+
Hi,
these are some patches for the
drivers/staging/bcm/DDRInit.c
file. The patchset contains:
* Indentation/Whitespace fixes
* Line length fixes
* Simplifying logic
Please notice: I'm not able to test these patches, as I do not have the
appropriate hardware. I compiled them, at
Hi,
these patches are still not applied, am I right?
Should I (rebase and) resend them?
On 12-06-2014 14:13:16, Matthias Beyer wrote:
> Hi,
>
> hopefully, this is the last version of the patchset.
>
> I fixed up my `sed` error and test compiled the module.
>
> Please notice: I cannot test the
Hi,
these patches are not applied yet, right?
Should I (rebase and) resend them?
On 11-06-2014 10:46:09, Matthias Beyer wrote:
> Hi,
>
> I fixed what Dan suggested.
>
> Kind regards,
> Matthias Beyer
>
> Matthias Beyer (2):
> Staging: bcm: Shortened lines
> Staging: bcm: InterfaceMisc.c:
On 2014-06-20 18:39, H Hartley Sweeten wrote:
As pointed out by Ian Abbott, the INSN_CONFIG_SET_CLOCK_SRC options should
be exposed in the user header comedi.h.
Signed-off-by: H Hartley Sweeten
Cc: Ian Abbott
Cc: Greg Kroah-Hartman
---
drivers/staging/comedi/comedi.h | 11 ++
On 2014-06-20 18:58, H Hartley Sweeten wrote:
Introduce an inline helper to return the number of bytes that are
ready to read from the comedi_async buffer. Use the helper in the
comedi drivers that currently do the calculation as part of the
(*poll) operation.
Also, use the helper in comedi_fops
On 2014-06-21 23:23, Chase Southwood wrote:
This patchset introduces a new private data struct for this driver, adds
all of the code required to support Change-of-State interrupts for the
digital input subdevice, and finally fixes
apci1564_interrupt() to service this type of interrupt correctly.
On 2014-06-20 19:25, H Hartley Sweeten wrote:
The mite module provides the DMA interface for the PCI MITE ASIC used on
many National Instruments (NI) DAQ boards. Many of the comedi NI drivers
do not support DMA and only depend on the mite driver in order to initialze
the MITE ASIC.
Add the code
On Sun, 2014-06-22 at 03:29 -0400, James A Shackleford wrote:
> This patch allocates a few pages and performs an ioread8_rep() from the bus
> address, which are then copied to userspace. This fixes the sparse warning:
>
> drivers/staging/goldfish/goldfish_audio.c:136:43: warning: incorrect type i
On 2014-06-20 20:01, H Hartley Sweeten wrote:
On the PCI-1720 board, bipolar analog output ranges use two's
complement data. Use comedi_offset_munge() to munge the comedi
offset binary values before writing them to the hardware.
Which manual are you looking at? According to Table 4.7 in the
P
This is a patch for the slicoss.c file that fixes four instances
of the following warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Vincent Heuken
---
drivers/staging/slicoss/slicoss.c | 22 --
1 file changed, 12 insertions(+), 10 deletions
On 2014-06-20 20:01, H Hartley Sweeten wrote:
These static arrays are used to program the range regiser with the
correct values to select the analog input range. The values in the
arrays are incrememntal for the bipolar and unipolar ranges and
have bit 4 set when the range is unipolar.
Modify th
On 2014-06-20 21:28, H Hartley Sweeten wrote:
The struct s626_enc_info is used in this driver to defined the callbacks and
other data needed to handle each encoder "channel". These "channel" numbers
are the same as the comedi_subdevice channels.
The channels specific stuff can easily be handled
> -Original Message-
> From: Olaf Hering [mailto:o...@aepfle.de]
> Sent: Monday, June 23, 2014 4:03 AM
> To: Haiyang Zhang
> Cc: Greg KH; net...@vger.kernel.org; jasow...@redhat.com; driverdev-
> de...@linuxdriverproject.org; linux-ker...@vger.kernel.org;
> da...@davemloft.net
> Subject:
On Mon, Jun 23, Haiyang Zhang wrote:
> > I think its reasonable to expect guest config changes on this new kind
> > of host. Would a link-down/link-up event work? I'm sure it will, there
> > is enough code floating around in the guests which handles cable unplug.
>
> Do you mean netif_carrier_off
Remove the now unnecessary memset too.
Signed-off-by: Joe Perches
---
drivers/vme/bridges/vme_ca91cx42.c | 6 ++
drivers/vme/bridges/vme_tsi148.c | 6 ++
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/vme/bridges/vme_ca91cx42.c
b/drivers/vme/bridges/vme_ca91cx4
Adding the helper reduces object code size as well as overall
source size line count.
It's also consistent with all the various zalloc mechanisms
in the kernel.
Done with a simple cocci script and some typing.
Joe Perches (22):
pci-dma-compat: Add pci_zalloc_consistent helper
atm: Use pci_za
Remove the now unnecessary memset too.
Signed-off-by: Joe Perches
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 15 ---
drivers/staging/rtl8192ee/pci.c | 37 +
drivers/staging/rtl8821ae/pci.c | 36 +
drivers/s
On 06/17/2014 05:49 PM, Greg KH wrote:
On Tue, Jun 03, 2014 at 09:43:35AM -0500, Ken Cox wrote:
The unisys drivers now properly check to make sure they are running
on the s-Par platform before they will initialize. This was fixed in
commit fcd0157ece so it is safe to allow the unisys drivers t
Hi All,
Thanks for your reply. I have been away the last few weeks.
Dan, I am using 64 bit Debian 3.2.57. So I am not using the latest
kernel. Would this be a problem?
Martyn, I don't have a bus analyzer. I believe I have understood a
few more things about my setup, but I still manage to han
On 2014-06-20 21:13, H Hartley Sweeten wrote:
The DT2821 series board have jumpers that set the output range for
the two Analog Output channels. The range_table for the Analog Output
subdevice provides all possible ranges to the user. When a unipolar
range is selected the board expects the data t
On Mon, Jun 23, 2014 at 09:12:32AM -0500, Ken Cox wrote:
>
> On 06/17/2014 05:49 PM, Greg KH wrote:
> >On Tue, Jun 03, 2014 at 09:43:35AM -0500, Ken Cox wrote:
> >>The unisys drivers now properly check to make sure they are running
> >>on the s-Par platform before they will initialize. This was f
On 2014-06-20 19:52, H Hartley Sweeten wrote:
Im not sure what was intended with this boardinfo.
It appears to be trying to handle the sink/source output type of the
boards supported by the driver. But, some of the boardinfo does not
match the actual hardware descriptions in the manual. The manu
On Mon, Jun 23, 2014 at 06:41:50AM -0700, Joe Perches wrote:
> Remove the now unnecessary memset too.
>
> Signed-off-by: Joe Perches
> ---
> drivers/vme/bridges/vme_ca91cx42.c | 6 ++
> drivers/vme/bridges/vme_tsi148.c | 6 ++
> 2 files changed, 4 insertions(+), 8 deletions(-)
Acked-b
On Mon, Jun 23, 2014 at 06:41:48AM -0700, Joe Perches wrote:
> Remove the now unnecessary memset too.
>
> Signed-off-by: Joe Perches
> ---
> drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 15 ---
> drivers/staging/rtl8192ee/pci.c | 37 +
> drivers/sta
On Mon, Jun 23, 2014 at 11:45:26AM +0200, Matthias Beyer wrote:
> Hi,
>
> these patches are still not applied, am I right?
You should have gotten an email if they went into my tree. If not, yes,
please rebase and resend after checking my tree to verify they are not
already there (same for your o
> -Original Message-
> From: Olaf Hering [mailto:o...@aepfle.de]
> Sent: Monday, June 23, 2014 9:17 AM
> To: Haiyang Zhang
> Cc: Greg KH; net...@vger.kernel.org; jasow...@redhat.com; driverdev-
> de...@linuxdriverproject.org; linux-ker...@vger.kernel.org;
> da...@davemloft.net
> Subject:
On Mon, Jun 23, Haiyang Zhang wrote:
> I have tried some possibilities, like IF_OPER_DORMANT then IF_OPER_UP with
> netdev_state_change() etc. but not able to trigger DHCP review. I will look
> at this
> further...
Is there a link down/up event anyway?
If the interface is configured to "on hotp
On Mon, Jun 23, 2014 at 04:09:59PM +, Haiyang Zhang wrote:
>
>
> > -Original Message-
> > From: Olaf Hering [mailto:o...@aepfle.de]
> > Sent: Monday, June 23, 2014 9:17 AM
> > To: Haiyang Zhang
> > Cc: Greg KH; net...@vger.kernel.org; jasow...@redhat.com; driverdev-
> > de...@linuxdri
On 2014-06-20 19:10, H Hartley Sweeten wrote:
The comedi core expects the first member of the boardinfo to be a
'const char *' to the board name. The comedi_recognize() function
uses this to locate the correct boardinfo when doing a legacy
(*attach) to a comedi driver.
It doesn't matter in this
On 2014-06-23 17:36, Ian Abbott wrote:
On 2014-06-20 19:10, H Hartley Sweeten wrote:
The comedi core expects the first member of the boardinfo to be a
'const char *' to the board name. The comedi_recognize() function
uses this to locate the correct boardinfo when doing a legacy
(*attach) to a co
On Monday, June 23, 2014 4:57 AM, Ian Abbott wrote:
> On 2014-06-20 20:01, H Hartley Sweeten wrote:
>> On the PCI-1720 board, bipolar analog output ranges use two's
>> complement data. Use comedi_offset_munge() to munge the comedi
>> offset binary values before writing them to the hardware.
>
> Whi
On Mon, Jun 23, 2014 at 06:41:28AM -0700, Joe Perches wrote:
> Adding the helper reduces object code size as well as overall
> source size line count.
>
> It's also consistent with all the various zalloc mechanisms
> in the kernel.
>
> Done with a simple cocci script and some typing.
Awesome, an
On Monday, June 23, 2014 8:10 AM, Ian Abbott wrote:
> On 2014-06-20 21:13, H Hartley Sweeten wrote:
>> The DT2821 series board have jumpers that set the output range for
>> the two Analog Output channels. The range_table for the Analog Output
>> subdevice provides all possible ranges to the user. W
> -Original Message-
> From: Olaf Hering [mailto:o...@aepfle.de]
> Sent: Monday, June 23, 2014 12:27 PM
> To: Haiyang Zhang
> Cc: Greg KH; net...@vger.kernel.org; jasow...@redhat.com; driverdev-
> de...@linuxdriverproject.org; linux-ker...@vger.kernel.org;
> da...@davemloft.net
> Subject:
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Monday, June 23, 2014 12:29 PM
> To: Haiyang Zhang
> Cc: Olaf Hering; net...@vger.kernel.org; jasow...@redhat.com; driverdev-
> de...@linuxdriverproject.org; linux-ker...@vger.kernel.org;
> da...@davemloft.net
> Subject:
On Mon, 2014-06-23 at 10:25 -0700, Luis R. Rodriguez wrote:
> On Mon, Jun 23, 2014 at 06:41:28AM -0700, Joe Perches wrote:
> > Adding the helper reduces object code size as well as overall
> > source size line count.
> >
> > It's also consistent with all the various zalloc mechanisms
> > in the ke
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 40 +++
1 file changed, 19 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 612e61e..f528a45 100644
--- a/drivers/stag
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 138 ++
1 file changed, 79 insertions(+), 59 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index eee4f47..fd73dbd 100644
--- a/drivers/stag
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 5060bad..9808f53 100644
--- a/drivers/staging/bcm/led_control.c
+++ b
This patch removes the never-entered else-block inside
blink_in_normal_bandwidth(). The condition of the preceding if statement
is always true.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 98 +++
1 file changed, 37 insertions(+), 61 d
Hi,
these patchset contains cleanup patches for the
drivers/staging/bcm/led_control.c
file.
The patchset contains:
* Whitespace / Indentation fixes
* Removing dead code
* Shortened lines / code
* Outsourcing code chunks
Please note: As I do not have the appropriate Hardwar
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 325 --
1 file changed, 171 insertions(+), 154 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index fd73dbd..3b3775b 100644
--- a/drivers/st
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 159 ++
1 file changed, 93 insertions(+), 66 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 085493a..5060bad 100644
--- a/drivers/stag
This patch fixes the TURN_{ON,OFF}_LED() macros by adding the adapter as
parameter.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 39 +++
drivers/staging/bcm/led_control.h | 16
2 files changed, 31 insertions(+), 24 del
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 28 ++--
1 file changed, 10 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 4cf4212..c0509ae 100644
--- a/drivers/staging/bcm/led
This patch replaces member accessing with a variable in
BcmGetGPIOPinInfo().
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 8e6bb6b..8353677 100644
--- a/drivers/staging/bcm/led_control.c
++
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 4077972..8e6bb6b 100644
--- a/drivers/staging/bcm/led_control.c
+++ b
Hi,
this is a resend of my patchset for the
drivers/staging/bcm/InterfaceInit.c
file. I rebased them onto the lastes 'staging-next' from greg k-hs repository
and verified they are not in the branch already.
Please notice: I cannot test the patches as I do not have the appropriate
hardware.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index d61a688..dd7b4a5 100644
--- a/drivers/staging/bcm/Interface
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 53 +++--
1 file changed, 27 insertions(+), 26 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index e1a3e6d..2f540d6 100644
--- a/drivers/
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 43 +++--
1 file changed, 17 insertions(+), 26 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index 2f540d6..bda33c8 100644
--- a/drivers/
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index 7c04c73..d61a688 100644
--- a/drivers/staging/bcm/InterfaceInit.c
+++ b/
This patch outsources a chunk of code which is for selecting an
alternate setting for highspeed modem into a new function.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 207 +++-
1 file changed, 112 insertions(+), 95 deletions(-)
diff --
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index dd7b4a5..e1a3e6d 100644
--- a/drivers/staging/bcm/Interf
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 47 ++---
1 file changed, 18 insertions(+), 29 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index 9f9045d..276825f 100644
--- a/drivers/
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index c0509ae..a387c6a 100644
--- a/drivers/staging/bcm/led
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/InterfaceInit.c | 25 +++--
1 file changed, 19 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/bcm/InterfaceInit.c
b/drivers/staging/bcm/InterfaceInit.c
index 276825f..bef13a6 100644
--- a/drivers/staging/bcm/I
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index a387c6a..8a59115 100644
--- a/drivers/staging/bcm/led_control.c
+++ b/drivers/
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/HandleControlPacket.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/bcm/HandleControlPacket.c
b/drivers/staging/bcm/HandleControlPacket.c
index 495fe3d..467b0cf 100644
--- a/drivers/staging/bcm/HandleCo
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/HandleControlPacket.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/bcm/HandleControlPacket.c
b/drivers/staging/bcm/HandleControlPacket.c
index 467b0cf..45d18de 100644
--- a/drivers/staging/bcm/HandleCont
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/HandleControlPacket.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/bcm/HandleControlPacket.c
b/drivers/staging/bcm/HandleControlPacket.c
index 6552b15..5455221 100644
--- a/drivers/staging/bcm/HandleControlPacket.c
+++
Hi,
this patchset contains cleanup patches for the
drivers/staging/bcm/HandleControlPacket.c
file.
Please note: As I do not have the appropriate hardware, I cannot test these
patches. I compiled them at least.
Kind regards,
Matthias Beyer
Matthias Beyer (4):
Staging: bcm: line length fi
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/HandleControlPacket.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/bcm/HandleControlPacket.c
b/drivers/staging/bcm/HandleControlPacket.c
index 45d18de..6552b15 100644
--- a/drivers/st
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 25 +
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index ae49a59..4077972 100644
--- a/drivers/staging/bcm/led_co
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/led_control.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/bcm/led_control.c
b/drivers/staging/bcm/led_control.c
index 8a59115..ae49a59 100644
--- a/drivers/staging/bcm/led_control.c
+++ b/dr
Hi,
this patchset contains cleanup patches for the
drivers/staging/bcm/hostmibs.c
file, containing fixes for:
* Line length
* member accessing replaced with variables
* outsourced code chunks in own functions
Please note: As I do not have the approrpiate Hardware, I'm not able
This commit includes renaming of variables which got outsourced into an
own function to fit the kernel styleguide.
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/hostmibs.c | 70 +++---
1 file changed, 39 insertions(+), 31 deletions(-)
diff --git a/dri
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/hostmibs.c | 39 ++-
1 file changed, 26 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/bcm/hostmibs.c b/drivers/staging/bcm/hostmibs.c
index 63bfb04..9ffd25b 100644
--- a/drivers/staging/bcm/ho
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/hostmibs.c | 26 +-
1 file changed, 17 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/bcm/hostmibs.c b/drivers/staging/bcm/hostmibs.c
index af3493c..13c1430 100644
--- a/drivers/staging/bcm/hostmibs.c
+++ b
Signed-off-by: Matthias Beyer
---
drivers/staging/bcm/hostmibs.c | 23 ---
1 file changed, 12 insertions(+), 11 deletions(-)
diff --git a/drivers/staging/bcm/hostmibs.c b/drivers/staging/bcm/hostmibs.c
index 42d9004..af3493c 100644
--- a/drivers/staging/bcm/hostmibs.c
+++ b/d
From: Olaf Hering
Date: Mon, 23 Jun 2014 15:17:23 +0200
> On Mon, Jun 23, Haiyang Zhang wrote:
>
>> > I think its reasonable to expect guest config changes on this new kind
>> > of host. Would a link-down/link-up event work? I'm sure it will, there
>> > is enough code floating around in the gues
From: Haiyang Zhang
Date: Mon, 23 Jun 2014 16:09:59 +
> So, what's the equivalent or similar command to "network restart" on SLES12?
> Could
> you update the command line for the usermodehelper when porting this patch to
> SLES
> 12?
No, you are not going to keep the usermodehelper invocat
From: Greg KH
Date: Mon, 23 Jun 2014 12:29:11 -0400
> Given that this change will fail on all future distro releases, and
> almost all of the community distros today, I don't see how this is
> acceptable at all. Nor would it be any better if you switch to a
> systemd command line script as well.
Hi Bob,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving cxt1e1 out of staging in over
a year. Are there any plans to clean it up and move it out soon? Because
otherwise we're going to have to delete the driver, as we don't want
st
Hi Sage,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving gdm72xx out of staging in
over a year. Are there any plans to clean it up and move it out soon?
Because otherwise we're going to have to delete the driver, as we don't
want
Hi Angelo,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving quickstart out of staging in
over a year. Are there any plans to clean it up and move it out soon?
Because otherwise we're going to have to delete the driver, as we don't
Hi Daniel,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving silicom out of staging in
over a year. Are there any plans to clean it up and move it out soon?
Because otherwise we're going to have to delete the driver, as we don't
wan
Hi Markus,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving line6 out of staging in over
a year. Are there any plans to clean it up and move it out soon? Because
otherwise we're going to have to delete the driver, as we don't want
Hi Evan,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving phison out of staging in over
a year. Are there any plans to clean it up and move it out soon? Because
otherwise we're going to have to delete the driver, as we don't want
s
Hi Bill,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving dgrp or serqt_usb2 out of
staging in over a year. Are there any plans to clean these drivers up
and move them out soon? Because otherwise we're going to have to delete
them,
Hi Mark,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving sep out of staging in over a
year. Are there any plans to clean it up and move it out soon? Because
otherwise we're going to have to delete the driver, as we don't want
stag
On Mon, Jun 23, 2014 at 11:42:28AM +0200, Matthias Beyer wrote:
> Signed-off-by: Matthias Beyer
Reviewed-by: Josh Triplett
> drivers/staging/bcm/DDRInit.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/bcm/DDRInit.c b/drivers/staging/bcm/DD
On Mon, Jun 23, 2014 at 11:42:29AM +0200, Matthias Beyer wrote:
> This patch fixes some indentation errors, where multi-line statements
> where not indented.
>
> Signed-off-by: Matthias Beyer
This patch also fixes some whitespace issues, such as a lack of space
before '/', which should probably
On Mon, Jun 23, 2014 at 11:42:30AM +0200, Matthias Beyer wrote:
> Signed-off-by: Matthias Beyer
Honestly, none of these seem like improvements. Checkpatch's line-width
warnings rarely result in improved code.
(Its "maximum indentation level" warnings, which prompt people to
refactor functions,
On Mon, Jun 23, 2014 at 11:42:31AM +0200, Matthias Beyer wrote:
> The inner if-statement was aligned just like the outer one. Why?
>
> This indention was introduced by
>
> f34c488c3894968e8cdbdc3b1ed617d78315cace
>
> which is a indention-fix patch itself. That's why I'm curious about it.
>
On Mon, Jun 23, 2014 at 11:42:33AM +0200, Matthias Beyer wrote:
> Signed-off-by: Matthias Beyer
As with the previous line-wrapping patch, this doesn't seem like a net
improvement; the lines seem far more readable un-wrapped.
> drivers/staging/bcm/DDRInit.c | 31 +--
>
On 23/06/14 23:41, Ben Chan wrote:
> Hi Kristina and Greg,
>
> I can help migrate the gdm72xx driver from staging to mainline. I
> believe the issues in TODO have been addressed.
That's great, thanks!
> Is there a migration process that I should follow?
I don't think there's anything special yo
From: Joe Perches
Date: Mon, 23 Jun 2014 06:41:28 -0700
> Adding the helper reduces object code size as well as overall
> source size line count.
>
> It's also consistent with all the various zalloc mechanisms
> in the kernel.
>
> Done with a simple cocci script and some typing.
For networking
On 06/23/2014 04:26 PM, Kristina Martšenko wrote:
Hi Bill,
I'm helping Greg do a bit of cleanup in the staging tree. I noticed that
nobody seems to have worked towards moving dgrp or serqt_usb2 out of
staging in over a year. Are there any plans to clean these drivers up
and move them out soon? B
On 21/06/14 20:52, Christian Lamparter wrote:
> On Saturday, June 21, 2014 06:45:07 PM Kristina Martšenko wrote:
>> On 20/06/14 23:52, Christian Lamparter wrote:
>>> rtl8192su development is chugging along. It just doesn't take place on
>>> driverdev list. The driver reached "feature parity" with r
On 21/06/14 21:24, Larry Finger wrote:
> On 06/21/2014 12:52 PM, Christian Lamparter wrote:
>> I think there's a reason why the development takes so long
>> (The RTL8192S* chips have been around since 2008, 2009)
>> and why we all have better things to do and write harsh
>> responses. As far as I c
1 - 100 of 107 matches
Mail list logo
