Skip to content

Commit 6d7ddd8

Browse files
committed
Merge tag 'soc-fixes-6.9-3' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc
Pull ARM SoC fixes from Arnd Bergmann: "These are a couple of last minute fixes that came in over the previous week, addressing: - A pin configuration bug on a qualcomm board that caused issues with ethernet and mmc - Two minor code fixes for misleading console output in the microchip firmware driver - A build warning in the sifive cache driver" * tag 'soc-fixes-6.9-3' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc: firmware: microchip: clarify that sizes and addresses are in hex firmware: microchip: don't unconditionally print validation success arm64: dts: qcom: sa8155p-adp: fix SDHC2 CD pin configuration cache: sifive_ccache: Silence unused variable warning
2 parents 1ab1a19 + 5549d1e commit 6d7ddd8

File tree

3 files changed

+19
-21
lines changed

3 files changed

+19
-21
lines changed

arch/arm64/boot/dts/qcom/sa8155p-adp.dts

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -367,6 +367,16 @@
367367
};
368368
};
369369

370+
&pmm8155au_1_gpios {
371+
pmm8155au_1_sdc2_cd: sdc2-cd-default-state {
372+
pins = "gpio4";
373+
function = "normal";
374+
input-enable;
375+
bias-pull-up;
376+
power-source = <0>;
377+
};
378+
};
379+
370380
&qupv3_id_1 {
371381
status = "okay";
372382
};
@@ -384,10 +394,10 @@
384394
&sdhc_2 {
385395
status = "okay";
386396

387-
cd-gpios = <&tlmm 4 GPIO_ACTIVE_LOW>;
397+
cd-gpios = <&pmm8155au_1_gpios 4 GPIO_ACTIVE_LOW>;
388398
pinctrl-names = "default", "sleep";
389-
pinctrl-0 = <&sdc2_on>;
390-
pinctrl-1 = <&sdc2_off>;
399+
pinctrl-0 = <&sdc2_on &pmm8155au_1_sdc2_cd>;
400+
pinctrl-1 = <&sdc2_off &pmm8155au_1_sdc2_cd>;
391401
vqmmc-supply = <&vreg_l13c_2p96>; /* IO line power */
392402
vmmc-supply = <&vreg_l17a_2p96>; /* Card power line */
393403
bus-width = <4>;
@@ -505,13 +515,6 @@
505515
bias-pull-up; /* pull up */
506516
drive-strength = <16>; /* 16 MA */
507517
};
508-
509-
sd-cd-pins {
510-
pins = "gpio96";
511-
function = "gpio";
512-
bias-pull-up; /* pull up */
513-
drive-strength = <2>; /* 2 MA */
514-
};
515518
};
516519

517520
sdc2_off: sdc2-off-state {
@@ -532,13 +535,6 @@
532535
bias-pull-up; /* pull up */
533536
drive-strength = <2>; /* 2 MA */
534537
};
535-
536-
sd-cd-pins {
537-
pins = "gpio96";
538-
function = "gpio";
539-
bias-pull-up; /* pull up */
540-
drive-strength = <2>; /* 2 MA */
541-
};
542538
};
543539

544540
usb2phy_ac_en1_default: usb2phy-ac-en1-default-state {

drivers/cache/sifive_ccache.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,7 @@ static int __init sifive_ccache_init(void)
290290
struct device_node *np;
291291
struct resource res;
292292
const struct of_device_id *match;
293-
unsigned long quirks;
293+
unsigned long quirks __maybe_unused;
294294
int rc;
295295

296296
np = of_find_matching_node_and_match(NULL, sifive_ccache_ids, &match);

drivers/firmware/microchip/mpfs-auto-update.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,10 +206,12 @@ static int mpfs_auto_update_verify_image(struct fw_upload *fw_uploader)
206206
if (ret | response->resp_status) {
207207
dev_warn(priv->dev, "Verification of Upgrade Image failed!\n");
208208
ret = ret ? ret : -EBADMSG;
209+
goto free_message;
209210
}
210211

211212
dev_info(priv->dev, "Verification of Upgrade Image passed!\n");
212213

214+
free_message:
213215
devm_kfree(priv->dev, message);
214216
free_response:
215217
devm_kfree(priv->dev, response);
@@ -265,7 +267,7 @@ static int mpfs_auto_update_set_image_address(struct mpfs_auto_update_priv *priv
265267
AUTO_UPDATE_DIRECTORY_WIDTH);
266268
memset(buffer + AUTO_UPDATE_BLANK_DIRECTORY, 0x0, AUTO_UPDATE_DIRECTORY_WIDTH);
267269

268-
dev_info(priv->dev, "Writing the image address (%x) to the flash directory (%llx)\n",
270+
dev_info(priv->dev, "Writing the image address (0x%x) to the flash directory (0x%llx)\n",
269271
image_address, directory_address);
270272

271273
ret = mtd_write(priv->flash, 0x0, erase_size, &bytes_written, (u_char *)buffer);
@@ -313,7 +315,7 @@ static int mpfs_auto_update_write_bitstream(struct fw_upload *fw_uploader, const
313315
erase.len = round_up(size, (size_t)priv->flash->erasesize);
314316
erase.addr = image_address;
315317

316-
dev_info(priv->dev, "Erasing the flash at address (%x)\n", image_address);
318+
dev_info(priv->dev, "Erasing the flash at address (0x%x)\n", image_address);
317319
ret = mtd_erase(priv->flash, &erase);
318320
if (ret)
319321
goto out;
@@ -323,7 +325,7 @@ static int mpfs_auto_update_write_bitstream(struct fw_upload *fw_uploader, const
323325
* will do all of that itself - including verifying that the bitstream
324326
* is valid.
325327
*/
326-
dev_info(priv->dev, "Writing the image to the flash at address (%x)\n", image_address);
328+
dev_info(priv->dev, "Writing the image to the flash at address (0x%x)\n", image_address);
327329
ret = mtd_write(priv->flash, (loff_t)image_address, size, &bytes_written, data);
328330
if (ret)
329331
goto out;

0 commit comments

Comments
 (0)