Add a property to configure maximum prediction error when
FBC is enabled.
Change-Id: Iad17d567e7e8b9a0bd73df8d59a34e364c2acf8e
Signed-off-by: Jeevan Shriram <jshriram@codeaurora.org>
We now support high resolutions like 4K that need higher b/w votes.
Increase b/w vote to 2GBps for the duration of the continuous splash
screen logo.
Change-Id: I2c73e98ea769b252bee51d953d7cdc24f2b6668a
Signed-off-by: Siddhartha Agrawal <agrawals@codeaurora.org>
While handing off pipes from LK to kernel during the continuous
splash screen support, we were iterating through the HW cursor
pipes causing a crash. Only iterate through non cursor pipes.
Change-Id: I1223d34b374c5517c68beaaaa1df8d21f0fe8b83
Signed-off-by: Siddhartha Agrawal <agrawals@codeaurora.org>
Framebuffer commpression(FBC) 2.0 has new bit fields added to the
registers. Configure FBC registers correctly based on the dtsi
configuration. FBC 2.0 has increased the compression ratio to 1/3
and has configurable block size(in lines).
Crs-Fixed: 723612
Change-Id: Icf3c9b29d31f0e3214d6c7d525369fc7398dd9bb
Signed-off-by: Siddhartha Agrawal <agrawals@codeaurora.org>
When entering doze mode, do not turn off the interface clocks since
display updates are expected in this mode. This also avoid unnecessary
delays when entering doze mode while trying to wait for the clocks to be
turned off.
Change-Id: Ie6658a561f5d49aff50881c561fef858f3d26c61
Signed-off-by: Aravind Venkateswaran <aravindh@codeaurora.org>
When doze mode is requested while the panel in powered off, the
panel needs to be first unblanked and then configured into low power
mode. While this sequence exists in the current implementation,
the execution sequence only calls MDP's on function while unblanking
the panel. The correct approach is to go through the full unblank
sequence which would also start the display commit thread. Without
this, screen updates while in doze mode will not work and could
lead to unexpected behaviors.
Change-Id: I4f65764acaf01d61d129e00179930af9ebb79c77
Signed-off-by: Aravind Venkateswaran <aravindh@codeaurora.org>
For video mode panels, MDSS hardware needs to be on as long as the
panel is on. When doze mode is requested for a video mode panel,
no special optimizations can be done in the display driver to
configure the hardware in a low power state. As such, when doze mode is
requested for video mode panels, simply unblank the panel if it is not
already on.
Change-Id: I0f279d78b9b5c8eebb2bf654df628acafa408f23
Signed-off-by: Aravind Venkateswaran <aravindh@codeaurora.org>
Add MDSS_DSI_LINK_READY support to mdp3 driver, since
8909 uses DSI 6g.
Change-Id: Iccb752329d2116f09c31a522c53f54c675da1cb7
Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
Move double bufferring mode support property for dsi controller
to dtsi as it is not supported on 8909.
Change-Id: I2d99c34af7d4b9eef8f4503bad6fcd1480b2bde6
Signed-off-by: Shivaraj Shetty <shivaraj@codeaurora.org>
This change adds mdss gdsc changes to mdp3 driver of
8909 platform and also throws error when iommu attach
fails in driver.
Change-Id: I6a0f18b19535e7450f0f9c3fad9b3c30178b9754
Signed-off-by: Shivaraj Shetty <shivaraj@codeaurora.org>
Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
[imaund@codeaurora.org: Resolved trivial context conflicts.]
Signed-off-by: Ian Maund <imaund@codeaurora.org>
This change updates the bandwidth calculation for mdp3
hardware based on 8909 requirements, otherwise under run
is seen when USB is disconnected.
Change-Id: I49f7c225b57c43eca5ba8c6ac7b889e53cce6037
Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
Set Correct unpack pattern for ARGB888, XRGB888 and BGR565
color formats to avoid the color swap.
Change-Id: I9e3aa5018ea87be34122b3c1a80b94fc8cec3967
Signed-off-by: Shivaraj Shetty <shivaraj@codeaurora.org>
In case of histogram collection call failure due to histogram is not
enabled, or histogram state is unknown, return ENODATA instead of
EINVALID.
Change-Id: I6996345a19a43752827195b9edf786d62f9cbc75
Signed-off-by: Ping Li <pingli@codeaurora.org>
If mixer swap or interface swap is enabled on split display, disable
synchronized flush to prevent under-runs.
Change-Id: I21592fd5ef682a2ee8f2bcdaba33d6bcf015e5d4
Signed-off-by: Ujwal Patel <ujwalp@codeaurora.org>
With many updates coming the postprocessing driver, there needs to be
a way to manage the multiple versions of programming sequences and
interface structures, without adding to the hardware agnostic code path.
The pp_feature_ops structure will be used to decouple the PP driver from
the PP hardware, such that the hardware-specific code is separated to a
different file
Change-Id: I604850c082c32f5741905bd509ae0e819e334749
Signed-off-by: Benet Clark <benetc@codeaurora.org>
[cip@codeaurora.org: Moved mdss_mdp_pp_v1_7.c file location]
Signed-off-by: Clarence Ip <cip@codeaurora.org>
Layer mixer count is used for configuring SSPP(source side post
processing), DSPP(Destination side post processing) modules. There can
be a mismatch between the number of layer mixers with SSPP, DSPP
modules. Driver should be using the sspp, dspp counts instead of using
the layer mixer count. This change udpates the logic to use the module
count specific to pipe instead of layer mixer count
Change-Id: I32e32ed233d9c7026135087d05458e8af13d72e7
Signed-off-by: Ping Li <pingli@codeaurora.org>
MDP hardware consists of postprocessing hardware modules which are part
of different layers. Offsets of the post processing block can change
based on the MDP version. Moving these module offsets into device tree
makes the driver agnostic of mdp version. This change adds support to
parse the offsets from device
Change-Id: I586c69d0f6217cda9bb7d4ce47aa563c827ea531
Signed-off-by: Ping Li <pingli@codeaurora.org>
MDP hardware supports inverse gamma correction(IGC) on the source pipes.
Driver will program the LUT(Look up table) on the source pipe and
enable the IGC. Register exposes a bit mask which is equal to number of
pipes for which IGC LUT table needs to updated.
Source pipe enum values are not contiguos in nature which was resulting
in incorrect programming of IGC module.
This change fixes the issue by assigning bit mask position instead of
relying on the source pipe enum values.
Change-Id: I219cc484edd42fdd563dcb7535ceacb11e9e2d38
Signed-off-by: Ping Li <pingli@codeaurora.org>
MDP_SMP_FORCE_ALLOC flag is used to allow SMP
allocations even when there is mismatch between
allocated and requested SMPs for a pipe.
User space can make use of this flag in extreme scenarios
where SMP allocations need to pass like GPU composition
and playback of protected or secure videos.
Change-Id: I369b4361e7e2bbfc8150add467678c4ef8d5cfb6
Signed-off-by: Justin Philip <jphili@codeaurora.org>
UI may update in doze mode and there may be tearing when going
from doze to unblank - with respect to synchronization, doze mode
must be treated as on.
Change-Id: I8635b0722bc6294f6850ab28ce43046f81ed49a5
Signed-off-by: Naseer Ahmed <naseer@codeaurora.org>
Update the capabilities sysfs node with the max mixer width
supported to have userspace read and add checks for pipe
configurations.
Change-Id: I1431b8d06538b68d0dab4ac1698bd2a93331ad1c
Signed-off-by: Jeevan Shriram <jshriram@codeaurora.org>
When partial update and sync-wait-broadcast enable, col_page
address dcs command are sent to panel on every frame update
which generates unnecessary interrupts and cause frame rate
to dropped. To avoid unnecessary interrupts, col_page address
dcs commands are sent only when frame size changed. Since
sync-wait-broadcast enabled, col_page address of both
left and right dsi controller need to be sent at same
trigger.
CRs-Fixed: 707502
Change-Id: I37b8f78edf4fa75fa4aea9ca5d57ae0efd10b36c
Signed-off-by: Kuogee Hsieh <khsieh@codeaurora.org>
At split display case, DSI panel initialization dcs
commands need to be sent to panel when both dsi links
are in LP-11 state. Add MDSS_EVENT_LINK_READY to have
both dsi controller's phy/pll initialized which brings
link to LP-11 state first so that panel initialization
dcs commands be received by panel successfully to bring
panel to display_on state.
Change-Id: I4b33fafd2673be1f068819bee25fc1a335eb0fe8
Signed-off-by: Kuogee Hsieh <khsieh@codeaurora.org>
For some command mode panels, the HW Vsync signal from the panel
stops or becomes irregular if the panel goes bad due to ESD attack.
For such panels, the Vsync signal irregularity can be considered
as a trigger for recovery due to ESD attack. The ESD thread
interval needs to be set based on the irregularity pattern seen
that is specific to the panel. Add support for this.
Change-Id: I8a2408ac1b2c0e063446f8af60ed6fac4c53cb8c
Signed-off-by: Padmanabhan Komanduru <pkomandu@codeaurora.org>
[imaund@codeaurora.org: Resolved trivial context conflicts.]
Signed-off-by: Ian Maund <imaund@codeaurora.org>
Change the final fence wait timeout to be 7s for a total of 10s
Change the display operation timeout to be 1s more than that
instead of the excessively large timeout.
Change-Id: If9ae04f9e5993d754b6ccbfdd9e80ec46960e73b
Signed-off-by: Naseer Ahmed <naseer@codeaurora.org>
Optional dtsi properties are not present for all targets
for MDP dtsi configuration. Parsing failure for such property
should be reported under debug message instead of error message.
Change-Id: I91c966ebffd1bab3d29e047fa2ff5446e7de1b33
Signed-off-by: Dhaval Patel <pdhaval@codeaurora.org>
Currently iommu map call is done in client's context during overlay
play, this requires iommu attachment to correctly map buffers.
This mapping can be delayed until the actual commit and avoid
unnecessary delays in client's context. Also, only unmap buffers if they
have been mapped, instead of checking if iommu is attached.
Modified to work one kernel 3.14, iommu changes.
Change-Id: I802ec83a5af52ce6f372e8bda3ceb2646fb9cc55
Signed-off-by: Terence Hampson <thampson@codeaurora.org>
Current driver does not configure the pipe if the requested
data is same as the data already in the pipe.
This is a problem and can cause underruns in cases where
the pipe is freed due some error and driver tries to
setup previous configuration.
These changes make sure that if pipe was freed due some error,
driver will not skip the re-configuration of the pipe.
Change-Id: I36fc1644b4bdf413b308f7f7b5edcf97dcc8ec0b
Signed-off-by: Ingrid Gallardo <ingridg@codeaurora.org>
Add support for DSI data lane workaround if the DSI data lanes
are in stop state and not active for MDSS v1.5 and v1.9.
Also, always check for the DSI lane status when the frame is
being sent to DSI host.
Change-Id: Id1cfcdcaf709c571840449e4517db760d5058c90
Signed-off-by: Jeevan Shriram <jshriram@codeaurora.org>
Currently, for dual DSI cases while turning off the DSI clocks,
the DSI clamps are getting enabled before turning off the DSI PLL.
Due to this, there are DSI PLL lock failures seen while enabling
the clocks back while coming out of idle screen on command
mode panels. Change the clock sequence by making sure to enable
the clamps only after the DSI PLL is disabled. Also while turning
ON the clocks, make sure to disable the clamps before the DSI PLL
is enabled.
Change-Id: I545f17b85866553d1ff1cea42a6eaa2a84f8f014
Signed-off-by: Padmanabhan Komanduru <pkomandu@codeaurora.org>
[imaund@codeaurora.org: Resolved minor merge conflict in
drivers/video/msm/mdss/msm_mdss_io_8974.c.]
Signed-off-by: Ian Maund <imaund@codeaurora.org>
when wfd is tear down, HW still fetch the address of buffer queue
which was already released and while cancelling the buffer cause the
iommu page fault. Call wait4comp before kickoff is called to avoid
iommu page fault.
Change-Id: Ia597b395e13072374cfe3d5fa8216363946bff16
Signed-off-by: Raghavendra Ambadas <rambad@codeaurora.org>
FBC lines prefill is option property but MDSS driver probe
returns failure if this property is not defined. This change
handles it as optional property.
Change-Id: I8d1813fbbc2f5aef9bf81535e1640c9c25c8a784
Signed-off-by: Dhaval Patel <pdhaval@codeaurora.org>
We were calling free_bootmem_late, which is a __init function,
from a none __init context. We are moving to free_reserved_page.
Change-Id: I53b3b781201d46a6894f8d9fdfab1c3c5f3802b8
Signed-off-by: Terence Hampson <thampson@codeaurora.org>
There are use-cases where pipe is reused with a lower resolution than
previous one. In such cases, it is possible that SMP requirement is lower
than before. Current implementation will reject pipe configuration where
any SMP change is requested. This may lead to GPU fall-back option and
eventually consume more power. But on high end targets we have enough
number of SMPs available for use such that we can still allow the use-case
and not run out of SMPs to use. Based on this knowledge, change the logic
to allow extra SMPs during pipe reuse.
Change-Id: Icad5ca284a6b5ec1810d65bb1755d2f9572db7f0
Signed-off-by: Ujwal Patel <ujwalp@codeaurora.org>
Currently, when BLANK is called to turn off the panel during suspend, we
set the backlight to level 0. The backlight pwm driver interprets this
as requiring pwm_disable() only. However, in order to prevent any
garbage data from being displayed on the panel during resume, we should
set the pwm level to 0 before disabling the pwm backlight node.
Change-Id: Ifc12c49cbfcac76a5ca07e8c5e16b0fd023e1eeb
CRs-Fixed: 711782
Signed-off-by: Benet Clark <benetc@codeaurora.org>
Add fudge factor to ib vote for scenarios with high
bandwidth request whenever a single pipe is present,
this is to account for inefficiencies as recommended.
Change-Id: I6bdfb8d71c61f58a26fae95bdbd5bc9a2d043957
Signed-off-by: Ingrid Gallardo <ingridg@codeaurora.org>
BWC decode works at 2 pixels/clk and MDP clock requirement needs to be
adjusted to account for this. If BWC decode is enabled and MDP clock is
not sufficient then real-time pipeline might suffer from throughput
delays and might result in under-runs. Prevent this by calculating MDP
clock rate when BWC decode is needed and then choose max MDP clock rate
by comparing normal MDP clock and bwc MDP clock.
Change-Id: I92afc5a3c7ec29602ffdeec1ada0d67c57c4497f
Signed-off-by: Ujwal Patel <ujwalp@codeaurora.org>
The Color Space Conversion hardware is only instantiated once as
CSC_1. There is no CSC_0 hardware in the MDSS. Remove any mention of CSC_0
in order to be clear.
Change-Id: Ie7f8ef4dbb6e5f6ba1550020938dc101fec64800
Signed-off-by: Benet Clark <benetc@codeaurora.org>
The play count will not be reset to 0 in static screen case.
However, we still need to reset the CSC setup because the
registers would be reset.
Change-Id: Id2f6d9317b4da561ec31b6454166704493880b81
CRs-Fixed: 716190
Signed-off-by: Benet Clark <benetc@codeaurora.org>
Enable partial update for MDP3. Use ROI provided by
HAL and program the DMA pipe accordingly.
Change-Id: I2dd2d59bf178383b6139a1432274efa516e21fcc
Signed-off-by: Dileep Kumar Reddi <dkumarre@codeaurora.org>
This code change modifies the mdp3 clock control
based on 8909 platform clock plan.
Change-Id: I316fd51ec05d4eb1d1d3a138bbb672b036f1bbc0
Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
When trying to boot a dual dsi panel with single dsi configs,
the kernel takes the pref-panel for dsi1. This change avoids the
pref-panel selection when the dsi1 panel string is none.
Change-Id: I433d81a3813f359ed0ba513a3d5412b1c476bccc
Signed-off-by: Veera Sundaram Sankaran <veeras@codeaurora.org>
Setting the correct return value for successful ion memory mapping
and handling the unmapping for ion memory.
Change-Id: I005ed3c4b09ddc9e559fa41929f11beb702d373d
Signed-off-by: Veera Sundaram Sankaran <veeras@codeaurora.org>
UBWC format is supported for pipe input and WB0 & WB1
output for MDPv(1.7). This new format also has specific
buffer configuration requirement for bit stream and
meta data. This change adds UBWC support in MDSS
driver and put checks for format support in various
use cases and register configuration.
Change-Id: I29c262a94461d7571bead63f60517875eaaa5e5b
Signed-off-by: Dhaval Patel <pdhaval@codeaurora.org>
MDSS driver defines new input format for all fetch
type modes Ex: linear vs tile mode. Such redifintion
in driver is not scallable for supporting new formats
in future. This cleans up fetch mode specific definition
from MDSS driver and make it ready to accept new input/
output image formats for future targets.
Change-Id: Ia2bb6684f7eb6c06323734c9c9bbdacb788c51e0
Signed-off-by: Dhaval Patel <pdhaval@codeaurora.org>
Remove logs which are showing up quite frequently and
doesn't help in debugging.
Change-Id: Ibdc24bce17c798cbd0697ef00718b478a2c20a32
Signed-off-by: Ajay Singh Parmar <aparmar@codeaurora.org>
Replace completion structures with waitq and track the completion of
ping pong transfer with a single kickoff count variable. This prevents
race conditions when using completion structure and provides a cleaner
way to identify whether wait is required.
CRs-Fixed: 720541
Change-Id: Iebb8077d520649db427470fc4963d892967a920c
Signed-off-by: Adrian Salido-Moreno <adrianm@codeaurora.org>
Added appropriate error code for hardware limitation on mdp overlay
vertical scaling. On encountering the error, it is expected to ignore
the error and fallback to GPU.
Change-Id: Ib3a7767a856ed4d80632aabd7e371d2766b53dc1
Signed-off-by: Veera Sundaram Sankaran <veeras@codeaurora.org>
