[Mesa-dev] [PATCH 08/24] i965/cnl: Add isl_gen10 header and source files

[Jason Ekstrand]

If everything truely is the same here, I see no reason why we should add a 
gen10 version.  I know this appears to contradict what I said before but 
the difference is that this file is handwritten and not just a recompile of 
an already multi-gen file.  We don't, for instance have separate gen7 and 
gen75 isl files.


On May 12, 2017 4:40:42 PM Anuj Phogat <anuj.phogat at gmail.com> wrote:

> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  src/intel/isl/isl_gen10.c | 41 +++++++++++++++++++++++++++++++++++++++++
>  src/intel/isl/isl_gen10.h | 45 +++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 86 insertions(+)
>  create mode 100644 src/intel/isl/isl_gen10.c
>  create mode 100644 src/intel/isl/isl_gen10.h
>
> diff --git a/src/intel/isl/isl_gen10.c b/src/intel/isl/isl_gen10.c
> new file mode 100644
> index 0000000..3652d58
> --- /dev/null
> +++ b/src/intel/isl/isl_gen10.c
> @@ -0,0 +1,41 @@
> +/*
> + * Copyright 2017 Intel Corporation
> + *
> + *  Permission is hereby granted, free of charge, to any person obtaining a
> + *  copy of this software and associated documentation files (the "Software"),
> + *  to deal in the Software without restriction, including without limitation
> + *  the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + *  and/or sell copies of the Software, and to permit persons to whom the
> + *  Software is furnished to do so, subject to the following conditions:
> + *
> + *  The above copyright notice and this permission notice (including the next
> + *  paragraph) shall be included in all copies or substantial portions of the
> + *  Software.
> + *
> + *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + *  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + *  THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + *  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + *  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER 
> DEALINGS
> + *  IN THE SOFTWARE.
> + */
> +
> +#include "isl_gen9.h"
> +#include "isl_gen10.h"
> +#include "isl_priv.h"
> +
> +void
> +isl_gen10_choose_image_alignment_el(const struct isl_device *dev,
> +                                    const struct isl_surf_init_info 
> *restrict info,
> +                                    enum isl_tiling tiling,
> +                                    enum isl_dim_layout dim_layout,
> +                                    enum isl_msaa_layout msaa_layout,
> +                                    struct isl_extent3d *image_align_el)
> +{
> +   /* No changes in image alignment fields from gen9 to gen10. So, just use
> +    * the gen9 function.
> +    */
> +   isl_gen9_choose_image_alignment_el(dev, info, tiling, dim_layout,
> +                                      msaa_layout, image_align_el);
> +}
> diff --git a/src/intel/isl/isl_gen10.h b/src/intel/isl/isl_gen10.h
> new file mode 100644
> index 0000000..35c12cf
> --- /dev/null
> +++ b/src/intel/isl/isl_gen10.h
> @@ -0,0 +1,45 @@
> +/*
> + * Copyright 2017 Intel Corporation
> + *
> + *  Permission is hereby granted, free of charge, to any person obtaining a
> + *  copy of this software and associated documentation files (the "Software"),
> + *  to deal in the Software without restriction, including without limitation
> + *  the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + *  and/or sell copies of the Software, and to permit persons to whom the
> + *  Software is furnished to do so, subject to the following conditions:
> + *
> + *  The above copyright notice and this permission notice (including the next
> + *  paragraph) shall be included in all copies or substantial portions of the
> + *  Software.
> + *
> + *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + *  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + *  THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + *  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + *  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER 
> DEALINGS
> + *  IN THE SOFTWARE.
> + */
> +
> +#ifndef ISL_GEN10_H
> +#define ISL_GEN10_H
> +
> +#include "isl.h"
> +
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> +void
> +isl_gen10_choose_image_alignment_el(const struct isl_device *dev,
> +                                    const struct isl_surf_init_info 
> *restrict info,
> +                                    enum isl_tiling tiling,
> +                                    enum isl_dim_layout dim_layout,
> +                                    enum isl_msaa_layout msaa_layout,
> +                                    struct isl_extent3d *image_align_el);
> +
> +#ifdef __cplusplus
> +}
> +#endif
> +
> +#endif /* ISL_GEN10_H */
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev