2019-02-21 10:38:27

by Horia Geanta

[permalink] [raw]
Subject: [PATCH] soc: fsl: guts: make fsl_guts_get_svr() static

The export of fsl_guts_get_svr() is a left-over, it's currently used
only internally and users needing SoC information should use the generic
soc_device infrastructure.

Signed-off-by: Horia Geantă <[email protected]>
---
drivers/soc/fsl/guts.c | 3 +--
include/linux/fsl/guts.h | 2 --
2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/soc/fsl/guts.c b/drivers/soc/fsl/guts.c
index 4f9655087bd7..63f6df86f9e5 100644
--- a/drivers/soc/fsl/guts.c
+++ b/drivers/soc/fsl/guts.c
@@ -115,7 +115,7 @@ static const struct fsl_soc_die_attr *fsl_soc_die_match(
return NULL;
}

-u32 fsl_guts_get_svr(void)
+static u32 fsl_guts_get_svr(void)
{
u32 svr = 0;

@@ -129,7 +129,6 @@ u32 fsl_guts_get_svr(void)

return svr;
}
-EXPORT_SYMBOL(fsl_guts_get_svr);

static int fsl_guts_probe(struct platform_device *pdev)
{
diff --git a/include/linux/fsl/guts.h b/include/linux/fsl/guts.h
index 941b11811f85..1fc0edd71c52 100644
--- a/include/linux/fsl/guts.h
+++ b/include/linux/fsl/guts.h
@@ -135,8 +135,6 @@ struct ccsr_guts {
u32 srds2cr1; /* 0x.0f44 - SerDes2 Control Register 0 */
} __attribute__ ((packed));

-u32 fsl_guts_get_svr(void);
-
/* Alternate function signal multiplex control */
#define MPC85xx_PMUXCR_QE(x) (0x8000 >> (x))

--
2.16.2



2019-02-22 08:09:52

by Yangbo Lu

[permalink] [raw]
Subject: RE: [PATCH] soc: fsl: guts: make fsl_guts_get_svr() static



> -----Original Message-----
> From: Horia Geantă <[email protected]>
> Sent: Thursday, February 21, 2019 6:38 PM
> To: Leo Li <[email protected]>
> Cc: Y.b. Lu <[email protected]>; [email protected];
> [email protected]; [email protected]
> Subject: [PATCH] soc: fsl: guts: make fsl_guts_get_svr() static
>
> The export of fsl_guts_get_svr() is a left-over, it's currently used only internally
> and users needing SoC information should use the generic soc_device
> infrastructure.
>

[Y.b. Lu] Acked-by: Yangbo Lu <[email protected]>

> Signed-off-by: Horia Geantă <[email protected]>
> ---
> drivers/soc/fsl/guts.c | 3 +--
> include/linux/fsl/guts.h | 2 --
> 2 files changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/soc/fsl/guts.c b/drivers/soc/fsl/guts.c index
> 4f9655087bd7..63f6df86f9e5 100644
> --- a/drivers/soc/fsl/guts.c
> +++ b/drivers/soc/fsl/guts.c
> @@ -115,7 +115,7 @@ static const struct fsl_soc_die_attr
> *fsl_soc_die_match(
> return NULL;
> }
>
> -u32 fsl_guts_get_svr(void)
> +static u32 fsl_guts_get_svr(void)
> {
> u32 svr = 0;
>
> @@ -129,7 +129,6 @@ u32 fsl_guts_get_svr(void)
>
> return svr;
> }
> -EXPORT_SYMBOL(fsl_guts_get_svr);
>
> static int fsl_guts_probe(struct platform_device *pdev) { diff --git
> a/include/linux/fsl/guts.h b/include/linux/fsl/guts.h index
> 941b11811f85..1fc0edd71c52 100644
> --- a/include/linux/fsl/guts.h
> +++ b/include/linux/fsl/guts.h
> @@ -135,8 +135,6 @@ struct ccsr_guts {
> u32 srds2cr1; /* 0x.0f44 - SerDes2 Control Register 0 */
> } __attribute__ ((packed));
>
> -u32 fsl_guts_get_svr(void);
> -
> /* Alternate function signal multiplex control */ #define
> MPC85xx_PMUXCR_QE(x) (0x8000 >> (x))
>
> --
> 2.16.2

2019-02-26 17:45:55

by Leo Li

[permalink] [raw]
Subject: Re: [PATCH] soc: fsl: guts: make fsl_guts_get_svr() static

On Fri, Feb 22, 2019 at 2:09 AM Y.b. Lu <[email protected]> wrote:
>
>
>
> > -----Original Message-----
> > From: Horia Geantă <[email protected]>
> > Sent: Thursday, February 21, 2019 6:38 PM
> > To: Leo Li <[email protected]>
> > Cc: Y.b. Lu <[email protected]>; [email protected];
> > [email protected]; [email protected]
> > Subject: [PATCH] soc: fsl: guts: make fsl_guts_get_svr() static
> >
> > The export of fsl_guts_get_svr() is a left-over, it's currently used only internally
> > and users needing SoC information should use the generic soc_device
> > infrastructure.
> >
>
> [Y.b. Lu] Acked-by: Yangbo Lu <[email protected]>

Applied for next. Thanks.

>
> > Signed-off-by: Horia Geantă <[email protected]>
> > ---
> > drivers/soc/fsl/guts.c | 3 +--
> > include/linux/fsl/guts.h | 2 --
> > 2 files changed, 1 insertion(+), 4 deletions(-)
> >
> > diff --git a/drivers/soc/fsl/guts.c b/drivers/soc/fsl/guts.c index
> > 4f9655087bd7..63f6df86f9e5 100644
> > --- a/drivers/soc/fsl/guts.c
> > +++ b/drivers/soc/fsl/guts.c
> > @@ -115,7 +115,7 @@ static const struct fsl_soc_die_attr
> > *fsl_soc_die_match(
> > return NULL;
> > }
> >
> > -u32 fsl_guts_get_svr(void)
> > +static u32 fsl_guts_get_svr(void)
> > {
> > u32 svr = 0;
> >
> > @@ -129,7 +129,6 @@ u32 fsl_guts_get_svr(void)
> >
> > return svr;
> > }
> > -EXPORT_SYMBOL(fsl_guts_get_svr);
> >
> > static int fsl_guts_probe(struct platform_device *pdev) { diff --git
> > a/include/linux/fsl/guts.h b/include/linux/fsl/guts.h index
> > 941b11811f85..1fc0edd71c52 100644
> > --- a/include/linux/fsl/guts.h
> > +++ b/include/linux/fsl/guts.h
> > @@ -135,8 +135,6 @@ struct ccsr_guts {
> > u32 srds2cr1; /* 0x.0f44 - SerDes2 Control Register 0 */
> > } __attribute__ ((packed));
> >
> > -u32 fsl_guts_get_svr(void);
> > -
> > /* Alternate function signal multiplex control */ #define
> > MPC85xx_PMUXCR_QE(x) (0x8000 >> (x))
> >
> > --
> > 2.16.2
>