<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/include/linux/ti_wilink_st.h, branch linux-5.1.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.</subtitle>
<id>https://universe.0xinfinity.dev/distro/kernel/atom?h=linux-5.1.y</id>
<link rel='self' href='https://universe.0xinfinity.dev/distro/kernel/atom?h=linux-5.1.y'/>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/'/>
<updated>2016-07-17T17:59:26Z</updated>
<entry>
<title>drivers: misc: ti-st: Use int instead of fuzzy char for callback status</title>
<updated>2016-07-17T17:59:26Z</updated>
<author>
<name>Geert Uytterhoeven</name>
<email>geert@linux-m68k.org</email>
</author>
<published>2016-06-06T09:02:03Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=4360fa22ad5b48a1d1e10e31ffb383ed8c977435'/>
<id>urn:sha1:4360fa22ad5b48a1d1e10e31ffb383ed8c977435</id>
<content type='text'>
On mips and parisc:

    drivers/bluetooth/btwilink.c: In function 'ti_st_open':
    drivers/bluetooth/btwilink.c:174:21: warning: overflow in implicit constant conversion [-Woverflow]
       hst-&gt;reg_status = -EINPROGRESS;

    drivers/nfc/nfcwilink.c: In function 'nfcwilink_open':
    drivers/nfc/nfcwilink.c:396:31: warning: overflow in implicit constant conversion [-Woverflow]
      drv-&gt;st_register_cb_status = -EINPROGRESS;

There are actually two issues:
  1. Whether "char" is signed or unsigned depends on the architecture.
     As the completion callback data is used to pass a (negative) error
     code, it should always be signed.
  2. EINPROGRESS is 150 on mips, 245 on parisc.
     Hence -EINPROGRESS doesn't fit in a signed 8-bit number.

Change the callback status from "char" to "int" to fix these.

Signed-off-by: Geert Uytterhoeven &lt;geert@linux-m68k.org&gt;
Acked-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
Acked-by: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
Signed-off-by: Marcel Holtmann &lt;marcel@holtmann.org&gt;
</content>
</entry>
<entry>
<title>ti-st: use worker instead of calling st_int_write in wake up</title>
<updated>2015-09-21T05:52:13Z</updated>
<author>
<name>Muhammad Hamza Farooq</name>
<email>mfarooq@visteon.com</email>
</author>
<published>2015-09-11T14:42:38Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=b05b7c7cc0324524dcda7fa7c2be1255290ee416'/>
<id>urn:sha1:b05b7c7cc0324524dcda7fa7c2be1255290ee416</id>
<content type='text'>
The wake up method is called with the port lock held. The st_int_write
method calls port-&gt;ops-&gt;write with tries to acquire the lock again,
causing CPU to wait infinitely. Right way to do is to write data to port
in worker thread.

Signed-off-by: Muhammad Hamza Farooq &lt;mfarooq@visteon.com&gt;
Signed-off-by: Jacob Siverskog &lt;jacob@teenage.engineering&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>Revert "ti-st: add device tree support"</title>
<updated>2015-08-05T20:24:12Z</updated>
<author>
<name>Rob Herring</name>
<email>robh@kernel.org</email>
</author>
<published>2015-07-22T18:17:15Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=c0bd1b9e58959c51a4c939505f89721dfbc73c44'/>
<id>urn:sha1:c0bd1b9e58959c51a4c939505f89721dfbc73c44</id>
<content type='text'>
This reverts commit 46d0d33350e9b32642d745a8b46a954910196b4d.

This binding is horrible and never should have been merged. It is not
documented nor are there any in tree users, so reverting it will not
break anything we care about. Lets revert it before we do have users.

The problems with it are:

- It is not documented.

- The GPIO connection is described with a custom property and uses Linux
GPIO numbering.

- The UART connection is described using the Linux tty device name.

Cc: Gigi Joseph &lt;gigi.joseph@ti.com&gt;
Cc: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
Signed-off-by: Rob Herring &lt;robh@kernel.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>ti-st: clean up data types (fix harmless memory corruption)</title>
<updated>2015-02-03T23:48:51Z</updated>
<author>
<name>Dan Carpenter</name>
<email>dan.carpenter@oracle.com</email>
</author>
<published>2015-01-15T11:42:27Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=63e144c9d6ffa791c1402f4ee4551c1b9f5a336a'/>
<id>urn:sha1:63e144c9d6ffa791c1402f4ee4551c1b9f5a336a</id>
<content type='text'>
The big issue here is:

	of_property_read_u32(np, "flow_cntrl", (u32 *)&amp;dt_pdata-&gt;flow_cntrl);

"-&gt;flow_cntrl" is a char so when we write a 32 bit number to it then it
corrupts past the end of the char.  It's probably hard to notice because
the struct has padding so the code works on little endian systems. But
on a big endian system the code would fail and on a 64 bit, big endian
systems then "nshutdown_gpio" and "baud_rate" would be buggy as well.

Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>ti-st: add device tree support</title>
<updated>2015-01-12T13:04:10Z</updated>
<author>
<name>Gigi Joseph</name>
<email>gigi.joseph@gmail.com</email>
</author>
<published>2015-01-09T03:45:02Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=46d0d33350e9b32642d745a8b46a954910196b4d'/>
<id>urn:sha1:46d0d33350e9b32642d745a8b46a954910196b4d</id>
<content type='text'>
When using device tree, driver configuration data need to be read from
device node.
Add support for getting the platform data information from the device
tree information stored in the .dtb file in case it exists.

