From nobody Sun Feb 8 19:08:40 2026 Received: from mail-ej1-f48.google.com (mail-ej1-f48.google.com [209.85.218.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 32F174C81 for ; Sun, 9 Nov 2025 19:18:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.48 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762715899; cv=none; b=fCD1bLh9YW6LTIR+Xc0c/MtqUqy+jmUpwWJxAgluzKk0f8qB6vwvIvSoM1Oox5u/motm51KpjZcGMwa6lo0BP9ogO8R/uZGJCA0/d9ePW8PHJwAar0ODOPOZ1W7WGltyVV2LpF4bkqjQvb69zT8ItTCI5wtzZFxr9gLMz8yrdoc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762715899; c=relaxed/simple; bh=4Tc5nMIJkCf+K1Yp/lS0TNl3kI2yP5RZlvH1VEVb//4=; h=Date:From:To:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition; b=rXkT/P0aCVw0x0X86wsSnUtk6pKOnPCFzvs7le462on5sYskEGQrPB1Wt2SzkuCMCoJ7BIRUhIKJ7+A2okyHWwb9TSKE7L7FTBMUapfDeeuCUed8KK8QRMxyp2GIaycoYtcvQJ4P4gzNuAMRbSizxhHcrPDXzsJLKk+nAVE3u8o= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=bOUox1EM; arc=none smtp.client-ip=209.85.218.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="bOUox1EM" Received: by mail-ej1-f48.google.com with SMTP id a640c23a62f3a-b626a4cd9d6so359649066b.3 for ; Sun, 09 Nov 2025 11:18:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1762715895; x=1763320695; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=Pt/WJLMW5CofgvBurApMG4dE+1UCGhjeZR+9TBOAEFA=; b=bOUox1EMgL6Y/1s/OW1tTX+n28pecYsTuXIdmflyy3d7XX97TV8kBOTywYBn8koFDe UYVHXNN0d5ZkR9Vnpm3VCuCXAm/Ljml0gXaL1cZeni7y9RehD1SM3/qNoUn19snfs6sa D834JOPOWtbOaYYruA7f8zkjdqymmb/6i6Ppj6DaV4ROCFFG8ILo4MBYmWVEHog6Puhd 7I3EmndZbrwPwmWqxBrMt9dzr3jw7k8BgpeRgbsLD7Gue60V/i4vkGwcNZuhTd1rCMsU 33oXa77jKS9GNaubL6vFUMnywboJWqPs7OfT9JARjEP3ai+qNYIHamIXYcVbsccx+9kx +2LA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762715895; x=1763320695; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Pt/WJLMW5CofgvBurApMG4dE+1UCGhjeZR+9TBOAEFA=; b=Qihh4XbWoisfDxM9NTuS5ylKVnaATAgqKYjg7EDgaBMLOm+J4YJ8dWUfLtG4xYtMPM zBl6JFtMzIoRiqwzHcvBUineWZ02f20JYgCfx0m6JKJQ4mETqyVozNAcumoFv+y1ZmLq VCuOY5omC6WZhLmZyHl15IWFvcFw5RvyzGd4Tv6mK93YId0VZCqCRrWETe6rcvBqcXr4 aG3zVcgHo8BN4why8iApdjCOfTYyuA/sh7wvzrGazd/TJLUHS3cDKDtubJXhVokXC4aG hJn5rV6OQV4bJhP7AE/g+YVqrarh5bqDWz9+pqS45rG8yVldEegmuK65yEJwzxxhLFxs wMJA== X-Forwarded-Encrypted: i=1; AJvYcCVvMvx1D79o3vGN+dx3gfoG0NcBF7uHRdKSx6E8KC7ca4hhKiOUzHJEVbkcivNpTfacp9BPCG20STyWKT0=@vger.kernel.org X-Gm-Message-State: AOJu0YyDFwMdB0eRXcRPMpIAf67VkQDV0EcEh3V184M6hBpcoaXukmTr BAzj3oLc2mOBZOwYBdHzwZPbBDJmnPkxK9SU+SL0+XSY809L9ghFKj94 X-Gm-Gg: ASbGncvxwIAQi7Sm7V/rWs52Og0+4d33nRIcjmrLmb+vyX7OosHEQYdye6OwU0X/Ii8 /Uy6X1q8HrH77xnatd/u/bTb6t1sTV3UNSNYP1GMo3PWkMW/UpBVfyIRyC+d9jBxR9QROc50qTU 2lM9gJyUavsdLnOwWoc8/g/VtGPj24QyFYKykNsKbYhct8xKF+z3NUubzxnydQ1j3iPPcMiDvxM AFx6iwr5iQMNzzm7QLfcFZGjPFfbIafUFIcNVwH1PJd5HS33agSaU7C1Bm1C4Y+hLvoPjfubJqW BDSz0+5psfZDRP903zvEzCaWqThjFKf8U6NmXgFkUbaw3P0xgiWcmN352HRzFQLZE4qbzbsOMt/ TmmGY/a6tuop55izB3rWH8oPlzTHcK1vhgj4CwLbeK3/oa6y1ABRLK7loUvYTIAfsd4Wc37zxc0 FVFjNM4cLD6LKVn9RFaMNHGx79/X2YJXp8KGJ1 X-Google-Smtp-Source: AGHT+IHuRc5rVpG/mviy3lNjbujL/a/ATQzKZYImG61QSsXht/oKQln9ULAg8h7RmQMZ+J58jigtQw== X-Received: by 2002:a17:906:6a19:b0:b72:9d56:ab52 with SMTP id a640c23a62f3a-b72e02d20c7mr560336266b.8.1762715895320; Sun, 09 Nov 2025 11:18:15 -0800 (PST) Received: from michalis-linux (adsl-75.176.58.251.tellas.gr. [176.58.251.75]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b72bfa24d1fsm859351866b.73.2025.11.09.11.18.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 09 Nov 2025 11:18:14 -0800 (PST) Date: Sun, 9 Nov 2025 21:18:13 +0200 From: Michail Tatas To: dpenkler@gmail.com, gregkh@linuxfoundation.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, outreachy@lists.linux.dev Subject: [PATCH] staging: gpib: converted // to /**/ comments Message-ID: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" This patch converts comments from // to /**/ to make them consistent across the file as suggested by the TODO file in drivers/staging/gpib/todo Signed-off-by: Michail Tatas --- .../gpib/agilent_82357a/agilent_82357a.c | 72 +++++++++---------- 1 file changed, 36 insertions(+), 36 deletions(-) diff --git a/drivers/staging/gpib/agilent_82357a/agilent_82357a.c b/drivers= /staging/gpib/agilent_82357a/agilent_82357a.c index 77c8e549b208..eafbd562f3ab 100644 --- a/drivers/staging/gpib/agilent_82357a/agilent_82357a.c +++ b/drivers/staging/gpib/agilent_82357a/agilent_82357a.c @@ -23,7 +23,7 @@ MODULE_DESCRIPTION("GPIB driver for Agilent 82357A/B usb = adapters"); =20 #define MAX_NUM_82357A_INTERFACES 128 static struct usb_interface *agilent_82357a_driver_interfaces[MAX_NUM_8235= 7A_INTERFACES]; -static DEFINE_MUTEX(agilent_82357a_hotplug_lock); // protect board inserti= on and removal +static DEFINE_MUTEX(agilent_82357a_hotplug_lock); /* protect board inserti= on and removal */ =20 static unsigned int agilent_82357a_update_status(struct gpib_board *board, unsigned int clear_mask); @@ -420,7 +420,7 @@ static int agilent_82357a_abort(struct agilent_82357a_p= riv *a_priv, int flush) return retval; } =20 -// interface functions +/* interface functions */ int agilent_82357a_command(struct gpib_board *board, u8 *buffer, size_t le= ngth, size_t *bytes_written); =20 @@ -449,8 +449,8 @@ static int agilent_82357a_read(struct gpib_board *board= , u8 *buffer, size_t leng if (!out_data) return -ENOMEM; out_data[i++] =3D DATA_PIPE_CMD_READ; - out_data[i++] =3D 0; // primary address when ARF_NO_ADDR is not set - out_data[i++] =3D 0; // secondary address when ARF_NO_ADDR is not set + out_data[i++] =3D 0; /* primary address when ARF_NO_ADDR is not set */ + out_data[i++] =3D 0; /* secondary address when ARF_NO_ADDR is not set */ out_data[i] =3D ARF_NO_ADDRESS | ARF_END_ON_EOI; if (a_priv->eos_mode & REOS) out_data[i] |=3D ARF_END_ON_EOS_CHAR; @@ -532,7 +532,7 @@ static int agilent_82357a_read(struct gpib_board *board= , u8 *buffer, size_t leng */ agilent_82357a_take_control_internal(board, 0); =20 - // FIXME check trailing flags for error + /* FIXME check trailing flags for error */ return retval; } =20 @@ -563,8 +563,8 @@ static ssize_t agilent_82357a_generic_write(struct gpib= _board *board, if (!out_data) return -ENOMEM; out_data[i++] =3D DATA_PIPE_CMD_WRITE; - out_data[i++] =3D 0; // primary address when AWF_NO_ADDRESS is not set - out_data[i++] =3D 0; // secondary address when AWF_NO_ADDRESS is not set + out_data[i++] =3D 0; /* primary address when AWF_NO_ADDRESS is not set */ + out_data[i++] =3D 0; /* secondary address when AWF_NO_ADDRESS is not set = */ out_data[i] =3D AWF_NO_ADDRESS | AWF_NO_FAST_TALKER_FIRST_BYTE; if (send_commands) out_data[i] |=3D AWF_ATN | AWF_NO_FAST_TALKER; @@ -632,7 +632,7 @@ static ssize_t agilent_82357a_generic_write(struct gpib= _board *board, if ((bsr & BSR_ATN_BIT) && !(bsr & (BSR_NDAC_BIT | BSR_NRFD_BIT))) { dev_dbg(&usb_dev->dev, "No listener on command\n"); clear_bit(TIMO_NUM, &board->status); - return -ENOTCONN; // no listener on bus + return -ENOTCONN; /* no listener on bus */ } } else { read_reg.address =3D ADSR; @@ -726,7 +726,7 @@ static int agilent_82357a_take_control(struct gpib_boar= d *board, int synchronous return -ETIMEDOUT; =20 agilent_82357a_take_control_internal(board, synchronous); - // busy wait until ATN is asserted + /* busy wait until ATN is asserted */ for (i =3D 0; i < timeout; ++i) { agilent_82357a_update_status(board, 0); if (test_bit(ATN_NUM, &board->status)) @@ -795,7 +795,7 @@ static void agilent_82357a_interface_clear(struct gpib_= board *board, int assert) int retval; =20 if (!a_priv->bus_interface) - return; // -ENODEV; + return; /* -ENODEV; */ =20 usb_dev =3D interface_to_usbdev(a_priv->bus_interface); write.address =3D AUXCR; @@ -817,7 +817,7 @@ static void agilent_82357a_remote_enable(struct gpib_bo= ard *board, int enable) int retval; =20 if (!a_priv->bus_interface) - return; //-ENODEV; + return; /* -ENODEV; */ =20 usb_dev =3D interface_to_usbdev(a_priv->bus_interface); write.address =3D AUXCR; @@ -828,7 +828,7 @@ static void agilent_82357a_remote_enable(struct gpib_bo= ard *board, int enable) if (retval) dev_err(&usb_dev->dev, "write_registers() returned error\n"); a_priv->ren_state =3D enable; - return;// 0; + return; /* 0; */ } =20 static int agilent_82357a_enable_eos(struct gpib_board *board, u8 eos_byte, @@ -876,22 +876,22 @@ static unsigned int agilent_82357a_update_status(stru= ct gpib_board *board, dev_err(&usb_dev->dev, "read_registers() returned error\n"); return board->status; } - // check for remote/local + /* check for remote/local */ if (address_status.value & HR_REM) set_bit(REM_NUM, &board->status); else clear_bit(REM_NUM, &board->status); - // check for lockout + /* check for lockout */ if (address_status.value & HR_LLO) set_bit(LOK_NUM, &board->status); else clear_bit(LOK_NUM, &board->status); - // check for ATN + /* check for ATN */ if (address_status.value & HR_ATN) set_bit(ATN_NUM, &board->status); else clear_bit(ATN_NUM, &board->status); - // check for talker/listener addressed + /* check for talker/listener addressed */ if (address_status.value & HR_TA) set_bit(TACS_NUM, &board->status); else @@ -926,7 +926,7 @@ static int agilent_82357a_primary_address(struct gpib_b= oard *board, unsigned int if (!a_priv->bus_interface) return -ENODEV; usb_dev =3D interface_to_usbdev(a_priv->bus_interface); - // put primary address in address0 + /* put primary address in address0 */ write.address =3D ADR; write.value =3D address & ADDRESS_MASK; retval =3D agilent_82357a_write_registers(a_priv, &write, 1); @@ -956,7 +956,7 @@ static int agilent_82357a_parallel_poll(struct gpib_boa= rd *board, u8 *result) if (!a_priv->bus_interface) return -ENODEV; usb_dev =3D interface_to_usbdev(a_priv->bus_interface); - // execute parallel poll + /* execute parallel poll */ writes[0].address =3D AUXCR; writes[0].value =3D AUX_CS | AUX_RPP; writes[1].address =3D HW_CONTROL; @@ -966,7 +966,7 @@ static int agilent_82357a_parallel_poll(struct gpib_boa= rd *board, u8 *result) dev_err(&usb_dev->dev, "write_registers() returned error\n"); return retval; } - udelay(2); // silly, since usb write will take way longer + udelay(2); /* silly, since usb write will take way longer */ read.address =3D CPTR; retval =3D agilent_82357a_read_registers(a_priv, &read, 1, 1); if (retval) { @@ -974,7 +974,7 @@ static int agilent_82357a_parallel_poll(struct gpib_boa= rd *board, u8 *result) return retval; } *result =3D read.value; - // clear parallel poll state + /* clear parallel poll state */ writes[0].address =3D HW_CONTROL; writes[0].value =3D a_priv->hw_control_bits | NOT_PARALLEL_POLL; writes[1].address =3D AUXCR; @@ -989,32 +989,32 @@ static int agilent_82357a_parallel_poll(struct gpib_b= oard *board, u8 *result) =20 static void agilent_82357a_parallel_poll_configure(struct gpib_board *boar= d, u8 config) { - // board can only be system controller - return;// 0; + /* board can only be system controller */ + return; /* 0; */ } =20 static void agilent_82357a_parallel_poll_response(struct gpib_board *board= , int ist) { - // board can only be system controller - return;// 0; + /* board can only be system controller */ + return;/* 0; */ } =20 static void agilent_82357a_serial_poll_response(struct gpib_board *board, = u8 status) { - // board can only be system controller - return;// 0; + /* board can only be system controller */ + return;/* 0; */ } =20 static u8 agilent_82357a_serial_poll_status(struct gpib_board *board) { - // board can only be system controller + /* board can only be system controller */ return 0; } =20 static void agilent_82357a_return_to_local(struct gpib_board *board) { - // board can only be system controller - return;// 0; + /* board can only be system controller */ + return;/* 0; */ } =20 static int agilent_82357a_line_status(const struct gpib_board *board) @@ -1269,7 +1269,7 @@ static int agilent_82357a_init(struct gpib_board *boa= rd) writes[14].value =3D HR_BOIE | HR_BIIE; writes[15].address =3D IMR1; writes[15].value =3D HR_SRQIE; - // turn off reset state + /* turn off reset state */ writes[16].address =3D AUXCR; writes[16].value =3D AUX_CHIP_RESET; writes[17].address =3D LED_CONTROL; @@ -1387,7 +1387,7 @@ static int agilent_82357a_go_idle(struct gpib_board *= board) struct agilent_82357a_register_pairlet writes[0x20]; int retval; =20 - // turn on tms9914 reset state + /* turn on tms9914 reset state */ writes[0].address =3D AUXCR; writes[0].value =3D AUX_CS | AUX_CHIP_RESET; a_priv->hw_control_bits &=3D ~NOT_TI_RESET; @@ -1448,7 +1448,7 @@ static struct gpib_interface agilent_82357a_gpib_inte= rface =3D { .parallel_poll =3D agilent_82357a_parallel_poll, .parallel_poll_configure =3D agilent_82357a_parallel_poll_configure, .parallel_poll_response =3D agilent_82357a_parallel_poll_response, - .local_parallel_poll_mode =3D NULL, // XXX + .local_parallel_poll_mode =3D NULL, /* XXX */ .line_status =3D agilent_82357a_line_status, .update_status =3D agilent_82357a_update_status, .primary_address =3D agilent_82357a_primary_address, @@ -1461,7 +1461,7 @@ static struct gpib_interface agilent_82357a_gpib_inte= rface =3D { .skip_check_for_command_acceptors =3D 1 }; =20 -// Table with the USB-devices: just now only testing IDs +/* Table with the USB-devices: just now only testing IDs */ static struct usb_device_id agilent_82357a_driver_device_table[] =3D { {USB_DEVICE(USB_VENDOR_ID_AGILENT, USB_DEVICE_ID_AGILENT_82357A)}, {USB_DEVICE(USB_VENDOR_ID_AGILENT, USB_DEVICE_ID_AGILENT_82357B)}, @@ -1626,15 +1626,15 @@ static int agilent_82357a_driver_resume(struct usb_= interface *interface) mutex_unlock(&agilent_82357a_hotplug_lock); return retval; } - // set/unset system controller + /* set/unset system controller */ retval =3D agilent_82357a_request_system_control(board, board->master); - // toggle ifc if master + /* toggle ifc if master */ if (board->master) { agilent_82357a_interface_clear(board, 1); usleep_range(200, 250); agilent_82357a_interface_clear(board, 0); } - // assert/unassert REN + /* assert/unassert REN */ agilent_82357a_remote_enable(board, a_priv->ren_state); =20 dev_dbg(&usb_dev->dev, --=20 2.43.0