DMA Engine Documentation: TX Descriptor and Submission

2019-01-28 Thread Federico Vaga
Hi, I have a new question concerning documentation. https://www.kernel.org/doc/html/latest/driver-api/dmaengine/client.html >From this document it is not really clear, at least to me, if clients can consider valid the `struct dma_async_tx_descriptor` after submission to the DMA engine.

Re: [PATCH V2 0/5] i2c:ocores: improvements

2019-01-15 Thread Federico Vaga
Hi there, I just want to ask if there has been any update about this patchset that I'm not aware off. Thanks On Tuesday, November 27, 2018 12:38:22 PM CET Wolfram Sang wrote: > > This patch set provides improvements to the i2c-ocore driver. > > > > [V1 -> V2] > > - replaced usleep_range() with

[PATCH V3] doc:it_IT: add translations in process/

2019-02-04 Thread Federico Vaga
This patch adds the Italian translation for the following documents in Documentation/process: - applying-patches - submit-checklist - submitting-drivers - changes - stable api nonsense Signed-off-by: Federico Vaga --- V3 - update according to recent change 8f7e6d134bda (doc/docs-next) doc

[PATCH] doc:dmaengine: clarify DMA desc. pointer after submission

2019-02-08 Thread Federico Vaga
It clarifies that the DMA description pointer returned by `dmaengine_prep_*` function should not be used after submission. Signed-off-by: Federico Vaga --- Documentation/driver-api/dmaengine/client.rst | 7 +++ 1 file changed, 7 insertions(+) diff --git a/Documentation/driver-api/dmaengine

[PATCH v3 2/5] i2c:ocores: do not handle IRQ if IF is not set

2019-02-08 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga Acked

[PATCH v3 0/5]

2019-02-08 Thread Federico Vaga
This patch set provides improvements to the i2c-ocore driver. [V2 -> V3] - fix error condition on platform_get_irq(). Copied from https://patchwork.ozlabs.org/patch/1038409/ [V1 -> V2] - replaced usleep_range() with udelay() so that the polling version can be used in atomic context. - added

[PATCH v3 1/5] i2c:ocores: stop transfer on timeout

2019-02-08 Thread Federico Vaga
a new function to handle timeout - modified the current ocores_process() function in order to be protected by the new spinlock Like this it is obvious at first sight that this locking serializes the execution of ocores_process() and ocores_process_timeout() Signed-off-by: Federico Vaga

[PATCH v3 5/5] i2c:ocores: checkpatch fixes

2019-02-08 Thread Federico Vaga
Miscellaneous style fixes from checkpatch Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 5b80190..ba35d2a 100644

[PATCH v3 4/5] i2c:ocores: add SPDX tag

2019-02-08 Thread Federico Vaga
It adds the SPDX tag and it removes the old text about the GPLv2. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index bbe3e96..5b80190

[PATCH v3 3/5] i2c:ocores: add polling interface

2019-02-08 Thread Federico Vaga
This driver assumes that an interrupt line is always available for the I2C master. This is not always the case and this patch adds support for a polling version. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 176 +++- 1 file changed, 156

