Re: [Outreachy kernel] [PATCH 1/2] staging: kpc2000: resolve various code style issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 21, 2020 at 10:11:52AM +0530, Vaishali Thakkar wrote:
> On Wed, Oct 21, 2020 at 8:33 AM Deepak R Varma <mh12gx2825@xxxxxxxxx> wrote:
> >
> > Multiple issues reported by checkpatch script around lines exceeding 100
> > columns, indentation of function parameters, extra blank lines. These
> > code formatting changes improves the code readability.
> 
> Please send separate patches while fixing different checkpatch
> warnings. It's a good idea to fix different warnings together if
> it's for the same block of code but otherwise your patch should
> handle one similar change at a time.
> 
Okay. Will change my old commit and resend the patch set.

Thank you,
Deepak.

> > Signed-off-by: Deepak R Varma <mh12gx2825@xxxxxxxxx>
> > ---
> >  drivers/staging/kpc2000/kpc2000/cell_probe.c  | 71 ++++++++++++++-----
> >  drivers/staging/kpc2000/kpc2000/core.c        |  3 +-
> >  .../staging/kpc2000/kpc2000/dma_common_defs.h |  3 +-
> >  3 files changed, 58 insertions(+), 19 deletions(-)
> >
> > diff --git a/drivers/staging/kpc2000/kpc2000/cell_probe.c b/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > index 738122afc2ae..e7e963d62699 100644
> > --- a/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > +++ b/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > @@ -30,9 +30,12 @@
> >   *
> >   */
> >
> > -#define KPC_OLD_DMA_CH_NUM(present, channel)   ((present) ? (0x8 | ((channel) & 0x7)) : 0)
> > -#define KPC_OLD_S2C_DMA_CH_NUM(cte)   KPC_OLD_DMA_CH_NUM(cte.s2c_dma_present, cte.s2c_dma_channel_num)
> > -#define KPC_OLD_C2S_DMA_CH_NUM(cte)   KPC_OLD_DMA_CH_NUM(cte.c2s_dma_present, cte.c2s_dma_channel_num)
> > +#define KPC_OLD_DMA_CH_NUM(present, channel)   \
> > +               ((present) ? (0x8 | ((channel) & 0x7)) : 0)
> > +#define KPC_OLD_S2C_DMA_CH_NUM(cte)   \
> > +               KPC_OLD_DMA_CH_NUM(cte.s2c_dma_present, cte.s2c_dma_channel_num)
> > +#define KPC_OLD_C2S_DMA_CH_NUM(cte)   \
> > +               KPC_OLD_DMA_CH_NUM(cte.c2s_dma_present, cte.c2s_dma_channel_num)
> >
> >  #define KP_CORE_ID_INVALID      0
> >  #define KP_CORE_ID_I2C          3
> > @@ -67,7 +70,8 @@ void  parse_core_table_entry_v0(struct core_table_entry *cte, const u64 read_val
> >  static
> >  void dbg_cte(struct kp2000_device *pcard, struct core_table_entry *cte)
> >  {
> > -       dev_dbg(&pcard->pdev->dev, "CTE: type:%3d  offset:%3d (%3d)  length:%3d (%3d)  s2c:%d  c2s:%d  irq_count:%d  base_irq:%d\n",
> > +       dev_dbg(&pcard->pdev->dev,
> > +               "CTE: type:%3d  offset:%3d (%3d)  length:%3d (%3d)  s2c:%d  c2s:%d  irq_count:%d  base_irq:%d\n",
> >                 cte->type,
> >                 cte->offset,
> >                 cte->offset / 4096,
> > @@ -107,7 +111,14 @@ static int probe_core_basic(unsigned int core_num, struct kp2000_device *pcard,
> >                 .ddna              = pcard->ddna,
> >         };
> >
> > -       dev_dbg(&pcard->pdev->dev, "Found Basic core: type = %02d  dma = %02x / %02x  offset = 0x%x  length = 0x%x (%d regs)\n", cte.type, KPC_OLD_S2C_DMA_CH_NUM(cte), KPC_OLD_C2S_DMA_CH_NUM(cte), cte.offset, cte.length, cte.length / 8);
> > +       dev_dbg(&pcard->pdev->dev,
> > +               "Found Basic core: type = %02d  dma = %02x / %02x  offset = 0x%x  length = 0x%x (%d regs)\n",
> > +               cte.type,
> > +               KPC_OLD_S2C_DMA_CH_NUM(cte),
> > +               KPC_OLD_C2S_DMA_CH_NUM(cte),
> > +               cte.offset,
> > +               cte.length,
> > +               cte.length / 8);
> >
> >         cell.platform_data = &core_pdata;
> >         cell.pdata_size = sizeof(struct kpc_core_device_platdata);
> > @@ -290,7 +301,14 @@ static int probe_core_uio(unsigned int core_num, struct kp2000_device *pcard,
> >         struct kpc_uio_device *kudev;
> >         int rv;
> >
> > -       dev_dbg(&pcard->pdev->dev, "Found UIO core:   type = %02d  dma = %02x / %02x  offset = 0x%x  length = 0x%x (%d regs)\n", cte.type, KPC_OLD_S2C_DMA_CH_NUM(cte), KPC_OLD_C2S_DMA_CH_NUM(cte), cte.offset, cte.length, cte.length / 8);
> > +       dev_dbg(&pcard->pdev->dev,
> > +               "Found UIO core:   type = %02d  dma = %02x / %02x  offset = 0x%x  length = 0x%x (%d regs)\n",
> > +               cte.type,
> > +               KPC_OLD_S2C_DMA_CH_NUM(cte),
> > +               KPC_OLD_C2S_DMA_CH_NUM(cte),
> > +               cte.offset,
> > +               cte.length,
> > +               cte.length / 8);
> >
> >         kudev = kzalloc(sizeof(*kudev), GFP_KERNEL);
> >         if (!kudev)
> > @@ -315,10 +333,14 @@ static int probe_core_uio(unsigned int core_num, struct kp2000_device *pcard,
> >
> >         kudev->uioinfo.mem[0].name = "uiomap";
> >         kudev->uioinfo.mem[0].addr = pci_resource_start(pcard->pdev, REG_BAR) + cte.offset;
> > -       kudev->uioinfo.mem[0].size = (cte.length + PAGE_SIZE - 1) & ~(PAGE_SIZE - 1); // Round up to nearest PAGE_SIZE boundary
> > +
> > +       // Round up to nearest PAGE_SIZE boundary
> > +       kudev->uioinfo.mem[0].size = (cte.length + PAGE_SIZE - 1) & ~(PAGE_SIZE - 1);
> >         kudev->uioinfo.mem[0].memtype = UIO_MEM_PHYS;
> >
> > -       kudev->dev = device_create(kpc_uio_class, &pcard->pdev->dev, MKDEV(0, 0), kudev, "%s.%d.%d.%d", kudev->uioinfo.name, pcard->card_num, cte.type, kudev->core_num);
> > +       kudev->dev = device_create(kpc_uio_class,
> > +                                  &pcard->pdev->dev, MKDEV(0, 0), kudev, "%s.%d.%d.%d",
> > +                                  kudev->uioinfo.name, pcard->card_num, cte.type, kudev->core_num);
> >         if (IS_ERR(kudev->dev)) {
> >                 dev_err(&pcard->pdev->dev, "%s: device_create failed!\n",
> >                         __func__);
> > @@ -341,7 +363,9 @@ static int probe_core_uio(unsigned int core_num, struct kp2000_device *pcard,
> >         return 0;
> >  }
> >
> > -static int  create_dma_engine_core(struct kp2000_device *pcard, size_t engine_regs_offset, int engine_num, int irq_num)
> > +static int  create_dma_engine_core(struct kp2000_device *pcard,
> > +                                  size_t engine_regs_offset,
> > +                                  int engine_num, int irq_num)
> >  {
> >         struct mfd_cell  cell = { .id = engine_num };
> >         struct resource  resources[2];
> > @@ -380,18 +404,28 @@ static int  kp2000_setup_dma_controller(struct kp2000_device *pcard)
> >
> >         // S2C Engines
> >         for (i = 0 ; i < 32 ; i++) {
> > -               capabilities_reg = readq(pcard->dma_bar_base + KPC_DMA_S2C_BASE_OFFSET + (KPC_DMA_ENGINE_SIZE * i));
> > +               capabilities_reg = readq(pcard->dma_bar_base +
> > +                                        KPC_DMA_S2C_BASE_OFFSET +
> > +                                        (KPC_DMA_ENGINE_SIZE * i));
> > +
> >                 if (capabilities_reg & ENGINE_CAP_PRESENT_MASK) {
> > -                       err = create_dma_engine_core(pcard, (KPC_DMA_S2C_BASE_OFFSET + (KPC_DMA_ENGINE_SIZE * i)), i,  pcard->pdev->irq);
> > +                       err = create_dma_engine_core(pcard, (KPC_DMA_S2C_BASE_OFFSET +
> > +                                                           (KPC_DMA_ENGINE_SIZE * i)),
> > +                                                    i, pcard->pdev->irq);
> >                         if (err)
> >                                 goto err_out;
> >                 }
> >         }
> >         // C2S Engines
> >         for (i = 0 ; i < 32 ; i++) {
> > -               capabilities_reg = readq(pcard->dma_bar_base + KPC_DMA_C2S_BASE_OFFSET + (KPC_DMA_ENGINE_SIZE * i));
> > +               capabilities_reg = readq(pcard->dma_bar_base +
> > +                                        KPC_DMA_C2S_BASE_OFFSET +
> > +                                        (KPC_DMA_ENGINE_SIZE * i));
> > +
> >                 if (capabilities_reg & ENGINE_CAP_PRESENT_MASK) {
> > -                       err = create_dma_engine_core(pcard, (KPC_DMA_C2S_BASE_OFFSET + (KPC_DMA_ENGINE_SIZE * i)), 32 + i,  pcard->pdev->irq);
> > +                       err = create_dma_engine_core(pcard, (KPC_DMA_C2S_BASE_OFFSET +
> > +                                                           (KPC_DMA_ENGINE_SIZE * i)),
> > +                                                    32 + i,  pcard->pdev->irq);
> >                         if (err)
> >                                 goto err_out;
> >                 }
> > @@ -433,10 +467,15 @@ int  kp2000_probe_cores(struct kp2000_device *pcard)
> >         // Then, iterate over the possible core types.
> >         for (current_type_id = 1 ; current_type_id <= highest_core_id ; current_type_id++) {
> >                 unsigned int core_num = 0;
> > -               // Foreach core type, iterate the whole table and instantiate subdevices for each core.
> > -               // Yes, this is O(n*m) but the actual runtime is small enough that it's an acceptable tradeoff.
> > +               /*
> > +                * Foreach core type, iterate the whole table and instantiate
> > +                * subdevices for each core.
> > +                * Yes, this is O(n*m) but the actual runtime is small enough
> > +                * that it's an acceptable tradeoff.
> > +                */
> >                 for (i = 0 ; i < pcard->core_table_length ; i++) {
> > -                       read_val = readq(pcard->sysinfo_regs_base + ((pcard->core_table_offset + i) * 8));
> > +                       read_val = readq(pcard->sysinfo_regs_base +
> > +                                        ((pcard->core_table_offset + i) * 8));
> >                         parse_core_table_entry(&cte, read_val, pcard->core_table_rev);
> >
> >                         if (cte.type != current_type_id)
> > diff --git a/drivers/staging/kpc2000/kpc2000/core.c b/drivers/staging/kpc2000/kpc2000/core.c
> > index 358d7b2f4ad1..6462a3059fb0 100644
> > --- a/drivers/staging/kpc2000/kpc2000/core.c
> > +++ b/drivers/staging/kpc2000/kpc2000/core.c
> > @@ -124,6 +124,7 @@ static ssize_t cpld_reconfigure(struct device *dev,
> >         writeq(wr_val, pcard->sysinfo_regs_base + REG_CPLD_CONFIG);
> >         return count;
> >  }
> > +
> >  static DEVICE_ATTR(cpld_reconfigure, 0220, NULL, cpld_reconfigure);
> >
> >  static ssize_t irq_mask_reg_show(struct device *dev,
> > @@ -367,7 +368,7 @@ static int kp2000_pcie_probe(struct pci_dev *pdev,
> >         dma_bar_phys_len = pci_resource_len(pcard->pdev, DMA_BAR);
> >
> >         pcard->dma_bar_base = ioremap(dma_bar_phys_addr,
> > -                                             dma_bar_phys_len);
> > +                                     dma_bar_phys_len);
> >         if (!pcard->dma_bar_base) {
> >                 dev_err(&pcard->pdev->dev,
> >                         "probe: DMA_BAR could not remap memory to virtual space\n");
> > diff --git a/drivers/staging/kpc2000/kpc2000/dma_common_defs.h b/drivers/staging/kpc2000/kpc2000/dma_common_defs.h
> > index 21450e3d408f..8bc78be3c259 100644
> > --- a/drivers/staging/kpc2000/kpc2000/dma_common_defs.h
> > +++ b/drivers/staging/kpc2000/kpc2000/dma_common_defs.h
> > @@ -6,8 +6,7 @@
> >  #define KPC_DMA_S2C_BASE_OFFSET     0x0000
> >  #define KPC_DMA_C2S_BASE_OFFSET     0x2000
> >  #define KPC_DMA_ENGINE_SIZE         0x0100
> > -#define  ENGINE_CAP_PRESENT_MASK            0x1
> > -
> > +#define ENGINE_CAP_PRESENT_MASK     0x1
> >
> >  #define KPC_DMA_CARD_IRQ_ENABLE                 (1 << 0)
> >  #define KPC_DMA_CARD_IRQ_ACTIVE                 (1 << 1)
> > --
> > 2.25.1
> >
> > --
> > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxx.
> > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/bce6824343f7a10919be5130b5a6276ed186701c.1603248984.git.mh12gx2825%40gmail.com.
> 
> 
> 
> -- 
> Vaishali
> 
> 
> -- 
> Vaishali
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux