Describe gpio-xilinx binding.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- Extend description
.../devicetree/bindings/gpio/gpio-xilinx.txt | 48 ++
1 file changed, 48 insertions(+)
create mode 100644 Documentation/devicetree/bindings/gpio
Enable gpio driver for usage on Xilinx ARM zynq platform.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- New patch in this series
drivers/gpio/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index
Simplification is done by using OF helper function
which increase readability of code and remove
(if (var) var = be32_to_cpup;) assignment.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- New patch in this series
drivers/gpio/gpio-xilinx.c | 24
endians directly on IP. But for the gpio case
without interrupt connected(it means without
interrupt logic) there are just 2 registers
data and tristate where auto detection can't be done.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- New patch in this series
I have chosen
Hi Hans and Greg,
can you please comment this?
Thanks,
Michal
On 05/29/2013 01:28 PM, Michal Simek wrote:
Hi Hans,
any comment on this?
Thanks,
Michal
On 05/23/2013 04:01 PM, Michal Simek wrote:
- Remove Userspace I/O platform driver without IRQ support
but add
Check that dma_ops are initialized correctly.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Functions dma_mmap_attrs(), dma_get_sgtable_attrs()
already have this checking.
---
include/asm-generic/dma-mapping-common.h | 12
1 file changed, 12 insertions(+)
diff --git
Hi Greg,
do you want to take these two patches through your tree?
Or should I take them through my microblaze tree?
Thanks,
Michal
On 05/23/2013 02:31 PM, Michal Simek wrote:
From: Graeme Smecher gsmec...@threespeedlogic.com
This fixes a bug introduced in 5d10302f46d, where device trees
Add support for cpuidle.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v3:
- Move driver to drivers/cpuidle/
- Check zynq compatible string suggested by Arnd
- Use zynq_ function prefix because of multiplatform kernel
- Incorporate comments from Daniel Lezcano
- Rebase
On 06/03/2013 04:22 PM, Greg Kroah-Hartman wrote:
On Mon, Jun 03, 2013 at 02:47:36PM +0200, Michal Simek wrote:
Hi Greg,
do you want to take these two patches through your tree?
Why would I be sending in drivers/block/* patches? I don't have a tree
for that, do I?
Ah ok. I took you
On 06/03/2013 04:03 PM, Daniel Lezcano wrote:
On 06/03/2013 03:40 PM, Michal Simek wrote:
Add support for cpuidle.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v3:
- Move driver to drivers/cpuidle/
- Check zynq compatible string suggested by Arnd
- Use zynq_ function
*/
- .resume = xuartps_resume, /* Resume after a suspend */
.driver = {
.owner = THIS_MODULE,
.name = XUARTPS_NAME, /* Driver name */
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer
mode 100644 drivers/clk/zynq/Makefile
create mode 100644 drivers/clk/zynq/clkc.c
create mode 100644 drivers/clk/zynq/pll.c
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt
On 05/13/2013 07:46 PM, Soren Brinkmann wrote:
I rebased this series on v3.10-rc1 eliminating all dependencies and
incorporated
Greg's feedback.
I'll push this also to my github repository.
https://github.com/sorenb-xlnx/linux-xlnx/tree/zynq/clkc
v2:
- dropped the 'sort #includes'
: this decimal constant
is unsigned only in ISO C90 [enabled by default]
if ((f-type == AUDIT_LOGINUID) (f-val == 4294967295)) {
Signed-off-by: Michal Simek michal.si...@xilinx.com
CC: Al Viro v...@zeniv.linux.org.uk
CC: Eric Paris epa...@redhat.com
---
kernel/auditfilter.c | 2 +-
1 file changed, 1
an unsigned type against a negative return code.
Signed-off-by: Graeme Smecher gsmec...@threespeedlogic.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
Acked-by: Grant Likely grant.lik...@secretlab.ca
---
drivers/block/xsysace.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
Fix sysace sparse warnings.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/block/xsysace.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/block/xsysace.c b/drivers/block/xsysace.c
index 3fd130f..1393b88 100644
--- a/drivers/block/xsysace.c
+++ b
Remove checkpatch warning:
WARNING: Use #include linux/io.h instead of asm/io.h
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/char/xilinx_hwicap/xilinx_hwicap.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/xilinx_hwicap/xilinx_hwicap.h
b
From: Michal Simek mon...@monstr.eu
s/regsiter/register/
Use origin comment from the first driver version
which also explain the usage of XHI_MAX_RETRIES better.
Signed-off-by: Michal Simek mon...@monstr.eu
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/char/xilinx_hwicap
This eliminates having an #ifdef returning NULL for the case
when OF is disabled.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/uio/uio_pdrv_genirq.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/uio/uio_pdrv_genirq.c b/drivers/uio
- Remove Userspace I/O platform driver without IRQ support
but add this functionality to genirq driver
- Remove code duplication from OF binding
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
The main reason for this change is to have one
compatibility string for UIO with and without
in futex cmpxchg (2013-05-27 12:29:35 +0200)
Meyer, Kirk (1):
microblaze: Reversed logic in futex cmpxchg
Michal Simek (2):
microblaze: Initialize temp variable to remove compilation warning
microblaze: Use proper casting
On 10/29/2013 09:26 AM, Kumar Gala wrote:
On Oct 28, 2013, at 5:17 PM, Tomasz Figa wrote:
diff --git a/Documentation/devicetree/bindings/clock/zynq-7000.txt
b/Documentation/devicetree/bindings/clock/zynq-7000.txt index
d99af878f5d7..11fdd146ec83 100644
---
is requested via cmd line ecc=on and runs on
appropriate ARM core.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Russell, Will: We discussed this at KS that will be good
to rephrase it or have different logic around this.
I am not sure if we can also test that this bit is
implemented
On 10/30/2013 02:07 PM, Russell King - ARM Linux wrote:
On Wed, Oct 30, 2013 at 01:46:18PM +0100, Michal Simek wrote:
Russell, Will: We discussed this at KS that will be good
to rephrase it or have different logic around this.
I am not sure if we can also test that this bit is
implemented
On 10/30/2013 03:23 PM, Michal Simek wrote:
On 10/30/2013 02:07 PM, Russell King - ARM Linux wrote:
On Wed, Oct 30, 2013 at 01:46:18PM +0100, Michal Simek wrote:
Russell, Will: We discussed this at KS that will be good
to rephrase it or have different logic around this.
I am not sure if we
Hi Russell,
On 10/30/2013 04:01 PM, Russell King - ARM Linux wrote:
On Wed, Oct 30, 2013 at 03:32:09PM +0100, Michal Simek wrote:
btw: passing ecc=on through command line will caused that ECC enabled
message will be there even on systems which don't implement this bit.
It is just side effect
is requested via cmd line ecc=on and runs on
appropriate ARM core.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- Show message all the time but show ECC enabled just for cases
where IMP bit is used.
---
arch/arm/mm/mmu.c | 4 ++--
1 file changed, 2 insertions(+), 2
;
}
}
Your fix is correct.
Tested-by: Michal Simek mon...@monstr.eu
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM
: xuartps: Fix build error when COMMON_CLK is not set
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
On 10/05/2013 08:56 AM, H. Peter Anvin wrote:
I would, but in my case it was employer-owned and closed.
ok. But I believe general concept for this can be shared.
If you used char device, sysfs, etc.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p
.
ok. How was that physical hardware connection to device you wanted to talk?
Was it any special IP with MMIO? Or gpio jtag emulation or similar?
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform
be also possible.
I think the question is if we can live with 2/3 user interfaces.
I tend to keep firmware one because it is covering a lot of common
use cases and it can be easily to use.
And then I don't have any preference if sysfs or char device
is better.
Thanks,
Michal
--
Michal Simek, Ing
and branch with all these patches will be helpful.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
On 10/09/2013 11:02 AM, Tomi Valkeinen wrote:
On 30/09/13 15:05, Michal Simek wrote:
Hi Tomi,
On 09/16/2013 12:34 PM, Tomi Valkeinen wrote:
On 16/09/13 13:33, Michal Simek wrote:
On 09/16/2013 11:51 AM, Tomi Valkeinen wrote:
On 12/09/13 08:54, Michal Simek wrote:
@@ -394,16 +382,12
devm_iounmap is called automatically that's why remove it from the code
dev_set_drvdata(dev, NULL) is called by generic code
after device_release or on probe failure.
Signed-off-by: Michal Simek michal.si...@xilinx.com
Reviewed-by: Jingoo Han jg1@samsung.com
---
Changes in v2:
Rebased on git
s/op/pdev/ in xilinxfb_of_probe().
No functional chagnes.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2: None
drivers/video/xilinxfb.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/video/xilinxfb.c b/drivers/video
Simplify driver probe and release function.
Signed-off-by: Michal Simek michal.si...@xilinx.com
Reviewed-by: Jingoo Han jg1@samsung.com
---
Changes in v2:
Rebased on git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git
for-next
drivers/video/xilinxfb.c | 7 +--
1 file changed
On 10/09/2013 01:06 PM, Mark Rutland wrote:
On Wed, Oct 09, 2013 at 11:52:12AM +0100, Michal Simek wrote:
s/op/pdev/ in xilinxfb_of_probe().
No functional chagnes.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2: None
drivers/video/xilinxfb.c | 18
On 10/10/2013 01:45 AM, Olof Johansson wrote:
On Tue, Oct 8, 2013 at 1:17 AM, Michal Simek mon...@monstr.eu wrote:
Hi,
On 10/04/2013 06:37 PM, Alexandre Courbot wrote:
Trusted Foundations is a TrustZone-based secure monitor for ARM that
can be invoked using the same SMC-based API on all
of a work around and having zero as
default would be desirable. I'd leave this to Michal.
No problem to have default value setup to 0. And use this workaround
when it is set in DTS.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
On 10/09/2013 05:13 PM, Joe Perches wrote:
On Wed, 2013-10-09 at 12:52 +0200, Michal Simek wrote:
Simplify driver probe and release function.
[]
diff --git a/drivers/video/xilinxfb.c b/drivers/video/xilinxfb.c
[]
@@ -423,7 +419,7 @@ static int xilinxfb_of_probe(struct platform_device
*pdev
devm_iounmap is called automatically that's why remove it from the code
dev_set_drvdata(dev, NULL) is called by generic code
after device_release or on probe failure.
Signed-off-by: Michal Simek michal.si...@xilinx.com
Reviewed-by: Jingoo Han jg1@samsung.com
---
Changes in v3: None
Changes
s/op/pdev/ in xilinxfb_of_probe().
No functional chagnes.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v3: None
Changes in v2: None
drivers/video/xilinxfb.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/video/xilinxfb.c b
Simplify driver probe and release function.
Signed-off-by: Michal Simek michal.si...@xilinx.com
Reviewed-by: Jingoo Han jg1@samsung.com
---
Changes in v3:
- Remove the unnecessary OOM message
Changes in v2:
- Rebased on git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git
for-next
ECC policy is just connected to L1 cache
(IMP - bit 9 - in L1 page table entry format)
and has no connection to other ECC capable devices
in the system. That's why declaring connection with
L1 cache make sense.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
IRC printk without KERN_
IRQ handler has to be register first before IRQ
is enabled in xiic_reinit().
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v3:
- New patch in this patchset based on my discussion with Wolfram
https://lkml.org/lkml/2013/10/4/148
Changes in v2: None
drivers/i2c/busses/i2c
From: Kedareswara rao Appana appana.durga@xilinx.com
Code changes to fix checkpatch warnings listed below.
- WARNING: please, no space before tabs
- WARNING: quoted string split across lines
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si
From: Kedareswara rao Appana appana.durga@xilinx.com
Simplified the probe and remove functions using devm_* functions
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v3:
- Remove error message which is already shown
):
microblaze: Set the default irq_domain
Michal Simek (7):
microblaze: pci: Remove duplicated header
microblaze: Do not use r6 in head.S
microblaze: Fix initrd support
microblaze: Do not select OPT_LIB_ASM by default
microblaze: Add support for new cpu versions
platforms are just correctly described.
The reaching this point I would suggest that for arm, arm-soc maintainers should
keep eyes on any dts binding change and all these changes require ACK from Rob
or Grant
(like device-tree maintainers).
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP
On 05/13/2013 01:59 PM, Ian Campbell wrote:
On Mon, 2013-05-13 at 08:02 +0100, Michal Simek wrote:
Just small overview it is a Xilinx soft core cpu where you can even setup
some parameters for core itself - multiplier, divider, BS, fpu, cache sizes,
etc.
You have to also compose the whole
of_match_ptr(_ptr) NULL
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian
is there an option to remove CONFIG_ARCH_AT91?
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT
-off-by: Maxime Ripard maxime.rip...@free-electrons.com
Acked-by: Michal Simek mon...@monstr.eu
---
arch/arm/mach-zynq/common.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm/mach-zynq/common.c b/arch/arm/mach-zynq/common.c
index 5bfe703..4c0199b8 100644
--- a/arch/arm/mach-zynq
Use devm_request_irq function.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/dma/pl330.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index fa645d8..1989e4c 100644
--- a/drivers/dma/pl330.c
+++ b
This patch is inpired by the patch for drvdata
device-core: Ensure drvdata = NULL when no driver is bound
(sha1: 0998d0631001288a5974afc0b2a5f568bcdecb4d)
Also it fixes all occurences in drivers.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
arch/arm/kernel/etm.c | 6
removing
Intc and timer fixes
Adding new MB versions
Minor fixes
Andrew Murray (1):
of/pci: Use of_pci_range_parser
Jason Wu (1):
microblaze: Add linux.bin.ub target
Michal Simek (13):
microblaze: Remove
On 09/06/2013 08:09 PM, Linus Torvalds wrote:
On Fri, Sep 6, 2013 at 2:09 AM, Michal Simek mon...@monstr.eu wrote:
Hi Linus,
please pull these Microblaze patches to your tree.
Hmm. You've started using signed tags, nice. I just wonder why you use
use one that seems to be a relatively weak
On 08/29/2013 03:37 PM, Arnd Bergmann wrote:
On Tuesday 27 August 2013, Sebastian Hesselbarth wrote:
@@ -58,10 +57,10 @@ static void __init zynq_init_machine(void)
of_platform_bus_probe(NULL, zynq_of_bus_ids, NULL);
}
-static void __init zynq_timer_init(void)
+static void __init
Hi.
-Original Message-
From: Arnd Bergmann [mailto:a...@arndb.de]
Sent: Thursday, August 29, 2013 7:20 PM
To: Michal Simek
Cc: Sebastian Hesselbarth; Soren Brinkmann; Mike Turquette; Russell King;
linux-arm-ker...@lists.infradead.org; linux-kernel@vger.kernel.org
Subject: Re
feature branches.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian
On 07/01/2013 07:20 PM, Greg Kroah-Hartman wrote:
On Mon, Jul 01, 2013 at 09:21:14AM +0200, Michal Simek wrote:
Hi Greg,
On 06/27/2013 01:17 AM, Pavel Machek wrote:
On Wed 2013-06-26 11:52:12, Michal Simek wrote:
The patch UIO: fix uio_pdrv_genirq with device tree but no interrupt
(sha1
On 07/05/2013 06:23 PM, Arnd Bergmann wrote:
The zynq platform code only supports DT based booting, so we
should use DT_MACHINE_START rather than MACHINE_START.
Signed-off-by: Arnd Bergmann a...@arndb.de
Cc: Michal Simek michal.si...@xilinx.com
---
arch/arm/mach-zynq/common.c | 2 +-
1
Use devm_request_irq() for irq allocation which
simplify driver code.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/spi/spi-xilinx.c | 23 ++-
1 file changed, 10 insertions(+), 13 deletions(-)
diff --git a/drivers/spi/spi-xilinx.c b/drivers/spi/spi-xilinx.c
It simplifies driver probing.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/spi/spi-xilinx.c | 14 +++---
1 file changed, 3 insertions(+), 11 deletions(-)
diff --git a/drivers/spi/spi-xilinx.c b/drivers/spi/spi-xilinx.c
index a6bb5b0..07a7bca 100644
--- a/drivers/spi
devm_ioremap_resource() automatically checks that
struct resource is initialized.
Also group platform_get_resource() and devm_ioremap_resource()
together.
And remove mem resource from struct xilinx_spi.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/spi/spi-xilinx.c | 13
dev.of_node is in struct device all the time.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/spi/spi-xilinx.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/spi/spi-xilinx.c b/drivers/spi/spi-xilinx.c
index 35503b4..f3de5e5 100644
--- a/drivers/spi/spi-xilinx.c
On 07/08/2013 04:49 PM, Mark Brown wrote:
On Mon, Jul 08, 2013 at 03:29:16PM +0200, Michal Simek wrote:
Use devm_request_irq() for irq allocation which
simplify driver code.
@@ -495,7 +493,6 @@ static int xilinx_spi_remove(struct platform_device
*pdev)
struct xilinx_spi *xspi
Hi Mark,
On 07/08/2013 04:51 PM, Mark Brown wrote:
On Mon, Jul 08, 2013 at 03:29:17PM +0200, Michal Simek wrote:
It simplifies driver probing.
Applied, thanks.
have you applied this patch?
I can't see it in your topic/xilinx branch.
https://git.kernel.org/cgit/linux/kernel/git/broonie
On 07/08/2013 06:26 PM, Mark Brown wrote:
On Mon, Jul 08, 2013 at 05:48:14PM +0200, Michal Simek wrote:
On 07/08/2013 04:49 PM, Mark Brown wrote:
Is it definitely safe to leave the IRQ hanging around after the master
has been freed - there's no possibility of a late error interrupt
On 07/09/2013 04:47 PM, Mark Brown wrote:
On Tue, Jul 09, 2013 at 04:15:13PM +0200, Michal Simek wrote:
4. spi_master_release()
5. devres_release() with irq
If interrupt happends between 4 and 5 than it can be the problem.
Do I understand you correctly?
Yes.
great
: What do you think?
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian
Use devm_request_irq() for irq allocation which
simplify driver code.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- Disable interrupts in xilinx_spi_remove() to be sure
that interrupt won't come between struct freeing
and devres irq unregistration.
---
drivers/spi
On 09/19/2013 05:18 PM, Yves Vandervennet wrote:
On Thu, Sep 19, 2013 at 5:55 AM, Michal Simek mon...@monstr.eu wrote:
Is there some way a per-device userspace helper can be added that can
handle adding the headers? Such that different fpga types get different
helpers?
What do you exactly
On 09/19/2013 07:28 PM, Jason Gunthorpe wrote:
On Thu, Sep 19, 2013 at 10:18:03AM -0500, Yves Vandervennet wrote:
On Thu, Sep 19, 2013 at 5:55 AM, Michal Simek mon...@monstr.eu wrote:
Is there some way a per-device userspace helper can be added that can
handle adding the headers
being one repo to store
patches for this subsystem.
Does it mean that you are able to see loading file there?
Can you share your script?
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http
have agreement on this another valid discussion is
if firmware interface is OK for this purpose or NOT. Or if make sense
to create different interface or have all of them.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux
On 09/23/2013 07:10 PM, Jason Gunthorpe wrote:
On Mon, Sep 23, 2013 at 03:10:11PM +0200, Michal Simek wrote:
1) The driver doesn't know what firmware to request. It just knows
how to send it to a FPGA.
But dts property in the manager driver which uses this as end driver
can know
by gpio interface and you can program another fpga from zynq (This can
be probably generic case for any combination in this 4 pin connection).
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu
what options there are for cplds.
But anyway AFAIK richer formats are not problem for us.
The same situation should be for partial bitstreams too.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng), OpenPGP - KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze
.
I acked-by the patches, so if someone is willing to take the patches I
am ok with that. I am ok also to pick them into my tree with the
acked-by maintainer. Just let me know ...
Yes, please add also this zynq specific patch through your tree.
Here is my ACK for that.
Acked-by: Michal Simek
Hi Jason,
On 09/18/2013 10:32 PM, Jason Gunthorpe wrote:
On Wed, Sep 18, 2013 at 03:15:17PM -0400, Jason Cooper wrote:
+ Jason Gunthorpe
Thanks, looks interesting, we could possibly use this interface if it
met our needs..
On Wed, Sep 18, 2013 at 05:56:39PM +0200, Michal Simek wrote
Some pl330 have per channel irq and it is necessary
to allocate all of them. Loop over irq assigned for this
device to support these pl330 IPs.
For example this IP is available on Xilinx Zynq platform.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Hi Vinod,
this is the patch I told
This patch is inpired by the patch for drvdata
device-core: Ensure drvdata = NULL when no driver is bound
(sha1: 0998d0631001288a5974afc0b2a5f568bcdecb4d)
Also it fixes all occurences in drivers.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
This patch has been sent as RFC
Xilinx Zynq pl330 dma driver has 9 irqs which all have to
be used by the driver to get it work properly.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Not sure if there are others IP with more than 9 irqs.
---
include/linux/amba/bus.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Driver core clears the driver data to NULL after device_release
or on probe failure, so just remove it from here.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/char/xilinx_hwicap/xilinx_hwicap.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/char/xilinx_hwicap
From: Kedareswara rao Appana appana.durga@xilinx.com
Simplified the probe and remove functions using devm_* functions
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/i2c/busses/i2c-xiic.c | 69
From: Kedareswara rao Appana appana.durga@xilinx.com
code changes to fix sparse warnings
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/i2c/busses/i2c-xiic.c | 23 +++
1 file changed, 11 insertions
From: Kedareswara rao Appana appana.durga@xilinx.com
In write operation, after filling address byte to tx fifo,
set the direction of transfer to tx using control register.
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
From: Michal Simek mon...@monstr.eu
Serial over jtag via mdm is compatible with uartlite driver
just need to extend time for reading chars because connection
is slow.
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
drivers/tty/serial/uartlite.c | 2 +-
1 file changed, 1 insertion(+), 1
Hi Tomi,
On 09/16/2013 12:34 PM, Tomi Valkeinen wrote:
On 16/09/13 13:33, Michal Simek wrote:
On 09/16/2013 11:51 AM, Tomi Valkeinen wrote:
On 12/09/13 08:54, Michal Simek wrote:
@@ -394,16 +382,12 @@ static int xilinxfb_release(struct device *dev)
/* Turn off the display
On 09/30/2013 12:23 PM, Wolfram Sang wrote:
On Mon, Sep 30, 2013 at 11:08:50AM +0200, Michal Simek wrote:
From: Kedareswara rao Appana appana.durga@xilinx.com
code changes to fix sparse warnings
Would be nice to either have them listed here or at least shortly
described.
I will send
On 09/30/2013 12:22 PM, Wolfram Sang wrote:
On Mon, Sep 30, 2013 at 11:08:51AM +0200, Michal Simek wrote:
From: Kedareswara rao Appana appana.durga@xilinx.com
In write operation, after filling address byte to tx fifo,
set the direction of transfer to tx using control register.
You
control register.
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2:
- Extend patch description as Wolfram asked for.
drivers/i2c/busses/i2c-xiic.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/i2c/busses
From: Kedareswara rao Appana appana.durga@xilinx.com
Code changes to fix checkpatch warnings listed below.
- WARNING: please, no space before tabs
- WARNING: quoted string split across lines
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si
From: Kedareswara rao Appana appana.durga@xilinx.com
Simplified the probe and remove functions using devm_* functions
Signed-off-by: Kedareswara rao Appana appa...@xilinx.com
Signed-off-by: Michal Simek michal.si...@xilinx.com
---
Changes in v2: None
drivers/i2c/busses/i2c-xiic.c | 69
On 09/30/2013 02:54 PM, Greg Kroah-Hartman wrote:
On Mon, Sep 30, 2013 at 01:50:35PM +0200, Michal Simek wrote:
From: Michal Simek mon...@monstr.eu
Serial over jtag via mdm is compatible with uartlite driver
just need to extend time for reading chars because connection
is slow.
Signed-off
+ Ohad
On 07/22/2013 03:47 AM, Rusty Russell wrote:
Michal Simek mon...@monstr.eu writes:
Hi Rusty and Jens,
I am getting problem with your patch which you have added to the kernel.
The problem is with my arm zynq remoteproc driver where
I use dma_declare_coherent_memory() to specify memory
On 07/23/2013 12:37 PM, Russell King - ARM Linux wrote:
On Tue, Jul 23, 2013 at 12:00:30PM +0930, Rusty Russell wrote:
Michal Simek mon...@monstr.eu writes:
Let me take some code from virtio_rpmsg_bus.c to show that problematic part.
bufs_va = dma_alloc_coherent(vdev-dev.parent-parent
501 - 600 of 4229 matches
Mail list logo