On Thu, Aug 11, 2016 at 10:45:00AM +0200, Oliver Hartkopp wrote:
> On 08/11/2016 09:14 AM, Andreas Werner wrote:
> >On Wed, Aug 10, 2016 at 10:28:45PM +0200, Oliver Hartkopp wrote:
>
> >>Just check 'git grep IFF_ECHO'. Even grcan.c and janz-ican3.c have IFF_ECHO
&g
On Thu, Aug 11, 2016 at 10:45:00AM +0200, Oliver Hartkopp wrote:
> On 08/11/2016 09:14 AM, Andreas Werner wrote:
> >On Wed, Aug 10, 2016 at 10:28:45PM +0200, Oliver Hartkopp wrote:
>
> >>Just check 'git grep IFF_ECHO'. Even grcan.c and janz-ican3.c have IFF_ECHO
&g
On Wed, Aug 10, 2016 at 10:28:45PM +0200, Oliver Hartkopp wrote:
> Hi Andreas,
>
> On 08/09/2016 08:10 AM, Andreas Werner wrote:
> >On Mon, Aug 08, 2016 at 04:35:34PM +0200, Wolfgang Grandegger wrote:
>
> >>>>>>You specify here one echo_skb but
On Wed, Aug 10, 2016 at 10:28:45PM +0200, Oliver Hartkopp wrote:
> Hi Andreas,
>
> On 08/09/2016 08:10 AM, Andreas Werner wrote:
> >On Mon, Aug 08, 2016 at 04:35:34PM +0200, Wolfgang Grandegger wrote:
>
> >>>>>>You specify here one echo_skb but
On Mon, Aug 08, 2016 at 08:23:55PM -0700, Benjamin Poirier wrote:
> On 2016/08/08 09:26, Andreas Werner wrote:
> [...]
> > > > +
> > > > + if (cf->can_dlc > 0)
> > > > + data[0] = be32_to_cpup((__be32 *)
On Mon, Aug 08, 2016 at 08:23:55PM -0700, Benjamin Poirier wrote:
> On 2016/08/08 09:26, Andreas Werner wrote:
> [...]
> > > > +
> > > > + if (cf->can_dlc > 0)
> > > > + data[0] = be32_to_cpup((__be32 *)
On Mon, Aug 08, 2016 at 04:35:34PM +0200, Wolfgang Grandegger wrote:
> Am 08.08.2016 um 16:05 schrieb Andreas Werner:
> >On Mon, Aug 08, 2016 at 02:28:39PM +0200, Wolfgang Grandegger wrote:
> >>Hello,
> >>
> >>Am 08.08.2016 um 13:39 schrieb Andreas Werner:
>
On Mon, Aug 08, 2016 at 04:35:34PM +0200, Wolfgang Grandegger wrote:
> Am 08.08.2016 um 16:05 schrieb Andreas Werner:
> >On Mon, Aug 08, 2016 at 02:28:39PM +0200, Wolfgang Grandegger wrote:
> >>Hello,
> >>
> >>Am 08.08.2016 um 13:39 schrieb Andreas Werner:
>
On Mon, Aug 08, 2016 at 03:06:33PM +0200, Kurt Van Dijck wrote:
>
> --- Original message ---
> > Date: Mon, 8 Aug 2016 14:28:39 +0200
> > From: Wolfgang Grandegger
> >
> [...]
> > >>>+
> > >>>+if (!(cf->can_id & CAN_RTR_FLAG)) {
> > >>>+
On Mon, Aug 08, 2016 at 03:06:33PM +0200, Kurt Van Dijck wrote:
>
> --- Original message ---
> > Date: Mon, 8 Aug 2016 14:28:39 +0200
> > From: Wolfgang Grandegger
> >
> [...]
> > >>>+
> > >>>+if (!(cf->can_id & CAN_RTR_FLAG)) {
> > >>>+writel(data[0],
On Mon, Aug 08, 2016 at 02:28:39PM +0200, Wolfgang Grandegger wrote:
> Hello,
>
> Am 08.08.2016 um 13:39 schrieb Andreas Werner:
> >On Mon, Aug 08, 2016 at 11:27:25AM +0200, Wolfgang Grandegger wrote:
> >>Hello Andreas,
> >>
> >>a first quick review..
On Mon, Aug 08, 2016 at 02:28:39PM +0200, Wolfgang Grandegger wrote:
> Hello,
>
> Am 08.08.2016 um 13:39 schrieb Andreas Werner:
> >On Mon, Aug 08, 2016 at 11:27:25AM +0200, Wolfgang Grandegger wrote:
> >>Hello Andreas,
> >>
> >>a first quick review..
On Mon, Aug 08, 2016 at 11:27:25AM +0200, Wolfgang Grandegger wrote:
> Hello Andreas,
>
> a first quick review
>
> Am 26.07.2016 um 11:16 schrieb Andreas Werner:
> >This CAN Controller is found on MEN Chameleon FPGAs.
> >
> >The driver/device
On Mon, Aug 08, 2016 at 11:27:25AM +0200, Wolfgang Grandegger wrote:
> Hello Andreas,
>
> a first quick review
>
> Am 26.07.2016 um 11:16 schrieb Andreas Werner:
> >This CAN Controller is found on MEN Chameleon FPGAs.
> >
> >The driver/device
On Sun, Aug 07, 2016 at 08:58:14PM -0700, Benjamin Poirier wrote:
> On 2016/07/26 11:16, Andreas Werner wrote:
> [...]
> > +
> > + /* Lock for CTL_BTR register access.
> > +* This register combines bittiming bits
> > +* and the operation mode bits.
> >
On Sun, Aug 07, 2016 at 08:58:14PM -0700, Benjamin Poirier wrote:
> On 2016/07/26 11:16, Andreas Werner wrote:
> [...]
> > +
> > + /* Lock for CTL_BTR register access.
> > +* This register combines bittiming bits
> > +* and the operation mode bits.
> >
parameters to configure the
buffer level interrupt for RX/TX as well as a RX timeout
interrupt.
With this configuration options, the driver/device
provides flexibility for different types of usecases.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/net/can/Kconfig
parameters to configure the
buffer level interrupt for RX/TX as well as a RX timeout
interrupt.
With this configuration options, the driver/device
provides flexibility for different types of usecases.
Signed-off-by: Andreas Werner
---
drivers/net/can/Kconfig| 10 +
drivers/net/can/Makefile
rovides flexibility for different types of usecases.
>
> Signed-off-by: Andreas Werner <andreas.wer...@men.de>
> ---
> drivers/net/can/Kconfig| 10 +
> drivers/net/can/Makefile | 1 +
> drivers/net/can/men_z192_can.c | 990
> +
rovides flexibility for different types of usecases.
>
> Signed-off-by: Andreas Werner
> ---
> drivers/net/can/Kconfig| 10 +
> drivers/net/can/Makefile | 1 +
> drivers/net/can/men_z192_can.c | 990
> +
> 3 files ch
-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mcb/mcb-internal.h | 9
drivers/mcb/mcb-parse.c| 125 -
2 files changed, 120 insertions(+), 14 deletions(-)
diff --git a/drivers/mcb/mcb-internal.h b/drivers/mcb/mcb-internal.h
index 5
-by: Andreas Werner
---
drivers/mcb/mcb-internal.h | 9
drivers/mcb/mcb-parse.c| 125 -
2 files changed, 120 insertions(+), 14 deletions(-)
diff --git a/drivers/mcb/mcb-internal.h b/drivers/mcb/mcb-internal.h
index 5254e02..d6e6933 100644
parameters to configure the
buffer level interrupt for RX/TX as well as a RX timeout
interrupt.
With this configuration options, the driver/device
provides flexibility for different types of usecases.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/net/can/Kconfig
parameters to configure the
buffer level interrupt for RX/TX as well as a RX timeout
interrupt.
With this configuration options, the driver/device
provides flexibility for different types of usecases.
Signed-off-by: Andreas Werner
---
drivers/net/can/Kconfig| 10 +
drivers/net/can/Makefile
Add support for MCB bases FPGAs connected to the LPC or
non PCI Bus.
This driver currently supports the SC24 board. The FPGA
is connected to the LPC bus and is identified using the BIOS
DMI string.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mcb/Kconfig
Add support for MCB bases FPGAs connected to the LPC or
non PCI Bus.
This driver currently supports the SC24 board. The FPGA
is connected to the LPC bus and is identified using the BIOS
DMI string.
Signed-off-by: Andreas Werner
---
drivers/mcb/Kconfig | 9 +++
drivers/mcb/Makefile | 1
The num_cells variable is only used in the dev_dbg print,
but we can directly use the ret variable which also includes the same
value.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mcb/mcb-pci.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/d
Replaced ioremap with devm_ioremap and request_mem_region with
devm_request_mem_region. This makes the code much more cleaner.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mcb/mcb-pci.c | 19 +++
1 file changed, 7 insertions(+), 12 deletions(-)
diff
The num_cells variable is only used in the dev_dbg print,
but we can directly use the ret variable which also includes the same
value.
Signed-off-by: Andreas Werner
---
drivers/mcb/mcb-pci.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/mcb/mcb-pci.c b/drivers
Replaced ioremap with devm_ioremap and request_mem_region with
devm_request_mem_region. This makes the code much more cleaner.
Signed-off-by: Andreas Werner
---
drivers/mcb/mcb-pci.c | 19 +++
1 file changed, 7 insertions(+), 12 deletions(-)
diff --git a/drivers/mcb/mcb-pci.c b
The first patch replace the ioremap an request_mem_region function
with the corresponding devm function to have a more cleaner code
and a better error handling.
The sencoder patch just delete a unused variable.
Andreas Werner (2):
mcb: Replace ioremap and request_region with the devm version
The first patch replace the ioremap an request_mem_region function
with the corresponding devm function to have a more cleaner code
and a better error handling.
The sencoder patch just delete a unused variable.
Andreas Werner (2):
mcb: Replace ioremap and request_region with the devm version
The bar number is found in reg2 within the gdd. Therefore
we need to change the assigment from reg1 to reg2 which
is the correct location.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mcb/mcb-parse.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/d
The bar number is found in reg2 within the gdd. Therefore
we need to change the assigment from reg1 to reg2 which
is the correct location.
Signed-off-by: Andreas Werner
---
drivers/mcb/mcb-parse.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mcb/mcb-parse.c b
Looks good to me. I have tested this on the MEN SC24 with
a MCB FPGA.
Reviewed-by: Andreas Werner <andreas.wer...@men.de>
Tested-by: Andreas Werner <andreas.wer...@men.de>
. I have tested this on the MEN SC24 with
a MCB FPGA.
Reviewed-by: Andreas Werner
Tested-by: Andreas Werner
ested this on the MEN SC24 AMD Board
with a MCB FPGA.
Reviewed-by: Andreas Werner <andreas.wer...@men.de>
Tested-by: Andreas Werner <andreas.wer...@men.de>
SC24 AMD Board
with a MCB FPGA.
Reviewed-by: Andreas Werner
Tested-by: Andreas Werner
com>
> CC: Andreas Werner <andreas.wer...@men.de>
> ---
Thanks for the Patch, looks good to me.
Reviewed-by: Andreas Werner <andreas.wer...@men.de>
On Tue, Apr 05, 2016 at 05:18:21PM +0530, Laxman Dewangan wrote:
> Use devm_mfd_add_devices() for mfd devices registration and get
> rid of .remove callback to remove mfd devices. This is done
> by managed device framework.
>
> Signed-off-by: Laxman Dewangan
> CC: Andreas Wer
The 16Z127 is a 32bit GPIO controller on a MCB FPGA.
Every single line can be configured as input and output.
Push pull and open drain are supported as well as setting
a debounce value for the input lines.
Signed-off-by: Andreas Werner <a...@wernerandy.de>
---
drivers/gpio/Kconfig
The 16Z127 is a 32bit GPIO controller on a MCB FPGA.
Every single line can be configured as input and output.
Push pull and open drain are supported as well as setting
a debounce value for the input lines.
Signed-off-by: Andreas Werner
---
drivers/gpio/Kconfig| 7 ++
drivers/gpio
This patch adds the MEN 16Z127 GPIO Controller driver, which
is 32bit wide and located within a MCB FPGA.
changes in v2:
- ported driver to use the generic MMIO library
- added open drain support
- added debounce support
Andreas Werner (1):
gpio: add driver for MEN
This patch adds the MEN 16Z127 GPIO Controller driver, which
is 32bit wide and located within a MCB FPGA.
changes in v2:
- ported driver to use the generic MMIO library
- added open drain support
- added debounce support
Andreas Werner (1):
gpio: add driver for MEN
Hi,
any comments for this patches?
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Hi,
any comments for this patches?
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
On Thu, Dec 17, 2015 at 06:57:51AM -0800, Guenter Roeck wrote:
> On Thu, Dec 17, 2015 at 12:48:08PM +0100, Jean Delvare wrote:
> > Hi Andreas,
> >
> > On Tue, 15 Dec 2015 16:11:24 +0100, Andreas Werner wrote:
> > > here is the register dump of the tmp461.
> &g
On Thu, Dec 17, 2015 at 06:57:51AM -0800, Guenter Roeck wrote:
> On Thu, Dec 17, 2015 at 12:48:08PM +0100, Jean Delvare wrote:
> > Hi Andreas,
> >
> > On Tue, 15 Dec 2015 16:11:24 +0100, Andreas Werner wrote:
> > > here is the register dump of the tmp461.
> &g
On Thu, Dec 17, 2015 at 12:48:08PM +0100, Jean Delvare wrote:
> Hi Andreas,
>
> On Tue, 15 Dec 2015 16:11:24 +0100, Andreas Werner wrote:
> > here is the register dump of the tmp461.
>
> Thanks.
>
> > It seemse that we really cannot detect if it is a tmp461 or a
On Thu, Dec 17, 2015 at 12:48:08PM +0100, Jean Delvare wrote:
> Hi Andreas,
>
> On Tue, 15 Dec 2015 16:11:24 +0100, Andreas Werner wrote:
> > here is the register dump of the tmp461.
>
> Thanks.
>
> > It seemse that we really cannot detect if it is a tmp461 or a
Signed-off-by: Andreas Werner
---
.../ABI/testing/sysfs-bus-i2c-devices-menf21bmc| 26 ++
1 file changed, 26 insertions(+)
diff --git a/Documentation/ABI/testing/sysfs-bus-i2c-devices-menf21bmc
b/Documentation/ABI/testing/sysfs-bus-i2c-devices-menf21bmc
index 28d1fa2
This patch adds additional sysfs entries to provide status
information to the userland.
The following informations are now provided:
- Get operation hours counter
- Get board slot address
- Get powercycle counter
- Set/get Hw Variant
Signed-off-by: Andreas Werner
to mechanism and add a sysfs interface to manually
exiting the production e.g. during the EOL test of the board.
Signed-off-by: Andreas Werner
---
drivers/mfd/menf21bmc.c | 65 +
1 file changed, 44 insertions(+), 21 deletions(-)
diff --git a/drive
This patch add the description of the "mode" sysfs interface
for the menf21bmc MFD driver.
Signed-off-by: Andreas Werner
---
Documentation/ABI/testing/sysfs-bus-i2c-devices-menf21bmc | 14 ++
1 file changed, 14 insertions(+)
create mode 100644 Documentation/ABI/testing
This patch set introduces new sysfs entries for BMC status information
as well as the ABI documention.
The mfd core driver now does not exit the production mode automatically.
The new sysfs entry allows the userland to exit the production mode
if required.
Andreas Werner (4):
mfd/menf21bmc
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
.../ABI/testing/sysfs-bus-i2c-devices-menf21bmc| 26 ++
1 file changed, 26 insertions(+)
diff --git a/Documentation/ABI/testing/sysfs-bus-i2c-devices-menf21bmc
b/Documentation/ABI/testing/sysfs-bus-i2c-d
This patch add the description of the "mode" sysfs interface
for the menf21bmc MFD driver.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
Documentation/ABI/testing/sysfs-bus-i2c-devices-menf21bmc | 14 ++
1 file changed, 14 insertions(+)
create mode 1006
to mechanism and add a sysfs interface to manually
exiting the production e.g. during the EOL test of the board.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/mfd/menf21bmc.c | 65 +
1 file changed, 44 insertions(+), 2
This patch adds additional sysfs entries to provide status
information to the userland.
The following informations are now provided:
- Get operation hours counter
- Get board slot address
- Get powercycle counter
- Set/get Hw Variant
Signed-off-by: Andreas Werner
This patch set introduces new sysfs entries for BMC status information
as well as the ABI documention.
The mfd core driver now does not exit the production mode automatically.
The new sysfs entry allows the userland to exit the production mode
if required.
Andreas Werner (4):
mfd/menf21bmc
On Sat, Dec 12, 2015 at 11:08:42AM +0100, Jean Delvare wrote:
> Hallo Andreas,
>
> On Thu, 10 Dec 2015 18:12:31 +0100, Andreas Werner wrote:
> > thanks for the register dump :-)
>
> Can you please share the register dump of your TMP461 with us?
>
> Thanks,
> --
&g
On Sat, Dec 12, 2015 at 11:08:42AM +0100, Jean Delvare wrote:
> Hallo Andreas,
>
> On Thu, 10 Dec 2015 18:12:31 +0100, Andreas Werner wrote:
> > thanks for the register dump :-)
>
> Can you please share the register dump of your TMP461 with us?
>
> Thanks,
> --
&g
>Hallo Andreas,
>
> On Thu, 10 Dec 2015 18:12:31 +0100, Andreas Werner wrote:
>> thanks for the register dump
>
> Can you please share the register dump of your TMP461 with us?
>
Hi,
yes for sure. Currently i have just a evaluation board my CPU without
this sensor, but
>Hallo Andreas,
>
> On Thu, 10 Dec 2015 18:12:31 +0100, Andreas Werner wrote:
>> thanks for the register dump
>
> Can you please share the register dump of your TMP461 with us?
>
Hi,
yes for sure. Currently i have just a evaluation board my CPU without
this sensor, but
...
>On Thu, Dec 10, 2015 at 09:49:43AM -0800, Guenter Roeck wrote:
> >
> >I have a DT based board yes, but i have also planed to submit my changes
> >and also wanted to implement the autodetection of the chip.
> >
> >I have also planned (or need) to implement the "n-Factor Correction" for the
>
...
> On Thu, Dec 10, 2015 at 09:43:53AM -0800, Guenter Roeck wrote:
> >
> >There is one difference. The temperature ranges differ:
> >
> >TMP451:
> >Standard Mode: 0 to +127
> >Extended Mode: -64 to +191
> >
> >TMP461:
> >Standard Mode: -40 to +127
> >Extended Mode: -64 to +191
> >
> >Therefore
On Thu, Dec 10, 2015 at 08:41:57AM -0800, Guenter Roeck wrote:
> On 12/10/2015 04:08 AM, Andreas Werner wrote:
> >Hi,
> >i have a temperature sensor device named "TI TMP461" which is quite the
> >same than the tmp451 which is already included in the lm90 dr
Hi,
i have a temperature sensor device named "TI TMP461" which is quite the
same than the tmp451 which is already included in the lm90 driver.
I just want to add the tmp461 to the driver but currently i have no way
to differ between the tmp461 and tmp451.
The main different is that the tmp461
Hi,
i have a temperature sensor device named "TI TMP461" which is quite the
same than the tmp451 which is already included in the lm90 driver.
I just want to add the tmp461 to the driver but currently i have no way
to differ between the tmp461 and tmp451.
The main different is that the tmp461
...
> On Thu, Dec 10, 2015 at 09:43:53AM -0800, Guenter Roeck wrote:
> >
> >There is one difference. The temperature ranges differ:
> >
> >TMP451:
> >Standard Mode: 0 to +127
> >Extended Mode: -64 to +191
> >
> >TMP461:
> >Standard Mode: -40 to +127
> >Extended Mode: -64 to +191
> >
> >Therefore
...
>On Thu, Dec 10, 2015 at 09:49:43AM -0800, Guenter Roeck wrote:
> >
> >I have a DT based board yes, but i have also planed to submit my changes
> >and also wanted to implement the autodetection of the chip.
> >
> >I have also planned (or need) to implement the "n-Factor Correction" for the
>
On Thu, Dec 10, 2015 at 08:41:57AM -0800, Guenter Roeck wrote:
> On 12/10/2015 04:08 AM, Andreas Werner wrote:
> >Hi,
> >i have a temperature sensor device named "TI TMP461" which is quite the
> >same than the tmp451 which is already included in the lm90 dr
On Mon, Dec 07, 2015 at 10:26:30AM -0500, Tejun Heo wrote:
> On Fri, Dec 04, 2015 at 06:12:24PM +0100, Andreas Werner wrote:
> > This patchset add a new ata port flag ATA_FLAG_NO_LOG_PAGE to be able
> > to blacklist ports/controller which e.g. locks up on a log page read.
>
On Mon, Dec 07, 2015 at 10:26:30AM -0500, Tejun Heo wrote:
> On Fri, Dec 04, 2015 at 06:12:24PM +0100, Andreas Werner wrote:
> > This patchset add a new ata port flag ATA_FLAG_NO_LOG_PAGE to be able
> > to blacklist ports/controller which e.g. locks up on a log page read.
>
Add CS polarity flag to be able to set the CS polarity
via the DT property spi-cs-high.
Signed-off-by: Andreas Werner
---
drivers/spi/spi-fsl-espi.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-fsl-espi.c b/drivers/spi/spi-fsl-espi.c
index c27124a
Add CS polarity flag to be able to set the CS polarity
via the DT property spi-cs-high.
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/spi/spi-fsl-espi.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/spi/spi-fsl-espi.c b/drivers/spi/s
: exception Emask 0x50 SAct 0x0 SErr 0x800 action 0x6 frozen
t4
[ 13.359281] ata1: SError: { HostInt }
[ 13.361644] ata1: hard resetting link
Signed-off-by: Andreas Werner
---
drivers/ata/sata_fsl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/ata/sata_fsl.c b
Some controller lockup on a ata_read_log_page.
Add new ata port flag ATA_FLAG_NO_LOG_PAGE which can used
to blacklist a controller.
If this flag is set, any attempt to read a log page returns an error
without actually issuing the command.
Signed-off-by: Andreas Werner
---
drivers/ata/libata
test robot
Andreas Werner (2):
libata-eh.c: Introduce new ata port flag for controller which lockup
on read log page
ata/sata_fsl.c: add ATA_FLAG_NO_LOG_PAGE to blacklist the controller
for log page reads
drivers/ata/libata-eh.c | 9 +
drivers/ata/sata_fsl.c | 3 ++-
include
Hi,
argh sorry send the wrong patches from a broken tree.
Will send a v2
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read
: exception Emask 0x50 SAct 0x0 SErr 0x800 action 0x6 frozen
t4
[ 13.359281] ata1: SError: { HostInt }
[ 13.361644] ata1: hard resetting link
Signed-off-by: Andreas Werner
---
drivers/ata/sata_fsl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/ata/sata_fsl.c b
Some controller lockup on a ata_read_log_page.
Add new ata port flag ATA_FLAG_NO_LOG_PAGE which can used
to blacklist a controller.
If this flag is set, any attempt to read a log page returns an error
without actually issuing the command.
Signed-off-by: Andreas Werner
---
drivers/ata/libata
.
The device failed during initialisation if the SATA device includes the
devslp feature.
With this patchset, we blacklist the fsl sata controller and return
a error on any attempt to read a log page. This allows us to access
the mSATA.
Andreas Werner (2):
libata-eh.c: Introduce new ata port flag
: exception Emask 0x50 SAct 0x0 SErr 0x800 action 0x6 frozen
t4
[ 13.359281] ata1: SError: { HostInt }
[ 13.361644] ata1: hard resetting link
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/ata/sata_fsl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff
Some controller lockup on a ata_read_log_page.
Add new ata port flag ATA_FLAG_NO_LOG_PAGE which can used
to blacklist a controller.
If this flag is set, any attempt to read a log page returns an error
without actually issuing the command.
Signed-off-by: Andreas Werner <andreas.wer...@men
.
The device failed during initialisation if the SATA device includes the
devslp feature.
With this patchset, we blacklist the fsl sata controller and return
a error on any attempt to read a log page. This allows us to access
the mSATA.
Andreas Werner (2):
libata-eh.c: Introduce new ata port flag
: exception Emask 0x50 SAct 0x0 SErr 0x800 action 0x6 frozen
t4
[ 13.359281] ata1: SError: { HostInt }
[ 13.361644] ata1: hard resetting link
Signed-off-by: Andreas Werner <andreas.wer...@men.de>
---
drivers/ata/sata_fsl.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff
Some controller lockup on a ata_read_log_page.
Add new ata port flag ATA_FLAG_NO_LOG_PAGE which can used
to blacklist a controller.
If this flag is set, any attempt to read a log page returns an error
without actually issuing the command.
Signed-off-by: Andreas Werner <andreas.wer...@men
test robot
Andreas Werner (2):
libata-eh.c: Introduce new ata port flag for controller which lockup
on read log page
ata/sata_fsl.c: add ATA_FLAG_NO_LOG_PAGE to blacklist the controller
for log page reads
drivers/ata/libata-eh.c | 9 +
drivers/ata/sata_fsl.c | 3 ++-
include
Hi,
argh sorry send the wrong patches from a broken tree.
Will send a v2
Regards
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read
On Thu, Dec 03, 2015 at 10:15:49AM -0500, Tejun Heo wrote:
> Hello,
>
> On Thu, Dec 03, 2015 at 10:33:55AM +0100, Andreas Werner wrote:
> > All the other "flag" fields in the structs are used and/or reserved
> > and it seems to be no good place for such flags.
On Wed, Dec 02, 2015 at 11:47:53AM -0500, Tejun Heo wrote:
> Hello, Andreas.
>
> On Wed, Dec 02, 2015 at 10:33:10AM +0100, Andreas Werner wrote:
> > Blacklisting the controller would be a solution yes, but I just
> > wanna wait the answer from the FAE to be sure we real
On Wed, Dec 02, 2015 at 11:47:53AM -0500, Tejun Heo wrote:
> Hello, Andreas.
>
> On Wed, Dec 02, 2015 at 10:33:10AM +0100, Andreas Werner wrote:
> > Blacklisting the controller would be a solution yes, but I just
> > wanna wait the answer from the FAE to be sure we real
On Thu, Dec 03, 2015 at 10:15:49AM -0500, Tejun Heo wrote:
> Hello,
>
> On Thu, Dec 03, 2015 at 10:33:55AM +0100, Andreas Werner wrote:
> > All the other "flag" fields in the structs are used and/or reserved
> > and it seems to be no good place for such flags.
On Tue, Dec 01, 2015 at 02:04:10PM -0500, Tejun Heo wrote:
> On Tue, Dec 01, 2015 at 11:17:15AM -0500, Tejun Heo wrote:
> > So, I suppose this is a fallout from 9faa643855df ("libata: use
> > READ_LOG_DMA_EXT"). The description just says "we should try use it"
> > but is there any benefit from
On Tue, Dec 01, 2015 at 02:04:10PM -0500, Tejun Heo wrote:
> On Tue, Dec 01, 2015 at 11:17:15AM -0500, Tejun Heo wrote:
> > So, I suppose this is a fallout from 9faa643855df ("libata: use
> > READ_LOG_DMA_EXT"). The description just says "we should try use it"
> > but is there any benefit from
Hi,
first sorry for the bad email format, but the IT decided to add my Email
signature as HTML
on the Exchange Server, this is why lkml rejected the email.
Therefore I will switch to my private address.
I do not think that the commit you mention is the root cause for this problem,
because it is
Hi,
first sorry for the bad email format, but the IT decided to add my Email
signature as HTML
on the Exchange Server, this is why lkml rejected the email.
Therefore I will switch to my private address.
I do not think that the commit you mention is the root cause for this problem,
because it is
1 - 100 of 322 matches
Mail list logo