* [PATCH] somedriver: remove the initialization of static pointers.
@ 2017-02-04 19:39 AbdAllah-MEZITI
2017-02-05 0:29 ` Greg Kroah-Hartman
0 siblings, 1 reply; 4+ messages in thread
From: AbdAllah-MEZITI @ 2017-02-04 19:39 UTC (permalink / raw)
To: linux-arm-kernel
In C a static pointer will be initialized to NULL (e.g: draft C99 standard $6.7.8):
"If an object that has static storage duration is not initialized
explicitly, then:
__ if it has pointer type, it is initialized to a null pointer;"
Signed-off-by: AbdAllah-MEZITI <abdallah.meziti.pro@gmail.com>
---
drivers/staging/bcm2835-audio/bcm2835.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/bcm2835-audio/bcm2835.c b/drivers/staging/bcm2835-audio/bcm2835.c
index a84d74d..265fe55 100644
--- a/drivers/staging/bcm2835-audio/bcm2835.c
+++ b/drivers/staging/bcm2835-audio/bcm2835.c
@@ -28,8 +28,8 @@
* to debug if we run into issues
*/
-static struct snd_card *g_card = NULL;
-static struct bcm2835_chip *g_chip = NULL;
+static struct snd_card *g_card;
+static struct bcm2835_chip *g_chip;
static int snd_bcm2835_free(struct bcm2835_chip *chip)
{
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] somedriver: remove the initialization of static pointers.
2017-02-04 19:39 [PATCH] somedriver: remove the initialization of static pointers AbdAllah-MEZITI
@ 2017-02-05 0:29 ` Greg Kroah-Hartman
2017-02-05 7:12 ` AbdAllah MEZITI
0 siblings, 1 reply; 4+ messages in thread
From: Greg Kroah-Hartman @ 2017-02-05 0:29 UTC (permalink / raw)
To: linux-arm-kernel
On Sat, Feb 04, 2017 at 08:39:21PM +0100, AbdAllah-MEZITI wrote:
> In C a static pointer will be initialized to NULL (e.g: draft C99 standard $6.7.8):
> "If an object that has static storage duration is not initialized
> explicitly, then:
> __ if it has pointer type, it is initialized to a null pointer;"
>
> Signed-off-by: AbdAllah-MEZITI <abdallah.meziti.pro@gmail.com>
Your subject is very odd :(
And is that the correct spelling of your name that you use for legal
documents? If not, please fix up when you resend this.
Also, fix the line-wrapping of your changelog, and the C99 standard is
not a "draft" anymore, it was released in 1999 :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] somedriver: remove the initialization of static pointers.
2017-02-05 0:29 ` Greg Kroah-Hartman
@ 2017-02-05 7:12 ` AbdAllah MEZITI
2017-02-06 17:33 ` Mathieu Poirier
0 siblings, 1 reply; 4+ messages in thread
From: AbdAllah MEZITI @ 2017-02-05 7:12 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, 5 Feb 2017 01:29:43 +0100
Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote:
> On Sat, Feb 04, 2017 at 08:39:21PM +0100, AbdAllah-MEZITI wrote:
> > In C a static pointer will be initialized to NULL (e.g: draft C99
> > standard $6.7.8): "If an object that has static storage duration is
> > not initialized explicitly, then:
> > __ if it has pointer type, it is initialized to a null pointer;"
> >
> > Signed-off-by: AbdAllah-MEZITI <abdallah.meziti.pro@gmail.com>
>
> Your subject is very odd :(
>
> And is that the correct spelling of your name that you use for legal
> documents? If not, please fix up when you resend this.
>
> Also, fix the line-wrapping of your changelog, and the C99 standard is
> not a "draft" anymore, it was released in 1999 :)
>
> thanks,
>
> greg k-h
Do not worry, this is just an answer to the Task 10 of the Eudyptula Challenge.
yes, this is the correct spelling of my name that i use for legal documents.
OK.
AbdAllah MEZITI.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] somedriver: remove the initialization of static pointers.
2017-02-05 7:12 ` AbdAllah MEZITI
@ 2017-02-06 17:33 ` Mathieu Poirier
0 siblings, 0 replies; 4+ messages in thread
From: Mathieu Poirier @ 2017-02-06 17:33 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, Feb 05, 2017 at 08:12:26AM +0100, AbdAllah MEZITI wrote:
> On Sun, 5 Feb 2017 01:29:43 +0100
> Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote:
>
> > On Sat, Feb 04, 2017 at 08:39:21PM +0100, AbdAllah-MEZITI wrote:
> > > In C a static pointer will be initialized to NULL (e.g: draft C99
> > > standard $6.7.8): "If an object that has static storage duration is
> > > not initialized explicitly, then:
> > > __ if it has pointer type, it is initialized to a null pointer;"
> > >
> > > Signed-off-by: AbdAllah-MEZITI <abdallah.meziti.pro@gmail.com>
> >
> > Your subject is very odd :(
> >
> > And is that the correct spelling of your name that you use for legal
> > documents? If not, please fix up when you resend this.
> >
> > Also, fix the line-wrapping of your changelog, and the C99 standard is
> > not a "draft" anymore, it was released in 1999 :)
> >
> > thanks,
> >
> > greg k-h
>
>
> Do not worry, this is just an answer to the Task 10 of the Eudyptula Challenge.
AbdAllah,
Patches submitted as part of the Eudyptula Challenge are fixing real problems
and most of the time accepted for queueing. As such the work you sumbit for it
is serious and should be as perfect as possible.
Happy times with the Challenge,
Mathieu
>
> yes, this is the correct spelling of my name that i use for legal documents.
>
> OK.
>
> AbdAllah MEZITI.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-02-06 17:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-04 19:39 [PATCH] somedriver: remove the initialization of static pointers AbdAllah-MEZITI
2017-02-05 0:29 ` Greg Kroah-Hartman
2017-02-05 7:12 ` AbdAllah MEZITI
2017-02-06 17:33 ` Mathieu Poirier
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).