[parisc-linux-cvs] linux deller

[Helge Deller]

--Boundary-00=_VgAH9pPDG+yqMb7
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 8bit
Content-Disposition: inline

On Friday 28 June 2002 08:42, Helge Deller wrote:
> CVSROOT:	/var/cvs
> Module name:	linux
> Changes by:	deller	02/06/28 00:42:07
>
> Modified files:
> 	drivers/hil    : hil_mlc.c
>
> Log message:
> removed unnecessary list_empty() and INIT_LIST_HEAD() calls.
> Thanks to Matthew Wilcox <willy@debian.org> for noticing.


--Boundary-00=_VgAH9pPDG+yqMb7
Content-Type: text/plain;
  charset="iso-8859-1";
  name="diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline; filename="diff"

Index: hil_mlc.c
===================================================================
RCS file: /var/cvs/linux/drivers/hil/hil_mlc.c,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -p -r1.3 -r1.4
--- hil_mlc.c	2002/06/27 21:46:55	1.3
+++ hil_mlc.c	2002/06/28 06:42:07	1.4
@@ -700,8 +700,7 @@ static void hil_mlcs_process(unsigned lo
 	struct list_head *tmp;
 
 	read_lock(&hil_mlcs_lock);
-	if (!list_empty(&hil_mlcs))
-	  list_for_each(tmp, &hil_mlcs) {
+	list_for_each(tmp, &hil_mlcs) {
 		struct hil_mlc *mlc = list_entry(tmp, hil_mlc, list);
 		while (hilse_donode(mlc) == 0) {
 #ifdef HIL_MLC_DEBUG
@@ -888,7 +887,6 @@ int hil_mlc_register(hil_mlc *mlc) {
 	mlc->tasklet = &hil_mlcs_tasklet;
 
 	write_lock_irqsave(&hil_mlcs_lock, flags);
-	INIT_LIST_HEAD(&mlc->list);
 	list_add_tail(&mlc->list, &hil_mlcs);
 	mlc->seidx = HILSEN_START;
 	write_unlock_irqrestore(&hil_mlcs_lock, flags);

--Boundary-00=_VgAH9pPDG+yqMb7--