On 2016-11-03 20:48, SF Markus Elfring wrote:
>> You could define an xprintf() macro that checks if the return value
>> is < 0 and simply calls perror() and exit(1) in such case.
>
> Does such a macro belong to any general header file from the Linux
> software library?
No.
Michal
On 2016-11-03 20:48, SF Markus Elfring wrote:
>> You could define an xprintf() macro that checks if the return value
>> is < 0 and simply calls perror() and exit(1) in such case.
>
> Does such a macro belong to any general header file from the Linux
> software library?
No.
Michal
On Thu, Nov 3, 2016 at 11:22 PM, Andrey Smirnov
wrote:
>>> This is good, you went faster than me !
>>>
>>> Small point, could you add Kconfig dependency on REGMAP ?
>
> Good catch! Will fix in v2 of the set.
OK I wait for a v2 with the ACK/test tags then I'll apply
On Thu, Nov 3, 2016 at 11:22 PM, Andrey Smirnov
wrote:
>>> This is good, you went faster than me !
>>>
>>> Small point, could you add Kconfig dependency on REGMAP ?
>
> Good catch! Will fix in v2 of the set.
OK I wait for a v2 with the ACK/test tags then I'll apply that.
Yours,
Linus Walleij
Ignore value of interrupt distribution mode for common interrupts in
IDU since setting an affinity using value from Device Tree is deprecated
in ARC. Originially it is done in idu_irq_xlate function and it is
semantically wrong and does not guaranty that an affinity value will be
set properly.
Ignore value of interrupt distribution mode for common interrupts in
IDU since setting an affinity using value from Device Tree is deprecated
in ARC. Originially it is done in idu_irq_xlate function and it is
semantically wrong and does not guaranty that an affinity value will be
set properly.
On 16-11-02 02:29 PM, Mark Lord wrote:
I have poked at it some more, and thus far it appears that it is
only necessary to disable TCP rx checksums. The system doesn't crash
when only IP/UDP checksums are enabled, but does when TCP checksums are on.
This happens regardless of whether RX_AGG is
On 16-11-02 02:29 PM, Mark Lord wrote:
I have poked at it some more, and thus far it appears that it is
only necessary to disable TCP rx checksums. The system doesn't crash
when only IP/UDP checksums are enabled, but does when TCP checksums are on.
This happens regardless of whether RX_AGG is
This reverts commit 05fd007e4629 ("console: don't prefer first registered
if DT specifies stdout-path").
The reverted commit changes existing behavior on which many ARM boards
rely. Many ARM small-board-computers, like e.g. the Raspberry Pi have
both a video output and a serial console. Depending
Hi All,
While booting 4.9-rc# for the first time on an Allwinner A33 tablet,
I noticed that after u-boot the LCD display stayed black. It turns out
that there was an issue which caused X to never get up, and all kernel
(and other startup) messages prior to that only went to ttyS0 which
consists
This reverts commit 05fd007e4629 ("console: don't prefer first registered
if DT specifies stdout-path").
The reverted commit changes existing behavior on which many ARM boards
rely. Many ARM small-board-computers, like e.g. the Raspberry Pi have
both a video output and a serial console. Depending
Hi All,
While booting 4.9-rc# for the first time on an Allwinner A33 tablet,
I noticed that after u-boot the LCD display stayed black. It turns out
that there was an issue which caused X to never get up, and all kernel
(and other startup) messages prior to that only went to ttyS0 which
consists
On Fri, 04 Nov 2016, Jarkko Sakkinen wrote:
> In order too make Documentation root directory cleaner move the tpm
> directory under Documentation/security.
FWIW I like this.
BR,
Jani.
>
> Signed-off-by: Jarkko Sakkinen
> ---
>
On Fri, 04 Nov 2016, Jarkko Sakkinen wrote:
> In order too make Documentation root directory cleaner move the tpm
> directory under Documentation/security.
FWIW I like this.
BR,
Jani.
>
> Signed-off-by: Jarkko Sakkinen
> ---
> Documentation/index.rst | 2 +-
>
Hi,
On 04-11-16 08:52, Jacek Anaszewski wrote:
Initially the claim about no need for lock in brightness_show()
was valid as the function was just returning unchanged
LED brightness. After the addition of led_update_brightness() this
is no longer true, as the function can change the brightness
Hi,
On 04-11-16 08:52, Jacek Anaszewski wrote:
Initially the claim about no need for lock in brightness_show()
was valid as the function was just returning unchanged
LED brightness. After the addition of led_update_brightness() this
is no longer true, as the function can change the brightness
The cache-uniphier is a full-custom outer-cache for
Socionext UniPhier SoC family.
This series includes SoC-specific cleanups and
a trivial coding style fix suggested by Documentation/CodingStyle.
Masahiro Yamada (3):
ARM: cache-uniphier: call kzalloc() after DT property parsing
ARM:
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Just in case, split it up into three labels because the CodingStyle
says "one err bugs" is a common type of bug (although, I do not
believe the current code includes such a bug).
During
The cache-uniphier is a full-custom outer-cache for
Socionext UniPhier SoC family.
This series includes SoC-specific cleanups and
a trivial coding style fix suggested by Documentation/CodingStyle.
Masahiro Yamada (3):
ARM: cache-uniphier: call kzalloc() after DT property parsing
ARM:
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Just in case, split it up into three labels because the CodingStyle
says "one err bugs" is a common type of bug (although, I do not
believe the current code includes such a bug).
During
Now, the active way setup function is called with a fixed value zero
for the second argument only when enabling the outer-cache.
The code can be simpler.
Signed-off-by: Masahiro Yamada
---
arch/arm/mm/cache-uniphier.c | 20 +++-
1 file changed, 7
Now, the active way setup function is called with a fixed value zero
for the second argument only when enabling the outer-cache.
The code can be simpler.
Signed-off-by: Masahiro Yamada
---
arch/arm/mm/cache-uniphier.c | 20 +++-
1 file changed, 7 insertions(+), 13 deletions(-)
Allocate memory after DT property parsing that has more possibility
of failure. This will decrease the number of bail-out points for
kfree().
Signed-off-by: Masahiro Yamada
---
arch/arm/mm/cache-uniphier.c | 34 --
1 file changed,
Allocate memory after DT property parsing that has more possibility
of failure. This will decrease the number of bail-out points for
kfree().
Signed-off-by: Masahiro Yamada
---
arch/arm/mm/cache-uniphier.c | 34 --
1 file changed, 16 insertions(+), 18
New Cadence GEM hardware support Large Segment Offload (LSO):
TCP segmentation offload (TSO) as well as UDP fragmentation
offload (UFO). Support for those features was added to the driver.
Signed-off-by: Rafal Ozieblo
---
Changed in v2:
macb_lso_check_compatibility() changed
New Cadence GEM hardware support Large Segment Offload (LSO):
TCP segmentation offload (TSO) as well as UDP fragmentation
offload (UFO). Support for those features was added to the driver.
Signed-off-by: Rafal Ozieblo
---
Changed in v2:
macb_lso_check_compatibility() changed to
On 2016-11-03 21:08, Al Viro wrote:
On Thu, Nov 03, 2016 at 04:50:55PM -0600, Ben Hutchings wrote:
On Wed, 2016-11-02 at 18:20 +0100, Sebastian Andrzej Siewior wrote:
Debian started to build the gcc with -fPIE by default so the kernel
build ends before it starts properly with:
On 2016-11-03 21:08, Al Viro wrote:
On Thu, Nov 03, 2016 at 04:50:55PM -0600, Ben Hutchings wrote:
On Wed, 2016-11-02 at 18:20 +0100, Sebastian Andrzej Siewior wrote:
Debian started to build the gcc with -fPIE by default so the kernel
build ends before it starts properly with:
The function populates the list of reserved regions with the
PCI host bridge windows and the MSI IOVA range.
At the moment an arbitray MSI IOVA window is set at 0x800
of size 1MB.
Signed-off-by: Eric Auger
---
RFC v1 -> v2: use defines for MSI IOVA base and length
The function populates the list of reserved regions with the
PCI host bridge windows and the MSI IOVA range.
At the moment an arbitray MSI IOVA window is set at 0x800
of size 1MB.
Signed-off-by: Eric Auger
---
RFC v1 -> v2: use defines for MSI IOVA base and length
---
From: Robin Murphy
IOMMU domain users such as VFIO face a similar problem to DMA API ops
with regard to mapping MSI messages in systems where the MSI write is
subject to IOMMU translation. With the relevant infrastructure now in
place for managed DMA domains, it's actually
A new callback is introduced, add_reserved_regions. This function
aims at populating the iommu domain reserved region list with
regions associated with the device. The function is called on
device attachment. The list is freed on iommu_domain_free().
Signed-off-by: Eric Auger
This patch allows the user-space to retrieve the reserved IOVA range(s),
if any. The implementation is based on capability chains, now also added
to VFIO_IOMMU_GET_INFO.
Signed-off-by: Eric Auger
---
RFC v1 -> v2:
- add resv_mutex lock/unlock
---
Implement the add_reserved_regions callback by registering
the [FEE0_h - FEF0_000h] 1MB range as a reserved region
(MSI address space).
Signed-off-by: Eric Auger
---
RFC v1 -> RFC v2:
- fix intel_iommu_add_reserved_regions name
- use IOAPIC_RANGE_START and
From: Robin Murphy
IOMMU domain users such as VFIO face a similar problem to DMA API ops
with regard to mapping MSI messages in systems where the MSI write is
subject to IOMMU translation. With the relevant infrastructure now in
place for managed DMA domains, it's actually really simple for
A new callback is introduced, add_reserved_regions. This function
aims at populating the iommu domain reserved region list with
regions associated with the device. The function is called on
device attachment. The list is freed on iommu_domain_free().
Signed-off-by: Eric Auger
---
This patch allows the user-space to retrieve the reserved IOVA range(s),
if any. The implementation is based on capability chains, now also added
to VFIO_IOMMU_GET_INFO.
Signed-off-by: Eric Auger
---
RFC v1 -> v2:
- add resv_mutex lock/unlock
---
drivers/vfio/vfio_iommu_type1.c | 70
Implement the add_reserved_regions callback by registering
the [FEE0_h - FEF0_000h] 1MB range as a reserved region
(MSI address space).
Signed-off-by: Eric Auger
---
RFC v1 -> RFC v2:
- fix intel_iommu_add_reserved_regions name
- use IOAPIC_RANGE_START and IOAPIC_RANGE_END defines
- return
Introduce a new iommu_reserved_region struct. This embodies
an IOVA reserved region that cannot be used along with the IOMMU
API. The list is protected by a dedicated mutex.
An iommu domain now owns a list of those.
Signed-off-by: Eric Auger
---
---
Introduce a new iommu_reserved_region struct. This embodies
an IOVA reserved region that cannot be used along with the IOMMU
API. The list is protected by a dedicated mutex.
An iommu domain now owns a list of those.
Signed-off-by: Eric Auger
---
---
drivers/iommu/iommu.c | 2 ++
Capability header next field is an offset relative to the start of
the INFO buffer. tmp->next is assigned the proper value but iterations
implemented in vfio_info_cap_add and vfio_info_cap_shift use next
as an offset between the headers. When coping with multiple capabilities
this leads to an
Capability header next field is an offset relative to the start of
the INFO buffer. tmp->next is assigned the proper value but iterations
implemented in vfio_info_cap_add and vfio_info_cap_shift use next
as an offset between the headers. When coping with multiple capabilities
this leads to an
Following Will & Robin's suggestions, this series attempts to propose
an alternative to [1] where the host would arbitrarily decide the
location of the IOVA MSI window and would be able to report to the
userspace the list of reserved IOVA regions that cannot be used
along with VFIO_IOMMU_MAP_DMA.
Following Will & Robin's suggestions, this series attempts to propose
an alternative to [1] where the host would arbitrarily decide the
location of the IOVA MSI window and would be able to report to the
userspace the list of reserved IOVA regions that cannot be used
along with VFIO_IOMMU_MAP_DMA.
Fix the size check within start_pfn and limit_pfn.
Signed-off-by: Eric Auger
---
the issue was observed when playing with 1 page iova domain with
higher iova reserved.
---
drivers/iommu/iova.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Fix the size check within start_pfn and limit_pfn.
Signed-off-by: Eric Auger
---
the issue was observed when playing with 1 page iova domain with
higher iova reserved.
---
drivers/iommu/iova.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iommu/iova.c
On Thu, Nov 03, 2016 at 03:01:53PM -0700, Aaron Miller wrote:
> The old 'csrowX' sysfs directories had per-csrow error counters, but the
> new 'dimmX' directories do not currently expose error counts.
>
> EDAC already keeps these counts, add them to sysfs so per-dimm counts
> are still available
On Thu, Nov 03, 2016 at 03:01:53PM -0700, Aaron Miller wrote:
> The old 'csrowX' sysfs directories had per-csrow error counters, but the
> new 'dimmX' directories do not currently expose error counts.
>
> EDAC already keeps these counts, add them to sysfs so per-dimm counts
> are still available
Hi,
On lun., oct. 31 2016, Gregory CLEMENT
wrote:
> Hello,
>
> This the second version of the series adding support for the SDHCI
> Xenon controller. It can be currently found on the Armada 37xx and the
> Armada 7K/8K but will be also used in more Marvell
Hi,
On lun., oct. 31 2016, Gregory CLEMENT
wrote:
> Hello,
>
> This the second version of the series adding support for the SDHCI
> Xenon controller. It can be currently found on the Armada 37xx and the
> Armada 7K/8K but will be also used in more Marvell SoC (and not only
> the mvebu ones
Hi Jaehoon,
Am Freitag, 4. November 2016, 19:21:30 CET schrieb Jaehoon Chung:
> On 11/04/2016 03:41 AM, Krzysztof Kozlowski wrote:
> > On Thu, Nov 03, 2016 at 03:21:32PM +0900, Jaehoon Chung wrote:
> >> In drivers/mmc/core/host.c, there is "max-frequency" property.
> >> It should be same
Hi Jaehoon,
Am Freitag, 4. November 2016, 19:21:30 CET schrieb Jaehoon Chung:
> On 11/04/2016 03:41 AM, Krzysztof Kozlowski wrote:
> > On Thu, Nov 03, 2016 at 03:21:32PM +0900, Jaehoon Chung wrote:
> >> In drivers/mmc/core/host.c, there is "max-frequency" property.
> >> It should be same
Am Freitag, 4. November 2016, 18:54:35 CET schrieb Andy Yan:
> Hi Heiko:
>
> On 2016年11月04日 18:03, Heiko Stuebner wrote:
> > Am Donnerstag, 3. November 2016, 20:43:59 CET schrieb Andy Yan:
> >> RK1108EVB is designed by Rockchip for CVR field.
> >> This patch add basic support for it, which can
Am Freitag, 4. November 2016, 18:54:35 CET schrieb Andy Yan:
> Hi Heiko:
>
> On 2016年11月04日 18:03, Heiko Stuebner wrote:
> > Am Donnerstag, 3. November 2016, 20:43:59 CET schrieb Andy Yan:
> >> RK1108EVB is designed by Rockchip for CVR field.
> >> This patch add basic support for it, which can
On Fri, 4 Nov 2016 12:55:36 +0200
"Michael S. Tsirkin" wrote:
> Virtio 1.0 spec says VIRTIO_F_ANY_LAYOUT and VIRTIO_NET_F_GSO are
> legacy-only feature bits. Do not negotiate them in virtio 1 mode. Note
> this is a spec violation so we need to backport it to stable/downstream
>
On Fri, 4 Nov 2016 12:55:36 +0200
"Michael S. Tsirkin" wrote:
> Virtio 1.0 spec says VIRTIO_F_ANY_LAYOUT and VIRTIO_NET_F_GSO are
> legacy-only feature bits. Do not negotiate them in virtio 1 mode. Note
> this is a spec violation so we need to backport it to stable/downstream
> kernels.
>
>
On 11/03/2016 06:50 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Thursday 03 November 2016 10:56 PM, Alexandre Bailon wrote:
>> On 11/03/2016 05:34 PM, Kishon Vijay Abraham I wrote:
>>> Hi,
>>>
>>> On Thursday 03 November 2016 08:56 PM, Alexandre Bailon wrote:
The USB PHY is able to
On 11/03/2016 06:50 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Thursday 03 November 2016 10:56 PM, Alexandre Bailon wrote:
>> On 11/03/2016 05:34 PM, Kishon Vijay Abraham I wrote:
>>> Hi,
>>>
>>> On Thursday 03 November 2016 08:56 PM, Alexandre Bailon wrote:
The USB PHY is able to
Hi,
On 04.11.2016 07:53, Joe Perches wrote:
CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to shortest
#446: FILE: drivers/net/ethernet/ethoc.c:446:
+ int size = bd.stat >> 16;
+ struct sk_buff *skb;
should not this case be valid?
Hi,
On 04.11.2016 07:53, Joe Perches wrote:
CHECK:REVERSE_XMAS_TREE: Prefer ordering declarations longest to shortest
#446: FILE: drivers/net/ethernet/ethoc.c:446:
+ int size = bd.stat >> 16;
+ struct sk_buff *skb;
should not this case be valid?
Please CC me on your future submissions, thanks.
On Fri, Nov 04, 2016 at 03:07:19PM +0800, He Chen wrote:
> The spec can be found in Intel Software Developer Manual or in
> Instruction Set Extensions Programming Reference.
This commit message is completely useless. Write commit messages in
the
Please CC me on your future submissions, thanks.
On Fri, Nov 04, 2016 at 03:07:19PM +0800, He Chen wrote:
> The spec can be found in Intel Software Developer Manual or in
> Instruction Set Extensions Programming Reference.
This commit message is completely useless. Write commit messages in
the
Virtio 1.0 spec says VIRTIO_F_ANY_LAYOUT and VIRTIO_NET_F_GSO are
legacy-only feature bits. Do not negotiate them in virtio 1 mode. Note
this is a spec violation so we need to backport it to stable/downstream
kernels.
Cc: sta...@vger.kernel.org
Signed-off-by: Michael S. Tsirkin
Virtio 1.0 spec says VIRTIO_F_ANY_LAYOUT and VIRTIO_NET_F_GSO are
legacy-only feature bits. Do not negotiate them in virtio 1 mode. Note
this is a spec violation so we need to backport it to stable/downstream
kernels.
Cc: sta...@vger.kernel.org
Signed-off-by: Michael S. Tsirkin
---
On 10/31/2016 05:56 PM, Neil Armstrong wrote:
> The Amlogic Meson GXL SoCs have an internal RMII PHY that is muxed with the
> external RGMII pins.
>
> In order to support switching between the two PHYs links, extended registers
> size for mdio-mux-mmioreg must be added.
>
> Finally, the internal
On 10/31/2016 05:56 PM, Neil Armstrong wrote:
> The Amlogic Meson GXL SoCs have an internal RMII PHY that is muxed with the
> external RGMII pins.
>
> In order to support switching between the two PHYs links, extended registers
> size for mdio-mux-mmioreg must be added.
>
> Finally, the internal
Hi Heiko:
On 2016年11月04日 18:03, Heiko Stuebner wrote:
Am Donnerstag, 3. November 2016, 20:43:59 CET schrieb Andy Yan:
RK1108EVB is designed by Rockchip for CVR field.
This patch add basic support for it, which can boot with
initramfs into shell.
Signed-off-by: Andy Yan
Hi Heiko:
On 2016年11月04日 18:03, Heiko Stuebner wrote:
Am Donnerstag, 3. November 2016, 20:43:59 CET schrieb Andy Yan:
RK1108EVB is designed by Rockchip for CVR field.
This patch add basic support for it, which can boot with
initramfs into shell.
Signed-off-by: Andy Yan
looks good, the only
Hi Stephen,
Krzysztof has left Samsung, but we would like to continue this task, because
the ABBA dead-locks related to global prepare lock becomes more and more
problematic for us to workaround.
On 2016-09-09 02:24, Stephen Boyd wrote:
On 08/16, Krzysztof Kozlowski wrote:
RFC, please, do not
Hi Stephen,
Krzysztof has left Samsung, but we would like to continue this task, because
the ABBA dead-locks related to global prepare lock becomes more and more
problematic for us to workaround.
On 2016-09-09 02:24, Stephen Boyd wrote:
On 08/16, Krzysztof Kozlowski wrote:
RFC, please, do not
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Signed-off-by: Masahiro Yamada
---
drivers/i2c/busses/i2c-uniphier-f.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Signed-off-by: Masahiro Yamada
---
drivers/i2c/busses/i2c-uniphier.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Signed-off-by: Masahiro Yamada
---
drivers/i2c/busses/i2c-uniphier-f.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-uniphier-f.c
Documentation/CodingStyle recommends to use label names which say
what the goto does or why the goto exists.
Signed-off-by: Masahiro Yamada
---
drivers/i2c/busses/i2c-uniphier.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/i2c/busses/i2c-uniphier.c
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/fsmc_nand.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/plat_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/plat_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/plat_nand.c
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/fsmc_nand.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/mtd/nand/fsmc_nand.c
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/mxc_nand.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/mxc_nand.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/mtd/nand/mxc_nand.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/ams-delta.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/omap2.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/orion_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/ams-delta.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/ams-delta.c
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/omap2.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/mtd/nand/omap2.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/orion_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/orion_nand.c
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/atmel_nand.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff
The nand_scan_ident/tail() returns an appropriate error value when
it fails. Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/atmel_nand.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/pasemi_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/pasemi_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/pasemi_nand.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/gpio.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/gpio.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/gpio.c b/drivers/mtd/nand/gpio.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/cmx270_nand.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/cmx270_nand.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/nand/cmx270_nand.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/cs553x_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENODEV.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/tmio_nand.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/cs553x_nand.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/cs553x_nand.c
The nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENODEV.
Signed-off-by: Masahiro Yamada
---
drivers/mtd/nand/tmio_nand.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/tmio_nand.c
nand_scan(), nand_scan_ident(), nand_scan_tail() return
an appropriate negative value on error.
Most of drivers return the value from them on error,
but some of them return the fixed error code -ENXIO
(and a few return -ENODEV).
This series make those drivers return more precise error code.
For this driver, there is nothing between nand_scan_ident() and
nand_scan_tail(). They can be merged into nand_scan().
Also, nand_scan() returns an appropriate error value when it fails.
Use it instead of the fixed error code -ENXIO.
Signed-off-by: Masahiro Yamada
701 - 800 of 996 matches
Mail list logo