Signed-off-by: Eyal Reizer &lt;eyalr@ti.com&gt;
Signed-off-by: bvijay &lt;bvijay@ti.com&gt;
Diff rendering mode:inlineside by side

Signed-off-by: Gigi Joseph &lt;gigi.joseph@ti.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>misc: st_kim: Increase size of dev_name buffer to incorporate termination</title>
<updated>2014-09-24T06:14:28Z</updated>
<author>
<name>Lee Jones</name>
<email>lee.jones@linaro.org</email>
</author>
<published>2014-08-28T13:14:09Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=ffdbb715fa0c53203b1ea2a6ecc54bdcc8951612'/>
<id>urn:sha1:ffdbb715fa0c53203b1ea2a6ecc54bdcc8951612</id>
<content type='text'>
Calling strncpy with a maximum size argument of 32 bytes on destination
array kim_gdata-&gt;dev_name of size 32 bytes might leave the destination
string unterminated.

Signed-off-by: Lee Jones &lt;lee.jones@linaro.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>drivers/misc/ti-st: remove sparse warnings</title>
<updated>2012-08-16T17:11:45Z</updated>
<author>
<name>Pavan Savoy</name>
<email>pavan_savoy@ti.com</email>
</author>
<published>2012-08-03T19:49:40Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=27712b3928bec9b1a889d7f60d718a35ca6c23b3'/>
<id>urn:sha1:27712b3928bec9b1a889d7f60d718a35ca6c23b3</id>
<content type='text'>
remove sparse warnings by assigning right storage specifiers to functions and
also clean-up the declarations in the include/linux/ti_wilink_st.h

Signed-off-by: Pavan Savoy &lt;pavan_savoy@ti.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
</content>
</entry>
<entry>
<title>ARM: OMAP: WiLink platform data for the PandaBoard</title>
<updated>2012-05-09T21:19:13Z</updated>
<author>
<name>Mircea Gherzan</name>
<email>mgherzan@gmail.com</email>
</author>
<published>2012-05-09T21:19:13Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=9624f615520696ffa845da2de56277197155b48c'/>
<id>urn:sha1:9624f615520696ffa845da2de56277197155b48c</id>
<content type='text'>
The "uim" deamon requires sysfs entries that are filled in using
this platform data.

Signed-off-by: Mircea Gherzan &lt;mgherzan@gmail.com&gt;
Signed-off-by: Tony Lindgren &lt;tony@atomide.com&gt;
</content>
</entry>
<entry>
<title>drivers:misc:ti-st: platform hooks for chip states</title>
<updated>2011-08-22T21:13:32Z</updated>
<author>
<name>Pavan Savoy</name>
<email>pavan_savoy@ti.com</email>
</author>
<published>2011-08-10T15:18:31Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=0d7c5f2572ccfa7bf83292b1506926663f2d164a'/>
<id>urn:sha1:0d7c5f2572ccfa7bf83292b1506926663f2d164a</id>
<content type='text'>
Certain platform specific or Host-WiLink Interface specific actions would be
required to be taken when the chip is being enabled and after the chip is
disabled such as configuration of the mux modes for the GPIO of host connected
to the nshutdown of the chip or relinquishing UART after the chip is disabled.

Similar actions can also be taken when the chip is in deep sleep or when the
chip is awake. Performance enhancements such as configuring the host to run
faster when chip is awake and slower when chip is asleep can also be made
here.

Signed-off-by: Pavan Savoy &lt;pavan_savoy@ti.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;
</content>
</entry>
<entry>
<title>drivers:misc:ti-st: handle delayed tty receive</title>
<updated>2011-04-23T00:01:09Z</updated>
<author>
<name>Pavan Savoy</name>
<email>pavan_savoy@ti.com</email>
</author>
<published>2011-04-08T09:57:42Z</published>
<link rel='alternate' type='text/html' href='https://universe.0xinfinity.dev/distro/kernel/commit/?id=764b0c4b3256ad4431cb52eaf99c0abe6df0a085'/>
<id>urn:sha1:764b0c4b3256ad4431cb52eaf99c0abe6df0a085</id>
<content type='text'>
When certain technologies shutdown their interface without waiting for
the acknowledgement from the chip. The receive_buf from the TTY would be
invoked a while after the relevant technology is unregistered.

This patch introduces a new flag "is_registered" which maintains the
state of protocols BT, FM or GPS and thereby removes the need to clear
the protocol data from ST when protocols gets unregistered.

This fixes corner cases when HCI RESET is sent down from bluetooth stack
and the receive_buf is called from tty after 250ms before which
bluetooth would have unregistered from the system.
OR - when FM application decides to close down the device without
sending a power-off FM command resulting in some RDS data or interrupt
data coming in after the driver is unregistered.

Signed-off-by: Pavan Savoy &lt;pavan_savoy@ti.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;
</content>
</entry>
</feed>
