linux-kernel - Re: [PATCH 1/3] Fixed checkpatch warning about unaligned function params

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20170408105735.GA23668@kroah.com>
Date:   Sat, 8 Apr 2017 12:57:35 +0200
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Andrea della Porta <sfaragnaus@...il.com>
Cc:     lee@...nel.org, eric@...olt.net, stefan.wahren@...e.com,
        f.fainelli@...il.com, rjui@...adcom.com, sbranden@...adcom.com,
        bcm-kernel-feedback-list@...adcom.com,
        linux-rpi-kernel@...ts.infradead.org,
        linux-arm-kernel@...ts.infradead.org, devel@...verdev.osuosl.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] Fixed checkpatch warning about unaligned function
 params

On Wed, Mar 29, 2017 at 06:15:35AM +0100, Andrea della Porta wrote:
> Fixed issues:
> CHECK: Alignment should match open parenthesis
> #71: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:71:
> +       audio_info("alsa_stream=%p substream=%p\n", alsa_stream,
> +               alsa_stream ? alsa_stream->substream : 0);
> 
> CHECK: Alignment should match open parenthesis
> #262: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:262:
> +static int snd_bcm2835_pcm_hw_params(struct snd_pcm_substream *substream,
> +       struct snd_pcm_hw_params *params)
> 
> CHECK: Alignment should match open parenthesis
> #315: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:315:
> +       err = bcm2835_audio_set_params(alsa_stream, channels,
> +               alsa_stream->params_rate,
> 
> CHECK: Alignment should match open parenthesis
> #384: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:384:
> +               audio_debug("bcm2835_AUDIO_TRIGGER_START running=%d\n",
> +                       alsa_stream->running);
> 
> CHECK: Alignment should match open parenthesis
> #391: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:391:
> +                                       bytes_to_frames(runtime,
> +                                       alsa_stream->pos);
> 
> CHECK: Alignment should match open parenthesis
> #436: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:436:
> +       audio_debug("pcm_pointer... (%d) hwptr=%d appl=%d pos=%d\n", 0,
> +               frames_to_bytes(runtime, runtime->status->hw_ptr),
> 
> CHECK: Alignment should match open parenthesis
> #447: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:447:
> +static int snd_bcm2835_pcm_lib_ioctl(struct snd_pcm_substream *substream,
> +       unsigned int cmd, void *arg)
> 
> CHECK: Alignment should match open parenthesis
> #452: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:452:
> +       audio_info(" .. substream=%p, cmd=%d, arg=%p (%x) ret=%d\n", substream,
> +               cmd, arg, arg ? *(unsigned *) arg : 0, ret);
> 
> CHECK: Alignment should match open parenthesis
> #544: FILE: drivers/staging/vc04_services/bcm2835-audio/bcm2835-pcm.c:544:
> +       snd_pcm_lib_preallocate_pages_for_all(pcm, SNDRV_DMA_TYPE_CONTINUOUS,
> +               snd_dma_continuous_data(GFP_KERNEL),
> 
> Signed-off-by: Andrea della Porta <sfaragnaus@...il.com>
> ---
>  .../vc04_services/bcm2835-audio/bcm2835-pcm.c      | 47 ++++++++++------------
>  1 file changed, 22 insertions(+), 25 deletions(-)

Your email subject lines do not match other patches for this driver
(staging: drivername: file: what is happening).

Please fix up and resend all of these.

Also, no need to put the full checkpatch output in the changelog,
summarize it for us.

thanks,

greg k-h