Merge request of s5pc210 universal board.

Kukjin Kim kgene.kim at samsung.com
Fri Aug 6 07:57:27 EDT 2010


Kyungmin Park wrote:
> 
> On Fri, Aug 6, 2010 at 4:22 PM, Kyungmin Park <kmpark at infradead.org>
wrote:
> > On Fri, Aug 6, 2010 at 4:11 PM, Kukjin Kim <kgene.kim at samsung.com>
wrote:
> >> Kyungmin Park wrote:
> >>>
> >>> On Fri, Aug 6, 2010 at 1:47 PM, Kyungmin Park <kmpark at infradead.org>
> >> wrote:
> >>> > On Fri, Aug 6, 2010 at 1:43 PM, Kukjin Kim <kgene.kim at samsung.com>
> >> wrote:
> >>> >> Kyungmin Park wrote:
> >>> >>>
> >>> >>> Dear all,
> >>> >>>
> >>> >> Hi,
> >>> >>
> >>> >>> Now I checked the next-v310 tree and know that only LSI required
codes
> >>> >>> are merged.
> >>> >>> http://git.kernel.org/?p=linux/kernel/git/kgene/linux-
> >>> >>> samsung.git;a=shortlog;h=refs/heads/next-s5pv310
> >>> >>>
> >>> >>> I can't find any reason to omit the universal board file. It's
posted
> >>> >>> before merge window and no difference with LSI SMDK board at
current
> >>> >>> status.
> >>> >>
> >>> >> Please answer my concern..I already replied to you about this.
> >>> >>
> >>> >>
> >>
http://lists.infradead.org/pipermail/linux-arm-kernel/2010-July/022087.html
> >>> >> ---
> >>> >>> >> +config MACH_UNIVERSAL
> >>> >>> >
> >>> >>> > As I still asking, do you _really_ want to use 'UNIVERSAL' as
the
> >>> >>> > board(machine) name?
> >>> >>>
> >>> >>> Yes, it's called 'universal' and it's really universal board at
here.
> >>> >>> I don't confuse the engineer who develop this board.
> >>> >>> Maybe your engineers saw and hear the brief description about
> >> universal
> >>> >> board.
> >>> >>>
> >>> >> Actually, I know that. But there are S5PC100, S5PC110 universal
board
> >>> >> also...
> >>> >> So I asked to you that you _really_ want to use 'universal'
here...and
> >> I'm
> >>> >> still concerning.
> >>> >
> >>> > No comment means "Of course want to merge it". thank you for your
> >>> > concerning. but I can handle it.
> >>>
> >> Hi,
> >>
> >>> After discuss with Mr. Kim. I changed to use non-generic name to
> >> universal_c210.
> >>>
> >> Thanks for your understanding...IMHO, 'MACH_UNIVERSAL_C210' is quite
> long...
> >> If you don't mind, how about 'MACH_UNI_C210' or 'MACH_UNIC210'...but it
> >> depends on your decision.
> >
> > FYI: These are also candidates but my team voted and decide to use it.
> >
> >>
> >>> Then do you resend the patch? or can you handle it by yourself?
> >>>
> >> I will apply this with changing it.
> Please modify the machine ID only. I want to use the universal prefix
> at variables as is.
> 
Ah...just now submitted with changing every stuff...
..ok...will update it ;-)

> Thank you,
> Kyungmin Park
> > Thanks
> >>
> >>> To Russell.
> >>> Sorry for bothering you.
> >>> Can you change the registered universal to universal_c210?
> >>> universal_c210          MACH_UNIVERSAL_C210     UNIVERSAL_C210
> 
> >>>
> >>> Thank you,
> >>> Kyungmin Park
> >>> >
> >>> >> ---
> >>> >>
> >>> >>> I try to support our board at mainline kernel but it seems to make
a
> >>> >>> difficult with unknown commit criteria of LSI.
> >>> >>>
> >>> >>
> >>


Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




More information about the linux-arm-kernel mailing list