Hi, On Mon, Dec 05, 2022 at 07:15:48PM +0100, Carl Philipp Klemm wrote: > > I don't follow. Can't you just use 'status = "disabled;"' for the > > unavailable nodes? > > Sure but unless im missing something, adding some nodes to a dts, one > for eatch led the device in question really has feals mutch better to > me than going arround and setting every led channel disabled for eatch > device that dosent use it. xt894 being the outlier here amoung the > cpcap devices we intend to support (xt894, xt860, xt875, xt910, xt912, > mb865, mz609 and mz617) in that it uses most of the extra led channels, > most of these use at most one (xt875, xt910, xt912, mb865) or zero > (mz609, mz617) extra channels. Just mark them all status = 'disabled'; in the common include and then enable them on a per board basis. Just the same way as all the SoC peripherals are handled nowadays. -- Sebastian
Attachment:
signature.asc
Description: PGP signature
