Driver logic is not being modified
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_mp_register.h |7 +-
drivers/staging/sb105x/sb_pci_mp.c | 390 ++-
drivers/staging/sb105x/sb_pci_mp.h | 165 ++---
drivers/staging/sb105x
of debug information, commented-out code, etc.
I added a TODO file in the last commit, to gather some of the work to be done.
Thanks,
Sam
Samuel Iglesias Gonsalvez (9):
staging/sb105x: coding style fixes
staging/sb105x: rename _INLINE_ to inline
staging/sb105x: delete debug printks
No change in the logic of the driver
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 974 +---
1 file changed, 449 insertions(+), 525 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 19 +--
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 7196412..9e30a0c 100644
--- a/drivers/staging
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 9e30a0c..cb27fb6 100644
Now the build finishes without any errors
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/Kconfig |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sb105x/Kconfig b/drivers/staging/sb105x/Kconfig
index 245e784..58cc52e 100644
No need to have a definition of inline. It doesn't harm anything if explicitly
specify
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 24
drivers/staging/sb105x/sb_pci_mp.h |9 -
2 files changed, 12 insertions(+), 21
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/TODO |6 ++
1 file changed, 6 insertions(+)
create mode 100644 drivers/staging/sb105x/TODO
diff --git a/drivers/staging/sb105x/TODO b/drivers/staging/sb105x/TODO
new file mode 100644
index 000..518346f
--- /dev
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 20
1 file changed, 20 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 4c88bfc..7196412 100644
--- a/drivers/staging/sb105x
Use struct tty_port wherever is needed, fixing compilation errors.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/sb105x/sb_pci_mp.c | 18 +++---
drivers/staging/sb105x/sb_ser_core.h |1 +
2 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers
Use struct tty_port wherever is needed, fixing compilation errors.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 18 +++---
drivers/staging/sb105x/sb_ser_core.h |1 +
2 files changed, 12 insertions(+), 7 deletions
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 20
1 file changed, 20 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 4c88bfc..7196412 100644
--- a/drivers
No need to have a definition of inline. It doesn't harm anything if explicitly
specify
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 24
drivers/staging/sb105x/sb_pci_mp.h |9 -
2 files changed, 12
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/TODO |6 ++
1 file changed, 6 insertions(+)
create mode 100644 drivers/staging/sb105x/TODO
diff --git a/drivers/staging/sb105x/TODO b/drivers/staging/sb105x/TODO
new file mode 100644
index 000
Now the build finishes without any errors
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/Kconfig |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sb105x/Kconfig b/drivers/staging/sb105x/Kconfig
index 245e784..58cc52e
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 19 +--
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 7196412..9e30a0c 100644
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers/staging/sb105x/sb_pci_mp.c
index 9e30a0c
No change in the logic of the driver
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_pci_mp.c | 974 +---
1 file changed, 449 insertions(+), 525 deletions(-)
diff --git a/drivers/staging/sb105x/sb_pci_mp.c
b/drivers
Driver logic is not being modified
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/sb105x/sb_mp_register.h |7 +-
drivers/staging/sb105x/sb_pci_mp.c | 390 ++-
drivers/staging/sb105x/sb_pci_mp.h | 165
of debug information, commented-out code, etc.
I added a TODO file in the last commit, to gather some of the work to be done.
Thanks,
Sam
Samuel Iglesias Gonsalvez (9):
staging/sb105x: coding style fixes
staging/sb105x: rename _INLINE_ to inline
staging/sb105x: delete debug printks
documentation says.
ipack_device_del() is added to remove the device.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/carriers/tpci200.c | 14 +-
drivers/ipack/ipack.c| 24 +--
include/linux/ipack.h| 39
Prepare everything for later use.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/ipack.c | 12
include/linux/ipack.h |3 +++
2 files changed, 15 insertions(+)
diff --git a/drivers/ipack/ipack.c b/drivers/ipack/ipack.c
index 599d4ff..bdac7f6 100644
--- a/drivers
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/ipack.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/ipack/ipack.c b/drivers/ipack/ipack.c
index 7ec6b20..599d4ff 100644
--- a/drivers/ipack/ipack.c
+++ b/drivers/ipack/ipack.c
@@ -24,6 +24,7 @@ static void
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/ipack.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/ipack/ipack.c b/drivers/ipack/ipack.c
index 7ec6b20..599d4ff 100644
--- a/drivers/ipack/ipack.c
+++ b/drivers/ipack/ipack.c
@@ -24,6 +24,7
documentation says.
ipack_device_del() is added to remove the device.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/carriers/tpci200.c | 14 +-
drivers/ipack/ipack.c| 24 +--
include/linux/ipack.h| 39
Prepare everything for later use.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/ipack.c | 12
include/linux/ipack.h |3 +++
2 files changed, 15 insertions(+)
diff --git a/drivers/ipack/ipack.c b/drivers/ipack/ipack.c
index 599d4ff..bdac7f6
put_device() must be called after device_register() fails,
since device_register() always initializes the refcount
on the device structure to one.
dev->id is free'd inside of ipack_device_release function.
So, it's not needed to do it here.
Signed-off-by: Samuel Iglesias Gonsalvez
---
driv
put_device() must be called after device_register() fails,
since device_register() always initializes the refcount
on the device structure to one.
dev-id is free'd inside of ipack_device_release function.
So, it's not needed to do it here.
Signed-off-by: Samuel Iglesias Gonsalvez sigles
There was a bug in the code when managing a GE IP-OCTAL-485 board. The RX would
be enabled but we have a wrong state in the rx_enable flag.
Then, if the user changes the terminal settings, RX would not be enabled again.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices
On Thu, Jan 17, 2013 at 01:54:20PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> Today's linux-next merge of the char-misc tree got a conflict in
> drivers/ipack/devices/ipoctal.c between commit 2e124b4a390c ("TTY: switch
> tty_flip_buffer_push") from the tty tree and commits 7e5730d7c222
>
On Thu, Jan 17, 2013 at 01:54:20PM +1100, Stephen Rothwell wrote:
Hi all,
Today's linux-next merge of the char-misc tree got a conflict in
drivers/ipack/devices/ipoctal.c between commit 2e124b4a390c (TTY: switch
tty_flip_buffer_push) from the tty tree and commits 7e5730d7c222
There was a bug in the code when managing a GE IP-OCTAL-485 board. The RX would
be enabled but we have a wrong state in the rx_enable flag.
Then, if the user changes the terminal settings, RX would not be enabled again.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers
Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 21 ++---
1 file changed, 2 insertions(+), 19 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 8f699b0..241db2f 100644
--- a/drivers/ipack/devices/ipoctal.c
+++ b/drivers
Don't block the TTY client when sending characters.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 19 +++
1 file changed, 3 insertions(+), 16 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index
In case of several characters present in RxFIFO, they will have the flag of the
previous one, no matter if the actual character was received properly or not.
This patch fixes this bug.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c |3 ++-
1 file changed, 2
-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 96467bb..45920f5 100644
--- a/drivers/ipack/devices/ipoctal.c
+++ b/drivers
will be waiting for a condition that will never be true.
Signed-off-by: Alberto Garcia
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices
We protect important data such as TX buffer pointer, nb_bytes counter and status
registers of the device, from accessing several times at the same time.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers
Due to the IRQ processing, we can generate another IRQ that can come before we
end the previous one, so we lost it. E.g. when transmitting a character.
To allow the processing in SMP machines, we ack the IRQ at the beginning of the
IRQ handler.
Signed-off-by: Samuel Iglesias Gonsalvez
Added shutdown callback to disable RX and TX when there is no other client
accesing the device.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack
Thus, we don't enable RX when a termios setup has been called, as it could be
disabled previously.
As the control registers (Rx, Tx flags specifically) cannot be read from the
device, we keep this info in rx_enable.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c
The function is already called in ipoctal_irq_rx()
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 8dff66e..f2284ec 100644
--- a/drivers
From: Alberto Garcia
It is an unsigned int so that check is pointless.
Signed-off-by: Alberto Garcia
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b
Depending of the device, it disables the TX mode in different places when there
is no more data to transmit.
This patch reorder them and disable the TX mode in the same place.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c | 18 +++---
1 file changed
From: Alberto Garcia agar...@igalia.com
It is an unsigned int so that check is pointless.
Signed-off-by: Alberto Garcia agar...@igalia.com
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c |4 +---
1 file changed, 1 insertion(+), 3 deletions
Depending of the device, it disables the TX mode in different places when there
is no more data to transmit.
This patch reorder them and disable the TX mode in the same place.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 18
The function is already called in ipoctal_irq_rx()
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 8dff66e..f2284ec
Added shutdown callback to disable RX and TX when there is no other client
accesing the device.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/ipack/devices
Thus, we don't enable RX when a termios setup has been called, as it could be
disabled previously.
As the control registers (Rx, Tx flags specifically) cannot be read from the
device, we keep this info in rx_enable.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack
We protect important data such as TX buffer pointer, nb_bytes counter and status
registers of the device, from accessing several times at the same time.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c |3 +++
1 file changed, 3 insertions
Due to the IRQ processing, we can generate another IRQ that can come before we
end the previous one, so we lost it. E.g. when transmitting a character.
To allow the processing in SMP machines, we ack the IRQ at the beginning of the
IRQ handler.
Signed-off-by: Samuel Iglesias Gonsalvez sigles
-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 96467bb..45920f5 100644
--- a/drivers/ipack/devices
,
otherwise we will be waiting for a condition that will never be true.
Signed-off-by: Alberto Garcia agar...@igalia.com
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/drivers
Don't block the TTY client when sending characters.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 19 +++
1 file changed, 3 insertions(+), 16 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices
In case of several characters present in RxFIFO, they will have the flag of the
previous one, no matter if the actual character was received properly or not.
This patch fixes this bug.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c |3
Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c | 21 ++---
1 file changed, 2 insertions(+), 19 deletions(-)
diff --git a/drivers/ipack/devices/ipoctal.c b/drivers/ipack/devices/ipoctal.c
index 8f699b0..241db2f 100644
--- a/drivers/ipack/devices
Move ipack header files to include/linux/ directory where they belong.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/carriers/tpci200.h |3 +--
drivers/ipack/devices/ipoctal.c |3 ++-
drivers/ipack/ipack.c|3 ++-
{drivers/ipack
Its contents are merged into ipack.h. So this file is not needed.
Doing that, it simplifies the ipack-related driver development.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/devices/ipoctal.c |1 -
drivers/ipack/ipack.c |1 -
drivers/ipack/ipack_ids.h
Move ipack header files to include/linux/ directory where they belong to.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/ipack/carriers/tpci200.h |3 +-
drivers/ipack/devices/ipoctal.c |2 +-
drivers/ipack/ipack.c|2 +-
drivers/ipack/ipack.h| 215
Add Samuel Iglesias Gonsalvez, Jens Taprogge and Greg Kroah-Hartman as
maintainers for the Industry Pack subsystem.
Cc: Jens Taprogge
Cc: Greg Kroah-Hartman
Signed-off-by: Samuel Iglesias Gonsalvez
---
MAINTAINERS | 11 +++
1 file changed, 11 insertions(+)
diff --git a/MAINTAINERS
Adapt the copyright clause to CERN guidelines, as it has the copyright over
them.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/carriers/tpci200.c |7 ---
drivers/staging/ipack/carriers/tpci200.h |7 ---
drivers/staging/ipack/devices/ipoctal.c |7
Adapt the copyright clause to CERN guidelines, as it has the copyright over
them.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/carriers/tpci200.c |7 ---
drivers/staging/ipack/carriers/tpci200.h |7 ---
drivers/staging/ipack/devices
Add Samuel Iglesias Gonsalvez, Jens Taprogge and Greg Kroah-Hartman as
maintainers for the Industry Pack subsystem.
Cc: Jens Taprogge jens.tapro...@taprogge.org
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
MAINTAINERS | 11
Move ipack header files to include/linux/ directory where they belong to.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/carriers/tpci200.h |3 +-
drivers/ipack/devices/ipoctal.c |2 +-
drivers/ipack/ipack.c|2 +-
drivers/ipack/ipack.h
Its contents are merged into ipack.h. So this file is not needed.
Doing that, it simplifies the ipack-related driver development.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/devices/ipoctal.c |1 -
drivers/ipack/ipack.c |1 -
drivers/ipack
Move ipack header files to include/linux/ directory where they belong.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/ipack/carriers/tpci200.h |3 +--
drivers/ipack/devices/ipoctal.c |3 ++-
drivers/ipack/ipack.c|3
: In function 'tpci200_request_irq':
drivers/staging/ipack/carriers/tpci200.c:217:2: error: implicit declaration of
function 'kzalloc' [-Werror=implicit-function-declaration]
Reported-by: Fengguang Wu
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/carriers/tpci200.c |1 +
1
: In function 'tpci200_request_irq':
drivers/staging/ipack/carriers/tpci200.c:217:2: error: implicit declaration of
function 'kzalloc' [-Werror=implicit-function-declaration]
Reported-by: Fengguang Wu fengguang...@intel.com
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers
off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 19 ++-
drivers/staging/ipack/ipack.c | 30 +-
drivers/staging/ipack/ipack.h | 15 +--
3 files changed, 36 insertions(+), 28 deletions(-)
d
From: Jens Taprogge
It is replicating information contained in ipack_device->bus->bus_nr.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 18 +-
drivers/staging/ipack/devices/ipoctal.c |2 +-
d
From: Jens Taprogge
The ordering was wrong.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/devices/ipoctal.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ipack/devices/ipoctal.c
b/drivers/staging
From: Jens Taprogge
Use the regular ioremap functions and their managed counterparts instead
of the ones provided through IPack callbacks.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/devices/ipoctal.c | 70
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/Kconfig |1 -
drivers/staging/ipack/devices/Kconfig |1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/ipack/bridges/Kconfig
b/drivers/staging/ipack/bridges/Kconfig
index 97c837e..33fdc24 100644
From: Jens Taprogge
Constant renames:
- Rename TPCI200_*_GAP to TPCI200_*_INTERVAL.
- Rename TPCI200_MEM*_* to TPCI200_MEM*_SPACE_* (to match the other SPACE
constants.
Make tpci200_status_timeout and tpci200_status_error const.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias
From: Jens Taprogge
Remove the unused pointers to these spaces.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 10 --
drivers/staging/ipack/bridges/tpci200.h |2 --
2 files changed, 12 deletions(-)
diff --git
of a region)
and use that as a replacement for struct ipack_addr_space.
struct ipack_region is defined in ipack.h because it is planned to later
expose the physical addressed to the IPack Module drivers.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack
From: Jens Taprogge
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 19 +++
drivers/staging/ipack/ipack.h |1 +
2 files changed, 20 insertions(+)
diff --git a/drivers/staging/ipack/bridges
From: Jens Taprogge
Now that we have the infrastructure to use the regular function in place
and all existing users are converted, remove the map and unmap callbacks
from the ipack_bus_device->ops.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/stag
From: Jens Taprogge
This will allow us to use the regular ioremop functions.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 22 ++
drivers/staging/ipack/ipack.h |6 --
2 files changed
From: Jens Taprogge
There also is a MEM16 space. This will make it clear which one is
which, once support for MEM16 space is added.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 24
drivers
From: Jens Taprogge
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/Kconfig | 13 -
drivers/staging/ipack/carriers/Kconfig |4 ++--
drivers/staging/ipack/ipack.h | 15 ---
3 files changed, 18
In case it is not possible to remap the memory, it returns 0 and
the driver thinks that everything went fine.
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/ipack.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ipack/ipack.c b/drivers
From: Jens Taprogge
tpci200_register is converted to use this.
A later patch will build on this.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/bridges/tpci200.c | 26 +-
drivers/staging/ipack/bridges/tpci200.h
From: Jens Taprogge
This is the name used by the standard.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/Kconfig|2 +-
drivers/staging/ipack/Makefile |2 +-
drivers/staging/ipack/bridges/Kconfig|7
From: Jens Taprogge
By doing so we can remove ipoctal_check_model() and we also no longer need
to map the IPACK_ID_SPACE.
Signed-off-by: Jens Taprogge
Signed-off-by: Samuel Iglesias Gonsalvez
---
drivers/staging/ipack/devices/ipoctal.c | 46 ++-
1 file changed
From: Jens Taprogge jens.tapro...@taprogge.org
By doing so we can remove ipoctal_check_model() and we also no longer need
to map the IPACK_ID_SPACE.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack
From: Jens Taprogge jens.tapro...@taprogge.org
This is the name used by the standard.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/Kconfig|2 +-
drivers/staging/ipack/Makefile
From: Jens Taprogge jens.tapro...@taprogge.org
tpci200_register is converted to use this.
A later patch will build on this.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 26
In case it is not possible to remap the memory, it returns 0 and
the driver thinks that everything went fine.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/ipack.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging
From: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/Kconfig | 13 -
drivers/staging/ipack/carriers/Kconfig |4 ++--
drivers
From: Jens Taprogge jens.tapro...@taprogge.org
There also is a MEM16 space. This will make it clear which one is
which, once support for MEM16 space is added.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers
From: Jens Taprogge jens.tapro...@taprogge.org
This will allow us to use the regular ioremop functions.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 22
-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 200 +--
drivers/staging/ipack/bridges/tpci200.h |4 -
drivers/staging/ipack/ipack.h | 17 ---
3 files changed, 1 insertion(+), 220 deletions(-)
diff --git
From: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 19 +++
drivers/staging/ipack/ipack.h |1 +
2 files
-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 41 ++-
drivers/staging/ipack/bridges/tpci200.h | 10
drivers/staging/ipack/ipack.h |9 ++-
3 files changed, 32 insertions(+), 28 deletions(-)
diff
From: Jens Taprogge jens.tapro...@taprogge.org
Remove the unused pointers to these spaces.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 10 --
drivers/staging/ipack
jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/tpci200.c | 24
drivers/staging/ipack/bridges/tpci200.h | 14 +++---
2 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/drivers
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/bridges/Kconfig |1 -
drivers/staging/ipack/devices/Kconfig |1 -
2 files changed, 2 deletions(-)
diff --git a/drivers/staging/ipack/bridges/Kconfig
b/drivers/staging/ipack/bridges/Kconfig
index
From: Jens Taprogge jens.tapro...@taprogge.org
Use the regular ioremap functions and their managed counterparts instead
of the ones provided through IPack callbacks.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers
From: Jens Taprogge jens.tapro...@taprogge.org
The ordering was wrong.
Signed-off-by: Jens Taprogge jens.tapro...@taprogge.org
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
drivers/staging/ipack/devices/ipoctal.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions
1 - 100 of 170 matches
Mail list logo