On 14/06/16 15:18, Thierry Reding wrote:
> From: Thierry Reding <treding@xxxxxxxxxx>
>
> There are some places where whitespace is used in very funky ways. Fix
> the most serious ones to make the code easier on the eye.
>
> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> ---
> drivers/dma/tegra20-apb-dma.c | 12 +++++-------
> 1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c
> index 01e316f73559..80c490faca85 100644
> --- a/drivers/dma/tegra20-apb-dma.c
> +++ b/drivers/dma/tegra20-apb-dma.c
> @@ -484,7 +484,7 @@ static void tegra_dma_configure_for_next(struct tegra_dma_channel *tdc,
> * load new configuration.
> */
> tegra_dma_pause(tdc, false);
> - status = tdc_read(tdc, TEGRA_APBDMA_CHAN_STATUS);
> + status = tdc_read(tdc, TEGRA_APBDMA_CHAN_STATUS);
>
> /*
> * If interrupt is pending then do nothing as the ISR will handle
> @@ -905,7 +905,6 @@ static int get_transfer_param(struct tegra_dma_channel *tdc,
> unsigned long *apb_seq, unsigned long *csr, unsigned int *burst_size,
> enum dma_slave_buswidth *slave_bw)
> {
> -
> switch (direction) {
> case DMA_MEM_TO_DEV:
> *apb_addr = tdc->dma_sconfig.dst_addr;
> @@ -948,8 +947,8 @@ static struct dma_async_tx_descriptor *tegra_dma_prep_slave_sg(
> {
> struct tegra_dma_channel *tdc = to_tegra_dma_chan(dc);
> struct tegra_dma_desc *dma_desc;
> - unsigned int i;
> - struct scatterlist *sg;
> + unsigned int i;
> + struct scatterlist *sg;
> unsigned long csr, ahb_seq, apb_ptr, apb_seq;
> struct list_head req_list;
> struct tegra_dma_sg_req *sg_req = NULL;
> @@ -1062,7 +1061,7 @@ static struct dma_async_tx_descriptor *tegra_dma_prep_dma_cyclic(
> {
> struct tegra_dma_channel *tdc = to_tegra_dma_chan(dc);
> struct tegra_dma_desc *dma_desc = NULL;
> - struct tegra_dma_sg_req *sg_req = NULL;
> + struct tegra_dma_sg_req *sg_req = NULL;
> unsigned long csr, ahb_seq, apb_ptr, apb_seq;
> int len;
> size_t remain_len;
> @@ -1204,7 +1203,6 @@ static void tegra_dma_free_chan_resources(struct dma_chan *dc)
> {
> struct tegra_dma_channel *tdc = to_tegra_dma_chan(dc);
> struct tegra_dma *tdma = tdc->tdma;
> -
> struct tegra_dma_desc *dma_desc;
> struct tegra_dma_sg_req *sg_req;
> struct list_head dma_desc_list;
> @@ -1305,7 +1303,7 @@ static const struct tegra_dma_chip_data tegra148_dma_chip_data = {
>
> static int tegra_dma_probe(struct platform_device *pdev)
> {
> - struct resource *res;
> + struct resource *res;
> struct tegra_dma *tdma;
> int ret;
> int i;
Looks good!
Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx>
Cheers
Jon
--
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
