Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 3 Feb 2022 16:34:01 +0300
From:   Dan Carpenter <dan.carpenter@oracle.com>
To:     Krzysztof Kozlowski <krzk@kernel.org>
Cc:     Sui Jingfeng <15330273260@189.cn>,
        Lucas Stach <l.stach@pengutronix.de>,
        Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Roland Scheidegger <sroland@vmware.com>,
        Zack Rusin <zackr@vmware.com>,
        Christian Gmeiner <christian.gmeiner@gmail.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Rob Herring <robh+dt@kernel.org>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        Andrey Zhizhikin <andrey.zhizhikin@leica-geosystems.com>,
        Sam Ravnborg <sam@ravnborg.org>,
        suijingfeng <suijingfeng@loongson.cn>,
        linux-mips@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        dri-devel@lists.freedesktop.org,
        Randy Dunlap <rdunlap@infradead.org>
Subject: Re: [PATCH v6 1/3] drm/lsdc: add drm driver for loongson display
 controller
Message-ID: <20220203133401.GF1978@kadam>
References: <20220203082546.3099-1-15330273260@189.cn>
 <20220203082546.3099-2-15330273260@189.cn>
 <CAJKOXPfCjx6UoLK6LgXBZCfixxq8k7E-RFXb8i-chyXc39WEoQ@mail.gmail.com>
 <20220203110751.GE1978@kadam>
 <CAJKOXPeLS+PZBCvEOeazc8N5w1D=WeO_D45yEazu-XQQeHNohQ@mail.gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAJKOXPeLS+PZBCvEOeazc8N5w1D=WeO_D45yEazu-XQQeHNohQ@mail.gmail.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?IEuK/HHPEAJR0LINNi4rJBpEpo3olhYiI0luiAXEyTo+6Jf7xYJFkl3DRPqd?=
 =?us-ascii?Q?l8Zj+GEEVPEqzYl0IAXyEMbhZmHBbE1/XpfeGXaO8Ngu01Z0myaLgfxs10QC?=
 =?us-ascii?Q?yttIsAtSZ2oRE6vB58/63xK/4brSJuwynwoZzouL7W8jeoZIYHR5O8sGaC9Q?=
 =?us-ascii?Q?dD+d7C6TrghYT280uY5tcD1AOBudfexlm7QPjhHAJNGRuT34RECUx/B5cF5U?=
 =?us-ascii?Q?3HqQxtoJ+7T12Rd00HUvJ0QLz62WfrX11k1nb+vzu5j+F0AoOCL5ul/uLlEN?=
 =?us-ascii?Q?QNBIQSKzX4KuCyl1emccOiqY0iT1uAXivvdwvyhfcxMICWH03mXqVxDki5JA?=
 =?us-ascii?Q?ucA8oWl3TyA1Fx+CA3TxeDzNiLKQRHUy46k19S+f1daiGIkcSAnfxhEx+zrz?=
 =?us-ascii?Q?UkZfQKZCHMALDQTPvENzQXazISX2Or/YKxGziMRMOBLrosbcpavumXz/WFSx?=
 =?us-ascii?Q?rg0YL5OSkGIoBZhdCOLmSUiP2PT1NBAqwh184S+Koiy2G6aSirnTjGdh9y+B?=
 =?us-ascii?Q?CNhWaJOWW0RD9BFi2BgCH9O2+UnYZipLbV9CiJF1XE0qPz5D8uyben0KQ1L8?=
 =?us-ascii?Q?Md/AfJy7j8Lo6GItbD2FCENXup52Wqa4PjLGMyoI4s/5N3lZh/nezSkbBlPO?=
 =?us-ascii?Q?FlSNN7FfHnVqcuxss6hoWQ4dQZACFaIHX9M5GTxYQ7frbuCBabqv4Y5VUK9z?=
 =?us-ascii?Q?/jXD8aRu/oUGAleoyIMeWH3yFzv7+Ky+1sjfOmrgyl6IU1haUAbCuj1YgdCD?=
 =?us-ascii?Q?pLdyyCsaMVarY4JbLGrWZBjvkY72+/dSJhWXh9dHJ6doJkJjPVnRo1LTJtXX?=
 =?us-ascii?Q?bIuZiw00pgyQutpXVHLDG+Q54Dbd0hkavpOib0OuvQYtk3J/Sk0+mwbmK9F+?=
 =?us-ascii?Q?EUzcMYzX/4VYkxmHRsFx5sTWl1GWZ5jU0vgZAszqAH61rvCfT3XfkCnVGa3j?=
 =?us-ascii?Q?vaQja6tmCJQNSnlzCRKfgIkPy84uLhIC7Usbo9QtciZboTGZvb49EP+g61t9?=
 =?us-ascii?Q?bprB5Gkq31EFaEP/qhFEEDQACTc0sJTbIZGcpUV80d8/vyRUgkf2pUQ6ekHp?=
 =?us-ascii?Q?ZJaQZOThXhVgL+YIJ1hpW2EPsLtBUoM7gwghHMWyqsfcFDUXbGSnWbwQvtve?=
 =?us-ascii?Q?/SBSnEGVxDs4fh/9w28AHc5GCu75VE4+R50IvT/AQbuqb4Rjcp/IxaRW1i6E?=
 =?us-ascii?Q?6f9qwt4ATeo11TyFCKgrhN498QOCnhNPmwCN+jvEivLekYfQcp+xX+l6NwjU?=
 =?us-ascii?Q?fLbD/KytwP9PdVrZnLEkiTPxyo+2xsvHd7f1CeHvmv5T/3EzygYPXL+KiXNX?=
 =?us-ascii?Q?02rr6s3W3vRhVetCshhvkhLErwQpWMzQ3QNeNrb+ITy0nR9G0IwTdplbYZ2p?=
 =?us-ascii?Q?Kx6xPgcasghuV+VnS+8oXvtOI4GlkI4yqfjX9qx7PG24uUyQB0+99jOAfN5L?=
 =?us-ascii?Q?cTZ8S1tes2SU2HA1BQObszS8h5ImfFySqAwFziYKy/ogjqA9RK0iKHzXgpFx?=
 =?us-ascii?Q?Mit1o2Jq3bSfEdjyiV3nrqDqjzLV8iOfQZkVekhEl8Oyjm4+beu/+FSc1dGv?=
 =?us-ascii?Q?sK4d5CgqkJT4B5pOaJorxcoetyY33g5QFLlEL2BR8U6cv/rBVgDa3s42mnqn?=
 =?us-ascii?Q?OPgBwfiSegajtCZW9teXkJr6VL8bfbdOLi3GDWTIxELxo8nIgU0foGK3Ur4Z?=
 =?us-ascii?Q?N87UGMBdyAykVTo84YullD+ItzU=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5e44626d-9a3e-47e1-e8da-08d9e719df64
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2365.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Feb 2022 13:34:15.9954
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 9aKtjRd5bamVbsLAM3A1o2bdpCzMhYzSD0xeyl22QjfVn6KzzJHZUfhVoFVApGO2fGyIeKrmrXhgqg+EBepiompdJ+QQwMTFxGAQt8l0As4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR10MB1470
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10246 signatures=673430
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 adultscore=0
 suspectscore=0 malwarescore=0 bulkscore=0 mlxscore=0 spamscore=0
 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2201110000 definitions=main-2202030084
