> Hi Dave,
>
> Warnings are fixed. The pull request...
Close but no cigar,
CC [M] drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.o
/home/airlied/devel/kernel/drm-next/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c:
In function âfsl_dcu_drm_probeâ:
/home/airlied/devel/kernel/drm-next/drivers/gpu/drm/fs
CC [M] drivers/gpu/drm/amd/amdgpu/amdgpu_cs.o
/home/airlied/devel/kernel/drm-next/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c:
In function âamdgpu_cs_parser_initâ:
/home/airlied/devel/kernel/drm-next/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c:211:21:
warning: cast to pointer from integer of different
> Due to some recent changes in
> drm_helper_probe_single_connector_modes_merge_bits(), old custom modes
> were not being pruned properly. In current kernels,
> drm_mode_validate_basic() is called to sanity-check each mode in the
> list. If the sanity-check passes, the mode's status gets set to to
From: Dave Airlie
We apparantly get a hotplug irq before we've initialised
modesetting,
[drm] Loading R100 Microcode
BUG: unable to handle kernel NULL pointer dereference at (null)
IP: [] __mutex_lock_slowpath+0x23/0x91
*pde =
Oops: 0002 [#1]
Modules linked in: radeon(+) drm_kms_helpe
On 19 August 2015 at 00:28, Meelis Roos wrote:
> Hi, I tried 4.2-rc7 and todays 4.2-rc7+git on a P4 PC with Intel 850
> chipset and old Radeon graphics. The machine crashes during boot and
> starts spamming dmesg as fast as it scrolls. Netconsole caught the
> dmesg. 4.1.0 worked fine.
>
> The firs
https://bugzilla.kernel.org/show_bug.cgi?id=102401
--- Comment #7 from Maxqia ---
Created attachment 185301
--> https://bugzilla.kernel.org/attachment.cgi?id=185301&action=edit
dmesg log
--
You are receiving this mail because:
You are watching the assignee of the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=102401
--- Comment #8 from Maxqia ---
It works with fglrx, also in the dmesg log it says
"[drm:drm_detect_monitor_audio] Monitor has basic audio support"
Which is when it should output true
--
You are receiving this mail because:
You are watching the
On 20 August 2015 at 00:48, Yakir Yang wrote:
>
> Hi all,
>The Samsung Exynos eDP controller and Rockchip RK3288 eDP controller
> share the same IP, so a lot of parts can be re-used. I split the common
> code into bridge directory, then rk3288 and exynos only need to keep
> some platform code.
All chagnes in CRTCs of Exynos are applied in H/W, only when the protection
for shadow register update is not enabled. Using these protection, every
change can be applied at once. Now exynos_crtc_atomic_{begin/flush} stops
or starts the update of shadow registers by calling each CRTC specific
atomi
Each CRTC's atomic_{begin/flush} must stop/start the update of shadow
registers to active register in the functions. This patch achieves these
purpose by moving the setting of protection bits to those functions from
{fimd/decon}_update_plane.
Signed-off-by: Hyungwon Hwang
---
drivers/gpu/drm/exy
Hi Emil,
Can I get you RB now? It looks like there was no more comment from others.
Regards,
Jammy
-Original Message-
From: Emil Velikov [mailto:emil.l.veli...@gmail.com]
Sent: Monday, August 17, 2015 10:13 PM
To: Zhou, Jammy
Cc: ML dri-devel
Subject: Re: [PATCH 1/4] drm: add interface
dri-devel/attachments/20150820/518ebe87/attachment.html>
On 20.08.2015 08:50, Dave Airlie wrote:
> CC [M] drivers/gpu/drm/amd/amdgpu/amdgpu_cs.o
> /home/airlied/devel/kernel/drm-next/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c:
> In function âamdgpu_cs_parser_initâ:
> /home/airlied/devel/kernel/drm-next/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c:211:21:
>
this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/1eaf6fbe/attachment-0001.html>
e bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/28acc5bb/attachment.html>
Hi Thierry, Lucas,
On 08/19/2015 08:32 PM, Thierry Reding wrote:
> On Wed, Aug 19, 2015 at 04:52:24PM +0200, Lucas Stach wrote:
>> Am Mittwoch, den 19.08.2015, 16:34 +0200 schrieb Thierry Reding:
>>> On Wed, Aug 19, 2015 at 04:17:08PM +0200, Lucas Stach wrote:
Hi Thierry, Archit,
>> [..
Hi,
On 08/19/2015 08:18 PM, Yakir Yang wrote:
>
> Hi all,
> The Samsung Exynos eDP controller and Rockchip RK3288 eDP controller
> share the same IP, so a lot of parts can be re-used. I split the common
> code into bridge directory, then rk3288 and exynos only need to keep
> some platform code
On 2015. 8. 20., at PM 1:29, Archit Taneja wrote:
>
> Hi,
>
>> On 08/19/2015 08:18 PM, Yakir Yang wrote:
>>
>> Hi all,
>>The Samsung Exynos eDP controller and Rockchip RK3288 eDP controller
>> share the same IP, so a lot of parts can be re-used. I split the common
>> code into bridge direct
Hi Jingoo & Archit,
On 08/20/2015 12:54 AM, Jingoo Han wrote:
> On 2015. 8. 20., at PM 1:29, Archit Taneja wrote:
>> Hi,
>>
>>> On 08/19/2015 08:18 PM, Yakir Yang wrote:
>>>
>>> Hi all,
>>> The Samsung Exynos eDP controller and Rockchip RK3288 eDP controller
>>> share the same IP, so a lot o
Hi, Tiago!
On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
> Hey Thomas, you haven't answered my email about making SYNC_* mandatory:
>
> http://lists.freedesktop.org/archives/dri-devel/2015-August/088376.html
Hmm, for some reason it doesn't show up in my mail app, but I found it
in the archives. A
On 2015. 8. 20., at PM 3:23, Yakir Yang wrote:
>
> Hi Jingoo & Archit,
>
>
>> On 08/20/2015 12:54 AM, Jingoo Han wrote:
>>> On 2015. 8. 20., at PM 1:29, Archit Taneja
>>> wrote:
>>> Hi,
>>>
On 08/19/2015 08:18 PM, Yakir Yang wrote:
Hi all,
The Samsung Exynos eDP contr
> On 19 August 2015 at 00:28, Meelis Roos wrote:
> > Hi, I tried 4.2-rc7 and todays 4.2-rc7+git on a P4 PC with Intel 850
> > chipset and old Radeon graphics. The machine crashes during boot and
> > starts spamming dmesg as fast as it scrolls. Netconsole caught the
> > dmesg. 4.1.0 worked fine.
>
Hi Jingoo,
On 08/20/2015 01:55 AM, Jingoo Han wrote:
> On 2015. 8. 20., at PM 3:23, Yakir Yang wrote:
>> Hi Jingoo & Archit,
>>
>>
>>> On 08/20/2015 12:54 AM, Jingoo Han wrote:
On 2015. 8. 20., at PM 1:29, Archit Taneja
wrote:
Hi,
> On 08/19/2015 08:18 PM, Yakir Yang wro
Hi Dave, one more batch of i915 fixes for v4.2.
Revert of a VBT parsing commit that should've been queued for drm-next,
not v4.2. The revert unbreaks Braswell among other things.
Also on Braswell removal of DP HBR2/TP3 and intermediate eDP frequency
support. The code was optimistically added bas
;ll pick this up.
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/2d958dbf/attachment.sig>
Hi Ravi,
I'm wondering is your e-mail come from eDP thread ? cause I see lots of
cc guys some as eDP emails :)
And for your question, I am not sure I understand rightly. Do you mean
that your ".ko" module not in
the same directory with driver source code?
If it's your question, I think you can
---
> 1 file changed, 13 insertions(+), 25 deletions(-)
Acked-by: Tomi Valkeinen
Tomi
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.f
gt; >On the other hand if you properly instantiate the DSI device you can
> >easily write a driver for it, and the driver will set up the correct
> >properties as implied by the compatible string. Once you have that you
> >can easily hook it up to the I2C control interface in whatever way you
> >like, be that an OF graph or just a simple unidirectional link by
> >phandle.
> >
>
> With the fused approach you suggested, we would have 2 drivers: one i2c
> and the other dsi. The i2c client driver would be more or less minimal,
> preparing an i2c_client device for the dsi driver to use. Is my
> understanding correct?
Correct. That's kind of similar to the way an HDMI encoder driver would
use an I2C adapter to query EDID. The i2c_client device would be a means
for the DSI driver to access the control interface.
> We can do without dummy dsi devices with this method. But, representing
> a device with 2 DT nodes seems a bit off. We'd also need to compatible
> strings for the same device, one for the i2c part, and the other for
> the dsi part.
I agree that this somewhat stretches the capabilities of device tree.
Another alternative I guess would be to not have a compatible string for
the I2C device at all (that's technically not valid, I guess) because we
really don't need an I2C driver for the device. What we really need is a
DSI driver with a means to talk over some I2C bus with some other part
of its device.
> From an adv75xx driver perspective, it should also support the ADV7511
> chip, which is a RGB/DPI to HDMI encoder. For adv7511, we don't need a
> DSI DT node. It would be a bit inconsistent to have the bindings
> require both DSI and I2C nodes for one chip, and only I2C node for the
> other, with both chips being supported by the same driver.
Why would that be inconsistent? That sounds like the most accurate
representation of the hardware to me.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/ffa63d2d/attachment.sig>
//lists.freedesktop.org/archives/dri-devel/attachments/20150820/47c780b1/attachment.html>
On Thu, Aug 20, 2015 at 2:48 AM, Thomas Hellstrom
wrote:
> Hi, Tiago!
>
> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
>> Hey Thomas, you haven't answered my email about making SYNC_* mandatory:
>>
>> http://lists.freedesktop.org/archives/dri-devel/2015-August/088376.html
>
> Hmm, for some reaso
On Thu, Aug 20, 2015 at 08:48:23AM +0200, Thomas Hellstrom wrote:
> Hi, Tiago!
>
> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
> > Hey Thomas, you haven't answered my email about making SYNC_* mandatory:
> >
> > http://lists.freedesktop.org/archives/dri-devel/2015-August/088376.html
>
> Hmm, fo
Seems like git send-email went awol when formatting this, and git am
wasn't able to pick it up :(
On the good side, I've noticed that we can/should drop amdgpu.h from
the LIBDRM_AMDGPU_FILES list. So I've amended that and pushed these
two.
Thank you Mauro.
Emil
Seems like git send-email went awol when formatting this, and git am
wasn't able to pick it up :(
On the good side, I've noticed that we can/should drop amdgpu.h from
the LIBDRM_AMDGPU_FILES list. So I've amended that and pushed these
two.
Thank you Mauro.
Emil
Hi Hyungwon,
On 19 August 2015 at 01:58, Hyungwon Hwang wrote:
> This patch seprates the code, which sorts proprty sets and eliminates
> duplicate properties, from drmModeAtomicCommit(). Now
> drmModeAtomicCleanup() has to do the job before calling
> drmModeAtomicCommit(), and drmModeAtomicCommit
On 19 August 2015 at 01:58, Hyungwon Hwang wrote:
> This patch makes 'struct _drmModeAtomicReqItem' and 'struct
> _drmModeAtomicReq' visible from outside. This is needed for userspace
> applications to use those structures when calling drmModeAtomicCommit().
>
Hmmm what is missing in the current A
On 20 August 2015 at 04:02, Zhou, Jammy wrote:
> Hi Emil,
>
> Can I get you RB now? It looks like there was no more comment from others.
>
I'll ping (again) one the possible users and if we haven't heard any
objection by tomorrow midday I'll commit it to master.
Cheers,
Emil
for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/1bb80cf8/attachment.html>
On 08/20/2015 04:33 PM, Rob Clark wrote:
> On Thu, Aug 20, 2015 at 2:48 AM, Thomas Hellstrom
> wrote:
>> Hi, Tiago!
>>
>> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
>>> Hey Thomas, you haven't answered my email about making SYNC_* mandatory:
>>>
>>> http://lists.freedesktop.org/archives/dri-de
On 08/20/2015 09:27 PM, Thomas Hellstrom wrote:
> On 08/20/2015 04:33 PM, Rob Clark wrote:
>> On Thu, Aug 20, 2015 at 2:48 AM, Thomas Hellstrom
>> wrote:
>>> Hi, Tiago!
>>>
>>> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
Hey Thomas, you haven't answered my email about making SYNC_* mandato
On 08/20/2015 04:53 PM, Jerome Glisse wrote:
> On Thu, Aug 20, 2015 at 08:48:23AM +0200, Thomas Hellstrom wrote:
>> Hi, Tiago!
>>
>> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
>>> Hey Thomas, you haven't answered my email about making SYNC_* mandatory:
>>>
>>> http://lists.freedesktop.org/archiv
This patch set implements support for i2s audio and new AMD GPUs.
The i2s codec is fed by a DMA engine on the GPU. To handle this
we create an mfd cell which we can hang the i2s codec on. Because
of this, this patch set covers two subsystems: drm and alsa. The
drm patches add support for the ACP
From: Maruthi Bayyavarapu
This adds the ACP (Audio CoProcessor) IP driver and wires
it up to the amdgpu driver. The ACP block provides the DMA
engine and bus for the i2s codec which is supported by an
alsa driver. This is required for audio on APUs that
utilize an i2s codec.
v2: integrate i2s/
From: Maruthi Srinivas Bayyavarapu
ACP IP block consists of dedicated DMA and I2S blocks. The PCM driver
provides the DMA and CPU DAI components to ALSA core.
v2: squash in Kconfig fix
v3: squash additional commits, convert to mfd, drop rt286 changes
v4: squash in naming fixes
Signed-off-by: Ma
for_each_entry(connector, &mode_config->connector_list,
> head)
> --
> 2.4.3
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
------ n
On Thu, Aug 20, 2015 at 09:39:12PM +0200, Thomas Hellstrom wrote:
> On 08/20/2015 04:53 PM, Jerome Glisse wrote:
> > On Thu, Aug 20, 2015 at 08:48:23AM +0200, Thomas Hellstrom wrote:
> >> Hi, Tiago!
> >>
> >> On 08/20/2015 12:33 AM, Tiago Vignatti wrote:
> >>> Hey Thomas, you haven't answered my em
ur
help.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/73acbcb0/attachment.html>
11 from the oibaf ppa
and a self-compiled xf86-video-amdgpu.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20
ed by commit e60b344f
> ("drm/amdgpu: optimize amdgpu_parser_init") need to stay.
>
Attached patch should fix it.
Alex
-- next part --
A non-text attachment was scrubbed...
Name: 0001-drm-amdgpu-fix-warnings-on-32bit-ARM.patch
Type: text/x-patch
Size: 1777 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/66ed9070/attachment.bin>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150820/6890073f/attachment-0001.html>
vel/attachments/20150820/40ac5ac2/attachment.html>
Signed-off-by: Bas Nieuwenhuizen
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h| 1 +
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index 2fc58e6..454d4e2 100644
Signed-off-by: Bas Nieuwenhuizen
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h| 1 +
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 16 +++-
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index 454d4e2
attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/03ec6889/attachment.html>
receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/d7b6923b/attachment.html>
On Thu, Aug 20, 2015 at 5:13 PM, Mark Brown wrote:
> On Thu, Aug 20, 2015 at 04:09:21PM -0400, Alex Deucher wrote:
>> From: Maruthi Srinivas Bayyavarapu
>>
>> ACP IP block consists of dedicated DMA and I2S blocks. The PCM driver
>> provides the DMA and CPU DAI components to ALSA core.
>>
>> v2: s
From: Maruthi Srinivas Bayyavarapu
ACP IP block consists of dedicated DMA and I2S blocks. The PCM driver
provides the DMA and CPU DAI components to ALSA core.
Signed-off-by: Maruthi Bayyavarapu
Reviewed-by: Alex Deucher
Reviewed-by: Murali Krishna Vemuri
Signed-off-by: Alex Deucher
---
v2:
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/235fa9ac/attachment.html>
On Thu, Aug 20, 2015 at 6:31 PM, Mark Brown wrote:
> On Thu, Aug 20, 2015 at 05:30:27PM -0400, Alex Deucher wrote:
>> On Thu, Aug 20, 2015 at 5:13 PM, Mark Brown wrote:
>
>> >> v4: squash in naming fixes
>
>> > To repeat what I said last time:
>
>> > | Please follow the patch submission process i
On Mon, Aug 17, 2015 at 10:02:04AM +0300, Jani Nikula wrote:
> On Fri, 14 Aug 2015, Rafael Antognolli wrote:
> > On Fri, Aug 14, 2015 at 02:56:55PM +0300, Jani Nikula wrote:
> >> On Wed, 12 Aug 2015, Thierry Reding wrote:
> >> > From: Thierry Reding
> >> >
> >> > The new drm_dp_dpcd_dump() helpe
From: Maruthi Srinivas Bayyavarapu
ACP IP block consists of dedicated DMA and I2S blocks. The PCM driver
provides the DMA and CPU DAI components to ALSA core.
Signed-off-by: Maruthi Bayyavarapu
Reviewed-by: Alex Deucher
Reviewed-by: Murali Krishna Vemuri
Signed-off-by: Alex Deucher
---
v2:
On Thu, Aug 20, 2015 at 7:18 PM, Mark Brown wrote:
> On Thu, Aug 20, 2015 at 05:36:34PM -0400, Alex Deucher wrote:
>> From: Maruthi Srinivas Bayyavarapu
>>
>> ACP IP block consists of dedicated DMA and I2S blocks. The PCM driver
>> provides the DMA and CPU DAI components to ALSA core.
>
> This is
On Thu, Aug 20, 2015 at 7:45 PM, Mark Brown wrote:
> On Thu, Aug 20, 2015 at 07:11:22PM -0400, Alex Deucher wrote:
>> On Thu, Aug 20, 2015 at 6:31 PM, Mark Brown wrote:
>
>> > DRM is a special case here since there has always been work to share
>> > bits of the code with other operating systems,
On Thu, Aug 20, 2015 at 8:10 PM, Mark Brown wrote:
> On Thu, Aug 20, 2015 at 07:50:34PM -0400, Alex Deucher wrote:
>> On Thu, Aug 20, 2015 at 7:45 PM, Mark Brown wrote:
>
>> > This is not a DRM driver, it is an ASoC driver, and like I say DRM is a
>> > bit special. I do note that AMD has contrib
.
Done.
- Yakir
>>
>> Thanks
>> Kishon
>>
>>
>>
>
------ next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/1c59cc4f/attachment-0001.html>
On 2015. 8. 19., at PM 11:50, Yakir Yang wrote:
>
> link_rate and lane_count already configed in analogix_dp_set_link_train(),
s/configed/configured
Also, the commit name such as "fix ... bug" is not good.
How about following?
drm: bridge/analogix_dp: remove duplicate configuration of link rat
On 08/19/2015 11:33 PM, Eric B Munson wrote:
> On Wed, 12 Aug 2015, Michal Hocko wrote:
>
>> On Sun 09-08-15 01:22:53, Eric B Munson wrote:
>>
>> I do not like this very much to be honest. We have only few bits
>> left there and it seems this is not really necessary. I thought that
>> LOCKONFAULT a
Hi Jingoo,
On 08/20/2015 02:22 AM, Jingoo Han wrote:
> On 2015. 8. 19., at PM 11:50, Yakir Yang wrote:
>> link_rate and lane_count already configed in analogix_dp_set_link_train(),
> s/configed/configured
>
> Also, the commit name such as "fix ... bug" is not good.
> How about following?
>
> drm:
Due to some recent changes in
drm_helper_probe_single_connector_modes_merge_bits(), old custom modes
were not being pruned properly. In current kernels,
drm_mode_validate_basic() is called to sanity-check each mode in the
list. If the sanity-check passes, the mode's status gets set to to
MODE_OK. I
next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/5050f3f4/attachment-0001.sig>
Hi,
On Wednesday 19 August 2015 08:21 PM, Yakir Yang wrote:
> Signed-off-by: Yakir Yang
where's the commit message?
> ---
> Changes in v3:
> - Take Heiko suggest, add rockchip dp phy driver,
> collect the phy clocks and power control.
>
> Changes in v2: None
>
> .../devicetree/bindings/phy/
On 2015. 8. 19., at PM 11:52, Yakir Yang wrote:
>
> Some edp screen with no hpd signal would need some delay time
> to ensure that screen would be ready for work, so we can expand
> the delay time in hpd detect function, it works prefectly on my
> rk3288 sdk board.
Then, this delay has a depende
RR(phy);
}
>> +
>> +return 0;
>> +}
>> +
>> +static int rockchip_dp_phy_probe(struct platform_device *pdev)
>> +{
>> +struct device *dev = &pdev->dev;
>> +struct phy_provider *phy_provider;
>> +struct rockchip_dp_phy *dp;
>> +struct resource *res;
>> +struct phy *phy;
>> +int ret;
>> +
>> +dp = devm_kzalloc(dev, sizeof(*dp), GFP_KERNEL);
>> +if (IS_ERR(dp))
>> +return -ENOMEM;
>> +
>> +dp->dev = dev;
>> +
>> +res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>> +dp->regs = devm_ioremap_resource(dev, res);
>> +if (IS_ERR(dp->regs))
>> +return PTR_ERR(dp->regs);
>> +
>> +ret = rockchip_dp_phy_init(dp);
>> +if (ret)
>> +return ret;
>> +
>> +phy = devm_phy_create(dev, NULL, &rockchip_dp_phy_ops, NULL);
>> +if (IS_ERR(phy)) {
>> +dev_err(dev, "failed to create phy\n");
>> +return PTR_ERR(phy);
>> +}
>> +phy_set_drvdata(phy, dp);
>> +
>> +phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
>> +
>> +return PTR_ERR_OR_ZERO(phy_provider);
>> +}
>> +
>> +static const struct of_device_id rockchip_dp_phy_dt_ids[] = {
>> +{ .compatible = "rockchip,rk3288-dp-phy" },
>> +{}
>> +};
>> +
>> +MODULE_DEVICE_TABLE(of, rockchip_dp_phy_dt_ids);
>> +
>> +static struct platform_driver rockchip_dp_phy_driver = {
>> +.probe = rockchip_dp_phy_probe,
>> +.driver = {
>> +.name = "rockchip-dp-phy",
>> +.owner = THIS_MODULE,
> owner is not required here.
>
> Thanks
> Kishon
>
>
>
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/dea79811/attachment-0001.html>
On 2015. 8. 19., at PM 11:52, Yakir Yang wrote:
>
What is the reason to make this patch?
Please make commit message including the reason.
Best regards,
Jingoo Han
> Signed-off-by: Yakir Yang
> ---
> Changes in v3:
> - move dp hpd detect to connector detect function.
>
> Changes in v2: None
On Wed 19-08-15 17:33:45, Eric B Munson wrote:
[...]
> The group which asked for this feature here
> wants the ability to distinguish between LOCKED and LOCKONFAULT regions
> and without the VMA flag there isn't a way to do that.
Could you be more specific on why this is needed?
> Do we know that
Hi Jingoo,
On 08/20/2015 01:11 AM, Jingoo Han wrote:
> On 2015. 8. 19., at PM 11:52, Yakir Yang wrote:
>> Some edp screen with no hpd signal would need some delay time
>> to ensure that screen would be ready for work, so we can expand
>> the delay time in hpd detect function, it works prefectly o
Hi Jingoo,
On 08/20/2015 02:49 AM, Jingoo Han wrote:
> On 2015. 8. 19., at PM 11:52, Yakir Yang wrote:
>
> What is the reason to make this patch?
>
> Please make commit message including the reason.
Okay, I think the below words would be okay :)
"This change just make a little clean to make cod
-
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/28fda8d8/attachment-0001.sig>
e_status_suspended(dev);
> + if (pm_rts == true) {
There seem to be a lot of variables in here that have only one
assignment and one user immediately next to them. I'm not sure it's
adding much.
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/03ad110f/attachment-0001.sig>
(the licensing
strategy seems very similar). I'd need to think through this carefully.
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/70b01bb0/attachment-0001.sig>
s direct contributions from AMD there AFAICT, not just outside
developers contributing support for AMD chips.
-- next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/4e04ac30/attachment-0001.sig>
t attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150820/d783c843/attachment.sig>
81 matches
Mail list logo
