From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Date: Fri, 15 May 2020 09:58:01 +0200 From: Oleksij Rempel Message-ID: <20200515075801.3a6667uzwqbbvvwx@pengutronix.de> References: <20200428174749.16079-1-o.rempel@pengutronix.de> <20200428174749.16079-3-o.rempel@pengutronix.de> <646feba5-c97b-f20b-26f6-c2c3753e4d87@pengutronix.de> MIME-Version: 1.0 In-Reply-To: <646feba5-c97b-f20b-26f6-c2c3753e4d87@pengutronix.de> Subject: Re: [DistroKit] [PATCH v1 3/3] MIPS: kernel: malta: remove flash partition table List-Id: DistroKit Mailinglist List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============1181158207==" Errors-To: distrokit-bounces@pengutronix.de Sender: "DistroKit" To: Ahmad Fatoum Cc: distrokit@pengutronix.de --===============1181158207== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="dyuh4shtyuaq3lw7" Content-Disposition: inline --dyuh4shtyuaq3lw7 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, May 15, 2020 at 09:42:42AM +0200, Ahmad Fatoum wrote: > Hello Oleksij, >=20 > On 4/28/20 7:47 PM, Oleksij Rempel wrote: > > it should be provided by the bootloader >=20 > barebox already deletes an existing partitions node on fixup, see of_part= ition_fixup(). > Does this not work for you? It is just warning made by analyzer. And in most cases it is indeed a bad idea to put project configuration in to a mainline kernel dts. > >=20 > > Signed-off-by: Oleksij Rempel > > --- > > ...2-MIPS-malta-remove-fixed-partitions.patch | 45 +++++++++++++++++++ > > .../platform-mips/patches/linux-5.6.2/series | 3 +- > > 2 files changed, 47 insertions(+), 1 deletion(-) > > create mode 100644 configs/platform-mips/patches/linux-5.6.2/0002-MIPS= -malta-remove-fixed-partitions.patch > >=20 > > diff --git a/configs/platform-mips/patches/linux-5.6.2/0002-MIPS-malta-= remove-fixed-partitions.patch b/configs/platform-mips/patches/linux-5.6.2/0= 002-MIPS-malta-remove-fixed-partitions.patch > > new file mode 100644 > > index 0000000..a920a4b > > --- /dev/null > > +++ b/configs/platform-mips/patches/linux-5.6.2/0002-MIPS-malta-remove-= fixed-partitions.patch > > @@ -0,0 +1,45 @@ > > +From: Oleksij Rempel > > +Date: Tue, 28 Apr 2020 14:55:25 +0200 > > +Subject: [PATCH] MIPS: malta: remove fixed partitions > > + > > +this partition table should be provided by the bootloader > > + > > +Signed-off-by: Oleksij Rempel > > +--- > > + arch/mips/boot/dts/mti/malta.dts | 23 ----------------------- > > + 1 file changed, 23 deletions(-) > > + > > +diff --git a/arch/mips/boot/dts/mti/malta.dts b/arch/mips/boot/dts/mti= /malta.dts > > +index f03279b1cde7..7642a7c53a3a 100644 > > +--- a/arch/mips/boot/dts/mti/malta.dts > > ++++ b/arch/mips/boot/dts/mti/malta.dts > > +@@ -57,29 +57,6 @@ flash@1e000000 { > > + bank-width =3D <4>; > > + #address-cells =3D <1>; > > + #size-cells =3D <1>; > > +- > > +- partitions { > > +- compatible =3D "fixed-partitions"; > > +- #address-cells =3D <1>; > > +- #size-cells =3D <1>; > > +- > > +- yamon@0 { > > +- label =3D "YAMON"; > > +- reg =3D <0x0 0x100000>; > > +- read-only; > > +- }; > > +- > > +- user-fs@100000 { > > +- label =3D "User FS"; > > +- reg =3D <0x100000 0x2e0000>; > > +- }; > > +- > > +- board-config@3e0000 { > > +- label =3D "Board Config"; > > +- reg =3D <0x3e0000 0x20000>; > > +- read-only; > > +- }; > > +- }; > > + }; > > +=20 > > + fpga_regs: system-controller@1f000000 { > > diff --git a/configs/platform-mips/patches/linux-5.6.2/series b/configs= /platform-mips/patches/linux-5.6.2/series > > index dadc772..a885d05 100644 > > --- a/configs/platform-mips/patches/linux-5.6.2/series > > +++ b/configs/platform-mips/patches/linux-5.6.2/series > > @@ -1,4 +1,5 @@ > > # generated by git-ptx-patches > > #tag:base --start-number 1 > > 0001-HACK-MIPS-disable-bogus-code.patch > > -# 1aabf59e6345434460b53d6d21377e5e - git-ptx-patches magic > > +0002-MIPS-malta-remove-fixed-partitions.patch > > +# 598cc0167d4c33db68cf711e58479093 - git-ptx-patches magic > >=20 >=20 > --=20 > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | >=20 > _______________________________________________ > DistroKit mailing list > DistroKit@pengutronix.de >=20 --=20 Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | --dyuh4shtyuaq3lw7 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEERBNZvwSgvmcMY/T74omh9DUaUbMFAl6+S4UACgkQ4omh9DUa UbM2Tg/+KyVJBmSTphiItW6h17D9KIphjn7YdcLA6eFy29v+R+YayR9tFLPvGTTi QKprtZ5R8t4DGQwPVcw7ES+LJyCue3hHMhAltywyXC2Z83hhdR/3NI7LOBIcwPvx ciwPz1eSjW1eR9jT/CsA7ebzIEzwdpSBrY78dq9cTozXfliYe3OL9jaNLBTF7oay Y27XZ0fvLZLpbSCXZEktt+oKngORcCteGnp4acoZmJwhCoEBPFvqm1XCysREPdXR BBOlmF9ojVW1DF/d73uDYP9jossMtM1wsyUEvVYHieKFNebYKLrMvCSf2h5y3ITC /lOyplVSqSB263Xit/tYb+I5z92M2hGLWSwG6hckvkfi0RKhvaUclvqpdVgbP2AK 2vwBgAogzAiWED1cjkX2kbSlN05QPLHzjbpj3qtdF4+i7kfwmVhb7Z/fxthLpyOU aKZ0S0OpII6lgJDF27LrbzBa9ez/c/UvJegoi63vWCIP5s3vrZquFLvcTzmbsTQZ J5S9qafgvsUhFY7gunAPp9Xyz+moTYcR/d+tq0rR4Co5KT+vFVAD5DExOuPwkyBg 2Y4e+svRJy4PE+7Y5SIMlibZhNJD/02MnxzSNi2htSIUP/P1rLVcT8EZzqf1kEYh CTJDw/X4ZdBPg7I41vAasWWd+gxJYzmOcijMDZnaaxlONrgDF1Y= =de8Y -----END PGP SIGNATURE----- --dyuh4shtyuaq3lw7-- --===============1181158207== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ DistroKit mailing list DistroKit@pengutronix.de --===============1181158207==--