Re: [PATCH/RFC] clocksource: Consolidate SH and ARM mach-shmobileKconfig bits

[Olof Johansson]

On Fri, Sep 27, 2013 at 01:43:25PM +0900, Simon Horman wrote:
> On Thu, Sep 26, 2013 at 08:37:09AM +0900, Magnus Damm wrote:
> > From: Magnus Damm <damm@xxxxxxxxxxxxx>
> > 
> > Based on request from Olof Johansson, move ARM mach-shmobile
> > clocksource entries to drivers/clocksource/Kconfig, and at
> > the same time make the SH architecture use these entries.
> > 
> > Kconfig entries for CMT, MTU2, TMU and STI are with this patch
> > now kept in one place. The SH SYS_SUPPORTS_nnn bits are dropped
> > to simplye let the user select the appropriate ones using Kconfig
> > instead of trying to filter available timers based on SoC.
> > 
> > Known issues:
> >  1) Not yet wrappend in menu/endmenu, populates the root driver menu
> >  -> proposal: Create separate patch to wrap all the clocksource entries?
> 
> That seems reasonable to me.
> 
> >  2) Should the entries be sorted?
> >  -> proposal: Ignore cosmetic bits
> 
> I propose that you sort the entries are adding
> but leave the rest of the entries as is.
> 
> >  3) What is correct dependency for multiplatform and single platform?
> >  -> proposal: Perhaps we should simply default to no?
> 
> I'm not sure that I understand the issue here.
> 
> >  4) Is single patch across 2 architectures and drivers/clocksource OK?
> >  -> proposal: Keep it like this if possible
> 
> Olof, I would like your opinion on this but from my pov this seems
> reasonable. Especially as one of the architectures (sh) is seeing very little
> change at this time.

(Sorry for the delayed response, the renesas volume email can be large and
I don't always read it when it comes in).


Yeah, as long as you have appropriate acks and don't expect conflicts.


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