[v2,4/9] fsl/fman: Add FMan MURAM support

Scott Wood scottwood at freescale.com
Fri Jun 26 11:59:09 AEST 2015


On Wed, 2015-06-24 at 22:34 +0300, igal.liberman at freescale.com wrote:
> +     struct muram_info *p_muram;

No Hungarian notation.

> +void fm_muram_free(struct muram_info *p_muram)
> +{
> +     /* Destroy pool */
> +     gen_pool_destroy(p_muram->pool);
> +     /* Unmap memory */
> +     iounmap(p_muram->vbase);
> +     /* Free pointer */
> +     kfree(p_muram);
> +}

This type of commenting is not useful.

> +     memset_io((void __iomem *)vaddr, 0, (int)size);

Unnecessary cast of size.

-Scott



More information about the Linuxppc-dev mailing list