* [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel
@ 2013-02-25 10:44 Manjunathappa, Prakash
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-25 10:44 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, nsekhar-l0cyMroinI0,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, Manjunathappa, Prakash,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Patch enables support for m25p80 SPI flash support on
da850-EVM.
Testing information:
da850-evm comes with partitions specified in DT blob.
Able to mount/umount and create/delete files on filesystem partition.
Depends on below patch in spi-next branch of git://git.secretlab.ca/git/linux-2.6.git.
"spi/davinci: add OF support for the spi controller"
Applies on top of below patch under community review:
http://www.mail-archive.com/davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org/msg25184.html
Manjunathappa, Prakash (4):
spi/davinci: fix module build error
ARM: davinci: da850: add SPI1 DT node
ARM: davinci: da850: override SPI DT node device name
ARM: davinci: da850-evm: add SPI flash support
arch/arm/boot/dts/da850-evm.dts | 40 ++++++++++++++++++++++++++++++++++++++
arch/arm/boot/dts/da850.dtsi | 18 +++++++++++++++++
arch/arm/mach-davinci/da8xx-dt.c | 1 +
drivers/spi/spi-davinci.c | 2 +-
4 files changed, 60 insertions(+), 1 deletions(-)
--
1.7.4.1
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/4] spi/davinci: fix module build error
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
@ 2013-02-25 10:44 ` Manjunathappa, Prakash
[not found] ` <1361789050-637-2-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-25 10:44 ` [PATCH 2/4] ARM: davinci: da850: add SPI1 DT node Manjunathappa, Prakash
` (3 subsequent siblings)
4 siblings, 1 reply; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-25 10:44 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
grant.likely-s3s/WqlpOiPyB63q8FvJNQ, hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Fix below module build error:
CC [M] drivers/spi/spi-davinci.o
drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
to undefined symbol 'davini_spi_of_match'
drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
to undefined symbol 'davini_spi_of_match'
make[2]: *** [drivers/spi/spi-davinci.o] Error 1
make[1]: *** [drivers/spi] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [drivers] Error 2
Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
---
drivers/spi/spi-davinci.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c
index 50bd2cd..b30064b 100644
--- a/drivers/spi/spi-davinci.c
+++ b/drivers/spi/spi-davinci.c
@@ -771,7 +771,7 @@ static const struct of_device_id davinci_spi_of_match[] = {
},
{ },
};
-MODULE_DEVICE_TABLE(of, davini_spi_of_match);
+MODULE_DEVICE_TABLE(of, davinci_spi_of_match);
/**
* spi_davinci_get_pdata - Get platform data from DTS binding
--
1.7.4.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 2/4] ARM: davinci: da850: add SPI1 DT node
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-25 10:44 ` [PATCH 1/4] spi/davinci: fix module build error Manjunathappa, Prakash
@ 2013-02-25 10:44 ` Manjunathappa, Prakash
2013-02-25 10:44 ` [PATCH 3/4] ARM: davinci: da850: override SPI DT node device name Manjunathappa, Prakash
` (2 subsequent siblings)
4 siblings, 0 replies; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-25 10:44 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, nsekhar-l0cyMroinI0,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, Manjunathappa, Prakash,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Patch adds SPI1 DT node along with pinmux data.
Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
---
arch/arm/boot/dts/da850.dtsi | 18 ++++++++++++++++++
1 files changed, 18 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index ebe7386..e921c82 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -76,6 +76,14 @@
0x28 0x00222222 0x00ffffff
>;
};
+ spi1_pins: pinmux_spi_pins {
+ pinctrl-single,bits = <
+ /* SIMO, SOMI, ENA,
+ * CLK, CS[0], CS[1]
+ */
+ 0x14 0x00111111 0x00ffffff
+ >;
+ };
};
serial0: serial@1c42000 {
compatible = "ns16550a";
@@ -126,6 +134,16 @@
interrupts = <16>;
status = "disabled";
};
+ spi1: spi@1f0e000 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ compatible = "ti,da8xx-spi";
+ reg = <0x30e000 0x1000>;
+ num-cs = <4>;
+ ti,davinci-spi-intr-line = <1>;
+ interrupts = <56>;
+ status = "disabled";
+ };
};
nand_cs3@62000000 {
compatible = "ti,davinci-nand";
--
1.7.4.1
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 3/4] ARM: davinci: da850: override SPI DT node device name
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-25 10:44 ` [PATCH 1/4] spi/davinci: fix module build error Manjunathappa, Prakash
2013-02-25 10:44 ` [PATCH 2/4] ARM: davinci: da850: add SPI1 DT node Manjunathappa, Prakash
@ 2013-02-25 10:44 ` Manjunathappa, Prakash
2013-02-25 10:44 ` [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support Manjunathappa, Prakash
2013-02-27 8:39 ` [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel Sekhar Nori
4 siblings, 0 replies; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-25 10:44 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
grant.likely-s3s/WqlpOiPyB63q8FvJNQ, hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Populate OF_DEV_AUXDATA with desired device name expected by spi-davinci
driver. Without this clk_get of spi-davinci DT driver fails.
Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
---
arch/arm/mach-davinci/da8xx-dt.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-davinci/da8xx-dt.c b/arch/arm/mach-davinci/da8xx-dt.c
index 2b740a9..d9c76009 100644
--- a/arch/arm/mach-davinci/da8xx-dt.c
+++ b/arch/arm/mach-davinci/da8xx-dt.c
@@ -42,6 +42,7 @@ struct of_dev_auxdata da850_auxdata_lookup[] __initdata = {
OF_DEV_AUXDATA("ti,davinci-wdt", 0x01c21000, "watchdog", NULL),
OF_DEV_AUXDATA("ti,davinci-mmc-da830", 0x01c40000, "davinci-mmc-da830.0",
NULL),
+ OF_DEV_AUXDATA("ti,da8xx-spi", 0x01f0e000, "spi_davinci.1", NULL),
{}
};
--
1.7.4.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
` (2 preceding siblings ...)
2013-02-25 10:44 ` [PATCH 3/4] ARM: davinci: da850: override SPI DT node device name Manjunathappa, Prakash
@ 2013-02-25 10:44 ` Manjunathappa, Prakash
[not found] ` <1361789050-637-5-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-27 8:39 ` [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel Sekhar Nori
4 siblings, 1 reply; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-25 10:44 UTC (permalink / raw)
To: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, nsekhar-l0cyMroinI0,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, Manjunathappa, Prakash,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Enable m25p80 SPI flash support on da850-EVM. Also
add partition information of SPI flash.
Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
---
arch/arm/boot/dts/da850-evm.dts | 40 +++++++++++++++++++++++++++++++++++++++
1 files changed, 40 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 78c8e54..1a28d43 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -46,6 +46,46 @@
pinctrl-names = "default";
pinctrl-0 = <&mmc0_pins>;
};
+ spi1: spi@1f0e000 {
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&spi1_pins>;
+ flash: m25p80@0 {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ compatible = "m25p80";
+ spi-max-frequency = <30000000>;
+ reg = <0>;
+ partition@0 {
+ label = "U-Boot-SPL";
+ reg = <0x00000000 0x00010000>;
+ read-only;
+ };
+ partition@1 {
+ label = "U-Boot";
+ reg = <0x00010000 0x00080000>;
+ read-only;
+ };
+ partition@2 {
+ label = "U-Boot-Env";
+ reg = <0x00090000 0x00010000>;
+ read-only;
+ };
+ partition@3 {
+ label = "Kernel";
+ reg = <0x000a0000 0x00280000>;
+ };
+ partition@4 {
+ label = "Filesystem";
+ reg = <0x00220000 0x00400000>;
+ };
+ partition@5 {
+ label = "MAC-Address";
+ reg = <0x00620000 0x00010000>;
+ read-only;
+ };
+ };
+ };
};
nand_cs3@62000000 {
status = "okay";
--
1.7.4.1
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support
[not found] ` <1361789050-637-5-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
@ 2013-02-27 8:25 ` Sekhar Nori
[not found] ` <512DC2E5.4080603-l0cyMroinI0@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Sekhar Nori @ 2013-02-27 8:25 UTC (permalink / raw)
To: Manjunathappa, Prakash
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> Enable m25p80 SPI flash support on da850-EVM. Also
> add partition information of SPI flash.
>
> Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
> ---
> arch/arm/boot/dts/da850-evm.dts | 40 +++++++++++++++++++++++++++++++++++++++
> 1 files changed, 40 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index 78c8e54..1a28d43 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -46,6 +46,46 @@
> pinctrl-names = "default";
> pinctrl-0 = <&mmc0_pins>;
> };
> + spi1: spi@1f0e000 {
> + status = "okay";
> + pinctrl-names = "default";
> + pinctrl-0 = <&spi1_pins>;
> + flash: m25p80@0 {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "m25p80";
Are you sure there is a m25p80 part on these boards? On my board using
this patch I get:
m25p80 spi32766.0: found m25p64, expected m25p80
I can access the flash just fine though.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/4] spi/davinci: fix module build error
[not found] ` <1361789050-637-2-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
@ 2013-02-27 8:27 ` Sekhar Nori
[not found] ` <512DC35B.20501-l0cyMroinI0@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Sekhar Nori @ 2013-02-27 8:27 UTC (permalink / raw)
To: Manjunathappa, Prakash
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> Fix below module build error:
>
> CC [M] drivers/spi/spi-davinci.o
> drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
> to undefined symbol 'davini_spi_of_match'
> drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
> to undefined symbol 'davini_spi_of_match'
> make[2]: *** [drivers/spi/spi-davinci.o] Error 1
> make[1]: *** [drivers/spi] Error 2
> make[1]: *** Waiting for unfinished jobs....
> make: *** [drivers] Error 2
>
> Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
Acked-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
` (3 preceding siblings ...)
2013-02-25 10:44 ` [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support Manjunathappa, Prakash
@ 2013-02-27 8:39 ` Sekhar Nori
[not found] ` <512DC65D.7080603-l0cyMroinI0@public.gmane.org>
4 siblings, 1 reply; 13+ messages in thread
From: Sekhar Nori @ 2013-02-27 8:39 UTC (permalink / raw)
To: Manjunathappa, Prakash
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
grant.likely-s3s/WqlpOiPyB63q8FvJNQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> Patch enables support for m25p80 SPI flash support on
> da850-EVM.
>
> Testing information:
> da850-evm comes with partitions specified in DT blob.
> Able to mount/umount and create/delete files on filesystem partition.
>
> Depends on below patch in spi-next branch of git://git.secretlab.ca/git/linux-2.6.git.
> "spi/davinci: add OF support for the spi controller"
>
> Applies on top of below patch under community review:
> http://www.mail-archive.com/davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org/msg25184.html
The series looks good to me except a minor comment I gave on 4/4. Patch
1/4 should be picked by Grant (ideally before v3.9-rc1). I will queue
rest of patches for v3.10.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support
[not found] ` <512DC2E5.4080603-l0cyMroinI0@public.gmane.org>
@ 2013-02-27 9:29 ` Manjunathappa, Prakash
0 siblings, 0 replies; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-27 9:29 UTC (permalink / raw)
To: Nori, Sekhar
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
grant.likely-s3s/WqlpOiPyB63q8FvJNQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, Feb 27, 2013 at 13:55:09, Nori, Sekhar wrote:
> On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> > Enable m25p80 SPI flash support on da850-EVM. Also
> > add partition information of SPI flash.
> >
> > Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
> > ---
> > arch/arm/boot/dts/da850-evm.dts | 40 +++++++++++++++++++++++++++++++++++++++
> > 1 files changed, 40 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> > index 78c8e54..1a28d43 100644
> > --- a/arch/arm/boot/dts/da850-evm.dts
> > +++ b/arch/arm/boot/dts/da850-evm.dts
> > @@ -46,6 +46,46 @@
> > pinctrl-names = "default";
> > pinctrl-0 = <&mmc0_pins>;
> > };
> > + spi1: spi@1f0e000 {
> > + status = "okay";
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&spi1_pins>;
> > + flash: m25p80@0 {
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + compatible = "m25p80";
>
> Are you sure there is a m25p80 part on these boards? On my board using
> this patch I get:
>
> m25p80 spi32766.0: found m25p64, expected m25p80
>
On my board also I see this warning. Also there is mistake in filesystem
partition information specified in this patch, will correct it in next
version of patch.
Thanks,
Prakash
> I can access the flash just fine though.
>
> Thanks,
> Sekhar
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel
[not found] ` <512DC65D.7080603-l0cyMroinI0@public.gmane.org>
@ 2013-02-27 11:32 ` Manjunathappa, Prakash
[not found] ` <A73F36158E33644199EB82C5EC81C7BC3EA47FEF-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
0 siblings, 1 reply; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-02-27 11:32 UTC (permalink / raw)
To: Nori, Sekhar, grant.likely-s3s/WqlpOiPyB63q8FvJNQ
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, Feb 27, 2013 at 14:09:57, Nori, Sekhar wrote:
> On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> > Patch enables support for m25p80 SPI flash support on
> > da850-EVM.
> >
> > Testing information:
> > da850-evm comes with partitions specified in DT blob.
> > Able to mount/umount and create/delete files on filesystem partition.
> >
> > Depends on below patch in spi-next branch of git://git.secretlab.ca/git/linux-2.6.git.
> > "spi/davinci: add OF support for the spi controller"
> >
Grant,
I could not find binding documentation changes submitted with above patch.
I checked in spi-next and linux-next.
> > Applies on top of below patch under community review:
> > http://www.mail-archive.com/davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org/msg25184.html
>
> The series looks good to me except a minor comment I gave on 4/4. Patch
> 1/4 should be picked by Grant (ideally before v3.9-rc1). I will queue
> rest of patches for v3.10.
>
Thanks for review Sekhar, I will put your Acked-by for 1/4 for v2 of this series.
Regards,
Prakash
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/4] spi/davinci: fix module build error
[not found] ` <512DC35B.20501-l0cyMroinI0@public.gmane.org>
@ 2013-03-02 23:11 ` Grant Likely
0 siblings, 0 replies; 13+ messages in thread
From: Grant Likely @ 2013-03-02 23:11 UTC (permalink / raw)
To: Sekhar Nori, Manjunathappa, Prakash
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, 27 Feb 2013 13:57:07 +0530, Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> wrote:
> On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> > Fix below module build error:
> >
> > CC [M] drivers/spi/spi-davinci.o
> > drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
> > to undefined symbol 'davini_spi_of_match'
> > drivers/spi/spi-davinci.c:774:1: error: '__mod_of_device_table' aliased
> > to undefined symbol 'davini_spi_of_match'
> > make[2]: *** [drivers/spi/spi-davinci.o] Error 1
> > make[1]: *** [drivers/spi] Error 2
> > make[1]: *** Waiting for unfinished jobs....
> > make: *** [drivers] Error 2
> >
> > Signed-off-by: Manjunathappa, Prakash <prakash.pm-l0cyMroinI0@public.gmane.org>
>
> Acked-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>
Applied, thanks.
g.
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel
[not found] ` <A73F36158E33644199EB82C5EC81C7BC3EA47FEF-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
@ 2013-03-02 23:13 ` Grant Likely
2013-03-04 13:08 ` Manjunathappa, Prakash
0 siblings, 1 reply; 13+ messages in thread
From: Grant Likely @ 2013-03-02 23:13 UTC (permalink / raw)
To: Manjunathappa, Prakash, Nori, Sekhar
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Wed, 27 Feb 2013 11:32:39 +0000, "Manjunathappa, Prakash" <prakash.pm-l0cyMroinI0@public.gmane.org> wrote:
> On Wed, Feb 27, 2013 at 14:09:57, Nori, Sekhar wrote:
> > On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> > > Patch enables support for m25p80 SPI flash support on
> > > da850-EVM.
> > >
> > > Testing information:
> > > da850-evm comes with partitions specified in DT blob.
> > > Able to mount/umount and create/delete files on filesystem partition.
> > >
> > > Depends on below patch in spi-next branch of git://git.secretlab.ca/git/linux-2.6.git.
> > > "spi/davinci: add OF support for the spi controller"
> > >
>
> Grant,
> I could not find binding documentation changes submitted with above patch.
> I checked in spi-next and linux-next.
It must have been missed then. It happens sometimes. Can someone submit
a fixup patch please?
Thanks,
g.
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel
2013-03-02 23:13 ` Grant Likely
@ 2013-03-04 13:08 ` Manjunathappa, Prakash
0 siblings, 0 replies; 13+ messages in thread
From: Manjunathappa, Prakash @ 2013-03-04 13:08 UTC (permalink / raw)
To: Grant Likely, Nori, Sekhar
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
linux-lFZ/pmaqli7XmaaqVzeoHQ,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
hs-ynQEQJNshbs,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Hi Grant,
On Sun, Mar 03, 2013 at 04:43:55, Grant Likely wrote:
> On Wed, 27 Feb 2013 11:32:39 +0000, "Manjunathappa, Prakash" <prakash.pm-l0cyMroinI0@public.gmane.org> wrote:
> > On Wed, Feb 27, 2013 at 14:09:57, Nori, Sekhar wrote:
> > > On 2/25/2013 4:14 PM, Manjunathappa, Prakash wrote:
> > > > Patch enables support for m25p80 SPI flash support on
> > > > da850-EVM.
> > > >
> > > > Testing information:
> > > > da850-evm comes with partitions specified in DT blob.
> > > > Able to mount/umount and create/delete files on filesystem partition.
> > > >
> > > > Depends on below patch in spi-next branch of git://git.secretlab.ca/git/linux-2.6.git.
> > > > "spi/davinci: add OF support for the spi controller"
> > > >
> >
> > Grant,
> > I could not find binding documentation changes submitted with above patch.
> > I checked in spi-next and linux-next.
>
> It must have been missed then. It happens sometimes. Can someone submit
> a fixup patch please?
>
I have taken care in my next version of this series.
Thanks,
Prakash
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2013-03-04 13:08 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-25 10:44 [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel Manjunathappa, Prakash
[not found] ` <1361789050-637-1-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-25 10:44 ` [PATCH 1/4] spi/davinci: fix module build error Manjunathappa, Prakash
[not found] ` <1361789050-637-2-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-27 8:27 ` Sekhar Nori
[not found] ` <512DC35B.20501-l0cyMroinI0@public.gmane.org>
2013-03-02 23:11 ` Grant Likely
2013-02-25 10:44 ` [PATCH 2/4] ARM: davinci: da850: add SPI1 DT node Manjunathappa, Prakash
2013-02-25 10:44 ` [PATCH 3/4] ARM: davinci: da850: override SPI DT node device name Manjunathappa, Prakash
2013-02-25 10:44 ` [PATCH 4/4] ARM: davinci: da850-evm: add SPI flash support Manjunathappa, Prakash
[not found] ` <1361789050-637-5-git-send-email-prakash.pm-l0cyMroinI0@public.gmane.org>
2013-02-27 8:25 ` Sekhar Nori
[not found] ` <512DC2E5.4080603-l0cyMroinI0@public.gmane.org>
2013-02-27 9:29 ` Manjunathappa, Prakash
2013-02-27 8:39 ` [PATCH 0/4] Enable SPI flash support on da850-evm DT kernel Sekhar Nori
[not found] ` <512DC65D.7080603-l0cyMroinI0@public.gmane.org>
2013-02-27 11:32 ` Manjunathappa, Prakash
[not found] ` <A73F36158E33644199EB82C5EC81C7BC3EA47FEF-Er742YJ7I/eIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2013-03-02 23:13 ` Grant Likely
2013-03-04 13:08 ` Manjunathappa, Prakash
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).