[PATCH 7/9] staging: csr: Reformat pointer type casts

Lisa Nguyen lisa at xenapiadmin.com
Sun Jun 2 06:32:07 UTC 2013


On Sat, Jun 1, 2013 at 1:31 AM, Dan Carpenter <dan.carpenter at oracle.com> wrote:
>
> On Fri, May 31, 2013 at 10:23:07PM -0700, Lisa Nguyen wrote:
> > Reformatted pointer type casts to resolve errors generated
> > by checkpatch.pl and meet kernel coding standards in
> > sme_userspace.c
> >
> > Signed-off-by: Lisa Nguyen <lisa at xenapiadmin.com>
> > ---
> >  drivers/staging/csr/sme_userspace.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/staging/csr/sme_userspace.c b/drivers/staging/csr/sme_userspace.c
> > index 8106779..b4f6d67 100644
> > --- a/drivers/staging/csr/sme_userspace.c
> > +++ b/drivers/staging/csr/sme_userspace.c
> > @@ -215,7 +215,7 @@ unifi_ta_indicate_protocol(void *ospriv,
> >               CsrWifiRouterCtrlProtocolDirection direction,
> >               const CsrWifiMacAddress *src_addr)
> >  {
> > -    unifi_priv_t *priv = (unifi_priv_t*)ospriv;
> > +    unifi_priv_t *priv = (unifi_priv_t *)ospriv;
>
> This is fine or if you wanted, you could just remove the cast
> entirely.
>
>         unifi_priv_t *priv = ospriv;
>
> regards,
> dan carpenter

Good to know. Makes sense. Thanks for letting me know, Dan.

-Lisa



More information about the devel mailing list