Re: [PATCH v3 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
On Saturday, February 9, 2019 10:33:53 PM CET Andrew Lunn wrote: > > +static int ocores_poll_wait(struct ocores_i2c *i2c) > > +{ > > + u8 mask; > > + int err; > > + > > + if (i2c->state == STATE_DONE || i2c->state == STATE_ERROR) { > > + /* transfer is over */ > > + mask

[PATCH v4 2/5] i2c:ocores: do not handle IRQ if IF is not set

2019-02-11 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga Acked

[PATCH v4 4/5] i2c:ocores: add SPDX tag

2019-02-11 Thread Federico Vaga
It adds the SPDX tag and it removes the old text about the GPLv2. Signed-off-by: Federico Vaga Reviewed-by: Andrew Lunn --- drivers/i2c/busses/i2c-ocores.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c

[PATCH v4 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
ted in ocores_wait() mostly not looping at all. But for reading an 4K AT24 EEPROM, it increased the read time by 10ms, from 424ms to 434ms. So we should probably keep with 8. Signed-off-by: Federico Vaga Tested-by: Andrew Lunn --- drivers/i2c/busses/i2c-ocores.c |

[PATCH v4 1/5] i2c:ocores: stop transfer on timeout

2019-02-11 Thread Federico Vaga
a new function to handle timeout - modified the current ocores_process() function in order to be protected by the new spinlock Like this it is obvious at first sight that this locking serializes the execution of ocores_process() and ocores_process_timeout() Signed-off-by: Federico Vaga Reviewed

[PATCH v4 0/5] i2c:ocores: improvements

2019-02-11 Thread Federico Vaga
This patch set provides improvements to the i2c-ocore driver. [V3 -> V4] - add reviews-by/tested-by - add comment to justify the formula in udelay((8 * 1000) / i2c->bus_clock_khz); [V2 -> V3] - fix particular error condition on platform_get_irq(). Copied from

[PATCH v4 5/5] i2c:ocores: checkpatch fixes

2019-02-11 Thread Federico Vaga
Miscellaneous style fixes from checkpatch Signed-off-by: Federico Vaga Reviewed-by: Andrew Lunn --- drivers/i2c/busses/i2c-ocores.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index

Re: [PATCH] doc:dmaengine: clarify DMA desc. pointer after submission

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 12:54:11 PM CET Vinod Koul wrote: > On 08-02-19, 16:30, Federico Vaga wrote: > > It clarifies that the DMA description pointer returned by > > `dmaengine_prep_*` function should not be used after submission. > > > > S

Re: [PATCH v4 1/5] i2c:ocores: stop transfer on timeout

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 11:44:46 AM CET Peter Rosin wrote: > On 2019-02-11 09:31, Federico Vaga wrote: > > > Detecting a timeout is ok, but we also need to assert a STOP command on > > the bus in order to prevent it from generating interrupts when there are > &g

Re: [PATCH v4 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 11:43:45 AM CET Peter Rosin wrote: > On 2019-02-11 09:31, Federico Vaga wrote: > > > This driver assumes that an interrupt line is always available for > > the I2C master. This is not always the case and this patch adds support > &g

Re: [PATCH v4 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 2:35:15 PM CET Peter Rosin wrote: > >>> @@ -294,7 +427,7 @@ static int ocores_init(struct device *dev, struct > >>> ocores_i2c *i2c) > >> > >> > >> > >>> > >>> > >>> > >>> /* Init the device */ > >>> oc_setreg(i2c, OCI2C_CMD, OCI2C_CMD_IACK); > >>> > >>> >

Re: [PATCH v4 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 11:25:26 AM CET Wolfram Sang wrote: > On Mon, Feb 11, 2019 at 09:31:20AM +0100, Federico Vaga wrote: > > This driver assumes that an interrupt line is always available for > > the I2C master. This is not always the case and this patch adds support &g

Re: [PATCH v4 1/5] i2c:ocores: stop transfer on timeout

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 3:01:38 PM CET Andrew Lunn wrote: > > Applied to for-next, thanks! > > Hi Wolfram > > Could you drop these patches and wait for a new version? I don't > think you have pushed it out yet? So it won't be a visible rebase. I will wait to send v5: full patch set, or

doc: stable-api-nonsense missing link

2019-01-17 Thread Federico Vaga
Hi, in this document Documentation/process/stable-api-nonsense.rst there is this note: Is it possible to get it fixed with the proper link? I searched for it without success. If that comment is impossible to find I will send a patch to remove that note, but first I wanted to ask you if you

[PATCH] doc:process: remove note from 'stable api nonsense'

2019-01-18 Thread Federico Vaga
The link referred by the note can't be retrieved: this patch just remove that old note. Signed-off-by: Federico Vaga --- Documentation/process/stable-api-nonsense.rst | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Documentation/process/stable-api-nonsense.rst b

Re: [PATCH v2] doc: Change LXR references to elixir.bootlin.com

2019-01-30 Thread Federico Vaga
to elixir.bootlin.com. [1]: https://bootlin.com/blog/free-electrons-becomes-bootlin/ [2]: https://github.com/free-electrons/elixir Signed-off-by: Jonathan Neuschäfer Acked-by: Federico Vaga --- v2: - Fix ident search URL, as suggested by Martin Kepplinger - Add trailing slash, which I accidentally dropped

Re: [PATCH] doc:it_IT: add translations in process/

2019-02-06 Thread Federico Vaga
On 2019-01-21 09:11, Federico Vaga wrote: On Monday, January 21, 2019 2:56:17 AM CET Jonathan Corbet wrote: On Sat, 19 Jan 2019 23:13:41 +0100 Federico Vaga wrote: > This patch adds the Italian translation for the following documents > in Documentation/process: > > - appl

fpga: fpga_mgr_free usage

2018-07-11 Thread Federico Vaga
Hi Alan, I have another point that I would like to discuss. It is about the usage of 'fpga_mgr_free()' which does not look like consistent. This function, according to the current implementation, can be used by an FPGA manager user and it is used by the FPGA manager itself on device release.

fpga: fpga_mgr_get() buggy ?

2018-06-21 Thread Federico Vaga
Hello, I believe that this patch fpga: manager: change api, don't use drvdata 7085e2a94f7df5f419e3cfb2fe809ce6564e9629 is incomplete and buggy. I completely agree that drvdata should not be used by the FPGA manager or any other subsystem like that. What is buggy is the function

Re: fpga: fpga_mgr_get() buggy ?

2018-06-22 Thread Federico Vaga
Hi Alan, inline comments On Friday, 22 June 2018 04:07:41 CEST Alan Tull wrote: > On Thu, Jun 21, 2018 at 8:13 AM, Federico Vaga > wrote: > > Hi Federico, > > Thanks for the analysis. I'll probably not be able to look into > this very much until next week. A few note

[PATCH 2/3] i2c:ocores: do not handle IRQ if IF is not set

2018-06-25 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga

[PATCH 3/3] i2c:ocores: add polling interface

2018-06-25 Thread Federico Vaga
This driver assumes that an interrupt line is always available for the I2C master. This is not always the case and this patch adds support for a polling version based on workqueue. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 94 ++--- 1

i2c:ocores: fixes and polling mechanism

2018-06-25 Thread Federico Vaga
The first two patches fix what I believe are bugs. The third patch add a polling mechanism for those systems where interrupts are not available. All these patches have been tested on a system without interrupt, this means that I used my third patch to validate also the other two. I would be nice

[PATCH 1/3] i2c:ocores: stop transfer on timeout

2018-06-25 Thread Federico Vaga
is taken is because we are in timeout, so there is no need to process the IRQ. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 28 ++-- 1 file changed, 26 insertions(+), 2 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c

Re: fpga: fpga_mgr_get() buggy ?

2018-06-27 Thread Federico Vaga
Hi Alan, On Tuesday, 26 June 2018 23:00:46 CEST Alan Tull wrote: > On Fri, Jun 22, 2018 at 2:53 AM, Federico Vaga > wrote: > > Hi Federico, > > >> > What is buggy is the function fpga_mgr_get(). > >> > That patch has been done to allow multiple FPGA manag

Re: fpga: fpga_mgr_get() buggy ?

2018-06-28 Thread Federico Vaga
On Wednesday, 27 June 2018 23:23:07 CEST Alan Tull wrote: > On Wed, Jun 27, 2018 at 4:25 AM, Federico Vaga wrote: > > Hi Alan, > > > > On Tuesday, 26 June 2018 23:00:46 CEST Alan Tull wrote: > >> On Fri, Jun 22, 2018 at 2:53 AM, Federico Vaga >

[PATCH 1/5] doc: typos and minor fixes

2018-05-27 Thread Federico Vaga
Signed-off-by: Federico Vaga --- Documentation/doc-guide/kernel-doc.rst| 2 +- Documentation/doc-guide/parse-headers.rst | 4 ++-- Documentation/doc-guide/sphinx.rst| 4 ++-- Documentation/index.rst | 2 +- Documentation/sphinx/parse-headers.pl | 2 +- 5 files

[PATCH 2/5] doc: add chapter labels

2018-05-27 Thread Federico Vaga
The idea is to make it easier to create references (doc-guide does the same). Signed-off-by: Federico Vaga --- Documentation/index.rst| 2 ++ Documentation/kernel-hacking/index.rst | 2 ++ 2 files changed, 4 insertions(+) diff --git a/Documentation/index.rst b/Documentation

doc: Italian translation

2018-05-27 Thread Federico Vaga
Ciao Jonathan, here the doc-guide translated in Italian. This set of patches includes some minor changes to the main one. The idea of this first set of patches is also to adjust the structure and our expectations. We tried to translate everything in **Italian**; which means that we avoided

[PATCH 3/5] doc: add Italian translation skeleton

2018-05-27 Thread Federico Vaga
Signed-off-by: Federico Vaga Signed-off-by: Alessia Mantegazza --- Documentation/index.rst| 8 ++ .../translations/it_IT/disclaimer-ita.rst | 11 +++ Documentation/translations/it_IT/index.rst | 101 + 3 files changed, 120

[PATCH 4/5] doc:it_IT: add doc-guide translation

2018-05-27 Thread Federico Vaga
Signed-off-by: Federico Vaga Signed-off-by: Alessia Mantegazza --- .../translations/it_IT/doc-guide/hello.dot | 3 + .../translations/it_IT/doc-guide/index.rst | 24 ++ .../translations/it_IT/doc-guide/kernel-doc.rst| 402 ++ .../translations/it_IT/doc

[PATCH] i2c: ocores: update HDL sources URL

2018-05-28 Thread Federico Vaga
The URL is broken. This patch fix it Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 8c42ca7107b2..14c5f6bbfd95 100644 --- a/drivers/i2c

[PATCH V2] i2c: ocores: update HDL sources URL

2018-05-28 Thread Federico Vaga
The URL is broken. This patch fix it Signed-off-by: Federico Vaga --- Documentation/i2c/busses/i2c-ocores | 2 +- drivers/i2c/busses/i2c-ocores.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/i2c/busses/i2c-ocores b/Documentation/i2c/busses/i2c-ocores

i2c: Open Core driver

2018-05-28 Thread Federico Vaga
Sorry for the SPAM, I re-send the patch because I missed the documentation file. This patch is (should) the good one :)

Re: doc: Italian translation

2018-05-28 Thread Federico Vaga
On Monday, 28 May 2018 15:08:28 CEST Jonathan Corbet wrote: > On Sun, 27 May 2018 16:55:55 +0200 > > Federico Vaga wrote: > > here the doc-guide translated in Italian. This set of patches > > includes some minor changes to the main one. The idea of this > > first set o

Re: fpga: fpga_mgr_get() buggy ?

2018-07-18 Thread Federico Vaga
Hi Alan, Thanks for your time, comments below On Wednesday, July 18, 2018 9:47:24 PM CEST Alan Tull wrote: > On Thu, Jun 28, 2018 at 2:50 AM, Federico Vaga wrote: > > On Wednesday, 27 June 2018 23:23:07 CEST Alan Tull wrote: > >> On Wed, Jun 27, 2018 at 4:25 AM, Federico Va

[PATCH 1/2] doc:process: add links where missing

2018-11-20 Thread Federico Vaga
Some documents are refering to others without links. With this patch I add those missing links. This patch affects only documents under process/ and labels where necessary. Signed-off-by: Federico Vaga --- Documentation/admin-guide/devices.rst| 1 + Documentation/dev-tools

Re: [PATCH 3/3] i2c:ocores: add polling interface

2018-10-29 Thread Federico Vaga
Hi Peter, On Friday, October 26, 2018 7:45:29 PM CET Peter Korsgaard wrote: > >>>>> "Federico" == Federico Vaga writes: > Hi, > > >> > - } else > >> > + } else { > >> > &g

Re: [PATCH 3/3] i2c:ocores: add polling interface

2018-10-29 Thread Federico Vaga
On Monday, October 29, 2018 2:04:13 PM CET Peter Korsgaard wrote: > > I think that something like this could be better > > > > (2) usleep_range(sleep_min, sleep_min * XXX); > > > > But. > > Since it is better to make this patch ready for xfer_irqless, then I will > > definitively go for

Re: [PATCH 2/3] i2c:ocores: do not handle IRQ if IF is not set

2018-10-29 Thread Federico Vaga
On Monday, October 29, 2018 9:53:01 AM CET Wolfram Sang wrote: > On Sun, Oct 21, 2018 at 04:12:10PM +0200, Peter Korsgaard wrote: > > On Mon, Jun 25, 2018 at 6:14 PM Federico Vaga wrote: > > > If the Interrupt Flag (IF) is not set, we should not handle the IRQ: > > &g

[PATCH V2 0/5] i2c:ocores: improvements

2018-10-29 Thread Federico Vaga
This patch set provides improvements to the i2c-ocore driver. [V1 -> V2] - replaced usleep_range() with udelay() so that the polling version can be used in atomic context. - added dedicated patch for minor style issues - fixed delay computation - use spin_lock_irqsave(), instead of

[PATCH V2 1/5] i2c:ocores: stop transfer on timeout

2018-10-29 Thread Federico Vaga
a new function to handle timeout - modified the current ocores_process() function in order to be protected by the new spinlock Like this it is obvious at first sight that this locking serializes the execution of ocores_process() and ocores_process_timeout() Signed-off-by: Federico Vaga

[PATCH V2 4/5] i2c:ocores: add SPDX tag

2018-10-29 Thread Federico Vaga
It adds the SPDX tag and it removes the old text about the GPLv2. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 2d71f11..e48173b

[PATCH V2 3/5] i2c:ocores: add polling interface

2018-10-29 Thread Federico Vaga
This driver assumes that an interrupt line is always available for the I2C master. This is not always the case and this patch adds support for a polling version. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 171 +++- 1 file changed, 151

[PATCH V2 2/5] i2c:ocores: do not handle IRQ if IF is not set

2018-10-29 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga Acked

[PATCH V2 5/5] i2c:ocores: checkpatch fixes

2018-10-29 Thread Federico Vaga
Miscellaneous style fixes from checkpatch Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index e48173b..52c062b 100644

[RFC] Proposal for FMC Subsystem Eradication

2018-10-29 Thread Federico Vaga
=== The fmc-bus Eradication === The *fmc-bus* Linux sub-system was developed under CERN supervision around 2012. During these years, the fmc-bus has evolved and drivers for particular FMC applications (FMC DEL, FMC ADC, FMC TDC, etc.) have been based on it.

Re: fpga: fpga_mgr_get() buggy ?

2018-08-16 Thread Federico Vaga
Hi alan, inline comments On Wednesday, August 15, 2018 11:02:12 PM CEST Alan Tull wrote: > On Wed, Jul 18, 2018 at 4:47 PM, Federico Vaga wrote: > > Hi Alan, > > > > Thanks for your time, comments below > > > > On Wednesday, July 18, 2018 9:47:24 PM CEST Ala

Re: i2c:ocores: fixes and polling mechanism

2018-08-11 Thread Federico Vaga
Hello, sorry to disturb you all but after one month and a half I never received any comment about this patch set and I fear it ended up in a forgotten corner. I would like to know if someone is considering it or not. Thanks :) On Monday, June 25, 2018 6:13:00 PM CEST Federico Vaga wrote

Device Description for FPGA Components on x86 system

2019-03-27 Thread Federico Vaga
Hello, I'm looking for guidance What I have: * Intel x86_64 computer * PCIe card with FPGA on it What I want to achieve: * load an FPGA bitstream on the card * load a device-tree like description for the FPGA devices contained in the bitstream This is achievable on ARM with DeviceTree,

Re: Device Description for FPGA Components on x86 system

2019-04-10 Thread Federico Vaga
something useful not only for CERN but for the entire community. Thank you On Wednesday, March 27, 2019 6:17:18 PM CEST Federico Vaga wrote: > Hello, > > I'm looking for guidance > > What I have: > * Intel x86_64 computer > * PCIe card with FPGA on it > > What I want t

Re: Device Description for FPGA Components on x86 system

2019-04-10 Thread Federico Vaga
m/vdsao/fpga-cfg > [2] https://marc.info/?l=linux-fpga=155078072107199=2 > [3] https://marc.info/?l=linux-fpga > > Am 10.04.2019 12:01, schrieb Federico Vaga: > > Hello, > > > > sorry to push for an answer but I do not want to take the risk of &

Re: Device Description for FPGA Components on x86 system

2019-04-10 Thread Federico Vaga
Hi Alan, thanks for your answer On Wednesday, April 10, 2019 4:21:09 PM CEST Alan Tull wrote: > On Wed, Apr 10, 2019 at 7:50 AM Federico Vaga wrote: > > Hi Federico, > > I wish I could point you to a complete solution, but there is a lot of > work to be done in this a

[PATCH] doc: fix typos in license-rules.rst

2019-02-20 Thread Federico Vaga
The patches fixes some typos in process/license-rules.rst Signed-off-by: Federico Vaga --- Documentation/process/license-rules.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/process/license-rules.rst b/Documentation/process/license-rules.rst index

[PATCH 1/2] doc: translations: sync translations 'remove info about -git patches'

2019-02-20 Thread Federico Vaga
Synchonise translations: CN, IT, JP, KR commit 2c71d305caf9 ("docs: process: Remove outdated info about -git patches") I can guarantee for the Italian translations, but since we are removing an entire chapter I think I did it right also for the other languages. Signed-off-by: Fed

[PATCH 2/2] doc: process: complete removal of info about -git patches

2019-02-20 Thread Federico Vaga
The following patch forgot to remove a reference to the -git patches commit 2c71d305caf9 ("docs: process: Remove outdated info about -git patches") This patch complete the removal and update all translations Signed-off-by: Federico Vaga --- Documentation/process

[PATCH v2 1/2] doc: translations: sync translations 'remove info about -git patches'

2019-02-21 Thread Federico Vaga
Synchonise translations: CN, IT, JP, KR commit 2c71d305caf9 ("docs: process: Remove outdated info about -git patches") I can guarantee for the Italian translations, but since we are removing an entire chapter I think I did it right also for the other languages. Signed-off-by: Fed

[PATCH v2 2/2] doc: process: complete removal of info about -git patches

2019-02-21 Thread Federico Vaga
The following patch forgot to remove a reference to the -git patches commit 2c71d305caf9 ("docs: process: Remove outdated info about -git patches") This patch complete the removal and update all translations Signed-off-by: Federico Vaga --- Documentation/process

Re: [PATCH 01/20] docs/zh_CN: add disclaimer file

2019-03-05 Thread Federico Vaga
: Documentation/translations/it_IT/disclaimer-ita.rst. Thanks a lot! Signed-off-by: Alex Shi Cc: Harry Wei Cc: Jonathan Corbet Cc: Li Zefan Cc: Shawn Guo Cc: Fengguang Wu Cc: Coly Li Cc: Federico Vaga --- Documentation/translations/zh_CN/disclaimer-zh_CN.rst | 11 +++ 1 file changed, 11

[PATCH v5 5/5] i2c:ocores: checkpatch fixes

2019-02-11 Thread Federico Vaga
Miscellaneous style fixes from checkpatch Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 30 +++--- 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 78085a8

[PATCH v5 4/5] i2c:ocores: add SPDX tag

2019-02-11 Thread Federico Vaga
It adds the SPDX tag and it removes the old text about the GPLv2. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 5dea7b9..78085a8

[PATCH v5 2/5] i2c:ocores: do not handle IRQ if IF is not set

2019-02-11 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga Acked

[PATCH v5 3/5] i2c:ocores: add polling interface

2019-02-11 Thread Federico Vaga
ted in ocores_wait() mostly not looping at all. But for reading an 4K AT24 EEPROM, it increased the read time by 10ms, from 424ms to 434ms. So we should probably keep with 8. Signed-off-by: Federico Vaga Tested-by: Andrew Lunn --- drivers/i2c/busses/i2c-ocores.c |

[PATCH v5 1/5] i2c:ocores: stop transfer on timeout

2019-02-11 Thread Federico Vaga
a new function to handle timeout - modified the current ocores_process() function in order to be protected by the new spinlock Like this it is obvious at first sight that this locking serializes the execution of ocores_process() and ocores_process_timeout() Signed-off-by: Federico Vaga

[PATCH v5 0/5] i2c:ocores: improvements

2019-02-11 Thread Federico Vaga
This patch set provides improvements to the i2c-ocore driver. [V4 -> V5] - deterministic status of IEN bit in register "CONTROL" at the end of ocores_init() - more style fixes [V3 -> V4] - add reviews-by/tested-by - add comment to justify the formula in udelay((8 * 1000) /

Re: [PATCH v5 5/5] i2c:ocores: checkpatch fixes

2019-02-11 Thread Federico Vaga
On Monday, February 11, 2019 5:12:23 PM CET Peter Rosin wrote: > On 2019-02-11 17:05, Federico Vaga wrote: > > > Miscellaneous style fixes from checkpatch > > > > Signed-off-by: Federico Vaga > > --- > > > > drivers/i2c/busses/i2c-ocores.c | 30

[PATCH v6 0/5] i2c: ocores: improvements

2019-02-11 Thread Federico Vaga
This patch set provides improvements to the i2c-ocore driver. [V5 -> V6] - remove redundant code introduced in V5 (double read control register) [V4 -> V5] - deterministic status of IEN bit in register "CONTROL" at the end of ocores_init() - more style fixes [V3 -> V4] - add

[PATCH v6 4/5] i2c: ocores: add SPDX tag

2019-02-11 Thread Federico Vaga
It adds the SPDX tag and it removes the old text about the GPLv2. Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 5dea7b9..78085a8

[PATCH v6 1/5] i2c: ocores: stop transfer on timeout

2019-02-11 Thread Federico Vaga
a new function to handle timeout - modified the current ocores_process() function in order to be protected by the new spinlock Like this it is obvious at first sight that this locking serializes the execution of ocores_process() and ocores_process_timeout() Signed-off-by: Federico Vaga

[PATCH v6 2/5] i2c: ocores: do not handle IRQ if IF is not set

2019-02-11 Thread Federico Vaga
If the Interrupt Flag (IF) is not set, we should not handle the IRQ: - the line can be shared with other devices - it can be a spurious interrupt To avoid reading twice the status register, the ocores_process() function expects it to be read by the caller. Signed-off-by: Federico Vaga Acked

[PATCH v6 5/5] i2c: ocores: checkpatch fixes

2019-02-11 Thread Federico Vaga
Miscellaneous style fixes from checkpatch Signed-off-by: Federico Vaga --- drivers/i2c/busses/i2c-ocores.c | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c index 78085a8..b32d67c

[PATCH v6 3/5] i2c: ocores: add polling interface

2019-02-11 Thread Federico Vaga
ted in ocores_wait() mostly not looping at all. But for reading an 4K AT24 EEPROM, it increased the read time by 10ms, from 424ms to 434ms. So we should probably keep with 8. Signed-off-by: Federico Vaga Tested-by: Andrew Lunn --- drivers/i2c/busses/i2c-ocores.c |

Re: [PATCH v6 4/5] i2c: ocores: add SPDX tag

2019-02-12 Thread Federico Vaga
On Monday, February 11, 2019 5:54:54 PM CET Wolfram Sang wrote: > On Mon, Feb 11, 2019 at 05:49:12PM +0100, Federico Vaga wrote: > > It adds the SPDX tag and it removes the old text about the GPLv2. > > > > Signed-off-by: Federico Vaga > > I can convert the platfor

[RFC PATCH] doc: translation disclaimer

2019-03-05 Thread Federico Vaga
This is only an example to propose a structure for translation's disclaimers. The actual text needs some thoughs. Signed-off-by: Federico Vaga --- Documentation/translations/index.rst | 18 ++ .../translations/it_IT/disclaimer-ita.rst | 13

Re: [RFC PATCH] doc: translation disclaimer

2019-03-06 Thread Federico Vaga
Hi Alex, On Wednesday, March 6, 2019 2:57:05 AM CET Alex Shi wrote: > Generally It looks good! A short guide to translator is good as well as a > disclaimer. > On 2019/3/6 6:06 上午, Federico Vaga wrote: > > This is only an example to propose a structure for translation's

Re: [PATCH] Documentation/process/howto: Update for 4.x -> 5.x versioning

2019-02-24 Thread Federico Vaga
hello, I have just a general observation for the community, not related to the content of this patch, but related with the idea behind. Is it really important to specify the major release number in the documents? . Can't we just use a generic x.y.z, or a more generic statement? When you open

[PATCH] doc:it_IT: translations for documents in process/

2019-02-24 Thread Federico Vaga
Translated documents: - stable-kernel-rules.rst - deprecated.rst - kernel-enforcement-statement.rst - license-rules.rst Added document to have valid links - netdev-FAQ.rst Modifications to main documentation - add label in deprecated.rst Signed-off-by: Federico Vaga --- Documentation/process

Re: DMA Engine Documentation: TX Descriptor and Submission

2019-02-01 Thread Federico Vaga
On February 1, 2019 4:17:50 AM UTC, Vinod Koul wrote: >On 28-01-19, 09:47, Federico Vaga wrote: >> Hi, >> >> I have a new question concerning documentation. >> >> >https://www.kernel.org/doc/html/latest/driver-api/dmaengine/client.html >> >

Re: dmaengine: usage of dmaengine_get() and dma_find_channel()

2019-01-23 Thread Federico Vaga
Thanks for your suggestion On Wednesday, January 23, 2019 1:15:21 PM CET Vinod Koul wrote: > On 21-01-19, 17:41, Federico Vaga wrote: > > Hello, > > > > I am a bit puzzle about the proper usage these two functions: > > > > void dmaengine_get(void) > >

[PATCH] doc:it_IT: update coding-style - expectations around bool

2019-01-23 Thread Federico Vaga
This patch translates in Italian the content of the following patch 7967656ffbfa coding-style: Clarify the expectations around bool Signed-off-by: Federico Vaga --- .../it_IT/process/coding-style.rst| 43 --- 1 file changed, 38 insertions(+), 5 deletions(-) diff

[PATCH] doc:it_IT: add translations in process/

2019-01-19 Thread Federico Vaga
This patch adds the Italian translation for the following documents in Documentation/process: - applying-patches - submit-checklist - submitting-drivers - changes - stable api nonsense Signed-off-by: Federico Vaga --- .../translations/it_IT/doc-guide/sphinx.rst | 2 + .../it_IT/process

[PATCH] doc:it_IT: documentation alignment

2019-01-19 Thread Federico Vaga
It aligns the italian translation with the latest changes: ae67ee6c5e1d docs: fix Co-Developed-by docs 3fe5dbfef47e Documentation/process/coding-style.rst: don't use "extern" with function prototypes Signed-off-by: Federico Vaga --- Documentation/translations/it_IT/process/coding

Re: [PATCH v5] coding-style: Clarify the expectations around bool

2019-01-20 Thread Federico Vaga
Signed-off-by: Jason Gunthorpe Acked-by: Federico Vaga --- Documentation/process/coding-style.rst | 38 +++--- scripts/checkpatch.pl | 13 - 2 files changed, 34 insertions(+), 17 deletions(-) v5: - Wording updates from JoeyP, MatthewW, FedericoV

[PATCH] doc:process: add missing internal link in stable-kernel-rules

2019-01-20 Thread Federico Vaga
Keep consistent the document. In the document, option references are always linked, except for the one I fixed with this patch Signed-off-by: Federico Vaga --- Documentation/process/stable-kernel-rules.rst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation

Re: [PATCH] doc:it_IT: add translations in process/

2019-01-21 Thread Federico Vaga
On Monday, January 21, 2019 2:56:17 AM CET Jonathan Corbet wrote: > On Sat, 19 Jan 2019 23:13:41 +0100 > > Federico Vaga wrote: > > This patch adds the Italian translation for the following documents > > in Documentation/process: > > > > - applyi

Re: [PATCH] doc:process: remove note from 'stable api nonsense'

2019-01-21 Thread Federico Vaga
On Monday, January 21, 2019 2:43:38 AM CET Jonathan Corbet wrote: > On Fri, 18 Jan 2019 22:58:04 +0100 > > Federico Vaga wrote: > > The link referred by the note can't be retrieved: this patch just > > remove that old note. > > > > Signed-off-by: Federico Vaga

dmaengine: usage of dmaengine_get() and dma_find_channel()

2019-01-21 Thread Federico Vaga
Hello, I am a bit puzzle about the proper usage these two functions: void dmaengine_get(void) struct dma_chan *dma_find_channel(enum dma_transaction_type tx_type) Looking into the kernel sources dma_find_channel is not used, despite the fact that it looks as one of the suggested mechanism to

[PATCH V2] doc:it_IT: add translations in process/

2019-01-21 Thread Federico Vaga
This patch adds the Italian translation for the following documents in Documentation/process: - applying-patches - submit-checklist - submitting-drivers - changes - stable api nonsense Signed-off-by: Federico Vaga --- .../translations/it_IT/doc-guide/sphinx.rst | 2 + .../it_IT/process

Re: [PATCH] doc:process: remove note from 'stable api nonsense'

2019-01-22 Thread Federico Vaga
On Monday, January 21, 2019 9:37:08 AM CET Greg KH wrote: > On Mon, Jan 21, 2019 at 09:14:00AM +0100, Federico Vaga wrote: > > On Monday, January 21, 2019 2:43:38 AM CET Jonathan Corbet wrote: > > > On Fri, 18 Jan 2019 22:58:04 +0100 > > > > > > Federico Va

Re: [PATCH v4] coding-style: Clarify the expectations around bool

2019-01-13 Thread Federico Vaga
} - # check for semaphores initialized locked if ($line =~ /^.\s*sema_init.+,\W?0\W?\)/) { WARN("CONSIDER_COMPLETION", -- Federico Vaga http://www.federicovaga.it/

<    1   2   3   4   5   >