<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-dev/drivers/net/fddi/skfp, branch master</title>
<subtitle>Linux kernel development work - see feature branches</subtitle>
<id>https://git.zx2c4.com/linux-dev/atom/drivers/net/fddi/skfp?h=master</id>
<link rel='self' href='https://git.zx2c4.com/linux-dev/atom/drivers/net/fddi/skfp?h=master'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/'/>
<updated>2022-08-12T10:20:29Z</updated>
<entry>
<title>skfp/h: fix repeated words in comments</title>
<updated>2022-08-12T10:20:29Z</updated>
<author>
<name>Jilin Yuan</name>
<email>yuanjilin@cdjrlc.com</email>
</author>
<published>2022-08-10T13:59:01Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=86d2155e48f6ce1aacbd01667176e5b915ae275c'/>
<id>urn:sha1:86d2155e48f6ce1aacbd01667176e5b915ae275c</id>
<content type='text'>
Delete the redundant word 'the'.

Signed-off-by: Jilin Yuan &lt;yuanjilin@cdjrlc.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>fddi/skfp: fix repeated words in comments</title>
<updated>2022-07-11T13:12:54Z</updated>
<author>
<name>Jilin Yuan</name>
<email>yuanjilin@cdjrlc.com</email>
</author>
<published>2022-07-08T15:03:54Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=edb2c3476db9898a63fb5d0011ecaa43ebf46c9b'/>
<id>urn:sha1:edb2c3476db9898a63fb5d0011ecaa43ebf46c9b</id>
<content type='text'>
Delete the redundant word 'test'.

Signed-off-by: Jilin Yuan &lt;yuanjilin@cdjrlc.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>net: fddi: skfp: smt: Remove extra parameters to vararg macro</title>
<updated>2022-05-22T22:05:56Z</updated>
<author>
<name>Tom Rix</name>
<email>trix@redhat.com</email>
</author>
<published>2022-05-22T13:36:27Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=2a11fb1d1b8501c430472c66ea5390cb719bbd8f'/>
<id>urn:sha1:2a11fb1d1b8501c430472c66ea5390cb719bbd8f</id>
<content type='text'>
cppcheck reports
[drivers/net/fddi/skfp/smt.c:750]: (warning) printf format string requires 0 parameters but 2 are given.

DB_SBAN is a vararg macro, like DB_ESSN.  Remove the extra args and the nl.

Signed-off-by: Tom Rix &lt;trix@redhat.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>net:Remove initialization of static variables to 0</title>
<updated>2021-12-27T14:53:00Z</updated>
<author>
<name>Wen Zhiwei</name>
<email>wenzhiwei@kylinos.cn</email>
</author>
<published>2021-12-27T08:22:01Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=b4aadd207322d243c3ceef107c495117898a36f1'/>
<id>urn:sha1:b4aadd207322d243c3ceef107c495117898a36f1</id>
<content type='text'>
Delete the initialization of three static variables
because it is meaningless.

Signed-off-by: Wen Zhiwei &lt;wenzhiwei@kylinos.cn&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>net: fddi: use swap() to make code cleaner</title>
<updated>2021-11-15T14:19:17Z</updated>
<author>
<name>Yihao Han</name>
<email>hanyihao@vivo.com</email>
</author>
<published>2021-11-15T06:58:16Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=311107bdecd158870ab8556fd1289a56a6087bad'/>
<id>urn:sha1:311107bdecd158870ab8556fd1289a56a6087bad</id>
<content type='text'>
Use the macro 'swap()' defined in 'include/linux/minmax.h' to avoid
opencoding it.

Signed-off-by: Yihao Han &lt;hanyihao@vivo.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>fddi: skfp: constify and use dev_addr_set()</title>
<updated>2021-10-22T17:16:05Z</updated>
<author>
<name>Jakub Kicinski</name>
<email>kuba@kernel.org</email>
</author>
<published>2021-10-21T13:12:08Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=2e0566aeb9ff83db9fb22bf6f0b994f03377b038'/>
<id>urn:sha1:2e0566aeb9ff83db9fb22bf6f0b994f03377b038</id>
<content type='text'>
Get it ready for constant netdev-&gt;dev_addr.

