linux.kernel - 26 new messages in 15 topics - digest

linux.kernel
http://groups.google.com/group/linux.kernel?hl=en

linux.kernel@googlegroups.com

Today's topics:

* ARM: s3c64xx: pm: Use name field of generic_pm_domain - 5 messages, 2
authors
http://groups.google.com/group/linux.kernel/t/63098c6f35af189c?hl=en
* drm/i2c: tda998x: use irq for connection status and EDID read - 1 messages,
1 author
http://groups.google.com/group/linux.kernel/t/bbeb5edcf598e1bd?hl=en
* Input: uinput: add full absinfo support - 2 messages, 1 author
http://groups.google.com/group/linux.kernel/t/414bf329d2e7aa1f?hl=en
* kvm: x86: Fix debug typo error in lapic - 1 messages, 1 author
http://groups.google.com/group/linux.kernel/t/d2f26e1c9fa67436?hl=en
* vlynq: fix another resource size off by 1 error - 1 messages, 1 author
http://groups.google.com/group/linux.kernel/t/fce98e37fbcdbec3?hl=en
* iio: tsl2563: Initialize channels - 4 messages, 3 authors
http://groups.google.com/group/linux.kernel/t/56e6b7f9b9c61c17?hl=en
* lockdep: Kill held_lock->check and "int check" arg of __lock_acquire() - 2
messages, 1 author
http://groups.google.com/group/linux.kernel/t/07e5799772466cd4?hl=en
* netfilter: nf_conntrack: fix RCU race in nf_conntrack_find_get (v3) - 1
messages, 1 author
http://groups.google.com/group/linux.kernel/t/1f5bd631cfffab0f?hl=en
* ASoC: wm8804: Allow control of master clock divider in PLL generation - 1
messages, 1 author
http://groups.google.com/group/linux.kernel/t/8c4c236a2ac2503e?hl=en
* misc: xgene: Add support for APM X-Gene SoC Queue Manager/Traffic Manager -
1 messages, 1 author
http://groups.google.com/group/linux.kernel/t/c3af64c6af02e0ba?hl=en
* ARM: shmobile: compile drivers/sh for CONFIG_ARCH_SHMOBILE_MULTI - 2
messages, 1 author
http://groups.google.com/group/linux.kernel/t/39053332c9e18780?hl=en
* YOUR ATTENTION PLEASE! - 1 messages, 1 author
http://groups.google.com/group/linux.kernel/t/b9497329d9f08412?hl=en
* power: clock_ops.c: fixup clk prepare/unprepare count - 1 messages, 1 author
http://groups.google.com/group/linux.kernel/t/41367c40b0de6d9b?hl=en
* mm, memcg: avoid oom notification when current needs access to memory
reserves - 2 messages, 1 author
http://groups.google.com/group/linux.kernel/t/ddffce70f2a0b5fc?hl=en
* ppc/cell: use get_unused_fd_flags(0) instead of get_unused_fd() - 1 messages,
1 author
http://groups.google.com/group/linux.kernel/t/dca6ac990a255e1e?hl=en

==============================================================================
TOPIC: ARM: s3c64xx: pm: Use name field of generic_pm_domain
http://groups.google.com/group/linux.kernel/t/63098c6f35af189c?hl=en
==============================================================================

== 1 of 5 ==
Date: Sun, Jan 12 2014 11:30 am
From: Tomasz Figa




On 12.01.2014 20:20, Mark Brown wrote:
> On Sat, Jan 11, 2014 at 08:42:43PM +0100, Tomasz Figa wrote:
>
>> static struct s3c64xx_pm_domain s3c64xx_pm_irom = {
>> - .name = "IROM",
>> .ena = S3C64XX_NORMALCFG_IROM_ON,
>> .pd = {
>> + .name = "domain_irom",
>
> This is nitpicking a bit but are you sure this is actually a
> beautification of the name? It's mainly the domain_ bit, mostly since
> I'd expect that if it's not obvious that this is a power domain the core
> logging would be changed rather than tweaking the name of every power
> domain user.
>

Hmm, that's a really good point. A separate patch might change this and
I could drop the "controversial" part of this patch ;).

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/




== 2 of 5 ==
Date: Sun, Jan 12 2014 11:30 am
From: Mark Brown



On Sun, Jan 12, 2014 at 08:03:50PM +0100, Tomasz Figa wrote:

> Also, there are multiple devices in most domains, so that would
> rather end up being "Domain_I (JPEG, Cam I/F)", "Domain_P (2D, TV
> Enc., Scaler)", "Domain_F (Rot, Post, LCD)".

