linux.git/drivers/staging/sm750fb, branch masterLinux Kernelhttps://git.etezian.org/cgit.cgi/linux.git/atom?h=master2016-11-30T11:59:53+00:00staging: sm750fb: fix tabstop style warning2016-11-30T11:59:53+00:00Andrea Ghittinoaghittino@gmail.com2016-11-29T21:57:21+00:00urn:sha1:f1706cb74e2ef85d7550aeafb5bb4a7534e06286
Fixes sm750fb tabstop style warning
found by checkpatch.pl tool
Signed-off-by: Andrea Ghittino <aghittino@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: fix a type issue in sm750_set_chip_type()2016-11-29T20:57:14+00:00Dan Carpenterdan.carpenter@oracle.com2016-11-26T10:20:58+00:00urn:sha1:d28fb1ffbaf4c3daffe10bf53f863dcb8845a3c1
"revId" needs to be unsigned because we use it to test:
if (revId == SM750LE_REVISION_ID) {
and SM750LE_REVISION_ID is ((unsigned char )0xfe).
Fixes: 81dee67e215b ("staging: sm750fb: add sm750 to staging")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: update license2016-11-29T20:57:14+00:00Sudip Mukherjeesudipm.mukherjee@gmail.com2016-11-26T09:07:42+00:00urn:sha1:4e0447cdaf60d00f0cabd78ef0ba1874e23ae464
The driver was actually released with BSD license. It also gained GPL
when it was submitted to be included in the kernel.
Cc: Teddy Wang <teddy.wang@siliconmotion.com>
Cc: gzhou1 <guojian.zhou@windriver.com>
Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Merge 4.9-rc5 into staging-next2016-11-14T07:53:56+00:00Greg Kroah-Hartmangregkh@linuxfoundation.org2016-11-14T07:53:56+00:00urn:sha1:ce0347c2b089eb6ad6c14ab628a0ba5eb42f1492
We want the staging/iio fixes in here as well to resolve issues and
merge problems.
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: prefix global identifiers2016-11-10T12:55:03+00:00Arnd Bergmannarnd@arndb.de2016-11-09T09:26:27+00:00urn:sha1:52d0744d751d8f13511df15e776460bfb95bcf5c
Renaming some symbols inside this driver caused a conflict with
an existing function, which in turn results in a link error:
drivers/staging/sm750fb/sm750fb.o: In function `enable_dma':
ddk750_hwi2c.c:(.text.enable_dma+0x0): multiple definition of `enable_dma'
This adds a sm750_ prefix to each global symbol in the sm750fb
driver that does not already have one. I manually looked for the
symbols and then converted the driver using
for i in calc_pll_value format_pll_reg set_power_mode set_current_gate \
enable_2d_engine enable_dma enable_gpio enable_i2c hw_set2dformat \
hw_de_init hw_fillrect hw_copyarea hw_imageblit hw_cursor_enable \
hw_cursor_disable hw_cursor_setSize hw_cursor_setPos \
hw_cursor_setColor hw_cursor_setData hw_cursor_setData2 ;
do
sed -i "s:\<$i\>:sm750_$i:" drivers/staging/sm750fb/*.[ch]
done
Fixes: 03140dabf584 ("staging: sm750fb: Replace functions CamelCase naming with underscores.")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: Replace pr_*() with dev_*().2016-11-07T10:31:12+00:00Elise Lennionelise.lennion@gmail.com2016-11-01T01:28:46+00:00urn:sha1:c56de0967a658cb773bf41c5e4ea26768267d8b1
dev_*() functions print identifying information about the struct device
and should be used instead of pr_*() whenever possible.
Signed-off-by: Elise Lennion <elise.lennion@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: Fix bugs introduced by early commits2016-11-07T09:27:40+00:00Huacai Chenchenhc@lemote.com2016-11-03T04:48:43+00:00urn:sha1:33c027ae3cfd8fefb6cccccc5c6b2c07d80891ce
Early commit 30ca5cb63c56965 ("staging: sm750fb: change definition of
PANEL_PLANE_TL fields") and 27b047bbe1ee9c0 ("staging: sm750fb: change
definition of PANEL_PLANE_BR fields") modify the register bit fields
definitions. But the modifications are wrong, because the bit mask of
"bit field 10:0" is not 0xeff, but 0x7ff. The wrong definition bugs
makes display very strange.
Signed-off-by: Huacai Chen <chenhc@lemote.com>
Cc: stable <stable@vger.kernel.org> # 4.6+
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: Shorten local variables names.2016-10-30T15:13:11+00:00Elise Lennionelise.lennion@gmail.com2016-10-28T00:24:27+00:00urn:sha1:c2b6028fd10617252c679b21b41204c531347a00
Shorter names are preferred by C variables naming convention, they are
easier to write and aren't more difficult to understand.
Signed-off-by: Elise Lennion <elise.lennion@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: Replace functions CamelCase naming with underscores.2016-10-30T15:13:11+00:00Elise Lennionelise.lennion@gmail.com2016-10-28T00:23:35+00:00urn:sha1:03140dabf584555e96440084d1ac426127bdf8c3
Replace CamelCase function names with underscores to comply with
the standard kernel coding style.
Signed-off-by: Elise Lennion <elise.lennion@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sm750fb: Refine code in set_current_gate().2016-10-30T15:13:11+00:00Elise Lennionelise.lennion@gmail.com2016-10-28T00:22:28+00:00urn:sha1:cefc2fc61e1f5c80c6240634da01a4b846e9a6db
The 'switch' statement in set_current_gate() had only two possible
scenarios, so it was replaced with an 'if' statement to make the code
shorter and easier to understand.
Signed-off-by: Elise Lennion <elise.lennion@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>