X-Proofpoint-ORIG-GUID: 1-3INV1221BKvuH5ZQWEj9gqyJP3geXd
X-Proofpoint-GUID: 1-3INV1221BKvuH5ZQWEj9gqyJP3geXd
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Feb 03, 2022 at 12:29:11PM +0100, Krzysztof Kozlowski wrote:
> On Thu, 3 Feb 2022 at 12:08, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> > >
> > > This does not look like compliant with GPL-2.0. You cannot call a
> > > license GPL-2.0 and restrict it with some other provisions.
> >
> > That's the MIT license.  It's not the GPL-2.0 license but it is
> > compliant.
> 
> It's compliant when included as "OR" for example in SPDX tag. The
> current solution - SPDX and MIT license text - is not the proper way
> to describe this. Otherwise one could argue that both licenses apply
> at the same time and one has to fulfill both of them, which is
> ridiculous. There is a SPDX tag for the proper case - GPL or MIT.

You're saying a bunch of different things.

We both agree that the SPDX text is confusing because it says GPL-2.0+
but it has the text from the MIT license.

"This does not look like compliant with GPL-2.0."

Wrong.  The MIT license is compatible with the GPL-2.0.

"You cannot call a license GPL-2.0 and restrict it with some other
provisions."

Wrong.  The MIT license just says you have to include the No Warranty
text.  The GPL has it's own list of requirements.  But you can combine
MIT and GPL code and easily comply with both requirements.  That's what
"compatible" means in this context.

In the kernel we have MIT licensed code which is dual licensed.  This
means that someone can take that driver and release it as closed source
software if they want.

// SPDX-License-Identifier: GPL-2.0 OR MIT

Then we also have code which was originally MIT licensed but now you
have to comply with the GPL as well.

// SPDX-License-Identifier: (GPL-2.0 WITH Linux-syscall-note) AND MIT

These examples were cut and paste from Documentation/process/license-rules.rst

regards,
dan carpenter