2019-11-06 18:48:50

by Daniel Almeida

Subject: [PATCH] media: dvb_dummy_fe: place EXPORT_SYMBOL below corresponding function

From: "Daniel W. S. Almeida" <[email protected]>

Suggested-by: Shuah Khan <[email protected]>
Signed-off-by: Daniel W. S. Almeida <[email protected]>
---
drivers/media/dvb-frontends/dvb_dummy_fe.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/media/dvb-frontends/dvb_dummy_fe.c b/drivers/media/dvb-frontends/dvb_dummy_fe.c
index 4db679cb70ad..909dac2345c4 100644
--- a/drivers/media/dvb-frontends/dvb_dummy_fe.c
+++ b/drivers/media/dvb-frontends/dvb_dummy_fe.c
@@ -121,6 +121,7 @@ struct dvb_frontend* dvb_dummy_fe_ofdm_attach(void)
state->frontend.demodulator_priv = state;
return &state->frontend;
}
+EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);

static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops;

@@ -138,6 +139,7 @@ struct dvb_frontend *dvb_dummy_fe_qpsk_attach(void)
state->frontend.demodulator_priv = state;
return &state->frontend;
}
+EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);

static const struct dvb_frontend_ops dvb_dummy_fe_qam_ops;

@@ -155,6 +157,7 @@ struct dvb_frontend *dvb_dummy_fe_qam_attach(void)
state->frontend.demodulator_priv = state;
return &state->frontend;
}
+EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);

static const struct dvb_frontend_ops dvb_dummy_fe_ofdm_ops = {
.delsys = { SYS_DVBT },
@@ -253,7 +256,3 @@ static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops = {
MODULE_DESCRIPTION("DVB DUMMY Frontend");
MODULE_AUTHOR("Emard");
MODULE_LICENSE("GPL");
-
-EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);
-EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);
-EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);
--
2.24.0

2019-11-06 19:10:15

by Shuah Khan

Subject: Re: [PATCH] media: dvb_dummy_fe: place EXPORT_SYMBOL below corresponding function

On 11/6/19 11:37 AM, Daniel W. S. Almeida wrote:
> From: "Daniel W. S. Almeida" <[email protected]>
>

You are missing the commit message. :)

> Suggested-by: Shuah Khan <[email protected]>
> Signed-off-by: Daniel W. S. Almeida <[email protected]>
> ---
> drivers/media/dvb-frontends/dvb_dummy_fe.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/media/dvb-frontends/dvb_dummy_fe.c b/drivers/media/dvb-frontends/dvb_dummy_fe.c
> index 4db679cb70ad..909dac2345c4 100644
> --- a/drivers/media/dvb-frontends/dvb_dummy_fe.c
> +++ b/drivers/media/dvb-frontends/dvb_dummy_fe.c
> @@ -121,6 +121,7 @@ struct dvb_frontend* dvb_dummy_fe_ofdm_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops;
>
> @@ -138,6 +139,7 @@ struct dvb_frontend *dvb_dummy_fe_qpsk_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_qam_ops;
>
> @@ -155,6 +157,7 @@ struct dvb_frontend *dvb_dummy_fe_qam_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_ofdm_ops = {
> .delsys = { SYS_DVBT },
> @@ -253,7 +256,3 @@ static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops = {
> MODULE_DESCRIPTION("DVB DUMMY Frontend");
> MODULE_AUTHOR("Emard");
> MODULE_LICENSE("GPL");
> -
> -EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);
> -EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);
> -EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);
>

2019-11-06 19:10:17

by Fabio Estevam

Subject: Re: [PATCH] media: dvb_dummy_fe: place EXPORT_SYMBOL below corresponding function

Hi Daniel,

On Wed, Nov 6, 2019 at 3:51 PM Daniel W. S. Almeida
<[email protected]> wrote:
>
> From: "Daniel W. S. Almeida" <[email protected]>

Please write a commit log.

> Suggested-by: Shuah Khan <[email protected]>
> Signed-off-by: Daniel W. S. Almeida <[email protected]>

2019-11-30 06:52:28

by Mauro Carvalho Chehab

Subject: Re: [PATCH] media: dvb_dummy_fe: place EXPORT_SYMBOL below corresponding function

Em Wed, 6 Nov 2019 15:37:16 -0300
"Daniel W. S. Almeida" <[email protected]> escreveu:

> From: "Daniel W. S. Almeida" <[email protected]>
>
> Suggested-by: Shuah Khan <[email protected]>
> Signed-off-by: Daniel W. S. Almeida <[email protected]>

Looks OK on my eyes.

> ---
> drivers/media/dvb-frontends/dvb_dummy_fe.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/media/dvb-frontends/dvb_dummy_fe.c b/drivers/media/dvb-frontends/dvb_dummy_fe.c
> index 4db679cb70ad..909dac2345c4 100644
> --- a/drivers/media/dvb-frontends/dvb_dummy_fe.c
> +++ b/drivers/media/dvb-frontends/dvb_dummy_fe.c
> @@ -121,6 +121,7 @@ struct dvb_frontend* dvb_dummy_fe_ofdm_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops;
>
> @@ -138,6 +139,7 @@ struct dvb_frontend *dvb_dummy_fe_qpsk_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_qam_ops;
>
> @@ -155,6 +157,7 @@ struct dvb_frontend *dvb_dummy_fe_qam_attach(void)
> state->frontend.demodulator_priv = state;
> return &state->frontend;
> }
> +EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);
>
> static const struct dvb_frontend_ops dvb_dummy_fe_ofdm_ops = {
> .delsys = { SYS_DVBT },
> @@ -253,7 +256,3 @@ static const struct dvb_frontend_ops dvb_dummy_fe_qpsk_ops = {
> MODULE_DESCRIPTION("DVB DUMMY Frontend");
> MODULE_AUTHOR("Emard");
> MODULE_LICENSE("GPL");
> -
> -EXPORT_SYMBOL(dvb_dummy_fe_ofdm_attach);
> -EXPORT_SYMBOL(dvb_dummy_fe_qam_attach);
> -EXPORT_SYMBOL(dvb_dummy_fe_qpsk_attach);



Cheers,
Mauro