From mboxrd@z Thu Jan 1 00:00:00 1970 From: b.zolnierkie@samsung.com (Bartlomiej Zolnierkiewicz) Date: Mon, 27 Jul 2015 16:48:59 +0200 Subject: [PATCH V2 08/11] opp: Add OPP sharing information to OPP library In-Reply-To: References: Message-ID: <1564699.gQjhqzf5VE@amdc1976> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Two very minor nits: On Monday, July 27, 2015 10:32:38 AM Viresh Kumar wrote: > An opp can be shared by multiple devices, for example its very common > for CPUs to share the OPPs, i.e. when they share clock/voltage rails. > > This patch adds support of shared OPPs to the OPP library. > > Instead of a single device, dev_opp will not contain a list of devices s/will not/will now/ ? > that use it. It also senses if the device (we are trying to initialize > OPPs for) shares OPPs with a device added earlier and in that case we > update the list of devices managed by OPPs instead of duplicating OPPs > again. > > The same infrastructure will be used for the old OPP bindings, with > later patches. > > Reviewed-by: Stephen Boyd > Signed-off-by: Viresh Kumar [...] > +static struct device_list_opp *_add_list_dev(const struct device *dev, > + struct device_opp *dev_opp) > +{ > + struct device_list_opp *list_dev; > + > + list_dev = kzalloc(sizeof(*list_dev), GFP_KERNEL); > + if (!list_dev) > + return NULL; > + > + /* Initialize list-dev */ > + list_add_rcu(&list_dev->node, &dev_opp->dev_list); > + list_dev->dev = dev; Probably doesn't matter currently but how's about: list_dev->dev = dev; list_add_rcu(&list_dev->node, &dev_opp->dev_list); ? > + > + return list_dev; > +} Otherwise: Reviewed-by: Bartlomiej Zolnierkiewicz Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics