From: David Daney
Date: Fri, 17 Feb 2017 12:04:12 -0800
> Some Cavium dev boards have firmware which doesn't supply a proper
> ethernet-phy-ieee802.3-c22" compatible property. Restore these boards
> to working order by whitelisting this compatible value.
>
>
From: David Daney
Date: Fri, 17 Feb 2017 12:04:12 -0800
> Some Cavium dev boards have firmware which doesn't supply a proper
> ethernet-phy-ieee802.3-c22" compatible property. Restore these boards
> to working order by whitelisting this compatible value.
>
> Signed-off-by: David Daney
From: David Wu
Date: Fri, 17 Feb 2017 20:55:11 +0800
> From: "david.wu"
>
> Add constants and callback functions for the dwmac on rk3328 socs.
> As can be seen, the base structure is the same, only registers and the
> bits in them moved
From: David Wu
Date: Fri, 17 Feb 2017 20:55:11 +0800
> From: "david.wu"
>
> Add constants and callback functions for the dwmac on rk3328 socs.
> As can be seen, the base structure is the same, only registers and the
> bits in them moved slightly.
>
> Signed-off-by: david.wu
Applied.
So there it is, the final 4.10 release. It's been quiet since rc8, but
we did end up fixing several small issues, so the extra week was all
good.
On the whole, 4.10 didn't end up as small as it initially looked.
After the huge release that was 4.9, I expected things to be pretty
quiet, but it
So there it is, the final 4.10 release. It's been quiet since rc8, but
we did end up fixing several small issues, so the extra week was all
good.
On the whole, 4.10 didn't end up as small as it initially looked.
After the huge release that was 4.9, I expected things to be pretty
quiet, but it
On Sun, 19 Feb 2017, Ondrej Zary wrote:
>
> These two patches are enough to make PDMA work with CD-ROM drives on
> 53C400(A), with IRQ enabled or not. They even improve transfer rates
> with HDDs because of bigger block size.
>
Nice! This is a great improvement.
> But DTC436 breaks with
On Sun, 19 Feb 2017, Ondrej Zary wrote:
>
> These two patches are enough to make PDMA work with CD-ROM drives on
> 53C400(A), with IRQ enabled or not. They even improve transfer rates
> with HDDs because of bigger block size.
>
Nice! This is a great improvement.
> But DTC436 breaks with
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:05:45 +0300
> On the kernel side, sockaddr_storage is #define'd to
> __kernel_sockaddr_storage. Replacing struct sockaddr_storage with
> struct __kernel_sockaddr_storage defined by fixes
> the following linux/rds.h userspace
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:46 +0300
> Include to fix the following linux/mroute.h userspace
> compilation errors:
>
> /usr/include/linux/mroute.h:58:18: error: field 'vifc_lcl_addr' has
> incomplete type
> struct in_addr vifc_lcl_addr; /*
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:05:45 +0300
> On the kernel side, sockaddr_storage is #define'd to
> __kernel_sockaddr_storage. Replacing struct sockaddr_storage with
> struct __kernel_sockaddr_storage defined by fixes
> the following linux/rds.h userspace compilation error:
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:46 +0300
> Include to fix the following linux/mroute.h userspace
> compilation errors:
>
> /usr/include/linux/mroute.h:58:18: error: field 'vifc_lcl_addr' has
> incomplete type
> struct in_addr vifc_lcl_addr; /* Local interface
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:05:13 +0300
> Consistently use types from linux/types.h to fix the following
> linux/rds.h userspace compilation errors:
...
> Signed-off-by: Dmitry V. Levin
Applied.
On Sun, Feb 19, 2017 at 9:00 AM, Alan Tull wrote:
> On Sat, Feb 18, 2017 at 2:45 PM, Moritz Fischer
> wrote:
>> On Sat, Feb 18, 2017 at 02:10:43PM -0600, Alan Tull wrote:
>>> On Sat, Feb 18, 2017 at 6:45 AM, Nadathur, Sundar
>>>
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:05:13 +0300
> Consistently use types from linux/types.h to fix the following
> linux/rds.h userspace compilation errors:
...
> Signed-off-by: Dmitry V. Levin
Applied.
On Sun, Feb 19, 2017 at 9:00 AM, Alan Tull wrote:
> On Sat, Feb 18, 2017 at 2:45 PM, Moritz Fischer
> wrote:
>> On Sat, Feb 18, 2017 at 02:10:43PM -0600, Alan Tull wrote:
>>> On Sat, Feb 18, 2017 at 6:45 AM, Nadathur, Sundar
>>> wrote:
>>>
>>> > Hi all,
>>> >Interesting discussion. The
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:14 +0300
> Include to fix the following linux/ipv6_route.h userspace
> compilation errors:
>
> /usr/include/linux/ipv6_route.h:42:19: error: field 'rtmsg_dst' has
> incomplete type
> struct in6_addr rtmsg_dst;
>
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:29 +0300
> Include to fix the following linux/mroute6.h userspace
> compilation errors:
>
> /usr/include/linux/mroute6.h:80:22: error: field 'mf6cc_origin' has
> incomplete type
> struct sockaddr_in6 mf6cc_origin; /*
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:14 +0300
> Include to fix the following linux/ipv6_route.h userspace
> compilation errors:
>
> /usr/include/linux/ipv6_route.h:42:19: error: field 'rtmsg_dst' has
> incomplete type
> struct in6_addr rtmsg_dst;
>
From: "Dmitry V. Levin"
Date: Thu, 16 Feb 2017 18:04:29 +0300
> Include to fix the following linux/mroute6.h userspace
> compilation errors:
>
> /usr/include/linux/mroute6.h:80:22: error: field 'mf6cc_origin' has
> incomplete type
> struct sockaddr_in6 mf6cc_origin; /* Origin of mcast */
>
Sparse emits several 'symbol not declared' warnings for various
functions and variables.
Add static keyword to functions and variables which have file scope
only.
Add header file with funciton declaration. Add preprocessor guard and
include header in ASM file. Include header in C file
Sparse emits several 'symbol not declared' warnings for various
functions and variables.
Add static keyword to functions and variables which have file scope
only.
Add header file with funciton declaration. Add preprocessor guard and
include header in ASM file. Include header in C file
Sparse emits several 'symbol not declared' warnings for various
functions and variables.
Add static keyword to functions and variables which have file scope
only.
Signed-off-by: Tobin C. Harding
---
arch/x86/purgatory/purgatory.c | 10 +-
1 file changed, 5 insertions(+),
Sparse emits several 'symbol not declared' warnings for various
functions and variables.
Add static keyword to functions and variables which have file scope
only.
Signed-off-by: Tobin C. Harding
---
arch/x86/purgatory/purgatory.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
Sparse emits warning, 'symbol not declared' for a function that has
neither file scope nor a forward declaration. The functions only call
site is an ASM file.
Add a header file with the function declaration. Include the
header file in the C source file defining the function in order
to fix the
Sparse emits warning, 'symbol not declared' for a function that has
neither file scope nor a forward declaration. The functions only call
site is an ASM file.
Add a header file with the function declaration. Include the
header file in the C source file defining the function in order
to fix the
Hi John and Pali,
On Sun, Feb 19, 2017 at 03:24:58PM +, John Preston wrote:
> On 2017-02-19 13:35, Pali Rohár wrote:
> > On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
> > > ---
> > > drivers/input/mouse/alps.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git
Hi John and Pali,
On Sun, Feb 19, 2017 at 03:24:58PM +, John Preston wrote:
> On 2017-02-19 13:35, Pali Rohár wrote:
> > On Sunday 19 February 2017 13:16:34 John L. Preston wrote:
> > > ---
> > > drivers/input/mouse/alps.c | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git
On 02/19/2017 06:33 PM, kbuild test robot wrote:
> drivers/spi/spi-lantiq-ssc.c:973:3-8: No need to set .owner here. The core
> will do it.
>
> Remove .owner field if calls are used which set it automatically
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> CC: Hauke
On 02/19/2017 06:33 PM, kbuild test robot wrote:
> drivers/spi/spi-lantiq-ssc.c:973:3-8: No need to set .owner here. The core
> will do it.
>
> Remove .owner field if calls are used which set it automatically
>
> Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
>
> CC: Hauke
Hi Bjorn,
Today's linux-next merge of the pci tree got conflicts in:
drivers/pci/dwc/pcie-designware-host.c
I have no idea why? I see this file has been split out from another that
was itself moved from another directory, but that all happened in the
pci tree.
I fixed it up (I just used the
Hi Bjorn,
Today's linux-next merge of the pci tree got conflicts in:
drivers/pci/dwc/pcie-designware-host.c
I have no idea why? I see this file has been split out from another that
was itself moved from another directory, but that all happened in the
pci tree.
I fixed it up (I just used the
Hi!
> > > And note that I am not making things different from others in tree,
> > > I am making the tsc2007 right (incl. following the touchscreen
> > > bindings which define the touchscreen size in "Pixels").
> >
> > Your touch screen is not in any way special, so it has to behave in
> > the
Hi!
> > > And note that I am not making things different from others in tree,
> > > I am making the tsc2007 right (incl. following the touchscreen
> > > bindings which define the touchscreen size in "Pixels").
> >
> > Your touch screen is not in any way special, so it has to behave in
> > the
> On Feb 18, 2017, at 11:04 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 15:16:54 +0100
>
> The script "checkpatch.pl" pointed information out like the following.
>
> WARNING: void function
> On Feb 18, 2017, at 11:04 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 15:16:54 +0100
>
> The script "checkpatch.pl" pointed information out like the following.
>
> WARNING: void function return statements are not generally useful
>
> Thus remove such
Hi,
On Sun, 19 Feb 2017 21:57:08 +0100
Pavel Machek wrote:
> Hi!
>
>
> > But as said I don't think we need float or fixed point for
> > practical systems at all.
>
> So you are going to loose precision. And if userspace decides to
> calibrate it slightly differently from
Hi,
On Sun, 19 Feb 2017 21:57:08 +0100
Pavel Machek wrote:
> Hi!
>
>
> > But as said I don't think we need float or fixed point for
> > practical systems at all.
>
> So you are going to loose precision. And if userspace decides to
> calibrate it slightly differently from kernel, lost
hi!
> >> But as said I don't think we need float or fixed point for practical
> >> systems
> >> at all.
> >
> > So you are going to loose precision. And if userspace decides to
> > calibrate it slightly differently from kernel, lost precision will
> > matter.
>
> Really?
Really.
> Example:
>
hi!
> >> But as said I don't think we need float or fixed point for practical
> >> systems
> >> at all.
> >
> > So you are going to loose precision. And if userspace decides to
> > calibrate it slightly differently from kernel, lost precision will
> > matter.
>
> Really?
Really.
> Example:
>
On 02/19/2017 03:00 PM, Stephen Rothwell wrote:
Hi Corey,
On Wed, 22 Jul 2015 11:03:31 +1000 Stephen Rothwell
wrote:
On Tue, 21 Jul 2015 18:45:33 -0500 Corey Minyard wrote:
On 07/21/2015 05:29 PM, Stephen Rothwell wrote:
While fetching the ipmi
On 02/19/2017 03:00 PM, Stephen Rothwell wrote:
Hi Corey,
On Wed, 22 Jul 2015 11:03:31 +1000 Stephen Rothwell
wrote:
On Tue, 21 Jul 2015 18:45:33 -0500 Corey Minyard wrote:
On 07/21/2015 05:29 PM, Stephen Rothwell wrote:
While fetching the ipmi tree
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
As I don't have the hardware, I'd be very pleased if
someone may test this patch.
Signed-off-by: Philippe Reynes
---
drivers/net/ethernet/qlogic/qlge/qlge_ethtool.c |
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
As I don't have the hardware, I'd be very pleased if
someone may test this patch.
Signed-off-by: Philippe Reynes
---
drivers/net/ethernet/qlogic/qlge/qlge_ethtool.c | 36
Hi!
> From: Philipp Zabel
>
> This driver can handle SoC internal and external video bus multiplexers,
> controlled either by register bit fields or by a GPIO. The subdevice
> passes through frame interval and mbus configuration of the active input
> to the output side.
Hi!
> From: Philipp Zabel
>
> This driver can handle SoC internal and external video bus multiplexers,
> controlled either by register bit fields or by a GPIO. The subdevice
> passes through frame interval and mbus configuration of the active input
> to the output side.
>
> Signed-off-by:
Hi Pavel,
> Am 19.02.2017 um 21:57 schrieb Pavel Machek :
>
> Hi!
>
>
>> But as said I don't think we need float or fixed point for practical systems
>> at all.
>
> So you are going to loose precision. And if userspace decides to
> calibrate it slightly differently from kernel,
Hi Pavel,
> Am 19.02.2017 um 21:57 schrieb Pavel Machek :
>
> Hi!
>
>
>> But as said I don't think we need float or fixed point for practical systems
>> at all.
>
> So you are going to loose precision. And if userspace decides to
> calibrate it slightly differently from kernel, lost precision
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
As I don't have the hardware, I'd be very pleased if
someone may test this patch.
Signed-off-by: Philippe Reynes
---
.../net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
As I don't have the hardware, I'd be very pleased if
someone may test this patch.
Signed-off-by: Philippe Reynes
---
.../net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c| 94
Unnecessary parentheses should be avoided as reported by checkpatch.pl.
Remove unnecessary parentheses, as reported by checkpatch as are nicer
to read.For example:-
It's often nicer to read if &(foo[0]) is converted to foo like:
memcpy(&(ap->bssid[0]), &(ap_info->bssid[0]), ETH_ALEN);
Unnecessary parentheses should be avoided as reported by checkpatch.pl.
Remove unnecessary parentheses, as reported by checkpatch as are nicer
to read.For example:-
It's often nicer to read if &(foo[0]) is converted to foo like:
memcpy(&(ap->bssid[0]), &(ap_info->bssid[0]), ETH_ALEN);
On Wed 2017-02-15 18:19:16, Steve Longerbeam wrote:
> v4l2_pipeline_inherit_controls() will add the v4l2 controls from
> all subdev entities in a pipeline to a given video device.
>
> Signed-off-by: Steve Longerbeam
Reviewed-by: Pavel Machek
--
On Wed 2017-02-15 18:19:16, Steve Longerbeam wrote:
> v4l2_pipeline_inherit_controls() will add the v4l2 controls from
> all subdev entities in a pipeline to a given video device.
>
> Signed-off-by: Steve Longerbeam
Reviewed-by: Pavel Machek
--
(english)
Change null comparisons of the form x == NULL to !x.
This was done using Coccinelle.
@@ expression e;
@@
- e == NULL
+ !e
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_TSProc.c| 2 +-
drivers/staging/rtl8192e/rtllib_crypt_ccmp.c | 2 +-
Change null comparisons of the form x == NULL to !x.
This was done using Coccinelle.
@@ expression e;
@@
- e == NULL
+ !e
Signed-off-by: simran singhal
---
drivers/staging/rtl8192e/rtl819x_TSProc.c| 2 +-
drivers/staging/rtl8192e/rtllib_crypt_ccmp.c | 2 +-
On Sat, Feb 18, 2017 at 12:22 PM, Logan Gunthorpe wrote:
> Hi,
>
> Please don't apply this patch and instead apply the switchtec driver as
> we submitted in v2. As per the discussion in [1], not using the cdev's
> kobj parent results in incorrect reference counting and a
On Sat, Feb 18, 2017 at 12:22 PM, Logan Gunthorpe wrote:
> Hi,
>
> Please don't apply this patch and instead apply the switchtec driver as
> we submitted in v2. As per the discussion in [1], not using the cdev's
> kobj parent results in incorrect reference counting and a possible use
> of the
On Wed 2017-02-15 18:19:14, Steve Longerbeam wrote:
> From: Philipp Zabel
>
> Signed-off-by: Philipp Zabel
>
> - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX
>
> Signed-off-by: Steve Longerbeam
This is slightly
On Wed 2017-02-15 18:19:14, Steve Longerbeam wrote:
> From: Philipp Zabel
>
> Signed-off-by: Philipp Zabel
>
> - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX
>
> Signed-off-by: Steve Longerbeam
This is slightly "interesting" format of changelog. Normally signoffs
go below.
> diff --git
On Sun, 19 Feb 2017 19:48:09 +0100,
Bhumika Goyal wrote:
>
> Declare snd_kcontrol_new structures as const as they are only passed as
> an argument to the function snd_ctl_new1. This argument is of type
> const, so snd_kcontrol_new structures having the same property can be
> made const too.
>
On Sun, 19 Feb 2017 19:48:09 +0100,
Bhumika Goyal wrote:
>
> Declare snd_kcontrol_new structures as const as they are only passed as
> an argument to the function snd_ctl_new1. This argument is of type
> const, so snd_kcontrol_new structures having the same property can be
> made const too.
>
Hi Corey,
On Wed, 22 Jul 2015 11:03:31 +1000 Stephen Rothwell
wrote:
>
> On Tue, 21 Jul 2015 18:45:33 -0500 Corey Minyard wrote:
> >
> > On 07/21/2015 05:29 PM, Stephen Rothwell wrote:
> > >
> > > While fetching the ipmi tree
> > >
Hi Corey,
On Wed, 22 Jul 2015 11:03:31 +1000 Stephen Rothwell
wrote:
>
> On Tue, 21 Jul 2015 18:45:33 -0500 Corey Minyard wrote:
> >
> > On 07/21/2015 05:29 PM, Stephen Rothwell wrote:
> > >
> > > While fetching the ipmi tree
> > > (git://git.code.sf.net/p/openipmi/linux-ipmi#for-next), I
Hi!
> But as said I don't think we need float or fixed point for practical systems
> at all.
So you are going to loose precision. And if userspace decides to
calibrate it slightly differently from kernel, lost precision will
matter.
> >>> and floating point in kernel is hard. Also
> >>> you'd
Hi!
> But as said I don't think we need float or fixed point for practical systems
> at all.
So you are going to loose precision. And if userspace decides to
calibrate it slightly differently from kernel, lost precision will
matter.
> >>> and floating point in kernel is hard. Also
> >>> you'd
Reorder error handling labels in order to match the way resources have
been allocated.
Signed-off-by: Christophe JAILLET
---
drivers/iio/adc/xilinx-xadc-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Reorder error handling labels in order to match the way resources have
been allocated.
Signed-off-by: Christophe JAILLET
---
drivers/iio/adc/xilinx-xadc-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iio/adc/xilinx-xadc-core.c
SMARC-sAMX6i is a SMARC (Smart Mobility Architecture) compliant
module.
Signed-off-by: Priit Laes
---
arch/arm/boot/dts/imx6q-smarc-sam6xi.dtsi | 434 ++
1 file changed, 434 insertions(+)
create mode 100644 arch/arm/boot/dts/imx6q-smarc-sam6xi.dtsi
SMARC-sAMX6i is a SMARC (Smart Mobility Architecture) compliant
module.
Signed-off-by: Priit Laes
---
arch/arm/boot/dts/imx6q-smarc-sam6xi.dtsi | 434 ++
1 file changed, 434 insertions(+)
create mode 100644 arch/arm/boot/dts/imx6q-smarc-sam6xi.dtsi
diff --git
The DuraCOR 310 is a tactical mission computer based on a
low-power four-core NXP i.MX6 Quad Arm processor equipped
with a rugged Flash disk and PCIe-Mini Card I/O expansion slots.
Add initial board support with CAN, eMMC, I2C, PCI, SATA, UART
and USB enabled.
Signed-off-by: Priit Laes
The DuraCOR 310 is a tactical mission computer based on a
low-power four-core NXP i.MX6 Quad Arm processor equipped
with a rugged Flash disk and PCIe-Mini Card I/O expansion slots.
Add initial board support with CAN, eMMC, I2C, PCI, SATA, UART
and USB enabled.
Signed-off-by: Priit Laes
---
CONFIG_FHANDLE is a must-have for newer distributions with systemd
support (including Debian stable).
Signed-off-by: Priit Laes
---
arch/arm/configs/imx_v6_v7_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/imx_v6_v7_defconfig
CONFIG_FHANDLE is a must-have for newer distributions with systemd
support (including Debian stable).
Signed-off-by: Priit Laes
---
arch/arm/configs/imx_v6_v7_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/imx_v6_v7_defconfig
Add basic devicetree files for Parvus Duracor C310 which is a custom
carrier with MXM3 connector for SMARC module.
SMARC module used in C310 is manufactured by Kontron AG.
Priit Laes (4):
arm: defconfig: imx6: Added CONFIG_FHANDLE
devicetree: Add vendor prefix for Kontron AG
arm: dts:
Kontron AG is a German-based company designing and manufacturing
embedded modules, computers and systems.
Signed-off-by: Priit Laes
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Add basic devicetree files for Parvus Duracor C310 which is a custom
carrier with MXM3 connector for SMARC module.
SMARC module used in C310 is manufactured by Kontron AG.
Priit Laes (4):
arm: defconfig: imx6: Added CONFIG_FHANDLE
devicetree: Add vendor prefix for Kontron AG
arm: dts:
Kontron AG is a German-based company designing and manufacturing
embedded modules, computers and systems.
Signed-off-by: Priit Laes
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Hi,
On Sun, Feb 19, 2017 at 01:07:26PM +0100, H. Nikolaus Schaller wrote:
> I don't really remember when we noticed it first. Maybe it was
> back in tslib times some years ago where setting the sensitivity
> threshold made problems. We then carried along our patch for a
> long time in our local
Hi,
On Sun, Feb 19, 2017 at 01:07:26PM +0100, H. Nikolaus Schaller wrote:
> I don't really remember when we noticed it first. Maybe it was
> back in tslib times some years ago where setting the sensitivity
> threshold made problems. We then carried along our patch for a
> long time in our local
On Thu, Feb 16, 2017 at 07:29:57PM +0100, Maxime Ripard wrote:
> On Thu, Feb 16, 2017 at 07:17:54AM +0100, Rask Ingemann Lambertsen wrote:
> > > > Supported features (+ means tested):
[...]
> > > > + SATA port on on-board SATA-to-USB bridge *
[...]
> > > > * Only shows up when a SATA device is
On Thu, Feb 16, 2017 at 07:29:57PM +0100, Maxime Ripard wrote:
> On Thu, Feb 16, 2017 at 07:17:54AM +0100, Rask Ingemann Lambertsen wrote:
> > > > Supported features (+ means tested):
[...]
> > > > + SATA port on on-board SATA-to-USB bridge *
[...]
> > > > * Only shows up when a SATA device is
> On Feb 18, 2017, at 11:12 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:32:41 +0100
>
> Do not use curly brackets at one source code place
> where a single statement should be sufficient.
>
> On Feb 18, 2017, at 11:12 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:32:41 +0100
>
> Do not use curly brackets at one source code place
> where a single statement should be sufficient.
>
> Signed-off-by: Markus Elfring
> ---
>
> On Feb 18, 2017, at 11:15 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 18:29:30 +0100
>
> Replace the specification of two data structures by pointer dereferences
> as the parameter for the
> On Feb 18, 2017, at 11:15 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 18:29:30 +0100
>
> Replace the specification of two data structures by pointer dereferences
> as the parameter for the operator "sizeof" to make the corresponding size
>
> On Feb 18, 2017, at 11:12 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:56:52 +0100
>
> * A multiplication for the size determination of a memory allocation
> indicated that an array data
> On Feb 18, 2017, at 11:12 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:56:52 +0100
>
> * A multiplication for the size determination of a memory allocation
> indicated that an array data structure should be processed.
> Thus use the corresponding
On Sun, Feb 19, 2017 at 08:04:38PM +0100, Christophe JAILLET wrote:
> According to the surrounding goto, it is likely that 'out_destroy_sq' was
> expected here.
s/surrounding/preceeding/ ?
But the patch itself looks good, thanks.
Reviewed-by: Christoph Hellwig
On Sun, Feb 19, 2017 at 08:04:38PM +0100, Christophe JAILLET wrote:
> According to the surrounding goto, it is likely that 'out_destroy_sq' was
> expected here.
s/surrounding/preceeding/ ?
But the patch itself looks good, thanks.
Reviewed-by: Christoph Hellwig
> On Feb 18, 2017, at 11:10 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:15:20 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> Link:
>
> On Feb 18, 2017, at 11:10 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:15:20 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> Link:
>
> On Feb 18, 2017, at 11:09 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:00:28 +0100
>
> Delete an assignment for the local variable "num_ports" at the beginning
> because it was initialised
> On Feb 18, 2017, at 11:09 PM, SF Markus Elfring
> wrote:
>
> From: Markus Elfring
> Date: Sat, 18 Feb 2017 16:00:28 +0100
>
> Delete an assignment for the local variable "num_ports" at the beginning
> because it was initialised with the same value.
>
> Signed-off-by: Markus Elfring
> ---
Hi Pavel,
> Am 19.02.2017 um 20:05 schrieb Pavel Machek :
>
> Hi!
>
>> Hi Pavel,
>> I love discussions with you :)
>
> Unfortunately I can't say the same.
>
>>> Am 19.02.2017 um 18:15 schrieb Pavel Machek :
>>>
>>>
> Solve it properly. That means passing
Hi Pavel,
> Am 19.02.2017 um 20:05 schrieb Pavel Machek :
>
> Hi!
>
>> Hi Pavel,
>> I love discussions with you :)
>
> Unfortunately I can't say the same.
>
>>> Am 19.02.2017 um 18:15 schrieb Pavel Machek :
>>>
>>>
> Solve it properly. That means passing calibration
> data from
Reorder 'out_free' and 'out_free_devt' error handling labels in order to
match the way resources have been allocated.
Fixes: 0dba1314d4f8 ("scsi, block: fix duplicate bdi name registration crashes")
Signed-off-by: Christophe JAILLET
---
drivers/scsi/sd.c | 4
Reorder 'out_free' and 'out_free_devt' error handling labels in order to
match the way resources have been allocated.
Fixes: 0dba1314d4f8 ("scsi, block: fix duplicate bdi name registration crashes")
Signed-off-by: Christophe JAILLET
---
drivers/scsi/sd.c | 4 ++--
1 file changed, 2
>> @@ -257,6 +257,8 @@ struct dentry *mount_pseudo_xattr(struct
>> file_system_type *fs_type, char *name,
>> + s->s_time_min = TIME64_MIN;
>> + s->s_time_max = TIME64_MAX;
>> @@ -518,6 +520,8 @@ int simple_fill_super(struct super_block *s, unsigned
>> long magic,
>> + s->s_time_min =
>> @@ -257,6 +257,8 @@ struct dentry *mount_pseudo_xattr(struct
>> file_system_type *fs_type, char *name,
>> + s->s_time_min = TIME64_MIN;
>> + s->s_time_max = TIME64_MAX;
>> @@ -518,6 +520,8 @@ int simple_fill_super(struct super_block *s, unsigned
>> long magic,
>> + s->s_time_min =
201 - 300 of 630 matches
Mail list logo