Signed-off-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
</content>
</entry>
<entry>
<title>fddi: use eth_hw_addr_set()</title>
<updated>2021-10-02T13:18:26Z</updated>
<author>
<name>Jakub Kicinski</name>
<email>kuba@kernel.org</email>
</author>
<published>2021-10-01T21:32:27Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=4d3d2c8dba360e5f8a9bf9d4b4f7d829e9812593'/>
<id>urn:sha1:4d3d2c8dba360e5f8a9bf9d4b4f7d829e9812593</id>
<content type='text'>
Convert from memcpy(), include is needed.

Signed-off-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>net: fddi: skfp: Fix a function name in comments</title>
<updated>2021-09-27T11:29:33Z</updated>
<author>
<name>Cai Huoqing</name>
<email>caihuoqing@baidu.com</email>
</author>
<published>2021-09-25T12:52:08Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=064d0171d7ee5005020da631135d4d6dc4b72695'/>
<id>urn:sha1:064d0171d7ee5005020da631135d4d6dc4b72695</id>
<content type='text'>
Use dma_map_single() instead of pci_map_single(),
because only dma_map_single() is called here.

Signed-off-by: Cai Huoqing &lt;caihuoqing@baidu.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>fddi: switch from 'pci_' to 'dma_' API</title>
<updated>2021-08-29T09:50:24Z</updated>
<author>
<name>Christophe JAILLET</name>
<email>christophe.jaillet@wanadoo.fr</email>
</author>
<published>2021-08-27T20:00:57Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=a3ba7fd1d3bf6ce68a06632e755c18ae56cb1a7a'/>
<id>urn:sha1:a3ba7fd1d3bf6ce68a06632e755c18ae56cb1a7a</id>
<content type='text'>
In [1], Christoph Hellwig has proposed to remove the wrappers in
include/linux/pci-dma-compat.h.

Some reasons why this API should be removed have been given by Julia
Lawall in [2].

A coccinelle script has been used to perform the needed transformation
Only relevant parts are given below.

It has been compile tested.

@@ @@
-    PCI_DMA_TODEVICE
+    DMA_TO_DEVICE

@@ @@
-    PCI_DMA_FROMDEVICE
+    DMA_FROM_DEVICE

@@
expression e1, e2, e3, e4;
@@
-    pci_map_single(e1, e2, e3, e4)
+    dma_map_single(&amp;e1-&gt;dev, e2, e3, e4)

@@
expression e1, e2, e3, e4;
@@
-    pci_unmap_single(e1, e2, e3, e4)
+    dma_unmap_single(&amp;e1-&gt;dev, e2, e3, e4)

[1]: https://lore.kernel.org/kernel-janitors/20200421081257.GA131897@infradead.org/
[2]: https://lore.kernel.org/kernel-janitors/alpine.DEB.2.22.394.2007120902170.2424@hadrien/

Signed-off-by: Christophe JAILLET &lt;christophe.jaillet@wanadoo.fr&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>fddi: use ndo_siocdevprivate</title>
<updated>2021-07-27T19:11:43Z</updated>
<author>
<name>Arnd Bergmann</name>
<email>arnd@arndb.de</email>
</author>
<published>2021-07-27T13:44:59Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=043393d8b4786534cd070e0c071a2716313c5558'/>
<id>urn:sha1:043393d8b4786534cd070e0c071a2716313c5558</id>
<content type='text'>
The skfddi driver has a private ioctl and passes the data correctly
through ifr_data, but the use of a pointer in s_skfp_ioctl is
broken in compat mode.

Change the driver to use ndo_siocdevprivate and disallow calling
it in compat mode until a conversion handler is added.

Cc: "Maciej W. Rozycki" &lt;macro@orcam.me.uk&gt;
Signed-off-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
</feed>
