Dmitry Torokhov wrote:

On Mon, Aug 02, 2010 at 03:30:57PM +0900, Kukjin Kim wrote:

quoted

Joonyoung Shim wrote:

quoted

On 8/2/2010 2:12 PM, Kukjin Kim wrote:

quoted

Joonyoung Shim wrote:

quoted

On 8/2/2010 12:38 PM, Kukjin Kim wrote:

quoted

Joonyoung Shim wrote:

quoted

Hi, I came back from vacation.

quoted

quoted

quoted

quoted

Do you have any feedback? If ok, i want to go to input tree

or

samsung

quoted

quoted

quoted

quoted

quoted

quoted

quoted

quoted

quoted

quoted

tree these added sched.h including.

I applied the patch adding samsung-keypad.c and the platform

data

quoted

quoted

quoted

quoted

quoted

quoted

quoted

quoted

quoted

arch/arm/plat-samsung/include/plat/keypad.h and I expect that
patches to actually enable keypad will go through appropriate
platform tree(s).

OK, the rest will go through samsung platform tree

Ok..will apply.

Kukjin, could you remove keypad.h on the commit "ARM: SAMSUNG:

Add

quoted

quoted

quoted

quoted

quoted

quoted

keypad device support" of the samsung platform tree. The keypad.h

file

quoted

quoted

quoted

quoted

quoted

quoted

was included on input tree.

(http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=commit;h=0fffed2
7f92d9d7

quoted

quoted

quoted

quoted

quoted

quoted

a34de9fe017b7082b5958bb93)

You mean in plat-samsung/dev-keypad.c?
If remove inclusion plat/keypad.h in there, following build error

happens.

quoted

quoted

quoted

quoted

quoted

No. I mean to remove arch/arm/plat-samsung/include/plat/keypad.h

file on

quoted

quoted

quoted

quoted

samsung platform tree, it was applied on input tree already. If

needs, i

quoted

quoted

quoted

quoted

can repost.

If it is not small change like to remove a file in a patch, I think

should be re-

quoted

quoted

submitted.

quoted

Will drop your previous patch in my tree.

quoted

quoted

arch/arm/plat-samsung/dev-keypad.c:41: warning: 'struct

samsung_keypad_platdata' declared inside parameter list

quoted

arch/arm/plat-samsung/dev-keypad.c:41: warning: its scope is only

this

quoted

quoted

definition

quoted

quoted

or declaration, which is probably not what you want

quoted

arch/arm/plat-samsung/dev-keypad.c: In function

'samsung_keypad_set_platdata':

quoted

arch/arm/plat-samsung/dev-keypad.c:50: error: invalid application

of

'sizeof'

quoted

quoted

to

quoted

quoted

incomplete type 'struct samsung_keypad_platdata'

quoted

arch/arm/plat-samsung/dev-keypad.c:54: error: dereferencing

pointer to

quoted

quoted

quoted

quoted

incomplete type

quoted

arch/arm/plat-samsung/dev-keypad.c:55: error: dereferencing

pointer to

quoted

quoted

quoted

quoted

incomplete type

quoted

arch/arm/plat-samsung/dev-keypad.c:55: error:

'samsung_keypad_cfg_gpio'

quoted

quoted

quoted

quoted

undeclared (first use in this function)

quoted

arch/arm/plat-samsung/dev-keypad.c:55: error: (Each undeclared

identifier

is

quoted

quoted

quoted

quoted

reported only once

quoted

arch/arm/plat-samsung/dev-keypad.c:55: error: for each function it

appears in.)

quoted

quoted

quoted

quoted

quoted

make[1]: *** [arch/arm/plat-samsung/dev-keypad.o] Error 1

As you know, released 35 today. So we have no much time for this

merge

quoted

quoted

quoted

quoted

window.
I'm not sure about merge rule, but I think the compile problem can

be

quoted

quoted

quoted

quoted

solved by merge to 36 of input tree and samsung platform tree.

Hmm...actually, already informed about that on linux-next.
If it's just build problem, I or Dmitry can fix it. However, this is

not just that.

quoted

quoted

quoted

I cannot understand how your 'plat/keypad.h' was included to both of

patch.

quoted

quoted

Please see the prior mail history. The keypad.h file is included only

at

quoted

quoted

the 1/3 patch in the original patch set, but Dmitry applied the 3/3
patch modified with keypad.h.

The basic problem is that each patches be applied on two tree. It

makes

quoted

quoted

compile errors at the one tree lacking keypad.h file because both tree
share keypad.h file.

There are two solutions, First, all patches go to one tree. Second, we
wait for merging to 36 of both tree then apply platform patches.

quoted

And I'm not sure, should be 'plat/keypad.h' in which tree...

Hi Dmitry,

How can/should I do for it?...Which way is better to us? :-)


Joonyoung,
Let's wait for Dmitry's reply.
I will holding drop your previous patch in my tree till that time.

Kgene,

I will be sending pull request to Linus in the next day or so, so it
will have plat/keypad.h as it is in my tree. Once you merge (with his or
mine) you have an option to patching the plat/keypad.h with hooks that I
removed, although I would prefer if you changed the way you initialize
your devices since having the these hooks defeats the purpose of having
config pointer in platform data. Another option would be to have a
separate include file for the hooks, one that
drivers/input/keyboard/samsung-keypad.c does not use.

Thanks.

Hi Dmitry,

Hmm...I saw your pull request to Linus...ok...will replace new one which
don't have duplicate 'plat/keypad.h' in my tree.
But..in this case, I'm not sure which way is better to us...

Thanks.

Best regards,
Kgene.
--
Kukjin Kim [off-list ref], Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.