From mboxrd@z Thu Jan 1 00:00:00 1970 From: tomasz.figa@gmail.com (Tomasz Figa) Date: Fri, 12 Apr 2013 22:47:16 +0200 Subject: [PATCH v5 03/14] clocksource: samsung-pwm: Add infrastructure to share PWM hardware In-Reply-To: <201304122242.36801.arnd@arndb.de> References: <1365794250-14436-1-git-send-email-t.figa@samsung.com> <1365794250-14436-4-git-send-email-t.figa@samsung.com> <201304122242.36801.arnd@arndb.de> Message-ID: <2367229.enzT7IXUEQ@flatron> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Arnd, On Friday 12 of April 2013 22:42:36 Arnd Bergmann wrote: > On Friday 12 April 2013, Tomasz Figa wrote: > > + > > +Samsung SoCs contain PWM timer blocks which can be used for system > > clock source +and clock event timers, as well as to drive SoC outputs > > with PWM signal. Each +PWM timer block provides 5 PWM channels (not > > all of them can drive physical +outputs - see SoC and board manual). > > + > > ... > > > +Optional properties: > > +- samsung,pwm-outputs: list of PWM channels used as PWM outputs on > > particular + platform - an array of up to 5 elements being indices > > of PWM channels + (from 0 to 4), the order does not matter. > > + > > There is probably a good reason, but can you explain why this is > actually required to be a property? Wouldn't it be enough to just not > refer to the PWM outputs from other devices when they are not used? It is needed for the clocksource driver, to select unused channels for clocksource and clock event purposes. Best regards, Tomasz