It's not just that there's multiple devices either, it's also the fact
that the groupings are a bit random and probably more to do with chip
layout or something than function.




== 3 of 5 ==
Date: Sun, Jan 12 2014 11:30 am
From: Mark Brown



On Sat, Jan 11, 2014 at 08:42:48PM +0100, Tomasz Figa wrote:

> This patch adds support for registering power domains of S3C64xx SoCs
> and binding devices to them using device tree.

> +#ifdef CONFIG_OF
> +static struct of_device_id s3c64xx_pd_matches[] = {
> + { .compatible = "samsung,s3c6400-clock", },
> + { .compatible = "samsung,s3c6410-clock", },
> + { },
> };

Looks like a rebasing issue? Otherwise this looks nice.




== 4 of 5 ==
Date: Sun, Jan 12 2014 11:30 am
From: Mark Brown



On Sat, Jan 11, 2014 at 08:42:43PM +0100, Tomasz Figa wrote:

> static struct s3c64xx_pm_domain s3c64xx_pm_irom = {
> - .name = "IROM",
> .ena = S3C64XX_NORMALCFG_IROM_ON,
> .pd = {
> + .name = "domain_irom",

This is nitpicking a bit but are you sure this is actually a
beautification of the name? It's mainly the domain_ bit, mostly since
I'd expect that if it's not obvious that this is a power domain the core
logging would be changed rather than tweaking the name of every power
domain user.




== 5 of 5 ==
Date: Sun, Jan 12 2014 11:40 am
From: Tomasz Figa


On 12.01.2014 20:29, Mark Brown wrote:
> On Sat, Jan 11, 2014 at 08:42:48PM +0100, Tomasz Figa wrote:
>
>> This patch adds support for registering power domains of S3C64xx SoCs
>> and binding devices to them using device tree.
>
>> +#ifdef CONFIG_OF
>> +static struct of_device_id s3c64xx_pd_matches[] = {
>> + { .compatible = "samsung,s3c6400-clock", },
>> + { .compatible = "samsung,s3c6410-clock", },
>> + { },
>> };
>
> Looks like a rebasing issue? Otherwise this looks nice.

Hmm, what's the problem here? Maybe I'm just blind, but I fail to see
anything wrong (except that the array should be const...).

Thanks for reviewing this.

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/





==============================================================================
TOPIC: drm/i2c: tda998x: use irq for connection status and EDID read
http://groups.google.com/group/linux.kernel/t/bbeb5edcf598e1bd?hl=en
==============================================================================

== 1 of 1 ==
Date: Sun, Jan 12 2014 11:30 am
From: Sebastian Hesselbarth


On 01/12/2014 07:51 PM, Jean-Francois Moine wrote:
> On Sat, 11 Jan 2014 19:35:21 +0100
> Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> wrote:
>
>> At least for the DT part, I'd suggest to not ask for interrupt directly
>> but use a proper gpios property. The can of course be converted to
>> priv->int_irq in some tda998x_dt_probe.
>
> May you give me more information?

Sure, see [1].

[1] http://lists.freedesktop.org/archives/dri-devel/2013-May/038822.html

Sebastian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/





==============================================================================
TOPIC: Input: uinput: add full absinfo support
http://groups.google.com/group/linux.kernel/t/414bf329d2e7aa1f?hl=en
==============================================================================

== 1 of 2 ==
Date: Sun, Jan 12 2014 11:40 am
From: Dmitry Torokhov


On Tue, Dec 17, 2013 at 04:48:51PM +0100, David Herrmann wrote:
> +
> +struct uinput_user_dev2 {
> + __u8 version;

It does not make sense to have version u8 since we going to have padding
(1 byte I believe) padding between name and id.

> + char name[UINPUT_MAX_NAME_SIZE];
> + struct input_id id;
> + __u32 ff_effects_max;
> + struct input_absinfo abs[ABS_CNT];
> +};
> +
>