linux-kernel - Re: [PATCH 03/11] drivers: char: add AXD Audio Processing IP driver

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20141028141038.GA18384@kroah.com>
Date:	Tue, 28 Oct 2014 22:10:38 +0800
From:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:	Qais Yousef <qais.yousef@...tec.com>
Cc:	linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
	Grant Likely <grant.likely@...aro.org>,
	Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
	alsa-devel@...a-project.org
Subject: Re: [PATCH 03/11] drivers: char: add AXD Audio Processing IP driver

On Tue, Oct 28, 2014 at 11:26:21AM +0000, Qais Yousef wrote:
> AXD is Audio Processing IP by Imagination Technologies that can
> perform decoding, encoding, equalisation, resampling, mixing,
> synchronisation and audio playback.

Isn't that a codec?  Why is this a "char" driver and not one that fits
into our existing audio subsystem?

> this patch adds defs and initialisation files

Spell check :)

> Signed-off-by: Qais Yousef <qais.yousef@...tec.com>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: Grant Likely <grant.likely@...aro.org>
> Cc: Rob Herring <robh+dt@...nel.org>
> Cc: <devicetree@...r.kernel.org>
> Cc: <alsa-devel@...a-project.org>
> ---
>  drivers/char/axd/axd_api.h    |  641 +++++++++++++++++++++++++
>  drivers/char/axd/axd_module.c | 1064 +++++++++++++++++++++++++++++++++++++++++
>  drivers/char/axd/axd_module.h |   99 ++++
>  include/linux/axd.h           |   32 ++
>  4 files changed, 1836 insertions(+)
>  create mode 100644 drivers/char/axd/axd_api.h
>  create mode 100644 drivers/char/axd/axd_module.c
>  create mode 100644 drivers/char/axd/axd_module.h
>  create mode 100644 include/linux/axd.h
> 
> diff --git a/drivers/char/axd/axd_api.h b/drivers/char/axd/axd_api.h
> new file mode 100644
> index 000000000000..0d732f173f55
> --- /dev/null
> +++ b/drivers/char/axd/axd_api.h
> @@ -0,0 +1,641 @@
> +/*
> + *  Copyright (C) 2011-2014 Imagination Technologies Ltd.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License
> + * as published by the Free Software Foundation; either version 2
> + * of the License, or (at your option) any later version.

Do you really mean "any later version"?


> + *
> + *  Main API to the AXD for access from the host.
> + */
> +#ifndef AXD_API_H_
> +#define AXD_API_H_
> +
> +#include <linux/types.h>
> +
> +
> +#define THREAD_COUNT 4
> +#define AXD_MAX_PIPES 3
> +
> +
> +#define AXD_DESCRIPTOR_READY_BIT	0x80000000
> +#define AXD_DESCRIPTOR_INUSE_BIT	0x40000000
> +#define AXD_DESCRIPTOR_EOS_BIT		0x20000000
> +#define AXD_DESCRIPTOR_SIZE_MASK	0x0000FFFF
> +
> +struct axd_buffer_desc {
> +	uint32_t status_size;
> +	uint32_t data_ptr;
> +	uint32_t pts_high;
> +	uint32_t pts_low;

Please always use standard Linux kernel types, "u32" in this case.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/