[Skiboot] [PATCH] Fix for typos

Frederic Bonnard frediz at linux.vnet.ibm.com
Mon Jun 13 19:36:39 AEST 2016


Thanks Mukesh,
added the typos you found as well.

F.

On Sat, 11 Jun 2016 11:59:04 +0530, Mukesh Ojha <mukesh02 at linux.vnet.ibm.com> wrote:
> 
> +Skiboot
> 
> -------- Forwarded Message --------
> Subject: 	Re: [Skiboot] [PATCH] Fix for typos
> Date: 	Sat, 11 Jun 2016 11:57:45 +0530
> From: 	Mukesh Ojha <mukesh02 at linux.vnet.ibm.com>
> To: 	Frédéric Bonnard <frediz at linux.vnet.ibm.com>
> 
> 
> 
> Hi Frédéric,
> 
> Few more typos pointed out below.
> 
> On Friday 10 June 2016 07:49 PM, Frédéric Bonnard wrote:
> > From: Frederic Bonnard <frediz at linux.vnet.ibm.com>
> >
> > While reviewing the Debian packaging, codespell found those.
> > Proposed fixes are based on codespell's default dictionnary.
> >
> > Signed-off-by: Frederic Bonnard <frediz at linux.vnet.ibm.com>
> > ---
> >   ccan/list/list.h                             |  8 ++++----
> >   doc/device-tree.txt                          |  8 ++++----
> >   doc/device-tree/ibm,opal.txt                 |  2 +-
> >   doc/error-logging.txt                        |  2 +-
> >   doc/nvlink.txt                               |  2 +-
> >   doc/opal-api/opal-pci-phb-mmio-enable-27.txt |  2 +-
> >   doc/opal-api/opal-rtc-read-3.txt             |  2 +-
> >   doc/release-notes/skiboot-5.1.0-beta2.txt    |  2 +-
> >   doc/release-notes/skiboot-5.1.0.txt          |  2 +-
> >   doc/release-notes/skiboot-5.1.12.txt         |  2 +-
> >   doc/release-notes/skiboot-5.1.2.txt          |  2 +-
> >   doc/release-notes/skiboot-5.2.2.txt          |  2 +-
> >   external/boot-tests/bmc_support.sh           |  6 +++---
> >   external/gard/gard.c                         |  2 +-
> >   external/mambo/mambo_utils.tcl               |  2 +-
> >   external/opal-prd/hostboot-interface.h       |  2 +-
> >   external/opal-prd/i2c.c                      |  2 +-
> >   external/opal-prd/opal-prd.8                 |  2 +-
> >   external/opal-prd/opal-prd.c                 |  2 +-
> >   external/opal-prd/test/test_pnor_ops.c       |  2 +-
> >   external/pflash/pflash.c                     |  6 +++---
> >   external/test/test.sh                        |  2 +-
> >   hdata/spira.h                                | 10 +++++-----
> >   hw/npu.c                                     |  2 +-
> >   hw/phb3.c                                    |  4 ++--
> >   include/spcn.h                               |  2 +-
> >   libc/stdio/vsnprintf.c                       |  2 +-
> >   libc/time.c                                  |  2 +-
> >   libpore/p8_pore_table_gen_api_fixed.C        |  2 +-
> >   libpore/pgas.h                               |  4 ++--
> >   libpore/pore_inline.h                        |  2 +-
> >   libpore/pore_inline_assembler.c              |  2 +-
> >   libpore/sbe_xip_image.c                      |  2 +-
> >   libpore/sbe_xip_image.h                      | 12 ++++++------
> >   34 files changed, 55 insertions(+), 55 deletions(-)
> >
> > diff --git a/ccan/list/list.h b/ccan/list/list.h
> > index 1a73510..5f46330 100644
> > --- a/ccan/list/list.h
> > +++ b/ccan/list/list.h
> > @@ -412,7 +412,7 @@ static inline const void *list_tail_(const struct list_head *h, size_t off)
> >   /**
> >    * list_for_each_off - iterate through a list of memory regions.
> >    * @h: the list_head
> > - * @i: the pointer to a memory region wich contains list node data.
> > + * @i: the pointer to a memory region which contains list node data.
> >    * @off: offset(relative to @i) at which list node data resides.
> >    *
> >    * This is a low-level wrapper to iterate @i over the entire list, used to
> > @@ -423,9 +423,9 @@ static inline const void *list_tail_(const struct list_head *h, size_t off)
> >    * nor care about the type of @i. The only assumtion made is that @i points
> >    * to a chunk of memory that at some @offset, relative to @i, contains a
> >    * properly filled `struct node_list' which in turn contains pointers to
> > - * memory chunks and it's turtles all the way down. Whith all that in mind
> > + * memory chunks and it's turtles all the way down. With all that in mind
> >    * remember that given the wrong pointer/offset couple this macro will
> > - * happilly churn all you memory untill SEGFAULT stops it, in other words
> > + * happilly churn all you memory until SEGFAULT stops it, in other words
> s/happilly/happily
> >    * caveat emptor.
> >    *
> >    * It is worth mentioning that one of legitimate use-cases for that wrapper
> > @@ -448,7 +448,7 @@ static inline const void *list_tail_(const struct list_head *h, size_t off)
> >    * list_for_each_safe_off - iterate through a list of memory regions, maybe
> >    * during deletion
> >    * @h: the list_head
> > - * @i: the pointer to a memory region wich contains list node data.
> > + * @i: the pointer to a memory region which contains list node data.
> >    * @nxt: the structure containing the list_node
> >    * @off: offset(relative to @i) at which list node data resides.
> >    *
> > diff --git a/doc/device-tree.txt b/doc/device-tree.txt
> > index a0e4457..ef5a744 100644
> > --- a/doc/device-tree.txt
> > +++ b/doc/device-tree.txt
> > @@ -115,7 +115,7 @@
> >   	 *
> >   	 * Unless a component (skiboot or Linux) specifically knows about a region (usually
> >   	 * based on its name) and decides to change or remove it, all these regions are
> > -	 * passed as-is to Linux and to subsequent kernels accross kexec and are kept
> > +	 * passed as-is to Linux and to subsequent kernels across kexec and are kept
> >   	 * preserved.
> >   	 *
> >   	 * NOTE: Do *NOT* copy the entries below, they are just an example and are actually
> > @@ -189,7 +189,7 @@
> >   			ibm,segment-page-sizes = <0xc 0x0 0x3 0xc 0x0 0x10 0x7 0x18 0x38 0x10 0x110 0x2 0x10 0x1 0x18 0x8 0x18 0x100 0x1 0x18 0x0 0x22 0x120 0x1 0x22 0x3>;
> >
> >   			/*
> > -			 * Similarily that might need to be reviewed later but will do for now...
> > +			 * Similarly that might need to be reviewed later but will do for now...
> >   			 */			
> >   			ibm,pa-features = [0x6 0x0 0xf6 0x3f 0xc7 0x0 0x80 0xc0];
> >
> > @@ -333,7 +333,7 @@
> >   	 * In order to be able to handle affinity propertly, we require that
> s/propertly/properly
> >   	 * a memory node is created for each range of memory that has a different
> >   	 * "affinity", which in practice means for each chip since we don't
> > -	 * support memory interleaved accross multiple chips on P8.
> > +	 * support memory interleaved across multiple chips on P8.
> >   	 *
> >   	 * Additionally, it is *not* required that one chip = one memory node,
> >   	 * it is perfectly acceptable to break down the memory of one chip into
> > @@ -345,7 +345,7 @@
> >
> >   		/*
> >   		 * We support multiple entries in the ibm,chip-id property for
> > -		 * memory nodes in case the memory is interleaved accross multiple
> > +		 * memory nodes in case the memory is interleaved across multiple
> >   		 * chips but that shouldn't happen on P8
> >   		 */
> >   		ibm,chip-id = <0x0>;
> > diff --git a/doc/device-tree/ibm,opal.txt b/doc/device-tree/ibm,opal.txt
> > index c89c916..d4dcfb8 100644
> > --- a/doc/device-tree/ibm,opal.txt
> > +++ b/doc/device-tree/ibm,opal.txt
> > @@ -3,7 +3,7 @@
> >   		#size-cells = <0x0>;
> >   		compatible = "ibm,opal-v2", "ibm,opal-v3";
> >
> > -; v2 is maintained for possible compatibilty with very, very old kernels
> > +; v2 is maintained for possible compatibility with very, very old kernels
> >   ; it will go away at some point in the future. Detect and rely on ibm,opal-v3
> >
> >   		ibm,associativity-reference-points = <0x4 0x3>;
> > diff --git a/doc/error-logging.txt b/doc/error-logging.txt
> > index 3d6b2a2..ba47785 100644
> > --- a/doc/error-logging.txt
> > +++ b/doc/error-logging.txt
> > @@ -52,7 +52,7 @@ Step 1: To report an error, invoke opal_elog_create() with required argument.
> >   		#define OPAL_INPUT_OUTPUT_ERR_EVT       0x02
> >   		/* RESOURCE_DEALLOC: Hotplug events and errors */
> >   		#define OPAL_RESOURCE_DEALLOC_ERR_EVT   0x03
> > -		/* MISC: Miscellanous error */
> > +		/* MISC: Miscellaneous error */
> >   		#define OPAL_MISC_ERR_EVT               0x04
> >
> >   	uint16_t component_id: Component ID of Sapphire component as
> > diff --git a/doc/nvlink.txt b/doc/nvlink.txt
> > index d871d20..5aef539 100644
> > --- a/doc/nvlink.txt
> > +++ b/doc/nvlink.txt
> > @@ -134,7 +134,7 @@ PCI Device Flag
> >
> >   Link Number
> >
> > -   Physical link number this emulated PCI device is assoicated
> > +   Physical link number this emulated PCI device is associated
> >      with. One of 0, 1, 4 or 5 (links 2 & 3 do not exist on Naples).
> >
> >   Reserved
> > diff --git a/doc/opal-api/opal-pci-phb-mmio-enable-27.txt b/doc/opal-api/opal-pci-phb-mmio-enable-27.txt
> > index c5b08f4..9bff057 100644
> > --- a/doc/opal-api/opal-pci-phb-mmio-enable-27.txt
> > +++ b/doc/opal-api/opal-pci-phb-mmio-enable-27.txt
> > @@ -13,7 +13,7 @@ used as a starting point for full documentation.
> >
> >   The host calls this function to enable or disable PHB decode of the PCI IO
> >   and Memory address spaces below that PHB. Window_num selects an mmio window
> > -within that addres space. Enable set to '1' enables the PHB to decode and
> > +within that address space. Enable set to '1' enables the PHB to decode and
> >   forward system real addresses to PCI memory, while enable set to '0' disables
> >   PHB decode and forwarding for the address range defined in a particular MMIO
> >   window.
> > diff --git a/doc/opal-api/opal-rtc-read-3.txt b/doc/opal-api/opal-rtc-read-3.txt
> > index 70f9520..d98103a 100644
> > --- a/doc/opal-api/opal-rtc-read-3.txt
> > +++ b/doc/opal-api/opal-rtc-read-3.txt
> > @@ -21,7 +21,7 @@ Calling:
> >   Since RTC calls can be pretty slow, OPAL_RTC_READ is likely to first return
> >   OPAL_BUSY_EVENT, requiring the caller to wait until the OPAL_EVENT_RTC event
> >   has been signaled. Once the event has been signalled, a subsequent
> s/signaled/signalled
> > -OPAL_RTC_READ call will retreive the time. Since the OPAL_EVENT_RTC event is
> > +OPAL_RTC_READ call will retrieve the time. Since the OPAL_EVENT_RTC event is
> >   used for both reading and writing the RTC, callers must be able to handle
> >   the event being signalled for a concurrent in flight OPAL_RTC_WRITE rather
> >   than this read request.
> > diff --git a/doc/release-notes/skiboot-5.1.0-beta2.txt b/doc/release-notes/skiboot-5.1.0-beta2.txt
> > index 3473d89..ebd7fc0 100644
> > --- a/doc/release-notes/skiboot-5.1.0-beta2.txt
> > +++ b/doc/release-notes/skiboot-5.1.0-beta2.txt
> > @@ -44,7 +44,7 @@ Over skiboot-5.1.0-beta1, the following bugs have been fixed:
> >        to have shared mode. So we have to cut off the first M64 segment,
> >        which corresponds to reserved PE#0 in kernel. If the first BAR
> >        (for example PF's IOV BAR) requires huge alignment in kernel, we
> > -     have to waste huge M64 space to accomodate the alignment. If we
> > +     have to waste huge M64 space to accommodate the alignment. If we
> >        have reserved PE#256, the waste of M64 space will be avoided.
> >
> >   Other changes:
> > diff --git a/doc/release-notes/skiboot-5.1.0.txt b/doc/release-notes/skiboot-5.1.0.txt
> > index 127c975..8558378 100644
> > --- a/doc/release-notes/skiboot-5.1.0.txt
> > +++ b/doc/release-notes/skiboot-5.1.0.txt
> > @@ -104,7 +104,7 @@ The following bugs have been fixed:
> >        to have shared mode. So we have to cut off the first M64 segment,
> >        which corresponds to reserved PE#0 in kernel. If the first BAR
> >        (for example PF's IOV BAR) requires huge alignment in kernel, we
> > -     have to waste huge M64 space to accomodate the alignment. If we
> > +     have to waste huge M64 space to accommodate the alignment. If we
> >        have reserved PE#256, the waste of M64 space will be avoided.
> >
> >   FSP-specific bugs fixed:
> > diff --git a/doc/release-notes/skiboot-5.1.12.txt b/doc/release-notes/skiboot-5.1.12.txt
> > index 6368473..49a7e25 100644
> > --- a/doc/release-notes/skiboot-5.1.12.txt
> > +++ b/doc/release-notes/skiboot-5.1.12.txt
> > @@ -25,7 +25,7 @@ POWER8 PHB (PCIe) specific:
> >   - hw/phb3: Flush cache line after updating P/Q bits
> >       When doing an MSI EOI, we update the P and Q bits in the IVE. That causes
> >       the corresponding cache line to be dirty in the L3 which will cause a
> > -    subsequent update by the PHB (upon recieving the next MSI) to get a few
> > +    subsequent update by the PHB (upon receiving the next MSI) to get a few
> >       retries until it gets flushed.
> >
> >       We improve the situation (and thus performance) by doing a dcbf
> > diff --git a/doc/release-notes/skiboot-5.1.2.txt b/doc/release-notes/skiboot-5.1.2.txt
> > index 6e112a5..e64932e 100644
> > --- a/doc/release-notes/skiboot-5.1.2.txt
> > +++ b/doc/release-notes/skiboot-5.1.2.txt
> > @@ -31,7 +31,7 @@ Over skiboot-5.1.1, we have the following changes:
> >   - build improvements
> >     - fixes for two compiler warnings were squashed in 5.1.1 commit,
> >       re-introduce the fixes.
> > -  - misc complier/static analysis warning fixes
> > +  - misc compiler/static analysis warning fixes
> >
> >   - gard utility:
> >     - If gard tool detects the GUARD PNOR partition is corrupted, it will
> > diff --git a/doc/release-notes/skiboot-5.2.2.txt b/doc/release-notes/skiboot-5.2.2.txt
> > index 603270c..3c65e11 100644
> > --- a/doc/release-notes/skiboot-5.2.2.txt
> > +++ b/doc/release-notes/skiboot-5.2.2.txt
> > @@ -9,7 +9,7 @@ first released August 17th, 2015.
> >
> >   Skiboot 5.2.2 replaces skiboot-5.2.1 as the current stable version, which was
> >   released on April 27th, 2016. Over skiboot-5.2.1, skiboot 5.2.2 contains
> > -one bug fix targetted at P8NVL systems, notably the Garrison platform.
> > +one bug fix targeted at P8NVL systems, notably the Garrison platform.
> >
> >   skiboot-5.2.2 contains all bug fixes as of skiboot-5.1.16.
> >
> > diff --git a/external/boot-tests/bmc_support.sh b/external/boot-tests/bmc_support.sh
> > index 0b4c675..f28c610 100644
> > --- a/external/boot-tests/bmc_support.sh
> > +++ b/external/boot-tests/bmc_support.sh
> > @@ -75,7 +75,7 @@ function flash {
> >   		msg "Flashing full PNOR"
> >   		$SSHCMD "$PFLASH_BINARY -E -f -p /tmp/image.pnor"
> >   		if [ "$?" -ne "0" ] ; then
> > -			error "An unexpected pflash error has occured";
> > +			error "An unexpected pflash error has occurred";
> >   		fi
> >   	fi
> >
> > @@ -83,7 +83,7 @@ function flash {
> >   		msg "Flashing PAYLOAD PNOR partition"
> >   		$SSHCMD "$PFLASH_BINARY -e -f -P PAYLOAD -p /tmp/skiboot.lid"
> >   		if [ "$?" -ne "0" ] ; then
> > -                        error "An unexpected pflash error has occured";
> > +                        error "An unexpected pflash error has occurred";
> >                   fi
> >   	fi
> >
> > @@ -91,7 +91,7 @@ function flash {
> >                   msg "Flashing BOOTKERNEL PNOR partition"
> >                   $SSHCMD "$PFLASH_BINARY -e -f -P BOOTKERNEL -p /tmp/bootkernel"
> >                   if [ "$?" -ne "0" ] ; then
> > -                        error "An unexpected pflash error has occured";
> > +                        error "An unexpected pflash error has occurred";
> >                   fi
> >           fi
> >   }
> > diff --git a/external/gard/gard.c b/external/gard/gard.c
> > index b2e814e..9da7496 100644
> > --- a/external/gard/gard.c
> > +++ b/external/gard/gard.c
> > @@ -127,7 +127,7 @@ static void show_flash_err(int rc)
> >   			fprintf(stderr, "libflash ecc invalid\n");
> >   			break;
> >   		default:
> > -			fprintf(stderr, "A libflash/libffs error has occured %d\n", rc);
> > +			fprintf(stderr, "A libflash/libffs error has occurred %d\n", rc);
> >   	}
> >   }
> >
> > diff --git a/external/mambo/mambo_utils.tcl b/external/mambo/mambo_utils.tcl
> > index 5e97302..65bf036 100644
> > --- a/external/mambo/mambo_utils.tcl
> > +++ b/external/mambo/mambo_utils.tcl
> > @@ -199,7 +199,7 @@ proc mywatch { } {
> >       while { [mysim memory display 0x700 8] != 0 } {
> >           mysim cycle 1
> >       }
> > -    puts "condition occured "
> > +    puts "condition occurred "
> >       ipc
> >   }
> >
> > diff --git a/external/opal-prd/hostboot-interface.h b/external/opal-prd/hostboot-interface.h
> > index 0393152..05fe052 100644
> > --- a/external/opal-prd/hostboot-interface.h
> > +++ b/external/opal-prd/hostboot-interface.h
> > @@ -181,7 +181,7 @@ struct host_interfaces {
> >   	 * @brief Write to Pnor
> >   	 * @param[in] i_proc: processor Id
> >   	 * @param[in] i_partitionName: name of the partition to write
> > -	 * @param[in] i_offset: offset withing the partition
> > +	 * @param[in] i_offset: offset within the partition
> >   	 * @param[in] i_data: pointer to the data to write
> >   	 * @param[in] i_sizeBytes: size of data to write
> >   	 * @retval rc - number of bytes written, or non-zero on error
> > diff --git a/external/opal-prd/i2c.c b/external/opal-prd/i2c.c
> > index 3c7b555..fbe2e9b 100644
> > --- a/external/opal-prd/i2c.c
> > +++ b/external/opal-prd/i2c.c
> > @@ -149,7 +149,7 @@ int i2c_write(uint32_t chip_id, uint8_t engine, uint8_t port,
> >
> >   	/* Not all kernel driver versions support breaking up a write into
> >   	 * two components (offset, data), so we coalesce them first and
> > -	 * issue a single write. The offset is layed out in BE format.
> > +	 * issue a single write. The offset is laid out in BE format.
> >   	 */
> >   	size = offset_size + length;
> >   	buf = malloc(size);
> > diff --git a/external/opal-prd/opal-prd.8 b/external/opal-prd/opal-prd.8
> > index f3aa797..72027f5 100644
> > --- a/external/opal-prd/opal-prd.8
> > +++ b/external/opal-prd/opal-prd.8
> > @@ -47,7 +47,7 @@ Note that the daemon must be running in the background here, as a separate
> >   process.
> >
> >   .PP
> > -Currently, there's one command available, 'occ', for controling the
> > +Currently, there's one command available, 'occ', for controlling the
> >   on-chip-controllers. That has 3 possible sub-commands: \fIreset\fP,
> >   \fIenable\fP, and \fIdisable\fP.
> >
> > diff --git a/external/opal-prd/opal-prd.c b/external/opal-prd/opal-prd.c
> > index d3dbc16..2c132ce 100644
> > --- a/external/opal-prd/opal-prd.c
> > +++ b/external/opal-prd/opal-prd.c
> > @@ -1636,7 +1636,7 @@ static int run_prd_daemon(struct opal_prd_ctx *ctx)
> >   	pr_debug("HBRT: calling hservices_init");
> >   	rc = hservices_init(ctx, ctx->code_addr);
> >   	if (rc) {
> > -		pr_log(LOG_ERR, "HBRT: Can't initiliase HBRT");
> > +		pr_log(LOG_ERR, "HBRT: Can't initialise HBRT");
> >   		goto out_close;
> >   	}
> >   	pr_debug("HBRT: hservices_init done");
> > diff --git a/external/opal-prd/test/test_pnor_ops.c b/external/opal-prd/test/test_pnor_ops.c
> > index 7ac471c..fd5e2c2 100644
> > --- a/external/opal-prd/test/test_pnor_ops.c
> > +++ b/external/opal-prd/test/test_pnor_ops.c
> > @@ -122,7 +122,7 @@ int main(int argc, char **argv)
> >   		perror("mkstemp");
> >   		return EXIT_FAILURE;
> >   	}
> > -	/* So the file dissapears when we exit */
> > +	/* So the file disappears when we exit */
> >   	unlink(filename);
> >
> >   	/* E for empty */
> > diff --git a/external/pflash/pflash.c b/external/pflash/pflash.c
> > index fddfe1d..c124356 100644
> > --- a/external/pflash/pflash.c
> > +++ b/external/pflash/pflash.c
> > @@ -404,7 +404,7 @@ static void enable_4B_addresses(void)
> >   	rc = arch_flash_4b_mode(bl, true);
> >   	if (rc) {
> >   		if (rc == -1) {
> > -			fprintf(stderr, "Switching address mode not availible on this architecture\n");
> > +			fprintf(stderr, "Switching address mode not available on this architecture\n");
> >   		} else {
> >   			fprintf(stderr, "Error %d enabling 4b mode\n", rc);
> >   		}
> > @@ -421,7 +421,7 @@ static void disable_4B_addresses(void)
> >   	rc = arch_flash_4b_mode(bl, false);
> >   	if (rc) {
> >   		if (rc == -1) {
> > -			fprintf(stderr, "Switching address mode not availible on this architecture\n");
> > +			fprintf(stderr, "Switching address mode not available on this architecture\n");
> >   		} else {
> >   			fprintf(stderr, "Error %d enabling 4b mode\n", rc);
> >   		}
> > @@ -480,7 +480,7 @@ static void print_help(const char *pname)
> >   	printf("\t\tto specify the size to read (or it will use the source\n");
> >   	printf("\t\tfile size if used in conjunction with -p and -s is not\n");
> >   	printf("\t\tspecified). When using -r together with -e or -p, the\n");
> > -	printf("\t\tread will be peformed first\n\n");
> > +	printf("\t\tread will be performed first\n\n");
> >   	printf("\t-E, --erase-all\n");
> >   	printf("\t\tErase entire flash chip\n");
> >   	printf("\t\t(Not supported on all chips/controllers)\n\n");
> > diff --git a/external/test/test.sh b/external/test/test.sh
> > index cfea786..827cd92 100755
> > --- a/external/test/test.sh
> > +++ b/external/test/test.sh
> > @@ -43,7 +43,7 @@ strip_version_from_result() {
> >   }
> >
> >   diff_with_result() {
> > -	# Explicitly diff a file with an arbitary result file
> > +	# Explicitly diff a file with an arbitrary result file
> >   	if [ "$#" -eq 1 ] ; then
> >   		if ! diff -u "$RESULT" "$1" ; then
> >   			fail_test;
> > diff --git a/hdata/spira.h b/hdata/spira.h
> > index bd10ae7..eabf7f9 100644
> > --- a/hdata/spira.h
> > +++ b/hdata/spira.h
> > @@ -475,14 +475,14 @@ struct msvpd_pmover_bsr_synchro {
> >    * Note that slots meant for the addition of GX+ adapters that
> >    * are currently unpopulated but support hotplug will have a
> >    * minimum "placeholder" entry, which will be fully populated
> > - * when the array is rebuild during concurrent maintainance.
> > + * when the array is rebuild during concurrent maintenance.
> >    * This "placeholder" is called a "reservation".
> >    *
> > - * WARNING: The array rebuild by concurrent maintainance is not
> > + * WARNING: The array rebuild by concurrent maintenance is not
> >    * guaranteed to be in the same order as the IPL array, not is
> > - * the order stable between concurrent maintainance operations.
> > + * the order stable between concurrent maintenance operations.
> >    *
> > - * There's also a child pointer to daugher card structures but
> > + * There's also a child pointer to daughter card structures but
> >    * we aren't going to handle that just yet.
> >    */
> >   #define CECHUB_FRU_HDIF_SIG	"IO HUB"
> > @@ -659,7 +659,7 @@ struct cpu_ctl_init_data {
> >    * child. A child has a pointer to its parent. Siblings are
> >    * consecutive entries.
> >    *
> > - * Note: If we ever support concurrent maintainance... this is
> > + * Note: If we ever support concurrent maintenance... this is
> >    * completely rebuilt, invalidating all indices, though other
> >    * structures that may reference SLCA by index will be rebuilt
> >    * as well.
> > diff --git a/hw/npu.c b/hw/npu.c
> > index 79b232a..02b5dae 100644
> > --- a/hw/npu.c
> > +++ b/hw/npu.c
> > @@ -1488,7 +1488,7 @@ static void npu_dev_create_cfg(struct npu_dev *dev)
> >   	/* 0x10 - BARs, always 64-bits non-prefetchable
> >   	 *
> >   	 * Each emulated device represents one link and therefore
> > -	 * there is one BAR for the assocaited DLTL region.
> > +	 * there is one BAR for the associated DLTL region.
> >   	 */
> >
> >   	/* Low 32-bits */
> > diff --git a/hw/phb3.c b/hw/phb3.c
> > index 156bd9e..5a90cd4 100644
> > --- a/hw/phb3.c
> > +++ b/hw/phb3.c
> > @@ -1204,7 +1204,7 @@ static int64_t phb3_pci_msi_eoi(struct phb *phb,
> >   	 * To avoid this race, we increment the generation count in
> >   	 * the IVT when we clear P. When software writes the IVC with
> >   	 * P cleared but with gen=n, the IVC won't actually clear P
> > -	 * becuase gen doesn't match what it just cached from the IVT.
> > +	 * because gen doesn't match what it just cached from the IVT.
> >   	 * Hence we don't lose P being set.
> >   	 */
> >
> > @@ -3342,7 +3342,7 @@ static void phb3_init_capp_regs(struct phb3 *p, bool dma_mode)
> >   		 * HW301991 - XSL sends PTE updates with nodal scope instead of
> >   		 * group scope. The workaround is to force all commands to
> >   		 * unlimited scope by setting bit 4. This may have a slight
> > -		 * performance impact, but it would be negligable on the XSL.
> > +		 * performance impact, but it would be negligible on the XSL.
> >   		 * To avoid the possibility it might impact other cards, key it
> >   		 * off DMA mode since the XSL based Mellanox CX4 is the only
> >   		 * card to use this mode in P8 timeframe:
> > diff --git a/include/spcn.h b/include/spcn.h
> > index 2f31728..0df9163 100644
> > --- a/include/spcn.h
> > +++ b/include/spcn.h
> > @@ -137,7 +137,7 @@ struct sensor_data {
> >   #define SENSOR_STATUS_FAULTED	0x0002
> >   #define SENSOR_STATUS_PRESENT	0x0001
> >
> > -/* Power sensor is retrieved thru a new PRS modifier 0x1C, data
> > +/* Power sensor is retrieved through a new PRS modifier 0x1C, data
> >    * response is as follows:
> >    *
> >    * Byte 0:
> > diff --git a/libc/stdio/vsnprintf.c b/libc/stdio/vsnprintf.c
> > index b9435b8..fbb84a0 100644
> > --- a/libc/stdio/vsnprintf.c
> > +++ b/libc/stdio/vsnprintf.c
> > @@ -240,7 +240,7 @@ print_format(char **buffer, size_t bufsize, const char *format, void *var)
> >
> >
> >   /*
> > - * The vsnprintf function prints a formated strings into a buffer.
> > + * The vsnprintf function prints a formatted strings into a buffer.
> >    * BUG: buffer size checking does not fully work yet
> >    */
> >   int
> > diff --git a/libc/time.c b/libc/time.c
> > index 9be6e11..3411bdc 100644
> > --- a/libc/time.c
> > +++ b/libc/time.c
> > @@ -57,7 +57,7 @@ struct tm *gmtime_r(const time_t *timep, struct tm *result)
> >   	 * Work out the year. We subtract one day for every four years
> >   	 * and every 400 years after 1969. However as leap years don't
> >   	 * occur every 100 years we add one day back to counteract the
> > -	 * the substraction for every 4 years.
> > +	 * the subtraction for every 4 years.
> >   	 */
> >   	Y = (D - (1+D/365)/4 + (69+D/365)/100 - (369+D/365)/400)/365;
> >
> > diff --git a/libpore/p8_pore_table_gen_api_fixed.C b/libpore/p8_pore_table_gen_api_fixed.C
> > index 60c0679..3163bc9 100644
> > --- a/libpore/p8_pore_table_gen_api_fixed.C
> > +++ b/libpore/p8_pore_table_gen_api_fixed.C
> > @@ -715,7 +715,7 @@ uint32_t p8_pore_gen_scom_fixed(void       *io_image,
> >     //   except the APPEND command, incl the translated REPLACE->APPEND, which will result
> >     //   in the previously mentioned code error being returned.
> >     // - The table can be full but still include NOOPs. If so, we can still APPEND since
> > -  //   we append at first occurrance of a NOOP or at the end of the table (at the RET).
> > +  //   we append at first occurrence of a NOOP or at the end of the table (at the RET).
> >     switch (i_section)  {
> >     case P8_SCOM_SECTION_NC:
> >       if ( ( (operation==P8_PORE_SCOM_APPEND && entriesCount==SLW_MAX_SCOMS_NC) &&
> > diff --git a/libpore/pgas.h b/libpore/pgas.h
> > index 3d985f8..5d1dbf3 100644
> > --- a/libpore/pgas.h
> > +++ b/libpore/pgas.h
> > @@ -34,7 +34,7 @@
> >   /// \file pgas.h
> >   /// \brief Pore GAS
> >   ///
> > -/// PGAS is documented in a seperate standalone document entitled <em> PGAS :
> > +/// PGAS is documented in a separate standalone document entitled <em> PGAS :
> >   /// PORE GAS (GNU Assembler) User's and Reference Manual </em>.
> >   ///
> >   /// This file defines support macros for the GNU PORE assembler, and the PORE
> > @@ -600,7 +600,7 @@
> >           // HW274735 documents that BC and BS are broken for the PORE-GPE0/1
> >           // pair. This bug is unfixed in POWER8, and by default we require BSI
> >           // and BCI to be implemented as macros on all engines. For
> > -        // compatability we continue to require that dx == D0.
> > +        // compatibility we continue to require that dx == D0.
> >
> >           .macro  bsi, dx:req, offset:req, base:req, imm:req
> >           ..d0    (\dx)
> > diff --git a/libpore/pore_inline.h b/libpore/pore_inline.h
> > index f74aa6f..214fca6 100644
> > --- a/libpore/pore_inline.h
> > +++ b/libpore/pore_inline.h
> > @@ -291,7 +291,7 @@ typedef uint32_t PoreInlineLocation;
> >   /// PORE inline assembler context
> >   ///
> >   /// See the documentation page \ref pore_inline_assembler and the function
> > -/// pore_inline_context_create() for futher details.
> > +/// pore_inline_context_create() for further details.
> >
> >   typedef struct {
> >
> > diff --git a/libpore/pore_inline_assembler.c b/libpore/pore_inline_assembler.c
> > index 44a93db..8c92ad7 100644
> > --- a/libpore/pore_inline_assembler.c
> > +++ b/libpore/pore_inline_assembler.c
> > @@ -108,7 +108,7 @@
> >   ///
> >   /// Since the PORE instruction APIs are effectivly predicates, linear code
> >   /// sequences are easily assembled using the C-language logical OR construct.
> > -/// Any non-0 return code will immediatly break the sequence and set the
> > +/// Any non-0 return code will immediately break the sequence and set the
> >   /// expression value to 1.  The failure code can then be recovered from the \a
> >   /// error field of the context.  This coding technique is illustrated in the
> >   /// following example of assembling a memory-memory copy sequence.
> > diff --git a/libpore/sbe_xip_image.c b/libpore/sbe_xip_image.c
> > index 7ee4886..3f32d65 100644
> > --- a/libpore/sbe_xip_image.c
> > +++ b/libpore/sbe_xip_image.c
> > @@ -906,7 +906,7 @@ xipValidateTocEntry(void* io_image, const SbeXipItem* i_item, void* io_arg)
> >   // "FNV hash algorithms and source code have been released into the public
> >   // domain. The authors of the FNV algorithmm look deliberate steps to disclose
> >   // the algorhtm (sic) in a public forum soon after it was invented. More than
> > -// a year passed after this public disclosure and the authors deliberatly took
> > +// a year passed after this public disclosure and the authors deliberately took
> >   // no steps to patent the FNV algorithm. Therefore it is safe to say that the
> >   // FNV authors have no patent claims on the FNV algorithm as published."
> >
> > diff --git a/libpore/sbe_xip_image.h b/libpore/sbe_xip_image.h
> > index 8ba55b4..955f3dc 100644
> > --- a/libpore/sbe_xip_image.h
> > +++ b/libpore/sbe_xip_image.h
> > @@ -215,7 +215,7 @@
> >   /// Final alignment constraint for SBE-XIP images.
> >   ///
> >   /// PORE images are required to be multiples of 8 bytes in length, to
> > -/// gaurantee that the PoreVe will be able to complete any 8-byte load/store.
> > +/// guarantee that the PoreVe will be able to complete any 8-byte load/store.
> >   #define SBE_XIP_FINAL_ALIGNMENT 8
> >
> >
> > @@ -270,7 +270,7 @@ typedef struct {
> >       /// unaligned instruction or data fetches.  Some sections and subsections
> >       /// must also be POWER cache-line aligned. The \a iv_alignment applies to
> >       /// the first byte of the section. PORE images are also required to be
> > -    /// multiples of 8 bytes in length, to gaurantee that the PoreVe will be
> > +    /// multiples of 8 bytes in length, to guarantee that the PoreVe will be
> >       /// able to complete any 8-byte load/store.  These constraints are checked
> >       /// by sbe_xip_validate() and enforced by sbe_xip_append(). The alignment
> >       /// constraints may force a section to be padded, which may create "holes"
> > @@ -811,7 +811,7 @@ sbe_xip_get_string(void *i_image, const char* i_id, char** o_data);
> >   /// images in terms of their relocatable PORE addresses.  The API checks that
> >   /// the \a i_poreAddress is properly aligned and contained in the image, then
> >   /// reads the contents of \a i_poreAddress into \a o_data, performing
> > -/// image-to-host endianess conversion if required.
> > +/// image-to-host endianness conversion if required.
> >   ///
> >   /// \retval 0 Success
> >   ///
> > @@ -926,7 +926,7 @@ sbe_xip_set_string(void *io_image, const char* i_id, const char* i_data);
> >   /// images in terms of their relocatable PORE addresses.  The API checks that
> >   /// the \a i_poreAddress is properly aligned and contained in the image, then
> >   /// updates the contents of \a i_poreAddress with \a i_data, performing
> > -/// host-to-image endianess conversion if required.
> > +/// host-to-image endianness conversion if required.
> >   ///
> >   /// \retval 0 Success
> >   ///
> > @@ -1332,7 +1332,7 @@ sbe_xip_pore2section(const void* i_image,
> >   /// \param[in] i_hostAddress A host address addressing data within the image.
> >   ///
> >   /// \param[out] o_poreAddress The API updates the location pointed to by \a
> > -/// o_poreAddress with the equivelent relocatable PORE address of the memory
> > +/// o_poreAddress with the equivalent relocatable PORE address of the memory
> >   /// addressed by i_hostAddress.  Since valid PORE addresses are always either
> >   /// 4-byte (code) or 8-byte (data) aligned, this API checks the aligment of
> >   /// the translated address and returns SBE_XIP_ALIGNMENT_ERROR if the PORE
> > @@ -1416,7 +1416,7 @@ sbe_xip_host2pore(const void* i_image,
> >   /// Attempt to grow the image past its defined memory allocation
> >   #define SBE_XIP_WOULD_OVERFLOW 14
> >
> > -/// Error associated with the disassembler occured.
> > +/// Error associated with the disassembler occurred.
> >   #define SBE_XIP_DISASSEMBLER_ERROR 15
> >
> >   /// hash collision creating the .fixed_toc section
> 
> Regards,
> Mukesh
> 
> 
> 
> 
> 
Non-text part: text/html



More information about the Skiboot mailing list