On 2017-01-10 11:42, Alexandre Belloni wrote: > On 10/01/2017 at 10:52:56 +0100, Peter Rosin wrote : >> On 2017-01-10 10:29, Alexandre Belloni wrote: >>> Hi, >>> >>> This needs a commit message, please add one. >> >> There's not all that much to say, but ok, I'll add something. >> > > It doesn't have to be long but it has to be present. Does it really? There are quite a few examples to contradict that, and checkpatch doesn't complain. That's no proof of course... As I said, I'll add something. >>> On 10/01/2017 at 09:08:51 +0100, Peter Rosin wrote : >> There's also the benefit of the increased chances of me getting >> notified of changes. I don't mind... >> > > Do you expect changes coming from third parties? I'm fine with it > anyway. Ok, I'll remember this and blame you for everything :-) >>>> +&main { >>>> + clock-frequency = <12000000>; >>>> +}; >>>> + >>> >>> I don't think this is needed >>> >>> >> >> "this"? The &main frequency, or all of them? >> > > I meant just main Ok, I'll test with that change and then resend. Thanks! Cheers, peda -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html