Re: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-09 Thread Bjorn Helgaas
On Tue, Feb 09, 2021 at 03:28:16PM +, Gustavo Pimentel wrote: > On Mon, Feb 8, 2021 at 22:53:54, Krzysztof Wilczyński > wrote: > > [...] > > > Thanks for your review. I will wait for a couple of days, before sending > > > a new version of this patch series based on your feedback. > > > >

Re: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-09 Thread Krzysztof Wilczyński
Hi Gustavo, [...] > > The code in question would be (exceprt from the patch): > > > > [...] > > +static int dw_xdata_pcie_probe(struct pci_dev *pdev, > > + const struct pci_device_id *pid) > > +{ > > + const struct dw_xdata_pcie_data *pdata = (void *)pid->driver_data;

RE: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-09 Thread Gustavo Pimentel
On Mon, Feb 8, 2021 at 22:53:54, Krzysztof Wilczyński wrote: > [+cc Bjorn] > > Hi Gustavo, > > [...] > > Thanks for your review. I will wait for a couple of days, before sending > > a new version of this patch series based on your feedback. > > Thank you! > > There might be one more

Re: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-08 Thread Krzysztof Wilczyński
[+cc Bjorn] Hi Gustavo, [...] > Thanks for your review. I will wait for a couple of days, before sending > a new version of this patch series based on your feedback. Thank you! There might be one more change, and improvement, to be done as per Bjorn's feedback, see:

RE: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-08 Thread Gustavo Pimentel
On Sun, Feb 7, 2021 at 1:36:15, Krzysztof Wilczyński wrote: > Hi Gustavo, > > Thank you for all the work here! > > A few suggestions. > > [...] > > +static void dw_xdata_stop(struct dw_xdata *dw) > > +{ > > + u32 burst = readl(&(__dw_xdara_regs(dw)->burst_cnt)); > > + > > + if (burst &

Re: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-06 Thread Krzysztof Wilczyński
Hi Gustavo, Thank you for all the work here! A few suggestions. [...] > +static void dw_xdata_stop(struct dw_xdata *dw) > +{ > + u32 burst = readl(&(__dw_xdara_regs(dw)->burst_cnt)); > + > + if (burst & BIT(31)) { > + burst &= ~(u32)BIT(31); > + writel(burst,

Re: [RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-03 Thread Greg Kroah-Hartman
On Wed, Feb 03, 2021 at 11:12:46PM +0100, Gustavo Pimentel wrote: > + /* Sysfs */ > + err = sysfs_create_group(>dev.kobj, _attr_group); > + if (err) > + return err; > + > + err = sysfs_create_link(kernel_kobj, >dev.kobj, > +

[RESEND v4 1/6] misc: Add Synopsys DesignWare xData IP driver

2021-02-03 Thread Gustavo Pimentel
Add Synopsys DesignWare xData IP driver. This driver enables/disables the PCI traffic generator module pertain to the Synopsys DesignWare prototype. Signed-off-by: Gustavo Pimentel --- drivers/misc/dw-xdata-pcie.c | 378 +++ 1 file changed, 378