From: kbuild test robot <lkp@intel.com>
To: Alexandru Ardelean <alexandru.ardelean@analog.com>
Cc: kbuild-all@01.org, dmaengine@vger.kernel.org, vkoul@kernel.org
Subject: [V2] dmaengine: axi-dmac: assign `copy_align` property
Date: Wed, 27 Feb 2019 01:43:00 +0800 [thread overview]
Message-ID: <201902270127.qKUjuTuF%fengguang.wu@intel.com> (raw)
Hi Alexandru,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.0-rc8 next-20190226]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Alexandru-Ardelean/dmaengine-axi-dmac-assign-copy_align-property/20190226-215236
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 8.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=8.2.0 make.cross ARCH=xtensa
All errors (new ones prefixed by >>):
drivers//dma/dma-axi-dmac.c: In function 'axi_dmac_probe':
>> drivers//dma/dma-axi-dmac.c:670:36: error: 'struct axi_dmac_chan' has no member named 'address_align_mask'; did you mean 'align_mask'?
dma_dev->copy_align = (dmac->chan.address_align_mask + 1);
^~~~~~~~~~~~~~~~~~
align_mask
vim +670 drivers//dma/dma-axi-dmac.c
606
607 static int axi_dmac_probe(struct platform_device *pdev)
608 {
609 struct device_node *of_channels, *of_chan;
610 struct dma_device *dma_dev;
611 struct axi_dmac *dmac;
612 struct resource *res;
613 int ret;
614
615 dmac = devm_kzalloc(&pdev->dev, sizeof(*dmac), GFP_KERNEL);
616 if (!dmac)
617 return -ENOMEM;
618
619 dmac->irq = platform_get_irq(pdev, 0);
620 if (dmac->irq < 0)
621 return dmac->irq;
622 if (dmac->irq == 0)
623 return -EINVAL;
624
625 res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
626 dmac->base = devm_ioremap_resource(&pdev->dev, res);
627 if (IS_ERR(dmac->base))
628 return PTR_ERR(dmac->base);
629
630 dmac->clk = devm_clk_get(&pdev->dev, NULL);
631 if (IS_ERR(dmac->clk))
632 return PTR_ERR(dmac->clk);
633
634 INIT_LIST_HEAD(&dmac->chan.active_descs);
635
636 of_channels = of_get_child_by_name(pdev->dev.of_node, "adi,channels");
637 if (of_channels == NULL)
638 return -ENODEV;
639
640 for_each_child_of_node(of_channels, of_chan) {
641 ret = axi_dmac_parse_chan_dt(of_chan, &dmac->chan);
642 if (ret) {
643 of_node_put(of_chan);
644 of_node_put(of_channels);
645 return -EINVAL;
646 }
647 }
648 of_node_put(of_channels);
649
650 pdev->dev.dma_parms = &dmac->dma_parms;
651 dma_set_max_seg_size(&pdev->dev, dmac->chan.max_length);
652
653 dma_dev = &dmac->dma_dev;
654 dma_cap_set(DMA_SLAVE, dma_dev->cap_mask);
655 dma_cap_set(DMA_CYCLIC, dma_dev->cap_mask);
656 dma_dev->device_free_chan_resources = axi_dmac_free_chan_resources;
657 dma_dev->device_tx_status = dma_cookie_status;
658 dma_dev->device_issue_pending = axi_dmac_issue_pending;
659 dma_dev->device_prep_slave_sg = axi_dmac_prep_slave_sg;
660 dma_dev->device_prep_dma_cyclic = axi_dmac_prep_dma_cyclic;
661 dma_dev->device_prep_interleaved_dma = axi_dmac_prep_interleaved;
662 dma_dev->device_terminate_all = axi_dmac_terminate_all;
663 dma_dev->device_synchronize = axi_dmac_synchronize;
664 dma_dev->dev = &pdev->dev;
665 dma_dev->chancnt = 1;
666 dma_dev->src_addr_widths = BIT(dmac->chan.src_width);
667 dma_dev->dst_addr_widths = BIT(dmac->chan.dest_width);
668 dma_dev->directions = BIT(dmac->chan.direction);
669 dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_DESCRIPTOR;
> 670 dma_dev->copy_align = (dmac->chan.address_align_mask + 1);
671 INIT_LIST_HEAD(&dma_dev->channels);
672
673 dmac->chan.vchan.desc_free = axi_dmac_desc_free;
674 vchan_init(&dmac->chan.vchan, dma_dev);
675
676 ret = clk_prepare_enable(dmac->clk);
677 if (ret < 0)
678 return ret;
679
680 axi_dmac_write(dmac, AXI_DMAC_REG_IRQ_MASK, 0x00);
681
682 ret = dma_async_device_register(dma_dev);
683 if (ret)
684 goto err_clk_disable;
685
686 ret = of_dma_controller_register(pdev->dev.of_node,
687 of_dma_xlate_by_chan_id, dma_dev);
688 if (ret)
689 goto err_unregister_device;
690
691 ret = request_irq(dmac->irq, axi_dmac_interrupt_handler, IRQF_SHARED,
692 dev_name(&pdev->dev), dmac);
693 if (ret)
694 goto err_unregister_of;
695
696 platform_set_drvdata(pdev, dmac);
697
698 return 0;
699
700 err_unregister_of:
701 of_dma_controller_free(pdev->dev.of_node);
702 err_unregister_device:
703 dma_async_device_unregister(&dmac->dma_dev);
704 err_clk_disable:
705 clk_disable_unprepare(dmac->clk);
706
707 return ret;
708 }
709
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
next reply other threads:[~2019-02-26 17:43 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-26 17:43 kbuild test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2019-02-27 10:55 [V2] dmaengine: axi-dmac: assign `copy_align` property Vinod Koul
2019-02-27 8:12 Ardelean, Alexandru
2019-02-26 8:52 Alexandru Ardelean
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201902270127.qKUjuTuF%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=alexandru.ardelean@analog.com \
--cc=dmaengine@vger.kernel.org \
--cc=kbuild-all@01.org \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox