All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-12 18:45 ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-12 18:45 UTC (permalink / raw)
  To: David Brown, Daniel Walker, Bryan Huntsman, Florian Tobias Schandinat
  Cc: linux-kernel, linux-arm-msm, linux-arm-kernel, linux-fbdev

The change
    commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
    Author: Gregory Bean <gbean@codeaurora.org>
    Date:   Wed Jul 28 10:22:13 2010 -0700

        video: msm: Fix section mismatch in mddi.c.

fixes a section mismatch between the board file and the driver's probe
function, however, it misses the additional mismatches between the
probe function and some routines it calls.  Fix these up as well.

Signed-off-by: David Brown <davidb@codeaurora.org>
---
 drivers/video/msm/mddi.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
index 4527cbf..b061d70 100644
--- a/drivers/video/msm/mddi.c
+++ b/drivers/video/msm/mddi.c
@@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
 	mddi_set_auto_hibernate(&mddi->client_data, 1);
 }
 
-static int __init mddi_get_client_caps(struct mddi_info *mddi)
+static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
 {
 	int i, j;
 
@@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
 
 static struct mddi_info mddi_info[2];
 
-static int __init mddi_clk_setup(struct platform_device *pdev,
-				 struct mddi_info *mddi,
-				 unsigned long clk_rate)
+static int __devinit mddi_clk_setup(struct platform_device *pdev,
+				    struct mddi_info *mddi,
+				    unsigned long clk_rate)
 {
 	int ret;
 
-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply related	[flat|nested] 11+ messages in thread
* [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-12 18:45 ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-12 18:45 UTC (permalink / raw)
  To: linux-arm-kernel

The change
    commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
    Author: Gregory Bean <gbean@codeaurora.org>
    Date:   Wed Jul 28 10:22:13 2010 -0700

        video: msm: Fix section mismatch in mddi.c.

fixes a section mismatch between the board file and the driver's probe
function, however, it misses the additional mismatches between the
probe function and some routines it calls.  Fix these up as well.

Signed-off-by: David Brown <davidb@codeaurora.org>
---
 drivers/video/msm/mddi.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
index 4527cbf..b061d70 100644
--- a/drivers/video/msm/mddi.c
+++ b/drivers/video/msm/mddi.c
@@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
 	mddi_set_auto_hibernate(&mddi->client_data, 1);
 }
 
-static int __init mddi_get_client_caps(struct mddi_info *mddi)
+static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
 {
 	int i, j;
 
@@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
 
 static struct mddi_info mddi_info[2];
 
-static int __init mddi_clk_setup(struct platform_device *pdev,
-				 struct mddi_info *mddi,
-				 unsigned long clk_rate)
+static int __devinit mddi_clk_setup(struct platform_device *pdev,
+				    struct mddi_info *mddi,
+				    unsigned long clk_rate)
 {
 	int ret;
 
-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.


^ permalink raw reply related	[flat|nested] 11+ messages in thread
* [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-12 18:45 ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-12 18:45 UTC (permalink / raw)
  To: linux-arm-kernel

The change
    commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
    Author: Gregory Bean <gbean@codeaurora.org>
    Date:   Wed Jul 28 10:22:13 2010 -0700

        video: msm: Fix section mismatch in mddi.c.

fixes a section mismatch between the board file and the driver's probe
function, however, it misses the additional mismatches between the
probe function and some routines it calls.  Fix these up as well.

Signed-off-by: David Brown <davidb@codeaurora.org>
---
 drivers/video/msm/mddi.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
index 4527cbf..b061d70 100644
--- a/drivers/video/msm/mddi.c
+++ b/drivers/video/msm/mddi.c
@@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
 	mddi_set_auto_hibernate(&mddi->client_data, 1);
 }
 
-static int __init mddi_get_client_caps(struct mddi_info *mddi)
+static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
 {
 	int i, j;
 
@@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
 
 static struct mddi_info mddi_info[2];
 
-static int __init mddi_clk_setup(struct platform_device *pdev,
-				 struct mddi_info *mddi,
-				 unsigned long clk_rate)
+static int __devinit mddi_clk_setup(struct platform_device *pdev,
+				    struct mddi_info *mddi,
+				    unsigned long clk_rate)
 {
 	int ret;
 
-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply related	[flat|nested] 11+ messages in thread
* [PATCH 2/2] ARM: msm: Fix section mismatches in proc_comm.c
  2012-04-12 18:45 ` David Brown
@ 2012-04-12 18:45   ` David Brown
  -1 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-12 18:45 UTC (permalink / raw)
  To: David Brown, Daniel Walker, Bryan Huntsman, Russell King
  Cc: linux-kernel, linux-arm-msm, linux-arm-kernel

The change
    commit 4416e9eb0b4859b3d28016c5fd0a609bdcbc8a2a
    Author: Gregory Bean <gbean@codeaurora.org>
    Date:   Wed Jul 28 10:22:12 2010 -0700

        arm: msm: Fix section mismatch in smd.c.

fixes a section mismatch between the board file and the smd driver's
probe function, however, it misses the additional mismatches between
the probe function and some routines it calls.  Fix these up as well.

Signed-off-by: David Brown <davidb@codeaurora.org>
---
 arch/arm/mach-msm/proc_comm.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-msm/proc_comm.c b/arch/arm/mach-msm/proc_comm.c
index 67e701c..9980dc7 100644
--- a/arch/arm/mach-msm/proc_comm.c
+++ b/arch/arm/mach-msm/proc_comm.c
@@ -121,7 +121,7 @@ int msm_proc_comm(unsigned cmd, unsigned *data1, unsigned *data2)
  * and unknown state. This function should be called early to
  * wait on the ARM9.
  */
-void __init proc_comm_boot_wait(void)
+void __devinit proc_comm_boot_wait(void)
 {
 	void __iomem *base = MSM_SHARED_RAM_BASE;
  
-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply related	[flat|nested] 11+ messages in thread
* [PATCH 2/2] ARM: msm: Fix section mismatches in proc_comm.c
@ 2012-04-12 18:45   ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-12 18:45 UTC (permalink / raw)
  To: linux-arm-kernel

The change
    commit 4416e9eb0b4859b3d28016c5fd0a609bdcbc8a2a
    Author: Gregory Bean <gbean@codeaurora.org>
    Date:   Wed Jul 28 10:22:12 2010 -0700

        arm: msm: Fix section mismatch in smd.c.

fixes a section mismatch between the board file and the smd driver's
probe function, however, it misses the additional mismatches between
the probe function and some routines it calls.  Fix these up as well.

Signed-off-by: David Brown <davidb@codeaurora.org>
---
 arch/arm/mach-msm/proc_comm.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-msm/proc_comm.c b/arch/arm/mach-msm/proc_comm.c
index 67e701c..9980dc7 100644
--- a/arch/arm/mach-msm/proc_comm.c
+++ b/arch/arm/mach-msm/proc_comm.c
@@ -121,7 +121,7 @@ int msm_proc_comm(unsigned cmd, unsigned *data1, unsigned *data2)
  * and unknown state. This function should be called early to
  * wait on the ARM9.
  */
-void __init proc_comm_boot_wait(void)
+void __devinit proc_comm_boot_wait(void)
 {
 	void __iomem *base = MSM_SHARED_RAM_BASE;
  
-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply related	[flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
  2012-04-12 18:45 ` David Brown
  (?)
@ 2012-04-23  6:16   ` Florian Tobias Schandinat
  -1 siblings, 0 replies; 11+ messages in thread
From: Florian Tobias Schandinat @ 2012-04-23  6:16 UTC (permalink / raw)
  To: David Brown
  Cc: Daniel Walker, Bryan Huntsman, linux-kernel, linux-arm-msm,
	linux-arm-kernel, linux-fbdev

On 04/12/2012 06:45 PM, David Brown wrote:
> The change
>     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
>     Author: Gregory Bean <gbean@codeaurora.org>
>     Date:   Wed Jul 28 10:22:13 2010 -0700
> 
>         video: msm: Fix section mismatch in mddi.c.
> 
> fixes a section mismatch between the board file and the driver's probe
> function, however, it misses the additional mismatches between the
> probe function and some routines it calls.  Fix these up as well.
> 
> Signed-off-by: David Brown <davidb@codeaurora.org>

Applied.


Thanks,

Florian Tobias Schandinat

> ---
>  drivers/video/msm/mddi.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
> index 4527cbf..b061d70 100644
> --- a/drivers/video/msm/mddi.c
> +++ b/drivers/video/msm/mddi.c
> @@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
>  	mddi_set_auto_hibernate(&mddi->client_data, 1);
>  }
>  
> -static int __init mddi_get_client_caps(struct mddi_info *mddi)
> +static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
>  {
>  	int i, j;
>  
> @@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
>  
>  static struct mddi_info mddi_info[2];
>  
> -static int __init mddi_clk_setup(struct platform_device *pdev,
> -				 struct mddi_info *mddi,
> -				 unsigned long clk_rate)
> +static int __devinit mddi_clk_setup(struct platform_device *pdev,
> +				    struct mddi_info *mddi,
> +				    unsigned long clk_rate)
>  {
>  	int ret;
>  

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-23  6:16   ` Florian Tobias Schandinat
  0 siblings, 0 replies; 11+ messages in thread
From: Florian Tobias Schandinat @ 2012-04-23  6:16 UTC (permalink / raw)
  To: linux-arm-kernel

On 04/12/2012 06:45 PM, David Brown wrote:
> The change
>     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
>     Author: Gregory Bean <gbean@codeaurora.org>
>     Date:   Wed Jul 28 10:22:13 2010 -0700
> 
>         video: msm: Fix section mismatch in mddi.c.
> 
> fixes a section mismatch between the board file and the driver's probe
> function, however, it misses the additional mismatches between the
> probe function and some routines it calls.  Fix these up as well.
> 
> Signed-off-by: David Brown <davidb@codeaurora.org>

Applied.


Thanks,

Florian Tobias Schandinat

> ---
>  drivers/video/msm/mddi.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
> index 4527cbf..b061d70 100644
> --- a/drivers/video/msm/mddi.c
> +++ b/drivers/video/msm/mddi.c
> @@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
>  	mddi_set_auto_hibernate(&mddi->client_data, 1);
>  }
>  
> -static int __init mddi_get_client_caps(struct mddi_info *mddi)
> +static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
>  {
>  	int i, j;
>  
> @@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
>  
>  static struct mddi_info mddi_info[2];
>  
> -static int __init mddi_clk_setup(struct platform_device *pdev,
> -				 struct mddi_info *mddi,
> -				 unsigned long clk_rate)
> +static int __devinit mddi_clk_setup(struct platform_device *pdev,
> +				    struct mddi_info *mddi,
> +				    unsigned long clk_rate)
>  {
>  	int ret;
>  


^ permalink raw reply	[flat|nested] 11+ messages in thread
* [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-23  6:16   ` Florian Tobias Schandinat
  0 siblings, 0 replies; 11+ messages in thread
From: Florian Tobias Schandinat @ 2012-04-23  6:16 UTC (permalink / raw)
  To: linux-arm-kernel

On 04/12/2012 06:45 PM, David Brown wrote:
> The change
>     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
>     Author: Gregory Bean <gbean@codeaurora.org>
>     Date:   Wed Jul 28 10:22:13 2010 -0700
> 
>         video: msm: Fix section mismatch in mddi.c.
> 
> fixes a section mismatch between the board file and the driver's probe
> function, however, it misses the additional mismatches between the
> probe function and some routines it calls.  Fix these up as well.
> 
> Signed-off-by: David Brown <davidb@codeaurora.org>

Applied.


Thanks,

Florian Tobias Schandinat

> ---
>  drivers/video/msm/mddi.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/video/msm/mddi.c b/drivers/video/msm/mddi.c
> index 4527cbf..b061d70 100644
> --- a/drivers/video/msm/mddi.c
> +++ b/drivers/video/msm/mddi.c
> @@ -420,7 +420,7 @@ static void mddi_resume(struct msm_mddi_client_data *cdata)
>  	mddi_set_auto_hibernate(&mddi->client_data, 1);
>  }
>  
> -static int __init mddi_get_client_caps(struct mddi_info *mddi)
> +static int __devinit mddi_get_client_caps(struct mddi_info *mddi)
>  {
>  	int i, j;
>  
> @@ -622,9 +622,9 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg)
>  
>  static struct mddi_info mddi_info[2];
>  
> -static int __init mddi_clk_setup(struct platform_device *pdev,
> -				 struct mddi_info *mddi,
> -				 unsigned long clk_rate)
> +static int __devinit mddi_clk_setup(struct platform_device *pdev,
> +				    struct mddi_info *mddi,
> +				    unsigned long clk_rate)
>  {
>  	int ret;
>  

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
  2012-04-23  6:16   ` Florian Tobias Schandinat
  (?)
@ 2012-04-23 16:19     ` David Brown
  -1 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-23 16:19 UTC (permalink / raw)
  To: Florian Tobias Schandinat
  Cc: Daniel Walker, Bryan Huntsman, linux-kernel, linux-arm-msm,
	linux-arm-kernel, linux-fbdev

On Mon, Apr 23, 2012 at 06:16:55AM +0000, Florian Tobias Schandinat wrote:
> On 04/12/2012 06:45 PM, David Brown wrote:
> > The change
> >     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
> >     Author: Gregory Bean <gbean@codeaurora.org>
> >     Date:   Wed Jul 28 10:22:13 2010 -0700
> > 
> >         video: msm: Fix section mismatch in mddi.c.
> > 
> > fixes a section mismatch between the board file and the driver's probe
> > function, however, it misses the additional mismatches between the
> > probe function and some routines it calls.  Fix these up as well.
> > 
> > Signed-off-by: David Brown <davidb@codeaurora.org>
> 
> Applied.

Hmm.  I guess it is somewhat my fault, since I sent the message To
both myself and you.  I've already applied this to the MSM tree, and I
believe it has been merged into Linus' tree.  I don't think it'll be
an issue, since git merge will handle the duplicate patch fine.

Next time, I'll try to be clearer about which tree I'm thinking for a
patch to go through.

David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-23 16:19     ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-23 16:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Apr 23, 2012 at 06:16:55AM +0000, Florian Tobias Schandinat wrote:
> On 04/12/2012 06:45 PM, David Brown wrote:
> > The change
> >     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
> >     Author: Gregory Bean <gbean@codeaurora.org>
> >     Date:   Wed Jul 28 10:22:13 2010 -0700
> > 
> >         video: msm: Fix section mismatch in mddi.c.
> > 
> > fixes a section mismatch between the board file and the driver's probe
> > function, however, it misses the additional mismatches between the
> > probe function and some routines it calls.  Fix these up as well.
> > 
> > Signed-off-by: David Brown <davidb@codeaurora.org>
> 
> Applied.

Hmm.  I guess it is somewhat my fault, since I sent the message To
both myself and you.  I've already applied this to the MSM tree, and I
believe it has been merged into Linus' tree.  I don't think it'll be
an issue, since git merge will handle the duplicate patch fine.

Next time, I'll try to be clearer about which tree I'm thinking for a
patch to go through.

David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 11+ messages in thread
* [PATCH 1/2] video: msm: Fix section mismatches in mddi.c
@ 2012-04-23 16:19     ` David Brown
  0 siblings, 0 replies; 11+ messages in thread
From: David Brown @ 2012-04-23 16:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Apr 23, 2012 at 06:16:55AM +0000, Florian Tobias Schandinat wrote:
> On 04/12/2012 06:45 PM, David Brown wrote:
> > The change
> >     commit 461cbe77d0a4f887c33a3a95ea68a7daf23b4302
> >     Author: Gregory Bean <gbean@codeaurora.org>
> >     Date:   Wed Jul 28 10:22:13 2010 -0700
> > 
> >         video: msm: Fix section mismatch in mddi.c.
> > 
> > fixes a section mismatch between the board file and the driver's probe
> > function, however, it misses the additional mismatches between the
> > probe function and some routines it calls.  Fix these up as well.
> > 
> > Signed-off-by: David Brown <davidb@codeaurora.org>
> 
> Applied.

Hmm.  I guess it is somewhat my fault, since I sent the message To
both myself and you.  I've already applied this to the MSM tree, and I
believe it has been merged into Linus' tree.  I don't think it'll be
an issue, since git merge will handle the duplicate patch fine.

Next time, I'll try to be clearer about which tree I'm thinking for a
patch to go through.

David

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

^ permalink raw reply	[flat|nested] 11+ messages in thread
end of thread, other threads:[~2012-04-23 16:19 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-12 18:45 [PATCH 1/2] video: msm: Fix section mismatches in mddi.c David Brown
2012-04-12 18:45 ` David Brown
2012-04-12 18:45 ` David Brown
2012-04-12 18:45 ` [PATCH 2/2] ARM: msm: Fix section mismatches in proc_comm.c David Brown
2012-04-12 18:45   ` David Brown
2012-04-23  6:16 ` [PATCH 1/2] video: msm: Fix section mismatches in mddi.c Florian Tobias Schandinat
2012-04-23  6:16   ` Florian Tobias Schandinat
2012-04-23  6:16   ` Florian Tobias Schandinat
2012-04-23 16:19   ` David Brown
2012-04-23 16:19     ` David Brown
2012-04-23 16:19     ` David Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.