summaryrefslogtreecommitdiffstats
path: root/drivers/staging/sm7xxfb
Commit message (Collapse)AuthorAgeFilesLines
* staging: sm7xxfb: merge sm712fb with fbdevSudip Mukherjee2015-08-075-1795/+0
| | | | | | | | Now since all cleanups are done and the code is ready to be merged lets move it out of staging into fbdev location. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: usr fb_read and fb_writeSudip Mukherjee2015-07-161-4/+0
| | | | | | | | | Now since the Big-Endian and Little-Endian based calculations are moved into a macro we can make fb_read() and fb_write() common for both Little-Endian and Big-Endian. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: define new macrosSudip Mukherjee2015-07-162-38/+29
| | | | | | | | Define and use some new macros to work with different situations based on little-endian and big-endian. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: use kernel commandlineSudip Mukherjee2015-07-141-2/+0
| | | | | | | | | | We were only using the kernel commandline to set the mode if this driver is builtin, but when it is built as a module we were not having any way to set the mode. Start using commandline even if it is built as a module. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix error handlingSudip Mukherjee2015-07-141-7/+8
| | | | | | | | | We were checking smtc_regbaseaddress and that too at a place where it can never be NULL. Real check should be on sfb->lfb immediately after we do ioremap. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove unused macrosSudip Mukherjee2015-07-141-20/+0
| | | | | | | These macros were only defined but not used anywhere. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix camelcaseSudip Mukherjee2015-06-172-2/+2
| | | | | | | Fix the checkpatch warning about CamelCase. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix indentionSudip Mukherjee2015-06-171-650/+650
| | | | | | | Fix the odd indention of vesa_mode_table. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: move mode tableSudip Mukherjee2015-06-172-660/+660
| | | | | | | | move vgamode table from the header file to the C file and mark it as const. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: reduce indentionSudip Mukherjee2015-06-171-91/+82
| | | | | | | reduce code indention keeping the logic same. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix alignmentSudip Mukherjee2015-06-171-10/+7
| | | | | | | Fix the alignment. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: no space after castSudip Mukherjee2015-06-171-3/+3
| | | | | | | fixes the checkpatch warning about using space after typecast. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: add newlineSudip Mukherjee2015-06-171-6/+6
| | | | | | | | new line was missing in some of the dev_* macros while printing the debug and error messages. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove numvgamodesSudip Mukherjee2015-06-172-3/+1
| | | | | | | | numvgamodes was only used in one place, so remove the #define and use its defined value. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: declare struct as constSudip Mukherjee2015-06-171-1/+1
| | | | | | | The vesa_mode_table is not to be modified, so declare it as const. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove unused functionsSudip Mukherjee2015-05-081-32/+0
| | | | | | | | removed the smtc_alloc_fb_info() and smtc_free_fb_info() functions which were not used anymore. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: use framebuffer_alloc and releaseSudip Mukherjee2015-05-081-98/+110
| | | | | | | | | | | | | | use the standard framebuffer_alloc() and framebuffer_release() instead of custom defined function. for making that change we had to change a member of the private structure from a variable to pointer and had to touch almost all places of the file. since fb was changed into a pointer so all instance of "sfb->fb." has been changed into "sfb->fb->". now we will get build warning about smtc_alloc_fb_info() and smtc_free_fb_info() to be unused which will be removed in the next patch of the series. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: disable pci deviceSudip Mukherjee2015-04-031-0/+1
| | | | | | | disable the pci device when the module exits. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: reserve PCI resourceSudip Mukherjee2015-04-031-0/+10
| | | | | | | | before starting to access any address inside the PCI region we should reserve the resource and release the resource when the module exits. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: add MODULE_DEVICE_TABLESudip Mukherjee2015-03-261-0/+2
| | | | | | | add MODULE_DEVICE_TABLE to support hot-plugging. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: change return of sm7xx_vga_setupSudip Mukherjee2015-03-201-5/+3
| | | | | | | | | | change return type of sm7xx_vga_setup() to void as we are not using the return type anywhere. if this function fails we fall back to the default configuration. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove unneeded __setupSudip Mukherjee2015-03-201-1/+0
| | | | | | | | as we start to use kernel boot parameters and fb_get_options() we donot need to use __setup() any more. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: use module init and exitSudip Mukherjee2015-03-201-1/+25
| | | | | | | | use module_init() and module_exit() instead of module_pci_driver and at the same time make way for use of kernel boot parameters. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: make vgamode staticSudip Mukherjee2015-03-061-1/+1
| | | | | | | | the variable vgamode is used only in this file and hence can be safely made as static. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix remaining CamelCaseSudip Mukherjee2015-02-072-38/+38
| | | | | | | | since mixed case names are not encouraged in coding, so those has been changed to their corresponding lowercase version. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix CamelCaseSudip Mukherjee2015-02-072-13/+13
| | | | | | | | since mixed case names are not encouraged in coding, so those has been changed to their corresponding lowercase version. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: make smtc_scr_info staticMax Perepelitsyn2015-02-071-1/+1
| | | | | | | | This symbol is never used anywhere else besides sm7xxfb.c Signed-off-by: Max Perepelitsyn <mperepelitsyn@gmail.com> Tested-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: fix alignmentSudip Mukherjee2015-01-251-3/+3
| | | | | | | checkpatch cleanup: alignment should match open parenthesis Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove unnecessary blank linesSudip Mukherjee2015-01-251-4/+0
| | | | | | | | checkpatch cleanup: blank lines are not necessary before closing brace and after opening brace. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: no space is necessary after a castSudip Mukherjee2015-01-251-6/+6
| | | | | | | checkpatch cleanup: space is not necessary after cast Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: add missing blank lineSudip Mukherjee2015-01-251-0/+2
| | | | | | | checkpatch cleanup to add missing blank line after declaration Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: update TODO fileSudip Mukherjee2015-01-251-3/+6
| | | | | | | | update the email addresses in the TODO file, also update the final destination of this driver. Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Revert "staging: sm7xxfb: remove driver"Sudip Mukherjee2015-01-255-0/+1828
| | | | | | | This reverts commit "dc93c85235efa5201e9a3c116bc3fbd1afc1a182" Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove driverGreg Kroah-Hartman2014-02-285-1828/+0
| | | | | | | | | | | It hasn't been worked on in a very long time, and the original author has moved on to a different product as this one is no longer being made. So remove the driver. If someone wants to resurect it, and clean it up and get it merged to the "proper" part of the kernel, this commit can be reverted. Cc: Teddy Wang <teddy.wang@siliconmotion.com.cn> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: sm7xxfb: Fix quoted string split across lines in sm7xxfb.cMonam Agarwal2014-02-271-4/+2
| | | | | | | | This patch fixes the following checkpatch.pl issues in sm7xxfb.c: WARNING: Quoted string split across lines Signed-off-by: Monam Agarwal <monamagarwal123@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* drivers: sm7xxfb: Mark function as static in sm7xxfb.cRashika Kheria2014-01-081-1/+1
| | | | | | | | | | | | | Mark function smtcfb_setmode() as static in sm7xxfb.c because it is not used outside this file. This eliminates the following warning in sm7xxfb.c: drivers/staging/sm7xxfb/sm7xxfb.c:588:6: warning: no previous prototype for ‘smtcfb_setmode’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria <rashika.kheria@gmail.com> Reviewed-by: Josh Triplett <josh@joshtriplett.org> Acked-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: remove DEFINE_PCI_DEVICE_TABLE macroJingoo Han2013-12-021-1/+1
| | | | | | | | Don't use DEFINE_PCI_DEVICE_TABLE macro, because this macro is not preferred. Signed-off-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: sm7xxfb: fix line over 80 characters in sm7xxfb.cEbru Akagunduz2013-10-111-1/+2
| | | | | | | | Fix checkpatch.pl issues with line over 80 characters in sm7xxfb.c Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: sm7xxfb: fixed line break coding style issuesMartin Berglund2013-09-301-7/+5
| | | | | | | Fixed coding style issues. Signed-off-by: Martin Berglund <martin@rogsta.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: Remove redundant pci_set_drvdataSachin Kamat2013-09-251-1/+0
| | | | | | | Driver core sets driver data to NULL upon failure or remove. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging/sm7xxfb: Convert to SIMPLE_DEV_PM_OPSPeter Huewe2013-03-111-9/+1
| | | | | | | | | Instead of assigning the pm_ops fields individually we can simply use SIMPLE_DEV_PM_OPS. Signed-off-by: Peter Huewe <peterhuewe@gmx.de> Acked-by: Javier Muñoz <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove use of __devexitBill Pemberton2012-11-211-1/+1
| | | | | | | | CONFIG_HOTPLUG is going away as an option so __devexit is no longer needed. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove use of __devinitBill Pemberton2012-11-211-1/+1
| | | | | | | | CONFIG_HOTPLUG is going away as an option so __devinit is no longer needed. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove use of __devexit_pBill Pemberton2012-11-211-1/+1
| | | | | | | | CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer needed. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: remove smtc_VRAMBaseAddressJavier M. Mellid2012-09-171-14/+7
| | | | | | | This patch erases smtc_VRAMBaseAddress variable. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: rename pFramebufferPhysical to mmio_baseJavier M. Mellid2012-09-171-7/+7
| | | | | | | This patch renames pFramebufferPhysical name to mmio_base. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: rename some smtcfb_info fieldsJavier M. Mellid2012-09-171-26/+26
| | | | | | | This patch improves naming style and comments on smtcfb_info fields. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: annotate iomem pointersJavier M. Mellid2012-09-172-8/+8
| | | | | | | This patch annotates iomem pointers. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: cleanup on smtc_alloc_fb_infoJavier M. Mellid2012-09-171-13/+7
| | | | | | | This patch improves code legibility after last changes. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: sm7xxfb: clean smtcfb_fix's id initializationJavier M. Mellid2012-09-171-6/+4
| | | | | | | | Setting up smtcfb_fix's id happens through smtc_alloc_fb_info. It adds complexity and unnecesary code. Signed-off-by: Javier M. Mellid <jmunhoz@igalia.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>