This patch series fixes some trivial and rudimentary spellings in the COMMENT
sections.
Bhaskar Chowdhury (30):
acpi-dma.c: Fix couple of typos
altera-msgdma.c: Couple of typos fixed
amba-pl08x.c: Fixed couple of typos
bcm-sba-raid.c: Few typos fixed
bcm2835-dma.c: Fix a typo
idma64.c: Fix couple of typos
iop-adma.c: Few typos fixed
mv_xor.c: Fix a typo
mv_xor.h: Fixed a typo
mv_xor_v2.c: Fix a typo
nbpfaxi.c: Fixed a typo
of-dma.c: Fixed a typo
s3c24xx-dma.c: Fix a typo
Revert "s3c24xx-dma.c: Fix a typo"
s3c24xx-dma.c: Few typos fixed
st_fdma.h: Fix couple of typos
ste_dma40_ll.h: Fix a typo
tegra20-apb-dma.c: Fixed a typo
xgene-dma.c: Few spello fixes
at_hdmac.c: Quite a few spello fixes
owl-dma.c: Fix a typo
at_hdmac_regs.h: Couple of typo fixes
dma-jz4780.c: Fix a typo
Kconfig: Change Synopsys to Synopsis
ste_dma40.c: Few spello fixes
dw-axi-dmac-platform.c: Few typos fixed
dpaa2-qdma.c: Fix a typo
usb-dmac.c: Fix a typo
edma.c: Fix a typo
xilinx_dma.c: Fix a typo
drivers/dma/Kconfig | 8 ++++----
drivers/dma/acpi-dma.c | 4 ++--
drivers/dma/altera-msgdma.c | 4 ++--
drivers/dma/amba-pl08x.c | 4 ++--
drivers/dma/at_hdmac.c | 14 +++++++-------
drivers/dma/at_hdmac_regs.h | 4 ++--
drivers/dma/bcm-sba-raid.c | 8 ++++----
drivers/dma/bcm2835-dma.c | 2 +-
drivers/dma/dma-jz4780.c | 2 +-
drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c | 8 ++++----
drivers/dma/idma64.c | 4 ++--
drivers/dma/iop-adma.c | 6 +++---
drivers/dma/mv_xor.c | 2 +-
drivers/dma/mv_xor.h | 2 +-
drivers/dma/mv_xor_v2.c | 2 +-
drivers/dma/nbpfaxi.c | 2 +-
drivers/dma/of-dma.c | 2 +-
drivers/dma/owl-dma.c | 2 +-
drivers/dma/s3c24xx-dma.c | 6 +++---
drivers/dma/sh/shdmac.c | 2 +-
drivers/dma/sh/usb-dmac.c | 2 +-
drivers/dma/st_fdma.h | 4 ++--
drivers/dma/ste_dma40.c | 10 +++++-----
drivers/dma/ste_dma40_ll.h | 2 +-
drivers/dma/tegra20-apb-dma.c | 2 +-
drivers/dma/ti/edma.c | 2 +-
drivers/dma/xgene-dma.c | 6 +++---
drivers/dma/xilinx/xilinx_dma.c | 2 +-
28 files changed, 59 insertions(+), 59 deletions(-)
--
2.26.3
s/exctract/extract/
s/avaiable/available/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/acpi-dma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/acpi-dma.c b/drivers/dma/acpi-dma.c
index 235f1396f968..4b1d5183bf64 100644
--- a/drivers/dma/acpi-dma.c
+++ b/drivers/dma/acpi-dma.c
@@ -100,7 +100,7 @@ static int acpi_dma_parse_resource_group(const struct acpi_csrt_group *grp,
}
/**
- * acpi_dma_parse_csrt - parse CSRT to exctract additional DMA resources
+ * acpi_dma_parse_csrt - parse CSRT to extract additional DMA resources
* @adev: ACPI device to match with
* @adma: struct acpi_dma of the given DMA controller
*
@@ -293,7 +293,7 @@ EXPORT_SYMBOL_GPL(devm_acpi_dma_controller_free);
* found.
*
* Return:
- * 0, if no information is avaiable, -1 on mismatch, and 1 otherwise.
+ * 0, if no information is available, -1 on mismatch, and 1 otherwise.
*/
static int acpi_dma_update_dma_spec(struct acpi_dma *adma,
struct acpi_dma_spec *dma_spec)
--
2.26.3
s/assosiated/associated/
s/oder/order/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/altera-msgdma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/altera-msgdma.c b/drivers/dma/altera-msgdma.c
index 9a841ce5f0c5..c3483bf2aaa7 100644
--- a/drivers/dma/altera-msgdma.c
+++ b/drivers/dma/altera-msgdma.c
@@ -152,7 +152,7 @@ struct msgdma_extended_desc {
/**
* struct msgdma_sw_desc - implements a sw descriptor
* @async_tx: support for the async_tx api
- * @hw_desc: assosiated HW descriptor
+ * @hw_desc: associated HW descriptor
* @node: node to move from the free list to the tx list
* @tx_list: transmit list node
*/
@@ -510,7 +510,7 @@ static void msgdma_copy_one(struct msgdma_device *mdev,
* of the DMA controller. The descriptor will get flushed to the
* FIFO, once the last word (control word) is written. Since we
* are not 100% sure that memcpy() writes all word in the "correct"
- * oder (address from low to high) on all architectures, we make
+ * order (address from low to high) on all architectures, we make
* sure this control word is written last by single coding it and
* adding some write-barriers here.
*/
--
2.26.3
s/Maibox/Mailbox/
s/alloced/allocated/
s/atleast/"at least"/
s/parallely/parallel/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/bcm-sba-raid.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c
index 64239da02e74..fba1585eb7ad 100644
--- a/drivers/dma/bcm-sba-raid.c
+++ b/drivers/dma/bcm-sba-raid.c
@@ -25,7 +25,7 @@
* number of hardware rings over one or more SBA hardware devices. By
* design, the internal buffer size of SBA hardware device is limited
* but all offload operations supported by SBA can be broken down into
- * multiple small size requests and executed parallely on multiple SBA
+ * multiple small size requests and executed parallel on multiple SBA
* hardware devices for achieving high through-put.
*
* The Broadcom SBA RAID driver does not require any register programming
@@ -143,7 +143,7 @@ struct sba_device {
u32 max_xor_srcs;
u32 max_resp_pool_size;
u32 max_cmds_pool_size;
- /* Maibox client and Mailbox channels */
+ /* Mailbox client and Mailbox channels */
struct mbox_client client;
struct mbox_chan *mchan;
struct device *mbox_dev;
@@ -328,7 +328,7 @@ static void sba_cleanup_nonpending_requests(struct sba_device *sba)
spin_lock_irqsave(&sba->reqs_lock, flags);
- /* Freeup all alloced request */
+ /* Freeup all allocated request */
list_for_each_entry_safe(req, req1, &sba->reqs_alloc_list, node)
_sba_free_request(sba, req);
@@ -1633,7 +1633,7 @@ static int sba_probe(struct platform_device *pdev)
sba->dev = &pdev->dev;
platform_set_drvdata(pdev, sba);
- /* Number of mailbox channels should be atleast 1 */
+ /* Number of mailbox channels should be at least 1 */
ret = of_count_phandle_with_args(pdev->dev.of_node,
"mboxes", "#mbox-cells");
if (ret <= 0)
--
2.26.3
s/busses/buses/
s/accound/account/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/amba-pl08x.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
index a24882ba3764..d15182634789 100644
--- a/drivers/dma/amba-pl08x.c
+++ b/drivers/dma/amba-pl08x.c
@@ -121,7 +121,7 @@ struct vendor_data {
/**
* struct pl08x_bus_data - information of source or destination
- * busses for a transfer
+ * buses for a transfer
* @addr: current address
* @maxwidth: the maximum width of a transfer on this bus
* @buswidth: the width of this bus in bytes: 1, 2 or 4
@@ -1010,7 +1010,7 @@ static inline u32 pl08x_lli_control_bits(struct pl08x_driver_data *pl08x,
/*
* Remove all src, dst and transfer size bits, then set the
* width and size according to the parameters. The bit offsets
- * are different in the FTDMAC020 so we need to accound for this.
+ * are different in the FTDMAC020 so we need to account for this.
*/
if (pl08x->vd->ftdmac020) {
retbits &= ~FTDMAC020_LLI_DST_WIDTH_MSK;
--
2.26.3
s/missmatch/mismatch/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/bcm2835-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/bcm2835-dma.c b/drivers/dma/bcm2835-dma.c
index 630dfbb01a40..e6baf17b0434 100644
--- a/drivers/dma/bcm2835-dma.c
+++ b/drivers/dma/bcm2835-dma.c
@@ -369,7 +369,7 @@ static struct bcm2835_desc *bcm2835_dma_create_cb_chain(
/* the last frame requires extra flags */
d->cb_list[d->frames - 1].cb->info |= finalextrainfo;
- /* detect a size missmatch */
+ /* detect a size mismatch */
if (buf_len && (d->size != buf_len))
goto error_cb;
--
2.26.3
s/transfered/transferred/ ...two different places
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/idma64.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/idma64.c b/drivers/dma/idma64.c
index f4c07ad3be15..3d0d48b12e28 100644
--- a/drivers/dma/idma64.c
+++ b/drivers/dma/idma64.c
@@ -282,7 +282,7 @@ static void idma64_desc_fill(struct idma64_chan *idma64c,
desc->length += hw->len;
} while (i);
- /* Trigger an interrupt after the last block is transfered */
+ /* Trigger an interrupt after the last block is transferred */
lli->ctllo |= IDMA64C_CTLL_INT_EN;
/* Disable LLP transfer in the last block */
@@ -356,7 +356,7 @@ static size_t idma64_active_desc_size(struct idma64_chan *idma64c)
if (!i)
return bytes;
- /* The current chunk is not fully transfered yet */
+ /* The current chunk is not fully transferred yet */
bytes += desc->hw[--i].len;
return bytes - IDMA64C_CTLH_BLOCK_TS(ctlhi);
--
2.26.3
s/capabilites/capabilities/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/mv_xor.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c
index 23b232b57518..a43388b6a30d 100644
--- a/drivers/dma/mv_xor.c
+++ b/drivers/dma/mv_xor.c
@@ -1074,7 +1074,7 @@ mv_xor_channel_add(struct mv_xor_device *xordev,
if (!mv_chan->dma_desc_pool_virt)
return ERR_PTR(-ENOMEM);
- /* discover transaction capabilites from the platform data */
+ /* discover transaction capabilities from the platform data */
dma_dev->cap_mask = cap_mask;
INIT_LIST_HEAD(&dma_dev->channels);
--
2.26.3
s/asynchrounous/asynchronous/
s/depedency/dependency/
s/capabilites/capabilities/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/iop-adma.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/dma/iop-adma.c b/drivers/dma/iop-adma.c
index 310b899d581f..81f32dc964e2 100644
--- a/drivers/dma/iop-adma.c
+++ b/drivers/dma/iop-adma.c
@@ -5,7 +5,7 @@
*/
/*
- * This driver supports the asynchrounous DMA copy and RAID engines available
+ * This driver supports the asynchronous DMA copy and RAID engines available
* on the Intel Xscale(R) family of I/O Processors (IOP 32x, 33x, 134x)
*/
@@ -243,7 +243,7 @@ static void iop_adma_tasklet(struct tasklet_struct *t)
struct iop_adma_chan *iop_chan = from_tasklet(iop_chan, t,
irq_tasklet);
- /* lockdep will flag depedency submissions as potentially
+ /* lockdep will flag dependency submissions as potentially
* recursive locking, this is not the case as a dependency
* submission will never recurse a channels submit routine.
* There are checks in async_tx.c to prevent this.
@@ -1302,7 +1302,7 @@ static int iop_adma_probe(struct platform_device *pdev)
adev->id = plat_data->hw_id;
- /* discover transaction capabilites from the platform data */
+ /* discover transaction capabilities from the platform data */
dma_dev->cap_mask = plat_data->cap_mask;
adev->pdev = pdev;
--
2.26.3
s/writen/written/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/mv_xor.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/mv_xor.h b/drivers/dma/mv_xor.h
index d86086b05b0e..c87cefd38a07 100644
--- a/drivers/dma/mv_xor.h
+++ b/drivers/dma/mv_xor.h
@@ -99,7 +99,7 @@ struct mv_xor_device {
* @common: common dmaengine channel object members
* @slots_allocated: records the actual size of the descriptor slot pool
* @irq_tasklet: bottom half where mv_xor_slot_cleanup runs
- * @op_in_desc: new mode of driver, each op is writen to descriptor.
+ * @op_in_desc: new mode of driver, each op is written to descriptor.
*/
struct mv_xor_chan {
int pending;
--
2.26.3
s/assosiated/associated/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/mv_xor_v2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/mv_xor_v2.c b/drivers/dma/mv_xor_v2.c
index 9b0d463f89bb..8a45861299a0 100644
--- a/drivers/dma/mv_xor_v2.c
+++ b/drivers/dma/mv_xor_v2.c
@@ -175,7 +175,7 @@ struct mv_xor_v2_device {
* struct mv_xor_v2_sw_desc - implements a xor SW descriptor
* @idx: descriptor index
* @async_tx: support for the async_tx api
- * @hw_desc: assosiated HW descriptor
+ * @hw_desc: associated HW descriptor
* @free_list: node of the free SW descriprots list
*/
struct mv_xor_v2_sw_desc {
--
2.26.3
s/propety/properly/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/of-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/of-dma.c b/drivers/dma/of-dma.c
index ec00b20ae8e4..e028acff7fe8 100644
--- a/drivers/dma/of-dma.c
+++ b/drivers/dma/of-dma.c
@@ -337,7 +337,7 @@ EXPORT_SYMBOL_GPL(of_dma_simple_xlate);
*
* This function can be used as the of xlate callback for DMA driver which wants
* to match the channel based on the channel id. When using this xlate function
- * the #dma-cells propety of the DMA controller dt node needs to be set to 1.
+ * the #dma-cells properly of the DMA controller dt node needs to be set to 1.
* The data parameter of of_dma_controller_register must be a pointer to the
* dma_device struct the function should match upon.
*
--
2.26.3
s/transfered/transferred/
s/desintation/destination/ ...two different places.
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/s3c24xx-dma.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c
index 8e14c72d03f0..045f2f531876 100644
--- a/drivers/dma/s3c24xx-dma.c
+++ b/drivers/dma/s3c24xx-dma.c
@@ -156,7 +156,7 @@ struct s3c24xx_sg {
* struct s3c24xx_txd - wrapper for struct dma_async_tx_descriptor
* @vd: virtual DMA descriptor
* @dsg_list: list of children sg's
- * @at: sg currently being transfered
+ * @at: sg currently being transferred
* @width: transfer width
* @disrcc: value for source control register
* @didstc: value for destination control register
@@ -920,7 +920,7 @@ static struct dma_async_tx_descriptor *s3c24xx_dma_prep_dma_cyclic(
}
/*
- * Always assume our peripheral desintation is a fixed
+ * Always assume our peripheral destination is a fixed
* address in memory.
*/
hwcfg |= S3C24XX_DISRCC_INC_FIXED;
@@ -1009,7 +1009,7 @@ static struct dma_async_tx_descriptor *s3c24xx_dma_prep_slave_sg(
}
/*
- * Always assume our peripheral desintation is a fixed
+ * Always assume our peripheral destination is a fixed
* address in memory.
*/
hwcfg |= S3C24XX_DISRCC_INC_FIXED;
--
2.26.3
s/superflous/superfluous/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/nbpfaxi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/nbpfaxi.c b/drivers/dma/nbpfaxi.c
index 9c52c57919c6..9eacaa20ddb3 100644
--- a/drivers/dma/nbpfaxi.c
+++ b/drivers/dma/nbpfaxi.c
@@ -898,7 +898,7 @@ static int nbpf_config(struct dma_chan *dchan,
/*
* We could check config->slave_id to match chan->terminal here,
* but with DT they would be coming from the same source, so
- * such a check would be superflous
+ * such a check would be superfluous
*/
chan->slave_dst_addr = config->dst_addr;
--
2.26.3
s/transfered/transferred/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/s3c24xx-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c
index 8e14c72d03f0..c98ae73cdcc3 100644
--- a/drivers/dma/s3c24xx-dma.c
+++ b/drivers/dma/s3c24xx-dma.c
@@ -156,7 +156,7 @@ struct s3c24xx_sg {
* struct s3c24xx_txd - wrapper for struct dma_async_tx_descriptor
* @vd: virtual DMA descriptor
* @dsg_list: list of children sg's
- * @at: sg currently being transfered
+ * @at: sg currently being transferred
* @width: transfer width
* @disrcc: value for source control register
* @didstc: value for destination control register
--
2.26.3
s/transferred/transfered/
This reverts commit a2ddb8aea8106bd5552f8516ad7a8a26b9282a8f.
---
drivers/dma/s3c24xx-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/s3c24xx-dma.c b/drivers/dma/s3c24xx-dma.c
index c98ae73cdcc3..8e14c72d03f0 100644
--- a/drivers/dma/s3c24xx-dma.c
+++ b/drivers/dma/s3c24xx-dma.c
@@ -156,7 +156,7 @@ struct s3c24xx_sg {
* struct s3c24xx_txd - wrapper for struct dma_async_tx_descriptor
* @vd: virtual DMA descriptor
* @dsg_list: list of children sg's
- * @at: sg currently being transferred
+ * @at: sg currently being transfered
* @width: transfer width
* @disrcc: value for source control register
* @didstc: value for destination control register
--
2.26.3
s/transfert/transfer/ ...two different places.
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/st_fdma.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/st_fdma.h b/drivers/dma/st_fdma.h
index fa15b97a3bab..702b21021124 100644
--- a/drivers/dma/st_fdma.h
+++ b/drivers/dma/st_fdma.h
@@ -41,8 +41,8 @@ struct st_fdma_generic_node {
* @saddr: Source address
* @daddr: Destination address
*
- * @generic: generic node for free running/paced transfert type
- * 2 others transfert type are possible, but not yet implemented
+ * @generic: generic node for free running/paced transfer type
+ * 2 others transfer type are possible, but not yet implemented
*
* The NODE structures must be aligned to a 32 byte boundary
*/
--
2.26.3
s/directy/directly/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/ste_dma40_ll.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/ste_dma40_ll.h b/drivers/dma/ste_dma40_ll.h
index c504e855eb02..2e30e9a94a1e 100644
--- a/drivers/dma/ste_dma40_ll.h
+++ b/drivers/dma/ste_dma40_ll.h
@@ -369,7 +369,7 @@ struct d40_phy_lli_bidir {
* @lcsp02: Either maps to register lcsp0 if src or lcsp2 if dst.
* @lcsp13: Either maps to register lcsp1 if src or lcsp3 if dst.
*
- * This struct must be 8 bytes aligned since it will be accessed directy by
+ * This struct must be 8 bytes aligned since it will be accessed directly by
* the DMA. Never add any none hw mapped registers to this struct.
*/
--
2.26.3
s/programing/programming/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/tegra20-apb-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c
index 71827d9b0aa1..e64789432587 100644
--- a/drivers/dma/tegra20-apb-dma.c
+++ b/drivers/dma/tegra20-apb-dma.c
@@ -475,7 +475,7 @@ static void tegra_dma_configure_for_next(struct tegra_dma_channel *tdc,
/*
* If interrupt is pending then do nothing as the ISR will handle
- * the programing for new request.
+ * the programming for new request.
*/
if (status & TEGRA_APBDMA_STATUS_ISE_EOC) {
dev_err(tdc2dev(tdc),
--
2.26.3
s/tranfers/transfers/
s/maxium/maximum/
s/previouly/previously/
s/broked/broken/
s/embedds/embeds/ .... three different places.
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/at_hdmac.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/dma/at_hdmac.c b/drivers/dma/at_hdmac.c
index 30ae36124b1d..02c09532a9e4 100644
--- a/drivers/dma/at_hdmac.c
+++ b/drivers/dma/at_hdmac.c
@@ -374,11 +374,11 @@ static int atc_get_bytes_left(struct dma_chan *chan, dma_cookie_t cookie)
* a second time. If the two consecutive read values of the DSCR
* are the same then we assume both refers to the very same
* child descriptor as well as the CTRLA value read inbetween
- * does. For cyclic tranfers, the assumption is that a full loop
+ * does. For cyclic transfers, the assumption is that a full loop
* is "not so fast".
* If the two DSCR values are different, we read again the CTRLA
* then the DSCR till two consecutive read values from DSCR are
- * equal or till the maxium trials is reach.
+ * equal or till the maximum trials is reach.
* This algorithm is very unlikely not to find a stable value for
* DSCR.
*/
@@ -403,7 +403,7 @@ static int atc_get_bytes_left(struct dma_chan *chan, dma_cookie_t cookie)
/*
* DSCR has changed inside the DMA controller, so the
- * previouly read value of CTRLA may refer to an already
+ * previously read value of CTRLA may refer to an already
* processed descriptor hence could be outdated.
* We need to update ctrla to match the current
* descriptor.
@@ -564,7 +564,7 @@ static void atc_handle_error(struct at_dma_chan *atchan)
spin_lock_irqsave(&atchan->lock, flags);
/*
* The descriptor currently at the head of the active list is
- * broked. Since we don't have any way to report errors, we'll
+ * broken. Since we don't have any way to report errors, we'll
* just have to scream loudly and try to carry on.
*/
bad_desc = atc_first_active(atchan);
@@ -870,7 +870,7 @@ atc_prep_dma_memcpy(struct dma_chan *chan, dma_addr_t dest, dma_addr_t src,
atc_desc_chain(&first, &prev, desc);
}
- /* First descriptor of the chain embedds additional information */
+ /* First descriptor of the chain embeds additional information */
first->txd.cookie = -EBUSY;
first->total_len = len;
@@ -1199,7 +1199,7 @@ atc_prep_slave_sg(struct dma_chan *chan, struct scatterlist *sgl,
/* set end-of-link to the last link descriptor of list*/
set_desc_eol(prev);
- /* First descriptor of the chain embedds additional information */
+ /* First descriptor of the chain embeds additional information */
first->txd.cookie = -EBUSY;
first->total_len = total_len;
@@ -1358,7 +1358,7 @@ atc_prep_dma_cyclic(struct dma_chan *chan, dma_addr_t buf_addr, size_t buf_len,
/* lets make a cyclic list */
prev->lli.dscr = first->txd.phys;
- /* First descriptor of the chain embedds additional information */
+ /* First descriptor of the chain embeds additional information */
first->txd.cookie = -EBUSY;
first->total_len = buf_len;
--
2.26.3
s/definations/definitions/ ....two different places.
s/Configue/Configure/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/xgene-dma.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/dma/xgene-dma.c b/drivers/dma/xgene-dma.c
index 3589b4ef50b8..9b0010b6e033 100644
--- a/drivers/dma/xgene-dma.c
+++ b/drivers/dma/xgene-dma.c
@@ -23,7 +23,7 @@
#include "dmaengine.h"
-/* X-Gene DMA ring csr registers and bit definations */
+/* X-Gene DMA ring csr registers and bit definitions */
#define XGENE_DMA_RING_CONFIG 0x04
#define XGENE_DMA_RING_ENABLE BIT(31)
#define XGENE_DMA_RING_ID 0x08
@@ -102,7 +102,7 @@
#define XGENE_DMA_BLK_MEM_RDY_VAL 0xFFFFFFFF
#define XGENE_DMA_RING_CMD_SM_OFFSET 0x8000
-/* X-Gene SoC EFUSE csr register and bit defination */
+/* X-Gene SoC EFUSE csr register and bit definition */
#define XGENE_SOC_JTAG1_SHADOW 0x18
#define XGENE_DMA_PQ_DISABLE_MASK BIT(13)
@@ -1741,7 +1741,7 @@ static int xgene_dma_probe(struct platform_device *pdev)
/* Initialize DMA channels software state */
xgene_dma_init_channels(pdma);
- /* Configue DMA rings */
+ /* Configure DMA rings */
ret = xgene_dma_init_rings(pdma);
if (ret)
goto err_clk_enable;
--
2.26.3
s/availlable/available/
s/signle/single/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/at_hdmac_regs.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/at_hdmac_regs.h b/drivers/dma/at_hdmac_regs.h
index 4d1ebc040031..46ecc40edaa8 100644
--- a/drivers/dma/at_hdmac_regs.h
+++ b/drivers/dma/at_hdmac_regs.h
@@ -338,7 +338,7 @@ static inline u8 convert_buswidth(enum dma_slave_buswidth addr_width)
* @ch_regs: memory mapped register base
* @clk: dma controller clock
* @save_imr: interrupt mask register that is saved on suspend/resume cycle
- * @all_chan_mask: all channels availlable in a mask
+ * @all_chan_mask: all channels available in a mask
* @dma_desc_pool: base of DMA descriptor region (DMA address)
* @chan: channels table to store at_dma_chan structures
*/
@@ -462,7 +462,7 @@ static inline int atc_chan_is_cyclic(struct at_dma_chan *atchan)
/**
* set_desc_eol - set end-of-link to descriptor so it will end transfer
- * @desc: descriptor, signle or at the end of a chain, to end chain on
+ * @desc: descriptor, single or at the end of a chain, to end chain on
*/
static void set_desc_eol(struct at_desc *desc)
{
--
2.26.3
s/proceeed/proceed/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/dma-jz4780.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/dma-jz4780.c b/drivers/dma/dma-jz4780.c
index ebee94dbd630..451bc754b609 100644
--- a/drivers/dma/dma-jz4780.c
+++ b/drivers/dma/dma-jz4780.c
@@ -379,7 +379,7 @@ static struct dma_async_tx_descriptor *jz4780_dma_prep_slave_sg(
if (i != (sg_len - 1) &&
!(jzdma->soc_data->flags & JZ_SOC_DATA_BREAK_LINKS)) {
- /* Automatically proceeed to the next descriptor. */
+ /* Automatically proceed to the next descriptor. */
desc->desc[i].dcm |= JZ_DMA_DCM_LINK;
/*
--
2.26.3
s/Eventhough/"Even though"/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/owl-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/owl-dma.c b/drivers/dma/owl-dma.c
index 1f0bbaed4643..3c7283afa443 100644
--- a/drivers/dma/owl-dma.c
+++ b/drivers/dma/owl-dma.c
@@ -1155,7 +1155,7 @@ static int owl_dma_probe(struct platform_device *pdev)
}
/*
- * Eventhough the DMA controller is capable of generating 4
+ * Even though the DMA controller is capable of generating 4
* IRQ's for DMA priority feature, we only use 1 IRQ for
* simplification.
*/
--
2.26.3
s/Synopsys/Synopsis/ .....two different places.
..and for some unknown reason it introduce a empty line deleted and added
back.
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
index 0c2827fd8c19..30e8cc26f43b 100644
--- a/drivers/dma/Kconfig
+++ b/drivers/dma/Kconfig
@@ -170,15 +170,15 @@ config DMA_SUN6I
Support for the DMA engine first found in Allwinner A31 SoCs.
config DW_AXI_DMAC
- tristate "Synopsys DesignWare AXI DMA support"
+ tristate "Synopsis DesignWare AXI DMA support"
depends on OF || COMPILE_TEST
depends on HAS_IOMEM
select DMA_ENGINE
select DMA_VIRTUAL_CHANNELS
help
- Enable support for Synopsys DesignWare AXI DMA controller.
+ Enable support for Synopsis DesignWare AXI DMA controller.
NOTE: This driver wasn't tested on 64 bit platform because
- of lack 64 bit platform with Synopsys DW AXI DMAC.
+ of lack 64 bit platform with Synopsis DW AXI DMAC.
config EP93XX_DMA
bool "Cirrus Logic EP93xx DMA support"
@@ -394,7 +394,7 @@ config MOXART_DMA
select DMA_VIRTUAL_CHANNELS
help
Enable support for the MOXA ART SoC DMA controller.
-
+
Say Y here if you enabled MMP ADMA, otherwise say N.
config MPC512X_DMA
--
2.26.3
s/ealier/earlier/
s/orignal/original/
s/manouver/maneuver/
s/transfer/transfer/
s/succees/success/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/ste_dma40.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c
index 265d7c07b348..39fa2fb74057 100644
--- a/drivers/dma/ste_dma40.c
+++ b/drivers/dma/ste_dma40.c
@@ -155,7 +155,7 @@ static __maybe_unused u32 d40_backup_regs[] = {
/*
* since 9540 and 8540 has the same HW revision
- * use v4a for 9540 or ealier
+ * use v4a for 9540 or earlier
* use v4b for 8540 or later
* HW revision:
* DB8500ed has revision 0
@@ -382,7 +382,7 @@ struct d40_desc {
*
* @base: The virtual address of LCLA. 18 bit aligned.
* @dma_addr: DMA address, if mapped
- * @base_unaligned: The orignal kmalloc pointer, if kmalloc is used.
+ * @base_unaligned: The original kmalloc pointer, if kmalloc is used.
* This pointer is only there for clean-up on error.
* @pages: The number of pages needed for all physical channels.
* Only used later for clean-up on error
@@ -1630,7 +1630,7 @@ static void dma_tasklet(struct tasklet_struct *t)
return;
check_pending_tx:
- /* Rescue manouver if receiving double interrupts */
+ /* Rescue maneuver if receiving double interrupts */
if (d40c->pending_tx > 0)
d40c->pending_tx--;
spin_unlock_irqrestore(&d40c->lock, flags);
@@ -1970,7 +1970,7 @@ static int d40_config_memcpy(struct d40_chan *d40c)
dma_has_cap(DMA_SLAVE, cap)) {
d40c->dma_cfg = dma40_memcpy_conf_phy;
- /* Generate interrrupt at end of transfer or relink. */
+ /* Generate interrupt at end of transfer or relink. */
d40c->dst_def_cfg |= BIT(D40_SREG_CFG_TIM_POS);
/* Generate interrupt on error. */
@@ -3415,7 +3415,7 @@ static int __init d40_lcla_allocate(struct d40_base *base)
base->lcla_pool.base = (void *)page_list[i];
} else {
/*
- * After many attempts and no succees with finding the correct
+ * After many attempts and no success with finding the correct
* alignment, try with allocating a big buffer.
*/
dev_warn(base->dev,
--
2.26.3
s/contoller/controller/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/sh/shdmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c
index 5aafe548ca5f..7b51b15b45b1 100644
--- a/drivers/dma/sh/shdmac.c
+++ b/drivers/dma/sh/shdmac.c
@@ -319,7 +319,7 @@ static void sh_dmae_setup_xfer(struct shdma_chan *schan,
}
/*
- * Find a slave channel configuration from the contoller list by either a slave
+ * Find a slave channel configuration from the controller list by either a slave
* ID in the non-DT case, or by a MID/RID value in the DT case
*/
static const struct sh_dmae_slave_config *dmae_find_slave(
--
2.26.3
s/configurarion/configuration/
s/inerrupts/interrupts/
s/chanels/channels/ ....two different places.
s/Synopsys/Synopsis/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c b/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c
index d9e4ac3edb4e..ef4da10361a7 100644
--- a/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c
+++ b/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c
@@ -35,7 +35,7 @@
/*
* The set of bus widths supported by the DMA controller. DW AXI DMAC supports
* master data bus width up to 512 bits (for both AXI master interfaces), but
- * it depends on IP block configurarion.
+ * it depends on IP block configuration.
*/
#define AXI_DMA_BUSWIDTHS \
(DMA_SLAVE_BUSWIDTH_1_BYTE | \
@@ -1056,10 +1056,10 @@ static irqreturn_t dw_axi_dma_interrupt(int irq, void *dev_id)
u32 status, i;
- /* Disable DMAC inerrupts. We'll enable them after processing chanels */
+ /* Disable DMAC interrupts. We'll enable them after processing channels */
axi_dma_irq_disable(chip);
- /* Poll, clear and process every chanel interrupt status */
+ /* Poll, clear and process every channel interrupt status */
for (i = 0; i < dw->hdata->nr_channels; i++) {
chan = &dw->chan[i];
status = axi_chan_irq_read(chan);
@@ -1511,5 +1511,5 @@ static struct platform_driver dw_driver = {
module_platform_driver(dw_driver);
MODULE_LICENSE("GPL v2");
-MODULE_DESCRIPTION("Synopsys DesignWare AXI DMA Controller platform driver");
+MODULE_DESCRIPTION("Synopsis DesignWare AXI DMA Controller platform driver");
MODULE_AUTHOR("Eugeniy Paltsev <[email protected]>");
--
2.26.3
s/descritpor/descriptor/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/sh/usb-dmac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/sh/usb-dmac.c b/drivers/dma/sh/usb-dmac.c
index 8f7ceb698226..7092a657932b 100644
--- a/drivers/dma/sh/usb-dmac.c
+++ b/drivers/dma/sh/usb-dmac.c
@@ -301,7 +301,7 @@ static struct usb_dmac_desc *usb_dmac_desc_get(struct usb_dmac_chan *chan,
struct usb_dmac_desc *desc = NULL;
unsigned long flags;
- /* Get a freed descritpor */
+ /* Get a freed descriptor */
spin_lock_irqsave(&chan->vc.lock, flags);
list_for_each_entry(desc, &chan->desc_freed, node) {
if (sg_len <= desc->sg_allocated_len) {
--
2.26.3
s/interace/interface/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/xilinx/xilinx_dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/xilinx/xilinx_dma.c b/drivers/dma/xilinx/xilinx_dma.c
index 3aded7861fef..63d2f447ad79 100644
--- a/drivers/dma/xilinx/xilinx_dma.c
+++ b/drivers/dma/xilinx/xilinx_dma.c
@@ -483,7 +483,7 @@ struct xilinx_dma_config {
* @ext_addr: Indicates 64 bit addressing is supported by dma device
* @pdev: Platform device structure pointer
* @dma_config: DMA config structure
- * @axi_clk: DMA Axi4-lite interace clock
+ * @axi_clk: DMA Axi4-lite interface clock
* @tx_clk: DMA mm2s clock
* @txs_clk: DMA mm2s stream clock
* @rx_clk: DMA s2mm clock
--
2.26.3
s/transfering/transferring/
Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
drivers/dma/ti/edma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/ti/edma.c b/drivers/dma/ti/edma.c
index 35d81bd857f1..5ad771e34457 100644
--- a/drivers/dma/ti/edma.c
+++ b/drivers/dma/ti/edma.c
@@ -1815,7 +1815,7 @@ static void edma_issue_pending(struct dma_chan *chan)
* This limit exists to avoid a possible infinite loop when waiting for proof
* that a particular transfer is completed. This limit can be hit if there
* are large bursts to/from slow devices or the CPU is never able to catch
- * the DMA hardware idle. On an AM335x transfering 48 bytes from the UART
+ * the DMA hardware idle. On an AM335x transferring 48 bytes from the UART
* RX-FIFO, as many as 55 loops have been seen.
*/
#define EDMA_MAX_TR_WAIT_LOOPS 1000
--
2.26.3
I really don't think these typo patchbomb are that useful. I'm all
for fixing typos when working with a subsystem, but I'm not sure these
patchbombs help anything.
On Mon, Mar 29, 2021 at 05:22:56AM +0530, Bhaskar Chowdhury wrote:
> This patch series fixes some trivial and rudimentary spellings in the COMMENT
> sections.
>
> Bhaskar Chowdhury (30):
> acpi-dma.c: Fix couple of typos
> altera-msgdma.c: Couple of typos fixed
> amba-pl08x.c: Fixed couple of typos
> bcm-sba-raid.c: Few typos fixed
> bcm2835-dma.c: Fix a typo
> idma64.c: Fix couple of typos
> iop-adma.c: Few typos fixed
> mv_xor.c: Fix a typo
> mv_xor.h: Fixed a typo
> mv_xor_v2.c: Fix a typo
> nbpfaxi.c: Fixed a typo
> of-dma.c: Fixed a typo
> s3c24xx-dma.c: Fix a typo
> Revert "s3c24xx-dma.c: Fix a typo"
> s3c24xx-dma.c: Few typos fixed
> st_fdma.h: Fix couple of typos
> ste_dma40_ll.h: Fix a typo
> tegra20-apb-dma.c: Fixed a typo
> xgene-dma.c: Few spello fixes
> at_hdmac.c: Quite a few spello fixes
> owl-dma.c: Fix a typo
> at_hdmac_regs.h: Couple of typo fixes
> dma-jz4780.c: Fix a typo
> Kconfig: Change Synopsys to Synopsis
> ste_dma40.c: Few spello fixes
> dw-axi-dmac-platform.c: Few typos fixed
> dpaa2-qdma.c: Fix a typo
> usb-dmac.c: Fix a typo
> edma.c: Fix a typo
> xilinx_dma.c: Fix a typo
>
> drivers/dma/Kconfig | 8 ++++----
> drivers/dma/acpi-dma.c | 4 ++--
> drivers/dma/altera-msgdma.c | 4 ++--
> drivers/dma/amba-pl08x.c | 4 ++--
> drivers/dma/at_hdmac.c | 14 +++++++-------
> drivers/dma/at_hdmac_regs.h | 4 ++--
> drivers/dma/bcm-sba-raid.c | 8 ++++----
> drivers/dma/bcm2835-dma.c | 2 +-
> drivers/dma/dma-jz4780.c | 2 +-
> drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c | 8 ++++----
> drivers/dma/idma64.c | 4 ++--
> drivers/dma/iop-adma.c | 6 +++---
> drivers/dma/mv_xor.c | 2 +-
> drivers/dma/mv_xor.h | 2 +-
> drivers/dma/mv_xor_v2.c | 2 +-
> drivers/dma/nbpfaxi.c | 2 +-
> drivers/dma/of-dma.c | 2 +-
> drivers/dma/owl-dma.c | 2 +-
> drivers/dma/s3c24xx-dma.c | 6 +++---
> drivers/dma/sh/shdmac.c | 2 +-
> drivers/dma/sh/usb-dmac.c | 2 +-
> drivers/dma/st_fdma.h | 4 ++--
> drivers/dma/ste_dma40.c | 10 +++++-----
> drivers/dma/ste_dma40_ll.h | 2 +-
> drivers/dma/tegra20-apb-dma.c | 2 +-
> drivers/dma/ti/edma.c | 2 +-
> drivers/dma/xgene-dma.c | 6 +++---
> drivers/dma/xilinx/xilinx_dma.c | 2 +-
> 28 files changed, 59 insertions(+), 59 deletions(-)
>
> --
> 2.26.3
---end quoted text---
On 07:29 Mon 29 Mar 2021, Christoph Hellwig wrote:
>I really don't think these typo patchbomb are that useful. I'm all
>for fixing typos when working with a subsystem, but I'm not sure these
>patchbombs help anything.
>
I am sure you are holding the wrong end of the wand and grossly failing to
understand.
Anyway, I hope I give a heads up ...find "your way" to fix those damn
thing...it's glaring....
>On Mon, Mar 29, 2021 at 05:22:56AM +0530, Bhaskar Chowdhury wrote:
>> This patch series fixes some trivial and rudimentary spellings in the COMMENT
>> sections.
>>
>> Bhaskar Chowdhury (30):
>> acpi-dma.c: Fix couple of typos
>> altera-msgdma.c: Couple of typos fixed
>> amba-pl08x.c: Fixed couple of typos
>> bcm-sba-raid.c: Few typos fixed
>> bcm2835-dma.c: Fix a typo
>> idma64.c: Fix couple of typos
>> iop-adma.c: Few typos fixed
>> mv_xor.c: Fix a typo
>> mv_xor.h: Fixed a typo
>> mv_xor_v2.c: Fix a typo
>> nbpfaxi.c: Fixed a typo
>> of-dma.c: Fixed a typo
>> s3c24xx-dma.c: Fix a typo
>> Revert "s3c24xx-dma.c: Fix a typo"
>> s3c24xx-dma.c: Few typos fixed
>> st_fdma.h: Fix couple of typos
>> ste_dma40_ll.h: Fix a typo
>> tegra20-apb-dma.c: Fixed a typo
>> xgene-dma.c: Few spello fixes
>> at_hdmac.c: Quite a few spello fixes
>> owl-dma.c: Fix a typo
>> at_hdmac_regs.h: Couple of typo fixes
>> dma-jz4780.c: Fix a typo
>> Kconfig: Change Synopsys to Synopsis
>> ste_dma40.c: Few spello fixes
>> dw-axi-dmac-platform.c: Few typos fixed
>> dpaa2-qdma.c: Fix a typo
>> usb-dmac.c: Fix a typo
>> edma.c: Fix a typo
>> xilinx_dma.c: Fix a typo
>>
>> drivers/dma/Kconfig | 8 ++++----
>> drivers/dma/acpi-dma.c | 4 ++--
>> drivers/dma/altera-msgdma.c | 4 ++--
>> drivers/dma/amba-pl08x.c | 4 ++--
>> drivers/dma/at_hdmac.c | 14 +++++++-------
>> drivers/dma/at_hdmac_regs.h | 4 ++--
>> drivers/dma/bcm-sba-raid.c | 8 ++++----
>> drivers/dma/bcm2835-dma.c | 2 +-
>> drivers/dma/dma-jz4780.c | 2 +-
>> drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c | 8 ++++----
>> drivers/dma/idma64.c | 4 ++--
>> drivers/dma/iop-adma.c | 6 +++---
>> drivers/dma/mv_xor.c | 2 +-
>> drivers/dma/mv_xor.h | 2 +-
>> drivers/dma/mv_xor_v2.c | 2 +-
>> drivers/dma/nbpfaxi.c | 2 +-
>> drivers/dma/of-dma.c | 2 +-
>> drivers/dma/owl-dma.c | 2 +-
>> drivers/dma/s3c24xx-dma.c | 6 +++---
>> drivers/dma/sh/shdmac.c | 2 +-
>> drivers/dma/sh/usb-dmac.c | 2 +-
>> drivers/dma/st_fdma.h | 4 ++--
>> drivers/dma/ste_dma40.c | 10 +++++-----
>> drivers/dma/ste_dma40_ll.h | 2 +-
>> drivers/dma/tegra20-apb-dma.c | 2 +-
>> drivers/dma/ti/edma.c | 2 +-
>> drivers/dma/xgene-dma.c | 6 +++---
>> drivers/dma/xilinx/xilinx_dma.c | 2 +-
>> 28 files changed, 59 insertions(+), 59 deletions(-)
>>
>> --
>> 2.26.3
>---end quoted text---
On Mon, Mar 29, 2021 at 11:25:11AM +0530, Bhaskar Chowdhury wrote:
> On 07:29 Mon 29 Mar 2021, Christoph Hellwig wrote:
> > I really don't think these typo patchbomb are that useful. I'm all
> > for fixing typos when working with a subsystem, but I'm not sure these
> > patchbombs help anything.
> >
> I am sure you are holding the wrong end of the wand and grossly failing to
> understand.
Please stop statements like this, it is not helpful and is doing nothing
but ensure that your patches will not be looked at in the future.
> Anyway, I hope I give a heads up ...find "your way" to fix those damn
> thing...it's glaring....
There is no requirement that anyone accept patches that are sent to
them. When you complain when receiving comments on them, that
shows you do not wish to work with others.
Sorry, but you are now on my local blacklist for a while, and I
encourage other maintainers to just ignore these patches as well.
thanks,
greg k-h
On 13:48 Mon 29 Mar 2021, Greg KH wrote:
>On Mon, Mar 29, 2021 at 11:25:11AM +0530, Bhaskar Chowdhury wrote:
>> On 07:29 Mon 29 Mar 2021, Christoph Hellwig wrote:
>> > I really don't think these typo patchbomb are that useful. I'm all
>> > for fixing typos when working with a subsystem, but I'm not sure these
>> > patchbombs help anything.
>> >
>> I am sure you are holding the wrong end of the wand and grossly failing to
>> understand.
>
>Please stop statements like this, it is not helpful and is doing nothing
>but ensure that your patches will not be looked at in the future.
>
Greg, don't you think you are bit harsh and have an one sided view? People can
say in better way if they don't like some work. I Have always try to get
along.
>> Anyway, I hope I give a heads up ...find "your way" to fix those damn
>> thing...it's glaring....
>
>There is no requirement that anyone accept patches that are sent to
>them. When you complain when receiving comments on them, that
>shows you do not wish to work with others.
>
Unfortunate you are only seeing my complains...I don't know why you are so
blindfolded.
>Sorry, but you are now on my local blacklist for a while, and I
>encourage other maintainers to just ignore these patches as well.
>
I can not overrule that ...I know my pathes are trivial ..but it seems some
other problems are looming large.
NOT good Greg....not good seriously.
>thanks,
>
>greg k-h
On 2021-03-29 00:53, Bhaskar Chowdhury wrote:
> s/Synopsys/Synopsis/ .....two different places.
Erm, that is definitely not a typo... :/
> ..and for some unknown reason it introduce a empty line deleted and added
> back.
Presumably your editor is configured to trim trailing whitespace on save.
Furthermore, there are several instances in the other patches where your
"corrections" are grammatically incorrect, I'm not sure what the deal is
with patch #14, and you've also used the wrong subsystem name (it should
be "dmaengine"). It's great to want to clean things up, but please pay a
bit of care and attention to what you're actually doing.
Robin.
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> drivers/dma/Kconfig | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
> index 0c2827fd8c19..30e8cc26f43b 100644
> --- a/drivers/dma/Kconfig
> +++ b/drivers/dma/Kconfig
> @@ -170,15 +170,15 @@ config DMA_SUN6I
> Support for the DMA engine first found in Allwinner A31 SoCs.
>
> config DW_AXI_DMAC
> - tristate "Synopsys DesignWare AXI DMA support"
> + tristate "Synopsis DesignWare AXI DMA support"
> depends on OF || COMPILE_TEST
> depends on HAS_IOMEM
> select DMA_ENGINE
> select DMA_VIRTUAL_CHANNELS
> help
> - Enable support for Synopsys DesignWare AXI DMA controller.
> + Enable support for Synopsis DesignWare AXI DMA controller.
> NOTE: This driver wasn't tested on 64 bit platform because
> - of lack 64 bit platform with Synopsys DW AXI DMAC.
> + of lack 64 bit platform with Synopsis DW AXI DMAC.
>
> config EP93XX_DMA
> bool "Cirrus Logic EP93xx DMA support"
> @@ -394,7 +394,7 @@ config MOXART_DMA
> select DMA_VIRTUAL_CHANNELS
> help
> Enable support for the MOXA ART SoC DMA controller.
> -
> +
> Say Y here if you enabled MMP ADMA, otherwise say N.
>
> config MPC512X_DMA
> --
> 2.26.3
>
> _______________________________________________
> iommu mailing list
> [email protected]
> https://lists.linuxfoundation.org/mailman/listinfo/iommu
>
On 29-03-21, 05:23, Bhaskar Chowdhury wrote:
> s/transferred/transfered/
>
> This reverts commit a2ddb8aea8106bd5552f8516ad7a8a26b9282a8f.
This is not upstream, why not squash in. Also would make sense to write
sensible changelog and not phrases and use the right subsystem
conventions!
Droped the series now
--
~Vinod
On 22:59 Tue 30 Mar 2021, Vinod Koul wrote:
>On 29-03-21, 05:23, Bhaskar Chowdhury wrote:
>> s/transferred/transfered/
>>
>> This reverts commit a2ddb8aea8106bd5552f8516ad7a8a26b9282a8f.
>
>This is not upstream, why not squash in. Also would make sense to write
>sensible changelog and not phrases and use the right subsystem
>conventions!
>
Changes like this don't deserve a history to tell, specific to changelog, one
line is suffice.
>Droped the series now
>
This is a bad commit slip in , not suppose to be there, thanks for catching
it.
Sorry for the noise.
>
>--
>~Vinod
On 12:43 Tue 30 Mar 2021, Robin Murphy wrote:
>On 2021-03-29 00:53, Bhaskar Chowdhury wrote:
>> s/Synopsys/Synopsis/ .....two different places.
>
>Erm, that is definitely not a typo... :/
>
>> ..and for some unknown reason it introduce a empty line deleted and added
>> back.
>
>Presumably your editor is configured to trim trailing whitespace on save.
>
>Furthermore, there are several instances in the other patches where your
>"corrections" are grammatically incorrect, I'm not sure what the deal is
>with patch #14, and you've also used the wrong subsystem name (it should
>be "dmaengine"). It's great to want to clean things up, but please pay a
>bit of care and attention to what you're actually doing.
Thank you so much, I shall be more careful.
>
>Robin.
>
>> Signed-off-by: Bhaskar Chowdhury <[email protected]>
>> ---
>> drivers/dma/Kconfig | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
>> index 0c2827fd8c19..30e8cc26f43b 100644
>> --- a/drivers/dma/Kconfig
>> +++ b/drivers/dma/Kconfig
>> @@ -170,15 +170,15 @@ config DMA_SUN6I
>> Support for the DMA engine first found in Allwinner A31 SoCs.
>>
>> config DW_AXI_DMAC
>> - tristate "Synopsys DesignWare AXI DMA support"
>> + tristate "Synopsis DesignWare AXI DMA support"
>> depends on OF || COMPILE_TEST
>> depends on HAS_IOMEM
>> select DMA_ENGINE
>> select DMA_VIRTUAL_CHANNELS
>> help
>> - Enable support for Synopsys DesignWare AXI DMA controller.
>> + Enable support for Synopsis DesignWare AXI DMA controller.
>> NOTE: This driver wasn't tested on 64 bit platform because
>> - of lack 64 bit platform with Synopsys DW AXI DMAC.
>> + of lack 64 bit platform with Synopsis DW AXI DMAC.
>>
>> config EP93XX_DMA
>> bool "Cirrus Logic EP93xx DMA support"
>> @@ -394,7 +394,7 @@ config MOXART_DMA
>> select DMA_VIRTUAL_CHANNELS
>> help
>> Enable support for the MOXA ART SoC DMA controller.
>> -
>> +
>> Say Y here if you enabled MMP ADMA, otherwise say N.
>>
>> config MPC512X_DMA
>> --
>> 2.26.3
>>
>> _______________________________________________
>> iommu mailing list
>> [email protected]
>> https://lists.linuxfoundation.org/mailman/listinfo/iommu
>>